Looks like there was a bug in the Paren code introduced with the JPA storage option refactoring. The code was doing String.equals(ParenType) which always returned false. I replaced ParenType with a enum named Type inside the Paren class and updated the logic to deal with that. I also switched the internal type field from a String to a Type. I don't know if this break the JPA storage option since there aren't any unit tests for it.

Drew could you review the JPA storage option and potentially add some basic unit tests?

Thanks,
-Eric

Jen Bourey wrote:
Hi all,

I've noticed that some of the DLM fragments don't seem to be appearing properly in the current uPortal trunk. Although the guest-lo fragment is imported correctly for the guest user, it seems like no fragments appear for logged in users. I'm not sure if this is related to the user being authenticated, or maybe a result of nested elements not being evalutated correctly. From a look at dlm.xml, it looks like the guest-lo fragment is the only one that doesn't have nested <paren> elements.

Any ideas?

- Jen
--

You are currently subscribed to [email protected] as: 
[email protected]
To unsubscribe, change settings or access archives, see 
http://www.ja-sig.org/wiki/display/JSG/uportal-dev

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature

Reply via email to