uPortal IRC Logs-2010-12-30

Owned by Apereo Infrastructure
Last updated: Dec 30, 2010Version comment

[09:21:37 CST(-0600)] <EricDalquist> yay for just spending a whole day on uportal/pluto
[10:41:00 CST(-0600)] <athena> hurray!
[10:41:07 CST(-0600)] <athena> my changes are almost ready for checkin
[10:41:10 CST(-0600)] <EricDalquist> great
[10:41:16 CST(-0600)] <athena> just a few bugs left in the portlet admin portlet that i haven't tracked down
[10:41:33 CST(-0600)] <athena> definitely glad you get to spend the day (smile)
[10:41:35 CST(-0600)] <EricDalquist> nickB is working on cookie support right now
[10:41:39 CST(-0600)] <athena> oh terrific (smile)
[10:41:42 CST(-0600)] <athena> sounds like a day of progress!
[10:41:51 CST(-0600)] <EricDalquist> https://wiki.jasig.org/display/UPC/Portlet+Cookie+Support#
[10:41:54 CST(-0600)] <EricDalquist> if you're interested
[10:42:05 CST(-0600)]

<EricDalquist> oh are you seeing "You are signed in as

Unknown macro: {0}

" in trunk?


[10:42:15 CST(-0600)]

<EricDalquist> actually I guess I see "You are signed in as

Unknown macro: {0}

 Admin User"


