uPortal IRC Logs-2010-11-19

[09:34:43 CST(-0600)] <EricDalquist> http://projectlombok.org
[09:34:51 CST(-0600)] <EricDalquist> looks like an AMAZING use of annotations ...
[10:06:26 CST(-0600)] <rickard> I'm working on adding some stuff to the helpdesk and CMDB software at work (from Computer Associates). Just took a look at the database. It has 2865 tables, 1388 views and 266 triggers.
[10:06:46 CST(-0600)] <EricDalquist> wow
[10:06:47 CST(-0600)] <rickard> Someone loves to normalize their data.
[10:06:51 CST(-0600)] <EricDalquist>
[10:07:11 CST(-0600)] <holdorph> 2865 tables. wow.
[10:07:50 CST(-0600)] <rickard> And they're all wonderfully named. For example ai_hwdef_login_lscandate_rel
[10:08:20 CST(-0600)] <holdorph> well it gets hard to name that many tables after a while, i imagine
[10:08:50 CST(-0600)] <rickard> I guess
[10:09:52 CST(-0600)] <rickard> Or it could be a plot to force us to bring in a consultant.
[10:54:22 CST(-0600)] <awills> update on header/footer content from fragments: fragment-admin-exit does what we want to to do if it's put in a <header> area with hidden="Y" in each frag-layout
[10:55:21 CST(-0600)] <awills> hidden=Y is meaningless when the channel is conjured in place with <xsl:copyof select="//channel[@fname=...]">
[11:00:07 CST(-0600)] <awills> (but hidden folders aren't copied over by DLM... forgot that part)
[11:00:49 CST(-0600)] <EricDalquist> is that in the structure or the theme?
[11:01:14 CST(-0600)] <EricDalquist> the big thing to check is the output of the structure transform
[11:01:36 CST(-0600)] <EricDalquist> and remember that every <channel> entry in that output results in the initiation of an async rendering thread
[11:01:40 CST(-0600)] <awills> it's getUserLayoutDOM()... well before any transform
[11:01:57 CST(-0600)] <EricDalquist> I was refering to <xsl:copyof select="//channel[@fname=...]">
[11:02:09 CST(-0600)] <awills> that's in the theme
[11:02:21 CST(-0600)] <awills> bb soon
[11:31:48 CST(-0600)] <awills> are there any remaining questions on their earlier topic SricDalquist?
[11:31:58 CST(-0600)] <awills> er EricDalquist
[11:32:02 CST(-0600)] <EricDalquist>
[11:32:42 CST(-0600)] <EricDalquist> my only question was if there are any extra/dupe <channel> elements coming out of the structure transform
[11:34:52 CST(-0600)] <awills> committing small data changes right now that squash them
[11:35:05 CST(-0600)] <EricDalquist> great
[13:23:12 CST(-0600)] <EricDalquist> athena: you have a minute for brain-storming around the channel list json data?
[13:23:24 CST(-0600)] <athena> sure, what's up?
[13:23:48 CST(-0600)] <EricDalquist> so after dealing with 3.2 a little more we have people asking for a better way to handle uncategorized channels
[13:24:10 CST(-0600)] <EricDalquist> my thought was having the channel list servlet create a fake "Uncategorized" category in the data model it writes out
[13:24:19 CST(-0600)] <EricDalquist> for the "manage" view
[13:24:20 CST(-0600)] <athena> that sounds reasonable
[13:24:23 CST(-0600)] <athena> wel
[13:24:44 CST(-0600)] <athena> what's supposed to happen is that those channels show up at the very root of the registry json
[13:24:49 CST(-0600)] <EricDalquist> right
[13:24:53 CST(-0600)] <EricDalquist> that is what is happening right now
[13:24:59 CST(-0600)] <EricDalquist> so I guess the question is
[13:25:13 CST(-0600)] <EricDalquist> if we want to do an uncategorized faux category
[13:25:23 CST(-0600)] <EricDalquist> do we do that in the channelList servlet or in the JS
[13:25:40 CST(-0600)] <athena> at which point they'd get automatically included in the ui - so that's definitely a bug
[13:26:12 CST(-0600)] <EricDalquist> well not really
[13:26:23 CST(-0600)] <EricDalquist> since the gallery UI asks for the list with no type parameter
[13:26:29 CST(-0600)] <EricDalquist> so uncategorized channels are never returned
[13:26:38 CST(-0600)] <EricDalquist> only the portlet manager asks for the list with type=manage
[13:26:46 CST(-0600)] <athena> hang on - phone
[13:26:46 CST(-0600)] <EricDalquist> which results in uncategorized channels being returned too
[13:26:51 CST(-0600)] <EricDalquist> no problem
[13:38:01 CST(-0600)] <athena> ooook back
[13:38:05 CST(-0600)] <EricDalquist>
[13:38:10 CST(-0600)] <EricDalquist> so I'm testing something out right now
[13:38:11 CST(-0600)] <athena> ok
[13:38:12 CST(-0600)] <EricDalquist> we'll see if it works
[13:38:12 CST(-0600)] <athena> so
[13:38:19 CST(-0600)] <athena> i think we need to back up and make some changes
[13:38:27 CST(-0600)] <athena> you shouldn't need the type=manage parameter
[13:38:44 CST(-0600)] <athena> what we should be doing is sending back uncategorized channels or not depending on your permissions
[13:38:45 CST(-0600)] <EricDalquist> you do right now if you want to see uncategorized channels in the portlet manager
[13:39:04 CST(-0600)] <athena> so if it's type=manage, we should be sending back uncategorized channels if you have the manage permission on the ALL_CHANNELS target
[13:39:23 CST(-0600)] <athena> if it's not, we should send back uncategorized channels if you have the subscribe permission on the ALL_CHANNELS target
[13:39:35 CST(-0600)] <athena> so i'd classify that as a bug we need to fix
[13:39:38 CST(-0600)] <EricDalquist> ok
[13:39:57 CST(-0600)] <athena> then from there we can decide whether it's more useful to have the uncategorized channels be at the root of the registry or in a fake group
[13:40:03 CST(-0600)] <EricDalquist> that won't result in showing uncategorized channels to most users will it?
[13:40:28 CST(-0600)] <EricDalquist> also theses are two related but independent issus
[13:40:34 CST(-0600)] <athena> yes, agreed
[13:40:43 CST(-0600)] <athena> so no, that shouldn't result in uncategorized channels showing up for most users
[13:40:51 CST(-0600)] <athena> they'd only show up for users with ALL_CHANNELS permissions
[13:41:10 CST(-0600)] <athena> most user permissions are scoped to a particular category or portlet
[13:41:26 CST(-0600)] <athena> want me to check in a fix for that part of the issue?
[13:42:01 CST(-0600)] <EricDalquist> just a sec … phone here
[13:42:06 CST(-0600)] <athena> sure
[13:43:47 CST(-0600)] <EricDalquist> sure that sounds good
[13:43:51 CST(-0600)] <EricDalquist> right now I'm working in 3.2.4 with the gallery patch
[13:44:00 CST(-0600)] <athena> oh yay
[13:44:01 CST(-0600)] <EricDalquist> and what I've done in up-portlet-registry.js is:
[13:44:21 CST(-0600)] <EricDalquist> http://uportal.pastebin.com/k0SnMpDa
[13:44:26 CST(-0600)] <EricDalquist> testing it now
[13:44:49 CST(-0600)] <athena> i think that makes sense
[13:45:01 CST(-0600)] <athena> and probably better display than putting them just at hte root
[13:45:06 CST(-0600)] <EricDalquist> yeah
[13:45:19 CST(-0600)] <athena> in some ways i think that's a nice feature because it'd be a way to find channels that have fallen out of the category heirarchy
[13:45:21 CST(-0600)] <athena> i like it
[13:45:33 CST(-0600)] <athena> we'll just have to make sure that we make the strings for the name and description injectable
[13:45:52 CST(-0600)] <athena> trying to complete my war on i18n issues
[13:46:09 CST(-0600)] <EricDalquist> yup
[13:46:11 CST(-0600)] <EricDalquist> that is our issue
[13:46:11 CST(-0600)] * athena waves a sword full of umlauts
[13:46:27 CST(-0600)] <EricDalquist> we have 39 uncategorized portlets out of 159
[13:46:40 CST(-0600)] <EricDalquist> and some of those should be categorized
[13:47:52 CST(-0600)] <athena> ee that's a lot
[13:47:55 CST(-0600)] <EricDalquist> JS error
[13:48:19 CST(-0600)] <EricDalquist> yeah
[13:58:22 CST(-0600)] <EricDalquist> yay got something working!
[13:58:38 CST(-0600)] <EricDalquist> so for these gallery fixes
[13:58:44 CST(-0600)] <EricDalquist> should I just get the committed back to trunk
[13:58:51 CST(-0600)] <EricDalquist> and make sure you let you know about the commits
[13:58:57 CST(-0600)] <EricDalquist> soyou can update the 3.2 patch?
[14:05:49 CST(-0600)] <EricDalquist> http://uportal.pastebin.com/mWFU31HA
[14:09:05 CST(-0600)] <athena> yeah that'd be great
[14:09:24 CST(-0600)] <athena> i think we can just keep adding stuff to the patch documentation page for now so we don't lose it?
[14:12:38 CST(-0600)] <athena> arrrgh
[14:12:43 CST(-0600)] <athena> i remember what the issue is now
[14:12:44 CST(-0600)] <athena> sigh
[14:24:12 CST(-0600)] <EricDalquist> ?
[14:44:33 CST(-0600)] <athena> so we don't have separate permissions for subscription and rendering
[14:44:39 CST(-0600)] <EricDalquist> oh yeah
[14:44:47 CST(-0600)] <athena> so i'll need to write some code to explicitly check the right permission based on lifecycle state
[14:44:48 CST(-0600)] <athena> annoying
[15:48:26 CST(-0600)] <JoeMoore> You folks post such great wireframes/mockups. What tool(s) do you use?
[15:49:32 CST(-0600)] <EricDalquist> what page(s) are you looking at?
[15:50:02 CST(-0600)] <JoeMoore> Just in general, presentations from Unicon etc
[15:50:11 CST(-0600)] <JoeMoore> Sometimes in JIRAs
[15:53:20 CST(-0600)] <JoeMoore> Of course, as a consumer, I don't always know the difference between a mock-up and a preview of code that is a work in progress
[15:54:24 CST(-0600)] <JoeMoore> One example was when the portlet administration portlet was still being written, the examples were referred to as wireframes
[15:54:32 CST(-0600)] <EricDalquist> ah
[15:54:37 CST(-0600)] <JoeMoore> but they looked just like screenshots.
[15:54:40 CST(-0600)] <EricDalquist> yeah I'm not sure what Gary/matt use
[15:55:14 CST(-0600)] <JoeMoore> It doesn't look like either of them are online, I'll drop them an e-mail
[15:55:17 CST(-0600)] <JoeMoore> Thanks!
[17:23:45 CST(-0600)] <EricDalquist> awills: in 3.2 I'm getting groovy.lang.MissingPropertyException: No such property: channelRegistryStore for class: Script16
[17:23:51 CST(-0600)] <EricDalquist> when trying to use the export portlet
[17:24:12 CST(-0600)] <EricDalquist> in importExportContext.xml I see the requestAttributes map contains channelRegistryStore
[17:32:53 CST(-0600)] <awills> ic... is that bad?
[17:33:04 CST(-0600)] <EricDalquist> well the export fails
[17:33:09 CST(-0600)] <EricDalquist> with a big ol err
[17:33:10 CST(-0600)] <awills> i mean, can we still use channelRegistryStore
[17:33:14 CST(-0600)] <awills> is it a thing
[17:33:19 CST(-0600)] <EricDalquist> yeah I believe so
[17:33:45 CST(-0600)] <awills> ok... let me reproduce and look
[17:33:53 CST(-0600)] <EricDalquist> trying from the command line to double check
[17:34:43 CST(-0600)] <EricDalquist> command line export works
[17:34:48 CST(-0600)] <EricDalquist> portlet export does not
[17:35:15 CST(-0600)] <awills> oooooh... this is in the portlet
[17:35:20 CST(-0600)] <EricDalquist> yeah
[17:35:45 CST(-0600)] <awills> ok, i think that can be cleaned up
[17:36:34 CST(-0600)] <EricDalquist> so I'm looking in doExport and thinking that it is a problem there in how it is loading the export scripts?
[17:36:44 CST(-0600)] <awills> yes, exactly
[17:41:48 CST(-0600)] <awills> this will likely do it: http://uportal.pastebin.com/yYEpEdJb
[17:42:11 CST(-0600)] <awills> i can verify in just a bit
[17:42:17 CST(-0600)] <EricDalquist> thanks awills I can test that shortly
[17:44:54 CST(-0600)] <awills> we could probably suggest & auto-complete that textbox with the new rest services
[17:47:20 CST(-0600)] <EricDalquist> so awills all of the export types in doExport will need a similar update then right?
[17:47:44 CST(-0600)] <awills> not all, though it's a good idea and I'll get on it
[17:48:07 CST(-0600)] <awills> we may not need to give them individual treatment any longer anyway
[17:48:27 CST(-0600)]

[17:48:28 CST(-0600)] <EricDalquist> well this is broken in 3.2 so I'll file a jira for it
[17:48:36 CST(-0600)] <awills> one sec
[17:48:50 CST(-0600)] <EricDalquist> just trying to fix up issues here for our 3.2 upgrade
[17:50:51 CST(-0600)] <awills> yeah... i thought there may have been a jira already, but i'm not seeing it
[17:51:35 CST(-0600)] <awills> this issue rings a bell
[17:53:22 CST(-0600)] <EricDalquist> that fix worked
[17:53:31 CST(-0600)] <EricDalquist> haven't tried any ofthe other types
[17:53:59 CST(-0600)] <awills> yeah that fix worked... was going to say
[17:54:15 CST(-0600)] <awills> well, several of the other types are already using that pattern
[17:54:20 CST(-0600)] <awills> more or less
[17:54:37 CST(-0600)]

<awills> it used to be that you could only pass a String to $

Unknown macro: {crn}

[17:54:51 CST(-0600)] <awills> but now you can pass a string or a live task object
[17:55:10 CST(-0600)] <awills> so there's like 100 lines of cruft that can come off that file
[17:56:07 CST(-0600)] <awills> and i believe all or nearly all imp/exp operations are now defined in that importExportContext.xml file
[17:56:12 CST(-0600)] <awills> and in that map
[17:56:14 CST(-0600)] <EricDalquist> ok
[17:56:40 CST(-0600)] <awills> i have to run to harry potter in a few min, otherwise i'd do it now
[17:57:38 CST(-0600)] <EricDalquist> no problem
[17:57:46 CST(-0600)] <EricDalquist> I'll file a jira and assign it to you?
[17:57:58 CST(-0600)] <awills> yes please
[17:58:04 CST(-0600)] <EricDalquist> sounds good
[17:58:05 CST(-0600)] <EricDalquist> thanks!
[17:58:11 CST(-0600)] <awills> absolutely