DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://nagoya.apache.org/bugzilla/show_bug.cgi?id=23901>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE.
http://nagoya.apache.org/bugzilla/show_bug.cgi?id=23901 [PATCH] [woody], adding <wd:on-phase> and moving load() and save() to Form. Summary: [PATCH] [woody], adding <wd:on-phase> and moving load() and save() to Form. Product: Cocoon 2 Version: Current CVS 2.1 Platform: PC OS/Version: Linux Status: NEW Severity: Normal Priority: Other Component: core AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] Hi, This is a patch file for the woody block, which contains : - adding <wd:on-phase> for the form definition, which define a phase listener on the form. - making load() and save() part of the Form class, with minimum changes to the woody interfaces. -------------------- the patch was created using a command : $COCOON_HOME/src/blocks/woody> cvs diff -u > woody-changes.diff ------------------- The changed files : - java/org/apache/cocoon/woody/flow/javascript/woody2.js - java/org/apache/cocoon/woody/formmodel/AbstractWidgetDefinitionBuilder.java - java/org/apache/cocoon/woody/formmodel/Form.java - java/org/apache/cocoon/woody/formmodel/FormDefinition.java - java/org/apache/cocoon/woody/formmodel/FormDefinitionBuilder.java ------------------ TODO : ------ - The patch didn't change any interface methods to maintain a comptability for other blocks (like Apples), I think if this patch is accepted, some interfaces might need changes for making the code better, like FormManager.createForm(Source source), to add the binding source as well, and remove the binding construction from woody2.js - cleaning up the TODO and FIXME added to show what changes were made. - Listeners are limited to only one listener per difinition ( single listener variable), that should be be fixed to allow multiple listsners from the same type? like many action listsner on the same action widget? - To add <wd:on-phase>, I had to change buildAnotherWidgetDefinition() method, This method was throwing exception if there's no builder for the the "wd" element, what do you think? - Make the avalon cycled widget listeners. - Write some documentations about the new stuff [partialy done].