[10:42:37 CST(-0600)] <athena> oh nice
[10:42:44 CST(-0600)] <athena> hm, i think i may have fixed that locally
[10:43:07 CST(-0600)] <athena> speaking of which i still haven't figured out a way to parameterize messages in the XSLT and include HTML
[10:43:15 CST(-0600)] <athena> the HTML always gets escaped
[10:43:22 CST(-0600)] <EricDalquist> hrm
[10:43:36 CST(-0600)] <EricDalquist> where is the java code that does the i18n in xslt?
[10:48:32 CST(-0600)] <nickblair> UP-2861 (portlet cookie support) underway; i'm about ready to put together some tests, wondering if I should mimic o.j.p.portlet.dao.jpa.JpaPortletDaoTest (which depends on deprecated parent class from spring)
[10:50:45 CST(-0600)] <EricDalquist> yeah, that was probably written pre-spring 3.0
[10:51:18 CST(-0600)] <nickblair> yep, the docs point to the org.springframework.test.context.junit38.AbstractJUnit38SpringContextTests oddly enough
[10:51:37 CST(-0600)] <athena> nick i think i should probably check in my work for you
[10:51:53 CST(-0600)] <nickblair> are we both working on portlet cookie support? (smile)
[10:51:55 CST(-0600)] <athena> since i've made a whole lot of changes to the JpaPortletDao, etc.
[10:51:56 CST(-0600)] <athena> no
[10:52:01 CST(-0600)] <nickblair> oh ok
[10:52:05 CST(-0600)] <athena> i was working on collapsing the portlet and channel models
[10:52:09 CST(-0600)] <nickblair> way cool
[10:52:18 CST(-0600)] <athena> things aren't perfect - in particular the portlet admin portlet doesn't quite work yet
[10:52:40 CST(-0600)] <athena> but maybe i should check in what i have so far so we won't have to sync up later?
[10:52:42 CST(-0600)] <nickblair> right now the portlet cookie support is not tied in to anything else; it's just a model and a basic dao for storing/updating/removing
[10:52:58 CST(-0600)] <EricDalquist> well it will affect your tests a bit
[10:53:04 CST(-0600)] <nickblair> ok
[10:53:15 CST(-0600)] <EricDalquist> since to create a PortletCookie you need to have a PortletEntity, PortletDefinition, & ChannelDefinition right now
[10:53:23 CST(-0600)] <athena> yeah
[10:53:29 CST(-0600)] <EricDalquist> after jen's commit the ChannelDefinition bit would be gone right?
[10:53:52 CST(-0600)] <nickblair> the only reference to anything else in the uportal project so far is the PortletCookie has a reference to a IPortletEntity
[10:54:11 CST(-0600)] <EricDalquist> right the runtime code won't be affected
[10:54:14 CST(-0600)] <EricDalquist> but your unit test will be
[10:54:45 CST(-0600)] <nickblair> ok, i can wait for jen's commit
[10:56:48 CST(-0600)] <athena> as you can imagine, it's pretty big
[11:02:15 CST(-0600)] <EricDalquist> athena: how close are you to your commit? I have a small commit to make but it will require you to do a mvn -U build to grab the latest pluto snapshot once I make it
[11:02:40 CST(-0600)] <athena> i can commit right now, actually
[11:02:52 CST(-0600)] <athena> just went through and did an audit of which things need committing
[11:02:53 CST(-0600)] <EricDalquist> ok, I'm fine with that (smile)
[11:02:55 CST(-0600)] <athena> ok
[11:03:10 CST(-0600)] <athena> so we'll just have to wait for me to click on everything (tongue)
[11:03:36 CST(-0600)] <EricDalquist> lol
[11:04:42 CST(-0600)] <nickblair> i just realized I have a local modification for universality's page.xsl related to UP-2851 that wasn't committed
[11:05:02 CST(-0600)] <EricDalquist> (smile)
[11:05:06 CST(-0600)] <nickblair> I'll wait until the big commit is in place
[11:06:13 CST(-0600)] <nickblair> with the latest maven plugin update, I've got a whole set of .classpath and .project files in every maven module of the project - does anyone else see this?
[11:06:25 CST(-0600)] <EricDalquist> yup
[11:06:32 CST(-0600)] <nickblair> should I accept the versions in the repo?
[11:06:35 CST(-0600)] <EricDalquist> because every module is is own project
[11:06:44 CST(-0600)] <EricDalquist> you can or you can ignore them
[11:07:02 CST(-0600)] <EricDalquist> doesn't really matter for uportal, just for how you want to set it up in eclipse
[11:07:11 CST(-0600)] <nickblair> ok
[11:07:30 CST(-0600)] <nickblair> mine don't have the attributes element on the MAVEN2_CLASSPATHCONTAINER element
[11:07:34 CST(-0600)] <nickblair> nor do they have:
[11:07:34 CST(-0600)] <nickblair> <classpathentry kind="con" path="org.eclipse.jst.j2ee.internal.web.container"/>
[11:07:35 CST(-0600)] <nickblair> <classpathentry kind="con" path="org.eclipse.jst.j2ee.internal.module.container"/>
[11:07:54 CST(-0600)] <EricDalquist> hrm, maybe different versions of eclipse and associated plugins?
[11:07:58 CST(-0600)] <nickblair> could be
[11:08:07 CST(-0600)] <EricDalquist> maybe it is time to revisit if we should even have eclipse project files in SVN
[11:08:13 CST(-0600)] <nickblair> I'll switch to the repo versions and see what happens
[11:08:15 CST(-0600)] <athena> argh
[11:08:33 CST(-0600)] <EricDalquist> uhoh
[11:08:44 CST(-0600)] <athena> oh, just subclipse being a pain
[11:09:06 CST(-0600)] <athena> svn: Cannot non-recursively commit a directory deletion of a directory with child nodes
[11:10:48 CST(-0600)] <EricDalquist> :/
[11:11:40 CST(-0600)] <nickblair> a number of the changes are to prefs files under the .settings directory for each project; the only change is a timestamp
[11:11:46 CST(-0600)] <athena> might have to do this one in parts
[11:14:46 CST(-0600)] <athena> ok, yeah
[11:14:55 CST(-0600)] <athena> going to do the java part, then the import files
[11:20:21 CST(-0600)] <athena> ok, looks like it's working now
[11:20:27 CST(-0600)] <athena> so obviously this requires an ant initdb
[11:20:36 CST(-0600)] <EricDalquist> (smile)
[11:20:40 CST(-0600)] <athena> and hopefully i haven't missed any important files
[11:20:52 CST(-0600)] <athena> though i did want to commit this while some other people were online so i can fix it if i missed something
[11:21:41 CST(-0600)] <EricDalquist> ok
[11:21:53 CST(-0600)] <EricDalquist> I'll give it a try before I commit my stuff
[11:23:47 CST(-0600)] <athena> ok, i think it shoudl be in now
[11:24:09 CST(-0600)] <athena> hmm.
[11:24:20 CST(-0600)] <athena> failing up updates due to .project files
[11:24:25 CST(-0600)] <EricDalquist> uhg
[11:24:39 CST(-0600)] <EricDalquist> think we should just stop putting those in SVN?
[11:24:43 CST(-0600)] <athena> yeah, i think so
[11:24:45 CST(-0600)] <EricDalquist> ok
[11:24:48 CST(-0600)] <EricDalquist> I'll do that shortly
[11:25:03 CST(-0600)] <nickblair> can that be done on the repo? e.g. reject .project/.classpath etc?
[11:25:04 CST(-0600)] <athena> i've been getting a lot of conflicts w/ them anyway
[11:26:16 CST(-0600)] <EricDalquist> well its up to each project
[11:26:32 CST(-0600)] <EricDalquist> I wouldn't want uPortal's decision to affect portlets, cas, bedework, etc
[11:26:40 CST(-0600)] <EricDalquist> I can just svn:ignore all of them
[11:27:39 CST(-0600)] <EricDalquist> shouldn't take more than a few minutes here ...
[11:29:46 CST(-0600)] <EricDalquist> ok they are all deleted from SVN
[11:29:53 CST(-0600)] <EricDalquist> I'll get the svn:ignores updated shortly
[11:32:13 CST(-0600)] <EricDalquist> and I am having no luck with my screencasts
[11:32:22 CST(-0600)] <EricDalquist> I got the ogv converted to h264
[11:32:37 CST(-0600)] <EricDalquist> but quicktime just shows black for the video'
[11:32:41 CST(-0600)] <EricDalquist> plays fine in vlc
[11:32:47 CST(-0600)] <athena> oh (sad)
[11:32:54 CST(-0600)] <athena> video encoding is such a pain
[11:33:06 CST(-0600)] <EricDalquist> but quicktime on snow leopard has built in screen capture!
[11:33:14 CST(-0600)] <athena> oh really?
[11:33:16 CST(-0600)] <EricDalquist> so now I just need to record it all for the 3rd time (tongue)
[11:33:16 CST(-0600)] <athena> i didn't realize that
[11:33:17 CST(-0600)] <EricDalquist> yeah
[11:33:18 CST(-0600)] <athena> lol
[11:33:21 CST(-0600)] <EricDalquist> just found that by accident
[11:33:22 CST(-0600)] <athena> that's actually great to know
[11:33:24 CST(-0600)] <athena> wow, awesome!
[11:33:27 CST(-0600)] <athena> it work well?
[11:33:29 CST(-0600)] <EricDalquist> yeah
[11:33:38 CST(-0600)] <EricDalquist> gives you a h264 video of your whole screen
[11:33:42 CST(-0600)] <EricDalquist> and imovie and crop it
[11:33:47 CST(-0600)] <EricDalquist> er can crop it
[11:34:25 CST(-0600)] <nickblair> giving up on GTK recordMyDesktop?
[11:35:19 CST(-0600)] <nickblair> i could never figure out why some of the movies it produced where just blank green screens
[11:35:52 CST(-0600)] <EricDalquist> I used it
[11:35:56 CST(-0600)] <EricDalquist> and it got me an OGV
[11:35:59 CST(-0600)] <EricDalquist> but I need to edit it
[11:36:02 CST(-0600)] <EricDalquist> and do voice over
[11:36:08 CST(-0600)] <EricDalquist> so I was hoping to use imovie for that
[11:36:18 CST(-0600)] <EricDalquist> used ffmpeg to convert the ogv to h264
[11:36:21 CST(-0600)] <EricDalquist> and that plays in VLC
[11:36:31 CST(-0600)] <EricDalquist> but quicktime based apps just play a black screen
[11:36:32 CST(-0600)] <athena> the trunk seem to be building ok?
[11:36:45 CST(-0600)] <EricDalquist> haven't checked yet … finishing up the svn:ignore stuff
[11:36:49 CST(-0600)] <nickblair> yep, I've updated and tests are passing
[11:36:53 CST(-0600)] <nickblair> initdb seems to be running ok
[11:38:24 CST(-0600)] <nickblair> starting uportal in tomcat fails...
[11:38:53 CST(-0600)] <nickblair> http://uportal.pastebin.com/YRxMwW6C
[11:40:39 CST(-0600)] <nickblair> unrelated note, I just committed that minor change to universality page.xsl for UP-2851
[11:43:19 CST(-0600)] <athena> let me do an initdb on my end and see if i missed something
[11:43:54 CST(-0600)] <nickblair> org.jasig.portal.security.xslt.XalanAuthorizationHelperBean still has a reference to IChannelRegistryStore
[11:44:09 CST(-0600)] <athena> ah
[11:44:12 CST(-0600)] <athena> thanks!
[11:44:17 CST(-0600)] <athena> let me get that committed
[11:44:21 CST(-0600)] <athena> missed teh grouphelper too
[11:44:58 CST(-0600)] <athena> there we go - committed
[11:47:53 CST(-0600)] <nickblair> there's a new compilation error, doesn't appear to be related to this change though
[11:48:03 CST(-0600)] <EricDalquist> what is it?
[11:48:05 CST(-0600)] <nickblair> org.jasig.portal.portlet.container.PortletRequestContextImpl
[11:48:07 CST(-0600)] <EricDalquist> yup
[11:48:18 CST(-0600)] <EricDalquist> you got the new pluto snapshot and I haven't made my change yet (smile)
[11:48:22 CST(-0600)] <nickblair> ah ha
[11:48:23 CST(-0600)] <EricDalquist> it is an easy fix though
[11:48:37 CST(-0600)] <EricDalquist> since im still a little bit away can you do it
[11:48:37 CST(-0600)] <EricDalquist> in org.jasig.portal.portlet.container.PortletRequestContextImpl
[11:48:46 CST(-0600)] <EricDalquist> there is a new method defined by the interface
[11:48:51 CST(-0600)] <EricDalquist> you can just implement it
[11:48:55 CST(-0600)] <EricDalquist> and it just returns null
[11:48:56 CST(-0600)] <nickblair> Object getAttribute(String arg0, ServletRequest arg1)
[11:49:27 CST(-0600)] <athena> oh ok, i was about to say i'm having an issue too (smile)
[11:49:31 CST(-0600)] <EricDalquist> yeah
[11:49:44 CST(-0600)] <EricDalquist> I made the change in pluto
[11:49:48 CST(-0600)] <EricDalquist> and deployed the new snapshot
[11:49:56 CST(-0600)] <EricDalquist> but handn't made the uportal change
[11:50:01 CST(-0600)] <EricDalquist> you can commit that too nick
[11:50:04 CST(-0600)] <nickblair> ok
[11:50:59 CST(-0600)] <nickblair> committed
[11:51:07 CST(-0600)] <nickblair> lunch time
[11:52:00 CST(-0600)] <nickblair> should I check in what I've got for portlet cookie - is anyone interested in reviewing?
[11:52:08 CST(-0600)] <nickblair> there are no tests yet and the dao is not complete
[11:52:17 CST(-0600)] <nickblair> but the data model is close if not done
[12:15:55 CST(-0600)] <nickblair> with the new pluto snapshot, a clean -shared and deploy-ear is required too
[12:16:01 CST(-0600)] <athena> ah!
[12:16:02 CST(-0600)] <athena> right.
[12:16:22 CST(-0600)] <EricDalquist> (smile)
[12:16:30 CST(-0600)] <nickblair> that caught me just now
[12:16:51 CST(-0600)] <EricDalquist> ok, finally have the eclipse project files all ignored
[12:18:25 CST(-0600)] <athena> hurray
[12:21:37 CST(-0600)] <nickblair> excellent
[12:24:02 CST(-0600)] <nickblair> uh oh
[12:24:33 CST(-0600)] <nickblair> src/main/java isn't a source folder anymore in my project; it's treated like a package, so every class is prefixed with src.main.java.org.jasig.portal...
[12:25:15 CST(-0600)] <EricDalquist> well you may have to re-do your m2eclipse project import
[12:25:30 CST(-0600)] <EricDalquist> since SVN would have forced all your .project/.classpath files to get nuked
[12:25:54 CST(-0600)] <nickblair> yep, that's what it looks like
[12:26:17 CST(-0600)] <nickblair> is your video to fix that up somewhere yet ? (smile)
[12:26:28 CST(-0600)] <EricDalquist> nope I have to record it for the THIRD time (tongue)
[12:26:43 CST(-0600)] <EricDalquist> I have two versions in ogv and one in mp4 (that quicktime won't touch) on my computer
[12:26:50 CST(-0600)] <EricDalquist> but all need editing work
[12:27:01 CST(-0600)] <EricDalquist> maybe I should just do that :/
[12:27:17 CST(-0600)] <nickblair> it looks like it can be repaired by clicking on each sub project in the package explorer, under Maven select Update Project Configuration
[12:27:24 CST(-0600)] <EricDalquist> yeah
[12:27:26 CST(-0600)] <EricDalquist> that would do it
[12:33:56 CST(-0600)] <EricDalquist> athena: I'm getting failures in initdb
[12:34:06 CST(-0600)] <EricDalquist> I think you need to start with a completely clean Database
[12:34:19 CST(-0600)] <EricDalquist> since an initdb won't remove existing tables that it isn't overwriting
[12:34:28 CST(-0600)] <EricDalquist> the layout import does stuff like:
[12:34:46 CST(-0600)] <athena> hmm interesting - i haven't actually blown mine away
[12:34:51 CST(-0600)] <EricDalquist> SELECT upc.chan_fname, upls.struct_id, upls.next_struct_id, upls.hidden
[12:34:51 CST(-0600)] <EricDalquist> FROM up_layout_struct upls
[12:34:52 CST(-0600)] <EricDalquist> LEFT JOIN up_channel upc ON upls.chan_id = upc.chan_id
[12:34:52 CST(-0600)] <EricDalquist> WHERE upls.user_id = ? AND upls.layout_id = 1
[12:35:04 CST(-0600)] <EricDalquist> and that works if you haven't done a DB nuke
[12:35:08 CST(-0600)] <EricDalquist> since up_channel will still be there
[12:35:09 CST(-0600)] <athena> oh ick
[12:35:10 CST(-0600)] <athena> yeah
[12:35:13 CST(-0600)] <EricDalquist> and will have generally the right data in it
[12:35:21 CST(-0600)] <athena> i guess i only grepped for upper case UP_CHANNEL
[12:35:33 CST(-0600)] <athena> which script is that in eri?
[12:35:59 CST(-0600)] <EricDalquist> /org/jasig/portal/io/load-limited-layout.crn
[12:36:19 CST(-0600)] <athena> ugh.
[12:36:22 CST(-0600)] <athena> what does that script even do?
[12:36:27 CST(-0600)] <EricDalquist> http://uportal.pastebin.com/qv0T8jRN
[12:36:30 CST(-0600)] <EricDalquist> full stack trace
[12:36:47 CST(-0600)] <EricDalquist> magic during layout imports? (wink)
[12:37:06 CST(-0600)] <EricDalquist> oh it loads a basic skeleton of a layout
[12:37:06 CST(-0600)] <athena> lol
[12:37:09 CST(-0600)] <athena> hmm
[12:37:12 CST(-0600)] <EricDalquist> so that the import can resolve DLM paths
[12:37:26 CST(-0600)] <EricDalquist> since during import/export DLM node refs are turned into XPath expressions to be portable
[12:38:24 CST(-0600)] <athena> ok, let me fix it
[12:39:40 CST(-0600)] <athena> yet another reason why we should drop all this manual sql stuff in import/export (smile)
[12:40:07 CST(-0600)] <EricDalquist> yup
[12:40:33 CST(-0600)] <EricDalquist> some day we'll do it all through JPA (tongue)
[12:42:37 CST(-0600)] <athena> yeah (smile)
[12:42:41 CST(-0600)] <athena> we're getting closer at least
[12:43:19 CST(-0600)] <EricDalquist> yup
[12:43:27 CST(-0600)] <EricDalquist> every step helps
[12:53:39 CST(-0600)] <nickblair> I've got the PortalCookie part working with a test of the lifecycle - should I check in for review?
[12:53:58 CST(-0600)] <nickblair> still have to implement PortletCookie storage
[12:54:03 CST(-0600)] <EricDalquist> sounds good
[12:55:38 CST(-0600)] <athena> ok, fix committed
[12:55:47 CST(-0600)] <athena> mind checking again to make sure it works for you?
[12:56:05 CST(-0600)] <EricDalquist> sure, just a sec
[12:56:40 CST(-0600)] <nickblair> just checked in UP-2861 work, I'll start implementing the PortletCookie portion now
[12:59:02 CST(-0600)] <nickblair> i'm a little unsure on how to establish the relationships between PortletCookieImpl and IPortletEntity with JPA annotations; likewise for PortalCookieImpl and PortletCookieImpl
[12:59:23 CST(-0600)] <nickblair> can someone validate I have those listed correctly?
[12:59:33 CST(-0600)] <EricDalquist> sure … I'll take a look in a few minutes
[13:01:05 CST(-0600)] <EricDalquist> import worked athena
[13:01:10 CST(-0600)] <athena> awesome
[13:01:16 CST(-0600)] <athena> i'm probably going to leave in a bit
[13:01:26 CST(-0600)] <athena> really nasty cold - think i'm going to just call it a day
[13:01:31 CST(-0600)] <EricDalquist> uhg
[13:01:33 CST(-0600)] <EricDalquist> sounds like a good plan
[13:01:33 CST(-0600)] <athena> but want to make sure everything's working first
[13:01:42 CST(-0600)] <athena> yeah, apparently that's what my sister got me for christmas (tongue)
[13:01:47 CST(-0600)] <EricDalquist> lol
[13:04:32 CST(-0600)] <EricDalquist> nickblair: I think in PortletCookieImpl you may need a ManyToOne mapping for portlet entity
[13:04:43 CST(-0600)] <EricDalquist> since one portlet entity can have more than one associated cookie
[13:04:52 CST(-0600)] <nickblair> that makes sense
[13:16:48 CST(-0600)] <nickblair> should the methods in the CookieDao that attach a PortletCookie to a PortalCookie take IPortletEntity or IPortletEntityId as arguments?
[13:17:10 CST(-0600)] <nickblair> the id might be convenient, but would require wiring a PortletEntityDao into the CookieDao implementation
[13:17:29 CST(-0600)] <nickblair> the caller will likely have access to the IPortletEntity already correct?
[13:19:10 CST(-0600)] <EricDalquist> I'd say the PortletCookieDao would need 3 args
[13:19:29 CST(-0600)] <EricDalquist> createPortletCookie(IPortalCookie, IPortletEntity, Cookie)
[13:19:43 CST(-0600)] <EricDalquist> those are the three things required to create a functional PortletCookie object
[13:20:00 CST(-0600)] <nickblair> right
[13:20:24 CST(-0600)] <nickblair> ok, I originally checked it in with the IPortletEntityId as the 2nd arg, but I'll refactor
[13:20:31 CST(-0600)] <nickblair> simpler to implement (smile)
[13:20:37 CST(-0600)] <nickblair> and test
[14:13:55 CST(-0600)] <athena> is it to be expected that the browser user-agent isn't coming through as a request property right now?
[14:14:39 CST(-0600)] <athena> i just see request method, remote address, and expiration cache
[14:16:11 CST(-0600)] <EricDalquist> hrm, yeah we probably missed that
[14:21:20 CST(-0600)] <nickblair> when using JPA/hibernate, is it valid to create a query that looks like: "from PortletCookieImpl portletCookie where portletCookie.portletEntity.id = :portletEntityId"
[14:21:34 CST(-0600)] <EricDalquist> I think so
[14:21:38 CST(-0600)] <nickblair> where portletEntity is a field on the portletCookie object?
[14:22:03 CST(-0600)] <EricDalquist> yup
[14:22:07 CST(-0600)] <EricDalquist> I believe that will work
[14:22:16 CST(-0600)] <nickblair> ok
[14:28:21 CST(-0600)] <nickblair> doesn't look like it
[14:28:27 CST(-0600)] <nickblair> perhaps I've got a typo
[14:30:58 CST(-0600)] <EricDalquist> you can also try "http://portletCookie.portletEntity.id/ = :portletEntity"
[14:32:41 CST(-0600)] <nickblair> http?
[14:33:04 CST(-0600)] <EricDalquist> thats just my ip client having fun with autocomplete
[14:33:46 CST(-0600)] <nickblair> ah
[14:34:11 CST(-0600)] <nickblair> java.lang.ClassCastException: org.jasig.portal.portlet.dao.jpa.PortletEntityIdImpl cannot be cast to java.lang.Long
[14:34:11 CST(-0600)] <nickblair> at org.hibernate.type.LongType.set(LongType.java:42)
[14:34:15 CST(-0600)] <nickblair> I'll pastebin it
[14:34:29 CST(-0600)] <EricDalquist> ah right
[14:34:33 CST(-0600)] <EricDalquist> so use my second example
[14:34:38 CST(-0600)] <nickblair> http://uportal.pastebin.com/sScPAWgq
[14:34:57 CST(-0600)] <EricDalquist> then hibernate should automaticaly figure out the IDs
[14:35:05 CST(-0600)] <EricDalquist> from PortletCookieImpl portletCookie where portletCookie.portletEntity = :portletEntity
[14:37:02 CST(-0600)] <nickblair> amended http://uportal.pastebin.com/3MUtkZSw - still fails, javax.persistence.PersistenceException: org.hibernate.PropertyAccessException: could not get a field value by reflection getter of org.jasig.portal.portlet.dao.jpa.PortletEntityImpl.internalPortletEntityId
[14:37:02 CST(-0600)] <nickblair>
[14:37:33 CST(-0600)] <EricDalquist> IPortletEntityId != IPortletEntity
[14:37:48 CST(-0600)] <EricDalquist> you need to pass in a IPortletEntity as your query parameter
[14:38:15 CST(-0600)] <nickblair> ah
[14:39:05 CST(-0600)] <nickblair> light bulb
[14:39:16 CST(-0600)] <EricDalquist> it seems weird passing in a whole object but hibernate understands it since it is an entity type
[14:39:21 CST(-0600)] <EricDalquist> and does the appropriate stuff
[14:58:56 CST(-0600)] <nickblair> ok, i've got the PortletCookieDao pretty much completed
[14:59:23 CST(-0600)] <nickblair> I've also got a local adjustment to the PortletEntityRegistryImpl that we should discuss first before I commit it?
[14:59:58 CST(-0600)] <EricDalquist> have you tried actually running it?
[15:00:05 CST(-0600)] <nickblair> not yet
[15:00:08 CST(-0600)] <nickblair> i'll do that now
[15:02:35 CST(-0600)] <nickblair> which portletEntityDao should the cookieDao implementation accept, the transient or the jpa?
[15:02:59 CST(-0600)] <EricDalquist> well it is only going to work with the JPA version
[15:03:08 CST(-0600)] <EricDalquist> I'm honestly not sure the best way to deal with all that stuff
[15:03:20 CST(-0600)] <EricDalquist> because the registry is going to have to have enough logic to convert entities back and forth
[15:03:24 CST(-0600)] <EricDalquist> it already does for preferences
[15:03:33 CST(-0600)] <EricDalquist> but now it is going to have to do it for preferences and cookies
[15:03:52 CST(-0600)] <nickblair> hmm
[15:07:07 CST(-0600)] <nickblair> ok, wired in the jpa (Qualifier("persistence"))
[15:07:51 CST(-0600)] <nickblair> for now