uPortal IRC Logs-2006-09-25

[07:41:59 CDT(-0500)] * andrewpetro (n=microcli@uni1.unicon.net) has joined ##uportal
[07:42:12 CDT(-0500)] <andrewpetro> Good morning JA-SIG world
[08:41:57 CDT(-0500)] * bszabo (n=bszabo@uni1.unicon.net) has joined ##uportal
[08:47:55 CDT(-0500)] * EricDalquist (n=dalquist@bohemia.doit.wisc.edu) has joined ##uportal
[08:48:28 CDT(-0500)] <EricDalquist> good morning
[08:48:42 CDT(-0500)] <bszabo> Good morning Eric
[08:48:55 CDT(-0500)] <andrewpetro> WSRP working for uPortal 2.5.3
[08:49:03 CDT(-0500)] <andrewpetro> Now we just need someone who actually wants to use this...
[08:49:05 CDT(-0500)] <EricDalquist> just saw that email
[08:49:20 CDT(-0500)] <andrewpetro> yeah
[08:49:35 CDT(-0500)] <andrewpetro> somewhere we went very wrong with getting multiple committers on this project
[08:49:41 CDT(-0500)] <andrewpetro> someone should own WSRP
[08:49:50 CDT(-0500)] <andrewpetro> smeone should own file upload / download
[08:50:05 CDT(-0500)] <andrewpetro> these someones should not be me (smile)
[08:50:13 CDT(-0500)] <EricDalquist> (smile)
[09:53:06 CDT(-0500)] * dmccallum (n=dmccallu@uni1.unicon.net) has joined ##uportal
[10:32:48 CDT(-0500)] <andrewpetro> incidentally, Eric, what are you up to on the jasig box that requires the pw?
[10:32:59 CDT(-0500)] <EricDalquist> publishing the dtds
[10:33:10 CDT(-0500)] <andrewpetro> cool
[10:35:34 CDT(-0500)] <andrewpetro> has UW given any thought to use cases around end users (of instances of the portlet) having a read-only view on a centrally configured bookmarks tree (PBookmarks)
[10:36:01 CDT(-0500)] <EricDalquist> nope
[10:36:04 CDT(-0500)] <andrewpetro> (thinking through how to make template users going away "ok" for CBookmarks
[10:36:04 CDT(-0500)] <andrewpetro> )
[10:36:13 CDT(-0500)] <EricDalquist> it would be pretty easy though
[10:36:19 CDT(-0500)] <andrewpetro> you do seed the portlet with some initial links?
[10:36:25 CDT(-0500)] <EricDalquist> just create a merging bookarmks DAO
[10:36:42 CDT(-0500)] <EricDalquist> then have one dao that persists the users customized bookmarks
[10:36:51 CDT(-0500)] <EricDalquist> and one that just returns the 'central' bookmarks
[10:36:53 CDT(-0500)] <EricDalquist> no
[10:36:57 CDT(-0500)] <EricDalquist> we don't seed anything
[10:37:23 CDT(-0500)] <EricDalquist> we did migrate the user's bookmarks from the previous portal
[10:37:27 CDT(-0500)] <EricDalquist> brb
[10:41:27 CDT(-0500)] <EricDalquist> back
[10:41:55 CDT(-0500)] <EricDalquist> so yeah, it shouldn't be that hard to have some 'defaultBookmarksDao' that bookmarks get merged in from
[10:43:02 CDT(-0500)] <andrewpetro> sure
[10:43:08 CDT(-0500)] <andrewpetro> or even is the only source of bookmarks
[10:43:16 CDT(-0500)] <EricDalquist> yup
[10:43:19 CDT(-0500)] <andrewpetro> I'm thinking the Sports tab has a cute tree of sports bookmarks
[10:43:24 CDT(-0500)] <EricDalquist> depending on how you want it to work
[10:43:25 CDT(-0500)] <andrewpetro> the library tab has a cute tree of library bookmarks
[10:43:26 CDT(-0500)] <andrewpetro> etc.
[10:44:05 CDT(-0500)] <andrewpetro> I think that's what CBookmarks was supposed to be doing with its ability to point at a template user to get the bookmarks
[10:44:37 CDT(-0500)] <EricDalquist> yeah that should be easy enough
[10:44:57 CDT(-0500)] <EricDalquist> the other logical part would be to enable a read-only mode for the portlet
[10:44:58 CDT(-0500)] <andrewpetro> cute
[10:45:06 CDT(-0500)] <andrewpetro> since CBookmarks doesn't actually ship with uP
[10:45:17 CDT(-0500)] <andrewpetro> it's probably not required to actually enhance PBookmarks to do this
[10:45:25 CDT(-0500)] <andrewpetro> so long as we cans state that it would be easy to do so
[10:45:26 CDT(-0500)] <EricDalquist> nope
[10:45:30 CDT(-0500)] <EricDalquist> people can do it if they want
[10:45:41 CDT(-0500)] <andrewpetro> and so that's the recommended path for anyone annoyed by the loss of template users in their CBookmarks context
[10:45:51 CDT(-0500)] <EricDalquist> yup
[10:47:26 CDT(-0500)] <andrewpetro> cool
[10:47:37 CDT(-0500)] <andrewpetro> so need to steal your portlet and plop it into our product
[10:47:49 CDT(-0500)] <EricDalquist> sounds good
[10:47:56 CDT(-0500)] <EricDalquist> maybe you'll write some neat addons
[10:54:07 CDT(-0500)] * peterk (i=[U2FsdGV@66.226.77.81) has joined ##uportal
[11:25:19 CDT(-0500)] <andrewpetro> looking it over, it sure looks like it would be easy to re-use a lot of this to implement the use case where the privileged few configure some bookmarks
[11:25:23 CDT(-0500)] <andrewpetro> for the consumption of the huddled masses
[11:25:30 CDT(-0500)] <EricDalquist> yup
[11:26:02 CDT(-0500)] <andrewpetro> heh, maybe even web2.0 style ability for end users to create bookmarks trees and talk their friends into subscribing to them...
[11:26:30 CDT(-0500)] <andrewpetro> just daydreaming about activities other than finalizing this release build...
[11:27:30 CDT(-0500)] <EricDalquist> (smile)
[11:27:40 CDT(-0500)] <EricDalquist> funny how it always works that way
[11:28:31 CDT(-0500)] <peterk> sometimes if you just start coding that "dream activity" you quickly get bogged down and return to mundane things, like cutting a release (smile)
[11:30:20 CDT(-0500)] <andrewpetro> yeah. I did the first increment of Spring WebMVC on uP 2.6 this weekend. It really cleaned up PortalSessionManager turning it into a Controller
[11:30:41 CDT(-0500)] <andrewpetro> and the uPortal error page is oh so much more sane as an error handler mapping, JSP than as the hard coded thing it is today
[12:52:16 CDT(-0500)] <EricDalquist> so peterk I've been reading the rendering attribute docs and I'm trying to figure out what related class I need to re-implement and what I can re-use
[12:52:53 CDT(-0500)] <EricDalquist> It looks like I could probably use the existing ElementResolverImpl to target only channel/portlet nodes
[12:53:17 CDT(-0500)] <EricDalquist> but I would likely need a custom IElementResolverKeyFactory to generate keys for the elements based on the portlet definition id?
[12:54:43 CDT(-0500)] <EricDalquist> bszabo you there?
[12:54:48 CDT(-0500)] <bszabo> Yes
[12:55:04 CDT(-0500)] <EricDalquist> could you make a request to NickB for me?
[12:55:08 CDT(-0500)] <bszabo> Sure
[12:55:20 CDT(-0500)] <EricDalquist> looking at his latest commit his log messages look like "log.debug("GroupsManagerBean::genericPortletRequestPreProcess..."
[12:55:53 CDT(-0500)] <EricDalquist> putting the class & method in the message just adds clutter, the log packages will provide that info if the log configurer wants it
[12:56:11 CDT(-0500)] <EricDalquist> the log statements should just have the message (& exception)
[12:57:16 CDT(-0500)] <bszabo> I passed it along.
[12:57:24 CDT(-0500)] <EricDalquist> thank you (smile)
[12:57:37 CDT(-0500)] <bszabo> np
[12:57:53 CDT(-0500)] * deuce_ (n=deuce@uni1.unicon.net) has joined ##uportal
[12:58:13 CDT(-0500)] <peterk> Eric: yah, you'd need to make a new resolver that would pick up window id from the xml stream and look up definition id based on it.
[12:58:19 CDT(-0500)] <deuce_> just being consistent with the other log message in legacy code .. it's going away...
[12:58:31 CDT(-0500)] <EricDalquist> np
[12:58:40 CDT(-0500)] <EricDalquist> thanks for taking it out
[12:58:42 CDT(-0500)] <peterk> btw., it may be worthwhile to consider adding IPortletWindow.getPortletEntity().getPortletDefinition( ) type of calls
[12:58:55 CDT(-0500)] <EricDalquist> yeah
[12:58:57 CDT(-0500)] <deuce_> i meant the code is going away... (smile)
[12:59:01 CDT(-0500)] <EricDalquist> ah ok
[12:59:04 CDT(-0500)] <peterk> ok, bbl
[12:59:06 CDT(-0500)] <EricDalquist> that works too
[12:59:22 CDT(-0500)] <EricDalquist> ok, so a new IElementResolver?
[12:59:26 CDT(-0500)] <EricDalquist> or just the key factory?
[13:00:13 CDT(-0500)] <EricDalquist> it looks like the IElementResolver is just responsible for picking the element to generate a key for from the stream and at that level we're just targeting portlet elements?
[14:01:16 CDT(-0500)] * shawnlonas (n=shawn@uni1.unicon.net) has joined ##uportal
[14:33:23 CDT(-0500)] * andrewpetro (n=microcli@uni1.unicon.net) has joined ##uportal
[15:00:09 CDT(-0500)] <peterk> Eric: you could fit this into a key factory, but you should make a new element resolver. Factories are just for creating ids based on the selected attributes, etc.
[15:01:30 CDT(-0500)] <EricDalquist> ok, should I follow the same pattern of using a IElementResolverKeyFactory to generate the key for the new resolver? I'm thinking the key will reference the portlet definition id?
[15:07:18 CDT(-0500)] <EricDalquist> peterk could I get a run down the interfaces you're thinking I need to re-implement. Looking at the code and the wiki documentation I'm seeing several possible ways I could implement the feature. I want to make sure I'm taking the 'correct' approach
[15:12:29 CDT(-0500)] <peterk> you should be able to get away with just implementing new IElementResolver and reusing existing components to configure an attribute provider that would persist attributes for a portlet definition
[15:12:44 CDT(-0500)] <EricDalquist> ok
[15:14:39 CDT(-0500)] <peterk> did you figure out what attributes you actually need this for ? (smile)
[15:15:36 CDT(-0500)] <EricDalquist> well thats the next part of this endevor. Is this how we're planning on dealing with things like minimized/maximized? Or should I look at things like icon/cssUri/jsUri ?
[15:17:30 CDT(-0500)] <peterk> minimized/maximized is done through rendering attributes, but those providers are specific to the theme/structure transforms. If you're making a provider that's scoped to a portlet definition, it should be used to define attributes that are constant for a particular portlet difinition. So icon/cssUri and such are potential candidates, but in uP2 context we don't have a structure/theme that can handle those anyway.
[15:18:01 CDT(-0500)] <EricDalquist> any suggestions for canidate attributes?
[15:20:30 CDT(-0500)] <peterk> well, there wasn't such a thing in uP2 (smile) ... I think dan (and whoever was working on portlet mode mappings) were envisioning some use for them.
[15:20:36 CDT(-0500)] <peterk> at the current stage
[15:20:51 CDT(-0500)] <peterk> we'll have plenty of uses once we start remaking the presentation layer
[15:22:00 CDT(-0500)] <EricDalquist> dmccallum, is there an envisioned use for portlet definition scoped rendering attributes for the short term uP3 release?
[15:22:01 CDT(-0500)] <EricDalquist> http://www.ja-sig.org/issues/browse/UPT-191
[15:37:15 CDT(-0500)] <dmccallum> It was on the short term road map. It's actual urgency, I assume, is open to debate.
[15:37:22 CDT(-0500)] <EricDalquist> ok
[15:47:09 CDT(-0500)] <EricDalquist> so peterk, sorry for the large number of questions but I'm having issues understanding how this is all going to fit together.
[15:47:17 CDT(-0500)] <EricDalquist> it looks like I'll also need to write my own persistence code
[15:47:28 CDT(-0500)] <EricDalquist> since the current persistence layer stores per-user attributes
[15:51:13 CDT(-0500)] <EricDalquist> also are IRenderingAttributeProviders meant to be thread safe?
[15:51:34 CDT(-0500)] <EricDalquist> If the *Parameter methods are refering to URL parameters it would appear that they are not
[16:01:16 CDT(-0500)] <peterk> Eric: parameters are name-value pairs not associated with a particular element (as in XSL params for the transformation preferences)
[16:01:45 CDT(-0500)] <peterk> I think you're right about the persistence code, since I've only implemented the user-scoped persistence.
[16:02:14 CDT(-0500)] <peterk> once again, I would clear up with the rest of the team why the wanted this done in the first phase.
[16:02:30 CDT(-0500)] <EricDalquist> ok
[16:03:18 CDT(-0500)] <EricDalquist> Can one of the guys in Chandler comment on why http://www.ja-sig.org/issues/browse/UPT-191 is on the short term road map?
[16:05:17 CDT(-0500)] <dmccallum> I think we'd need to involve John Lewis in that discussion. I don't think any of us Chandler guys other than John were involved in deciding what was in and what was out.
[16:05:46 CDT(-0500)] <dmccallum> i'll ping him
[16:06:11 CDT(-0500)] <EricDalquist> ok
[16:06:13 CDT(-0500)] <EricDalquist> thanks
[16:07:45 CDT(-0500)] <andrewpetro> it sure looks puntable to me. Maybe a concern that if we don't engineer it in now it will get harder to do later?
[16:08:30 CDT(-0500)] <EricDalquist> could be, but it sure would be easier to engineer if I had an actuall use-case(s) to think about
[16:09:59 CDT(-0500)] <dmccallum> john's not responding. we may want to circle back to this when he has some free time
[16:10:05 CDT(-0500)] <EricDalquist> ok
[16:10:16 CDT(-0500)] <EricDalquist> I'll send an email about it to him and charise
[16:14:47 CDT(-0500)] <andrewpetro> sigh, anyone remember how to update www.uportal.org news?
[16:14:55 CDT(-0500)] <EricDalquist> nope
[16:15:03 CDT(-0500)] <andrewpetro> sometimes I really miss the days when that was just HTML in CVS
[16:15:11 CDT(-0500)] <andrewpetro> I was pretty confident in updating that
[16:20:23 CDT(-0500)] <andrewpetro> well, I guess the socially well adjusted thing to do would be to document this
[16:20:24 CDT(-0500)] <andrewpetro> http://www.ja-sig.org/wiki/display/JSG/Using+Hypercontent+to+post+news+to+JA-SIG.org
[16:33:26 CDT(-0500)] * johnalewis (n=jlewis@uni1.unicon.net) has joined ##uportal
[16:34:20 CDT(-0500)] <johnalewis> Eric, Dan mentioned you had a question about UPT-191
[16:34:45 CDT(-0500)] <EricDalquist> yes, just wondering why it is on the short term road map
[16:35:11 CDT(-0500)] <EricDalquist> we were talking about usage, and the only thing I'm aware of is features for the uP3 rendering context which isn't part of the short term deliverable
[16:35:31 CDT(-0500)] <johnalewis> honestly, I'm not sure. Probably best as a question for Peter. I imagine we put it there due to his input. It may be there are other features that depend on it.
[16:35:49 CDT(-0500)] <EricDalquist> he is the one that originally asked why it was on the short term road map (smile)
[16:36:07 CDT(-0500)] <johnalewis> Could be that we wanted it there in order to be ready for the uP3 context – I agree that we can move it to mid-term.
[16:36:24 CDT(-0500)] <EricDalquist> ok
[16:36:38 CDT(-0500)] <EricDalquist> I'm out for the day but I'll touch base with him on that later and likely move the task
[16:36:40 CDT(-0500)] <EricDalquist> thanks
[16:37:49 CDT(-0500)] * johnalewis (n=jlewis@uni1.unicon.net) has left ##uportal
[17:31:59 CDT(-0500)] * EricDalquist (n=dalquist@ppp-70-226-174-248.dsl.mdsnwi.ameritech.net) has joined ##uportal
[19:03:11 CDT(-0500)] * dmccallum (n=dmccallu@uni1.unicon.net) has left ##uportal
[19:38:48 CDT(-0500)] * shawnlonas (n=shawn@uni1.unicon.net) has left ##uportal
[20:19:19 CDT(-0500)] * ajanovski (n=Miranda@80.77.149.170) has joined ##uportal
[20:41:48 CDT(-0500)] * ajanovski5882 (n=Miranda@80.77.149.170) has joined ##uportal
[21:04:34 CDT(-0500)] * andrewpetro (n=microcli@uni1.unicon.net) has left ##uportal