This is my first real commit.. anyone care to comment about anything they don't like? I wasn't especially happy about changing the MethodAnnotationEnhancementWorker interface to take in an IPropertySource but I couldn't think of any other way to get at the new org.apache.tapestry.default-property-persistence-strategy application property inside of PersistAnnotationWorker.
I welcome any comments, I won't take it personally even if it means reverting the change.. I'm doing my best to remain consistent with the quality of this codebase, which is quite impressive I think :) Thanks, Ben ---------- Forwarded message ---------- From: [EMAIL PROTECTED] <[EMAIL PROTECTED]> Date: Jul 25, 2007 4:40 PM Subject: svn commit: r559620 - in /tapestry/tapestry4/trunk: tapestry-annotations/src/descriptor/META-INF/ tapestry-annotations/src/java/org/apache/tapestry/annotations/ tapestry-annotations/src/test/org/apache/tapestry/annotations/ tapestry-framework/src/descr... To: [EMAIL PROTECTED] Author: bdotte Date: Wed Jul 25 14:40:07 2007 New Revision: 559620 URL: http://svn.apache.org/viewvc?view=rev&rev=559620 Log: Resolves TAPESTRY-1448. Add the ability to specify a default @Persist property persistence strategy in the .application file using the new org.apache.tapestry.default-property-persistence-strategy property rather than hardcoding the default on the annotation. <<snipped>> --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
