All,

I have been doing some tests and I think we should
upgrade Torque (to 3.2-rc1) and OJB (to 1.0.3) for the
following reasons:

1. Torque:  Now supports derby for SQL generation. 
Also, issues with cascading delete and HSQL have been
fixed.  This will address
http://issues.apache.org/jira/browse/JS2-227.

2. OJB: Quite a few bugs fixed. Also, I noticed a
couple of weird behaviors with the current version:
As I mentioned in a previous email:

- AbstractRegistryTest: the assertion on line 198
(assertNotNull("Web app did NOT persist its
description", webApp.getDescription(Locale.FRENCH));)
works no matter what locale is passed. I tried
Locale.getDefault(), as well as Locale.FRENCH and it
does not seem to matter.  With 1.0.3, the assertion
only works for Locale.FRENCH.  Which the right correct
behavior.
- TestUserInfoManager: When testing with
ExtendedPortletMetadata, I do not quite understand how

it ever worked without calling
updatePortletApplication on the registry.  With OJB
1.0.3, you need to update the portlet application with
the new loaded metadata for the test to pass.  Just
seem to make sense 
to me.

I have applied the changes locally, I re-ran all the
tests and everything is working, including the
deployed application.

I am still seeing a problem that Santiago found, where
for the jetspeed user only (all other users work
fine), primary navigation seems broken. It looks like
it may be related the role-fallback profiler rule and
occurs before or after the upgrade, therefore seems
unrelated.

Let me know if you want me to commit the upgrade.  I
have create a JIRA issue to track the changes
(http://issues.apache.org/jira/browse/JS2-318).

Regards,

David Le Strat.

________________________
David Le Strat
Blogging @ http://dlsthoughts.blogspot.com


                
____________________________________________________
Start your day with Yahoo! - make it your home page 
http://www.yahoo.com/r/hs 
 

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to