uPortal IRC Logs-2009-03-05

[05:05:47 EST(-0500)] * higmad (n=chatzill@pcit-8752.HIG.SE) has joined ##uportal
[08:19:11 EST(-0500)] * dstn (n=dstn@unaffiliated/dstn) has joined ##uportal
[08:37:17 EST(-0500)] * EricDalquist (n=EricDalq@216.143.72.2) has joined ##uportal
[09:03:53 EST(-0500)] * anastasiac (n=stasia@142.150.154.189) has joined ##uportal
[09:37:20 EST(-0500)] <dstn> for the logs, EricDalquist: just finished a full export on our data with rel-2-6-export and it completed without errors. Total time: 60 minutes 48 seconds. I haven't verified that the export is valid yet.
[09:48:13 EST(-0500)] * michelled (n=team@142.150.154.193) has joined ##uportal
[09:54:36 EST(-0500)] * lennard1 (n=sparhk@wsip-98-174-242-39.ph.ph.cox.net) has joined ##uportal
[09:58:43 EST(-0500)] * EricDalquist (n=EricDalq@216.143.72.100) has joined ##uportal
[10:04:26 EST(-0500)] <EricDalquist> good morning everyone
[10:07:02 EST(-0500)] * athena7 (n=athena7@99.129.100.66) has joined ##uportal
[10:20:01 EST(-0500)] * Arvids (n=arvids@213.175.95.54) has joined ##uportal
[10:20:48 EST(-0500)] <atpa8a> hello
[10:21:21 EST(-0500)] <EricDalquist> hello
[10:22:10 EST(-0500)] <Arvids> hi, i commented http://www.ja-sig.org/issues/browse/UP-2328
[10:22:25 EST(-0500)] <athena7> thanks Arvids, i saw that
[10:22:29 EST(-0500)] <athena7> i'll take a look at it tonight
[10:22:45 EST(-0500)] <Arvids> great! That bug annoys when creating a new layout owner
[10:22:46 EST(-0500)] <athena7> which user were you logging in as?
[10:22:55 EST(-0500)] <athena7> i actually wasn't able to reproduce it
[10:23:01 EST(-0500)] <Arvids> i created a new layout owner
[10:23:04 EST(-0500)] <Arvids> without any tabs
[10:23:31 EST(-0500)] <athena7> ah ok
[10:23:42 EST(-0500)] <athena7> so this wasn't switching to one of the existing fragment owners?
[10:24:03 EST(-0500)] <Arvids> no, but i'm sure that you can reproduce it by deleting all tabs from existing one
[10:24:27 EST(-0500)] <athena7> i have to admit i'm not even sure what uportal does when you delete all the tabs
[10:24:32 EST(-0500)] <athena7> does it automatically add one back in?
[10:24:40 EST(-0500)] * holdorph (n=holdorph@uni1.unicon.net) has joined ##uportal
[10:24:51 EST(-0500)] <Arvids> i don't think so
[10:25:10 EST(-0500)] <athena7> if there are no more tabs though, what are you left with?
[10:26:42 EST(-0500)] <Arvids> there aren't any tabs at the navigation
[10:26:51 EST(-0500)] <Arvids> but controls to the right are still there
[10:27:35 EST(-0500)] <athena7> interesting
[10:27:50 EST(-0500)] <athena7> i wonder what's supposed to happen in that case
[10:28:13 EST(-0500)] <athena7> like if uportal expects that case, or if it expects users to add a tab before adding content, etc.
[10:28:28 EST(-0500)] <athena7> i mean i can fix the js bug, but sort of more wondering about the larger behavior
[10:28:42 EST(-0500)] <athena7> thanks a lot for the debugging info - i hadn't realized it was a new fragment, not one of the existing ones
[10:29:52 EST(-0500)] <Arvids> not sure about behaviour, but i suppose there shouldn't be "Add content" "Page layout" controls
[10:31:06 EST(-0500)] <athena7> yeah
[10:31:19 EST(-0500)] <athena7> i'll have to check and see what the legacy tool does
[10:38:45 EST(-0500)] <dstn> EricDalquist, you see my comments in the logs?
[10:38:52 EST(-0500)] <EricDalquist> no
[10:38:56 EST(-0500)] <EricDalquist> I'll go check ...
[10:39:42 EST(-0500)] <EricDalquist> dstn: great
[10:39:53 EST(-0500)] <EricDalquist> I guess now you just need to double check the data
[10:40:02 EST(-0500)] <EricDalquist> did you get any non zero byte .layout files?
[10:42:06 EST(-0500)] <dstn> ya, they are all > 0 bytes now
[10:42:22 EST(-0500)] <dstn> whatever you did in the last commit fixed it I guess?
[10:42:22 EST(-0500)] <EricDalquist> so no 0 byte layout files?
[10:42:27 EST(-0500)] <EricDalquist> great
[10:55:42 EST(-0500)] * awills (n=awills@wsip-98-174-242-39.ph.ph.cox.net) has joined ##uportal
[11:08:42 EST(-0500)] <EricDalquist> awills: so dstn confirmed that the scripts backported into the 2.6-export branch are working
[11:10:41 EST(-0500)] <awills> now that he has 2 missing files and the null layout business is cleared up? that's great
[11:10:50 EST(-0500)] <EricDalquist> yup
[11:11:58 EST(-0500)] <awills> this is a good solution then... the next school that uses it to upgrade will probably have a more seamless experience than cal poly
[11:15:45 EST(-0500)] <EricDalquist> yeah
[11:16:48 EST(-0500)] <EricDalquist> I still need to finish the backport into 2.5
[11:17:00 EST(-0500)] <EricDalquist> and I'm going to wait for a bit more testing from yale
[11:17:11 EST(-0500)] <EricDalquist> but next week we can look at packaging up releases of both of these
[11:17:58 EST(-0500)] <awills> sounds great
[11:18:24 EST(-0500)] <EricDalquist> I went through and added the null checks to all of the write-document calls in 2.6-export
[11:18:37 EST(-0500)] <EricDalquist> but that approach needs to get ported into trunk and the 3.x patches code
[11:19:23 EST(-0500)] <dstn> question, what is the difference between permission and permission_set?
[11:20:14 EST(-0500)] <EricDalquist> a permission_set is just a bunch of permissions in one fioe
[11:20:15 EST(-0500)] <dstn> no permission_set's were generated from our export...that's what I was curious
[11:20:16 EST(-0500)] <EricDalquist> file
[11:20:25 EST(-0500)] <EricDalquist> and permission_sets are only in 3.1
[11:20:27 EST(-0500)] <dstn> that's why*
[11:20:30 EST(-0500)] <dstn> ok
[11:20:38 EST(-0500)] <dstn> thanks
[11:20:38 EST(-0500)] <EricDalquist> I backported the code so it would work as an import into 3.0 or 3.1
[11:23:11 EST(-0500)] <awills> where did you move import.props and export.props in trunk? i'm glad you did, i was thinking there's got to be a better place
[11:23:26 EST(-0500)] <EricDalquist> directly under properties
[11:24:05 EST(-0500)] <awills> perfect
[11:39:27 EST(-0500)] <atpa8a> hmm
[11:39:29 EST(-0500)] <dstn> so there are potentially more hooks for is-user-included than just all-layouts and all-users, for instance memberships
[11:40:03 EST(-0500)] <EricDalquist> ah yes
[11:40:14 EST(-0500)] <dstn> this could be added to export-memberships.crn
[11:40:24 EST(-0500)] * dstn is pastebinning
[11:40:45 EST(-0500)] <atpa8a> got a question on the use of org.jasig.services.persondir.support.jdbc.SingleRowJdbcPersonAttributeDao...
[11:40:55 EST(-0500)] <dstn> http://uportal.pastebin.com/mdd43bf6
[11:40:55 EST(-0500)] <atpa8a> always getting 'DEBUG [org.jasig.services.persondir.support.jdbc.SingleRowJdbcPersonAttributeDao] - <The seed does not cont
[11:40:58 EST(-0500)] <atpa8a> ain the required information to run the query, returning null.>'
[11:41:01 EST(-0500)] <dstn> it's all mangled but you get the idea
[11:41:02 EST(-0500)] <dstn> brb
[11:41:34 EST(-0500)] <EricDalquist> looks good dstn
[11:44:27 EST(-0500)] <EricDalquist> atpa8a: can you pastebin your person directory config?
[11:45:24 EST(-0500)] <atpa8a> EricDalquist: it's not really person directory... it's CAS
[11:45:47 EST(-0500)] <EricDalquist> ah
[11:45:52 EST(-0500)] <EricDalquist> well it uses person directory
[11:45:58 EST(-0500)] <atpa8a> but it uses it
[11:46:08 EST(-0500)] <atpa8a> let me paste the relevant config...
[11:46:13 EST(-0500)] <atpa8a> or the whole config
[11:46:29 EST(-0500)] <EricDalquist> relevant config is fine
[11:47:06 EST(-0500)] <atpa8a> http://uportal.pastebin.com/m4481c98c
[11:47:19 EST(-0500)] <atpa8a> tell me if you need more info
[11:47:35 EST(-0500)] <awills> this looks a bit off: http://uportal.pastebin.com/d673adf74
[11:47:56 EST(-0500)] <EricDalquist> awills: yeah
[11:48:04 EST(-0500)] <EricDalquist> our svn needs some content type pruning
[11:49:18 EST(-0500)] <EricDalquist> I can't remember enough about it ... I'd go ask on the cas email list
[11:51:47 EST(-0500)] <atpa8a> ok
[11:51:49 EST(-0500)] <atpa8a> thanks
[11:53:28 EST(-0500)] <dstn> "I am in a deep soup.. please help me out..." <--- from the user list
[11:53:32 EST(-0500)] <dstn> wow
[11:53:33 EST(-0500)] <dstn> lol
[11:53:34 EST(-0500)] <EricDalquist> yup
[11:56:04 EST(-0500)] <dstn> so I was just asked for our new uPortal instance whether they should install java 5 or java 6...I know uPortal compiles as 1.5 but are there others here running on 6?
[11:56:39 EST(-0500)] <EricDalquist> we're on 5 right now
[11:56:50 EST(-0500)] <EricDalquist> 6 would likely perform a little better
[11:57:10 EST(-0500)] <EricDalquist> as sun generally has performance improvements when you upgrade JVMs
[12:04:44 EST(-0500)] * dstn didn't know that
[12:04:46 EST(-0500)] <dstn> thanks
[12:05:13 EST(-0500)] <dstn> committed that user check in export-memberships, rev 45280
[12:05:48 EST(-0500)] <holdorph> 6 also has some slightly better jconsole/jmx conveniences
[12:06:23 EST(-0500)] <EricDalquist> dstn: in which branch?
[12:09:10 EST(-0500)] <EricDalquist> athena7: are there any 3.1 issues that I should assign to Gary?
[12:09:30 EST(-0500)] <athena7> i think i've assigned most of them over to him already
[12:09:56 EST(-0500)] <athena7> talked to him about them this morning - i think he may be able to address those
[12:10:04 EST(-0500)] <EricDalquist> great
[12:10:07 EST(-0500)] <EricDalquist> thanks
[12:10:13 EST(-0500)] <athena7> yep
[12:10:13 EST(-0500)] <athena7> if not, i'll take a look at the css
[12:10:21 EST(-0500)] <athena7> but me fixing css is much more inefficient
[12:10:36 EST(-0500)] <athena7> i have a few things to update tonight
[12:10:37 EST(-0500)] <EricDalquist> no problem
[12:10:51 EST(-0500)] <athena7> mostly things i've already done but haven't had sufficient net access to commit
[12:11:03 EST(-0500)] <athena7> found a bunch of image paths that needed to be updated in the admin tools in particular
[12:11:24 EST(-0500)] <EricDalquist> ah yeah
[12:17:14 EST(-0500)] * mrogers (n=mrogers@cabinlake.cc.umanitoba.ca) has joined ##uportal
[12:46:22 EST(-0500)] <dstn> EricDalquist, sorry, went to lunch. rel-2-6-export branch
[12:46:29 EST(-0500)] <EricDalquist> got it
[12:48:21 EST(-0500)] <awills> anyone know if there are any current issues with <expiration-cache>0</expiration-cache>?
[12:48:43 EST(-0500)] <EricDalquist> no
[12:48:52 EST(-0500)] <awills> didn't think so... thanks
[12:48:55 EST(-0500)] <EricDalquist> no issues that I know of
[13:19:46 EST(-0500)] * Arvids (n=arvids@213.175.95.54) has left ##uportal
[13:31:41 EST(-0500)] <EricDalquist> awills: you around?
[13:40:10 EST(-0500)] <awills> yeah, what's up?
[13:40:28 EST(-0500)] <EricDalquist> found a couple issues around the JPA dao for dlm evaluators
[13:40:36 EST(-0500)] <EricDalquist> just a sec for jira links
[13:40:47 EST(-0500)] <awills> sure
[13:42:31 EST(-0500)] <awills> let me look at them, but then probably just assign them to me and I can start on them in the very near term (like today)
[13:42:42 EST(-0500)] <EricDalquist> ok
[13:42:59 EST(-0500)] <EricDalquist> one is that the DAO needs to be defined by an interface
[13:43:12 EST(-0500)] <EricDalquist> we don't by default use CGLIB in uPortal
[13:43:22 EST(-0500)] <awills> yeah i did see that one go by... that's for aspects?
[13:43:25 EST(-0500)] <EricDalquist> so without an interface the transactional annotation is meaningless
[13:43:50 EST(-0500)] <EricDalquist> also the export scripts have SQL directly against the JPA managed table
[13:43:57 EST(-0500)] <EricDalquist> instead of using methods on the JPA daos
[13:44:26 EST(-0500)] <awills> one sec...
[13:44:30 EST(-0500)] <EricDalquist> so when we fixed some other issues related to column names and such
[13:44:39 EST(-0500)] <EricDalquist> when jira starts behaving again I'll add that one too
[13:46:29 EST(-0500)] <awills> yeah give it to me... i made sure to use the DAO (and thought it was so cool that i could) except i guess i used the table to get the list of fragments in existance :?
[13:46:54 EST(-0500)] <EricDalquist> yeah
[13:47:25 EST(-0500)] <awills> it probably started w/ a copy-paste from another entity... this is a good thing to clear up
[13:49:19 EST(-0500)] <awills> yeah i'll take 'em both, certainly, and i'd like to help with some of the other small things that are surfacing w/ the RC process if i can
[13:55:32 EST(-0500)] <dstn> <!-- Ignore channels, their group membership information is handled by the chanpub files -->
[13:55:42 EST(-0500)] <dstn> what are chanpub files...are these the .channel?
[13:55:47 EST(-0500)] <EricDalquist> yes
[13:55:56 EST(-0500)] <EricDalquist> chanpub == .channel
[14:01:32 EST(-0500)] <dstn> added is-user-included to export-group_membership in rel-2-6-export, rev 45284
[14:01:43 EST(-0500)] <EricDalquist> great
[14:34:33 EST(-0500)] <EricDalquist> dstn: I just made a few tweaks in the 2.6 export scripts too
[15:05:37 EST(-0500)] <dstn> EricDalquist, k. I'm adding one more tweak in export-preferences, a null check for PORTLET_DATA[2] which is the channel fname...this will get rid of preferences for orphaned channels
[15:05:46 EST(-0500)] <EricDalquist> ah
[15:05:49 EST(-0500)] <EricDalquist> good call
[15:15:24 EST(-0500)] <dstn> is there a way to configure eclipse to use spaces when I tab?
[15:15:32 EST(-0500)] <EricDalquist> yes
[15:15:47 EST(-0500)] <EricDalquist> in preferences
[15:16:03 EST(-0500)] <EricDalquist> type 'tab' in the search box in the upper left
[15:16:10 EST(-0500)] <EricDalquist> General > Editors > Text Editors
[15:16:26 EST(-0500)] <dstn> got it
[15:16:28 EST(-0500)] <dstn> thanks
[15:25:52 EST(-0500)] <dstn> ok, that's checked in to the branch, 45294
[15:26:05 EST(-0500)] <EricDalquist> great
[15:27:40 EST(-0500)] <dstn> what were the tweaks you did btw?
[15:48:59 EST(-0500)] <EricDalquist> dstn: ah, adding a few more is-user-included checks
[15:49:06 EST(-0500)] <EricDalquist> and adding the system user
[15:49:13 EST(-0500)] <EricDalquist> so they are always included
[15:49:25 EST(-0500)] <EricDalquist> thought
[15:49:27 EST(-0500)] <EricDalquist> though*
[15:49:31 EST(-0500)] <EricDalquist> that may not be nessesary
[15:49:53 EST(-0500)] <EricDalquist> we probably want people to use the system user and layout that comes with the version of uportal they are upgrading to
[16:02:36 EST(-0500)] <dstn> ya, that's back to that sort of required entities thing
[16:02:42 EST(-0500)] <EricDalquist> yeah
[16:09:15 EST(-0500)] <dstn> what is the result of this? will the users be missing content or changes or what? – [java] WARNING: DLM Node Reference n456 could not be completely resolved and will be ignored: [user, null, ]
[16:23:14 EST(-0500)] <dstn> interesting how one builds maven
[16:23:43 EST(-0500)] <EricDalquist> hrm
[16:24:17 EST(-0500)] <EricDalquist> I'm not sure, it probably depends on what was calling the dlm noderef lookup script
[16:26:40 EST(-0500)] <dstn> alright, I'll look into it more...I'm getting a lot of em
[16:27:35 EST(-0500)] <EricDalquist> yeah ... so what should probably happen
[16:28:11 EST(-0500)] <EricDalquist> actually
[16:28:14 EST(-0500)] <EricDalquist> searching on that string
[16:28:20 EST(-0500)] <EricDalquist> looks like it is in export-layout.crn
[16:28:46 EST(-0500)] <EricDalquist> so that would likely be a dlm move or delete referencing a node that no longer exists in the fragment owner's layut
[16:30:08 EST(-0500)] <dstn> ok, that's normal then
[16:47:39 EST(-0500)] <atpa8a> hmm
[16:48:16 EST(-0500)] <atpa8a> i hate that SingleRowJdbcPersonAttributeDao and all superclasses...
[16:48:45 EST(-0500)] <atpa8a> is it the same in 1.1.2? or whatever the latest person directory is?
[16:48:56 EST(-0500)] <EricDalquist> the JAR file should have a version number on it
[16:49:06 EST(-0500)] <atpa8a> yeah
[16:49:14 EST(-0500)] <atpa8a> in my repo the latest is 1.1.2
[16:49:28 EST(-0500)] <EricDalquist> http://developer.jasig.org/projects/person-directory/1.1.2/apidocs/index.html
[16:49:57 EST(-0500)] <atpa8a> i'd usully cut the tips of the fingers of my developers for code like that
[16:50:17 EST(-0500)] <EricDalquist> what about it is being problematic?
[16:50:28 EST(-0500)] <atpa8a> somebody really likes collections
[16:51:06 EST(-0500)] <EricDalquist> ah yeah
[16:51:13 EST(-0500)] <EricDalquist> the joy of legacy api support
[16:52:41 EST(-0500)] <atpa8a> yeah
[16:53:20 EST(-0500)] <atpa8a> i was kinda hoping you'd say 'NO! it was rewritten using objects!'
[16:53:49 EST(-0500)] <EricDalquist> PD 1.5 has a few more strongly typed objectsw
[16:54:04 EST(-0500)] <EricDalquist> for a lot of the collections code your objects would just be reimplenting a Map or a Set
[16:54:11 EST(-0500)] <EricDalquist> or a very simple wrapper class
[16:56:26 EST(-0500)] <atpa8a> POJOs man should be simple POJOs
[16:56:51 EST(-0500)] <EricDalquist> so :
[16:57:13 EST(-0500)]

