uPortal IRC Logs-2011-02-10

Owned by Apereo Infrastructure
Last updated: Feb 10, 2011Version comment
4 min read

[12:10:25 CST(-0600)] <EricDalquist> athena: you're working on the PortletDefinitionImpl right now aren't you?
[12:10:35 CST(-0600)] <athena> yeah, although my work is mostly finished
[12:10:40 CST(-0600)] <EricDalquist> ok
[12:10:45 CST(-0600)] <athena> you looking for me to check in changes before you do something?
[12:10:56 CST(-0600)] <EricDalquist> so I realized a few days ago that we're 3 versions behind with hibernate (tongue)
[12:11:00 CST(-0600)] <EricDalquist> and could be using JPA2
[12:11:08 CST(-0600)] <athena> yeah you know i was wondering that
[12:11:24 CST(-0600)] <EricDalquist> so I started looking at it and it should be pretty easy to switch
[12:11:39 CST(-0600)] <EricDalquist> one thing I'm wondering about is if we want to use the type-safe criteria builder stuff
[12:11:53 CST(-0600)] <EricDalquist> it is neat in that your queries are never based on strings, everything is a strong reference
[12:12:11 CST(-0600)] <athena> yeah that'd be nice (smile)
[12:12:14 CST(-0600)] <EricDalquist> so if you rename a field in a JPA managed object you have to fix the criteria builder too or your project wont compile
[12:12:21 CST(-0600)] <athena> sounds terrific to me!
[12:12:30 CST(-0600)] <EricDalquist> the down side is the only reasonable way to do this is with a maven plugin that generates the JPA2 metamodel classes
[12:12:35 CST(-0600)] <EricDalquist> I don't think that will be a problem
[12:12:37 CST(-0600)] <athena> ah gotcha
[12:12:41 CST(-0600)] <EricDalquist> but developers will just have to be aware of it
[12:12:48 CST(-0600)] <EricDalquist> the m2eclipse plugin appears to handle it correctly
[12:13:28 CST(-0600)] <EricDalquist> it adds "target/metamodel" as a source folder in eclipse
[12:13:46 CST(-0600)] <EricDalquist> which is generated by a maven plugin from all of the objects with an @Entity annotation
[12:13:54 CST(-0600)] <athena> gotcha
[12:14:33 CST(-0600)] <EricDalquist> so I'll go ahead with it but it should be easy to switch backed to JQL queries if it turns out to be too much of a pain
[12:14:41 CST(-0600)] <athena> sounds good
[12:15:00 CST(-0600)] <EricDalquist> the other annoying thing is the criteria builder is verbose
[12:15:02 CST(-0600)] <athena> i'm going to be kind of heads-down-panic-mode on a couple demos until mid-day tomorrow
[12:15:08 CST(-0600)] <EricDalquist> but we don't have to use it for a ton
[12:15:08 CST(-0600)] <athena> so i may just avoid pulling down an update until then
[12:15:09 CST(-0600)] <EricDalquist> thats fine
[12:15:11 CST(-0600)] <athena> ah, that does suck
[12:15:30 CST(-0600)] <EricDalquist> http://uportal.pastebin.com/4UVgS3UL
[12:15:48 CST(-0600)] <athena> gotcha
[12:15:50 CST(-0600)] <EricDalquist> that is building the same thing that "from PortletEntityImpl portEnt where portEnt.channelSubscribeId = :channelSubscribeId and portEnt.userId = :userId" does
[12:15:54 CST(-0600)] <athena> yeah
[12:16:18 CST(-0600)] <EricDalquist> so I'm going to keep working on this but I'll hold off on committing until you get yourchanges in
[12:16:28 CST(-0600)] <EricDalquist> it should be easier for me to do the JPA2 side of the merge
[12:16:32 CST(-0600)] <EricDalquist> than the other way around
[12:17:21 CST(-0600)] <athena> ok, sounds good
[12:17:35 CST(-0600)] <athena> i'll make sure to at least get my changes in by the end of tomorrow - that work ok for you?
[12:17:49 CST(-0600)] <EricDalquist> thats great
[12:17:50 CST(-0600)] <EricDalquist> thanks
[12:18:07 CST(-0600)] <athena> yep!
[13:54:56 CST(-0600)] <EricDalquist> athena: is the IPermissionOwnerDao in use yet?
[13:55:28 CST(-0600)] <athena> yes
[13:55:58 CST(-0600)] <EricDalquist> I'm not seeing any code that actually populates permissions
[13:56:01 CST(-0600)] <EricDalquist> or is that all in CRN?
[13:56:46 CST(-0600)] <EricDalquist> ah yeah … ok found it all in CRN
[17:10:12 CST(-0600)] <EricDalquist> athena: NewsReaderPortlet in trunk appears to be hard-coded to use mysql
[17:10:27 CST(-0600)] <athena> yeah, i've noticed that
[17:10:30 CST(-0600)] <athena> needs to be fixed
[17:10:33 CST(-0600)] <athena> as do a bunch of other things
[17:10:38 CST(-0600)] <EricDalquist> (smile)
[17:10:48 CST(-0600)] <EricDalquist> just checking if you saw the same thing (smile)
[17:10:51 CST(-0600)] <athena> yeah
[17:10:58 CST(-0600)] <athena> i'll make sure it gets fixed
[17:37:26 CST(-0600)] <athena> hmm
[17:37:45 CST(-0600)] <athena> it looks like portlet parameters aren't coming through to the structure transformation?
[17:38:04 CST(-0600)] <EricDalquist> the stuff that used to be channel parameters?
[17:38:08 CST(-0600)] <athena> yeah
[17:38:15 CST(-0600)] <athena> now portlet parameters, i guess
[17:38:20 CST(-0600)] <EricDalquist> yup
[17:38:28 CST(-0600)] <athena> is that intentional?
[17:38:28 CST(-0600)] <EricDalquist> hrm, I can never remember how that stuff gets added into the layout dom
[17:38:34 CST(-0600)] <EricDalquist> probably not
[17:39:02 CST(-0600)] <athena> we do probably want them to go through, since our biggest use case for them is changing the display of a portlet
[17:39:08 CST(-0600)] <EricDalquist> right
[17:39:13 CST(-0600)] <EricDalquist> I think that is now there primary use
[17:39:18 CST(-0600)] <EricDalquist> their*
[17:40:12 CST(-0600)] <athena> ok, so just something that needs to get fixed?
[17:40:31 CST(-0600)] <EricDalquist> yeah
[17:40:34 CST(-0600)] <athena> ok
[17:40:55 CST(-0600)] <EricDalquist> I can't remember where in the chain the <channel> element gets populated in the layout from the definition object
[17:41:03 CST(-0600)] <athena> gotcha
[18:15:17 CST(-0600)] <EricDalquist> checking out another instance of uportal trunk, time to see if my local mods are causing the problems or if they already existed (tongue)
[18:15:36 CST(-0600)] <EricDalquist> I should just resign myself to having two instances of trunk all the time