[16:58:21 EST(-0500)] <atpa8a> Set<UserAttributes>
[16:58:43 EST(-0500)] <atpa8a> especially... those things are processed one at a time anyway
[16:59:39 EST(-0500)] <EricDalquist> what is in a UserAttributes?
[17:02:40 EST(-0500)] <atpa8a> columnName (corresponds to the query) and attributeName
[17:02:57 EST(-0500)] <atpa8a> attributeValue possibly
[17:03:16 EST(-0500)] <atpa8a> might be some collections
[17:04:08 EST(-0500)] <atpa8a> Set<Attribute> where Attribute is a name-value pairs
[17:04:21 EST(-0500)] <EricDalquist> that would work
[17:04:53 EST(-0500)] <atpa8a> would be nice and transparent
[17:05:05 EST(-0500)] <atpa8a> especially with spring
[17:05:38 EST(-0500)] <EricDalquist> well it would be more verbose in the spring config
[17:06:02 EST(-0500)] <EricDalquist> since you'd switch from lots of <entry> to lots of <bean class="Attribute><stuff></bean>
[17:06:14 EST(-0500)] <atpa8a> right
[17:06:21 EST(-0500)] <atpa8a> which is ok by me
[17:06:33 EST(-0500)] <atpa8a> i don't mind spring contexts
[17:06:53 EST(-0500)] <atpa8a> i hate overuse of collections
[17:07:11 EST(-0500)] * athena7 suspects that many people do mind excessively long spring configs
[17:07:48 EST(-0500)] <atpa8a> athena7: split them up?
[17:08:31 EST(-0500)] <atpa8a> rather trivial
[17:08:43 EST(-0500)] <holdorph> i don't think it's simply the length of the file, but rather the 'amount of spring config' she was getting at
[17:08:43 EST(-0500)] <atpa8a> CAS started doing just that btw
[17:08:48 EST(-0500)] <athena7> yes
[17:09:00 EST(-0500)] <holdorph> uportal already has several spring config files
[17:09:07 EST(-0500)] <athena7> in fact, i think it's probably the complaint i hear most about spring in general
[17:09:43 EST(-0500)] <atpa8a> the ones that complain probably didn't spend hours trying to figure out what maps to what in what map
[17:10:46 EST(-0500)] <atpa8a> like i just did
[17:11:35 EST(-0500)] <holdorph> actually you're the first I've seen complain about the overuse of collections in PD. Not saying i agree or disagree with that sentiment, just the first I've seen of it.
[17:11:54 EST(-0500)] <holdorph> however, I have seen lots of complaints over the length/amount of spring config
[17:12:41 EST(-0500)] <atpa8a> spring is good
[17:13:54 EST(-0500)] <athena7> yeah, i'm not sure i've seen that complaint before either
[17:14:17 EST(-0500)] <athena7> i think the times i've found collections to be irritating and confusing have generally been when they're poorly named and don't use generics
[17:14:23 EST(-0500)] <atpa8a> it's an opinion if you will
[17:14:35 EST(-0500)] <athena7> there's nothing i hate more than trying to figure out what random bucket of stuff to expect
[17:14:57 EST(-0500)] <atpa8a> athena7: as is the case with attributes
[17:15:09 EST(-0500)] <atpa8a> sans generics
[17:30:28 EST(-0500)] * anastasiac (n=stasia@142.150.154.189) has left ##uportal
[18:23:41 EST(-0500)] * EricDalquist (n=EricDalq@216.143.72.2) has joined ##uportal
[19:10:49 EST(-0500)] * lennard1 (n=sparhk@wsip-98-174-242-39.ph.ph.cox.net) has left ##uportal