http://nagoya.apache.org/bugzilla/show_bug.cgi?id=2413 *** shadow/2413 Mon Jul 2 01:56:34 2001 --- shadow/2413.tmp.20681 Mon Jul 2 01:56:34 2001 *************** *** 0 **** --- 1,102 ---- + +============================================================================+ + | AbstractAction implements configure method & Configuration resources are r | + +----------------------------------------------------------------------------+ + | Bug #: 2413 Product: Cocoon 2 | + | Status: NEW Version: 2.1alpha CVS | + | Resolution: Platform: PC | + | Severity: Enhancement OS/Version: Linux | + | Priority: Other Component: general components | + +----------------------------------------------------------------------------+ + | Assigned To: [EMAIL PROTECTED] | + | Reported By: [EMAIL PROTECTED] | + | CC list: Cc: | + +----------------------------------------------------------------------------+ + | URL: | + +============================================================================+ + | DESCRIPTION | + Prerequisite + + The patches are made using "diff -uNr" against a clean snapshot of today's + CVS (monday, 2001-07-02) + + Some patches superceede patches that I have submitted on thursday 21st + but that seem to have gone unoticed. Please, do not apply them + anymore. + + Some patches require changes to the same files. I have separated those + to files named "shared_PATCH1__PATCH2_patch". I couldn't figure out + how to make them independent enough so that they could be applied in + any order. Please edit manually or check that the shared patch has + already been applied. It should be straight forward. + + The actual patch can be found at + http://www.dvs1.informatik.tu-darmstadt.de/~haul/misc/c2/actions_patch + + +http://www.dvs1.informatik.tu-darmstadt.de/~haul/misc/c2/shared_actions__session_state_patch + + + + patches + org.apache.cocoon.acting.AbstractAction + org.apache.cocoon.acting.AbstractComplimentaryConfigurableAction + org.apache.cocoon.acting.DatabaseAddAction + org.apache.cocoon.acting.DatabaseAuthenticatorAction + org.apache.cocoon.acting.DatabaseDeleteAction + org.apache.cocoon.acting.DatabaseUpdateAction + org.apache.cocoon.acting.FormValidatorAction + + reason + Feature enhancement + + requires + patch to org.apache.cocoon.Constants shared with session state + patch + + summary + 1) AbstractAction implements configure method to parse global + configuration parameters + 2) Configuration resources are reloadable + + description + 1) Before AbstractAction has an empty configure method and + sofar no supplied action allows global(*) configuration + options eg. for the descriptor resource thus duplicating + configuration options throughout the sitemap. The provided + configure method parses the configuration nodeset and puts + the names and values of all top level <map:parameter/> + elements into a HashMap "settings" for easy access by the + derived action. + + (*) global to this action instance as opposed to local to a + specific invocation of the action. + + If a derived action needs a more complex global + configuration, it can override the configure method or + extend it. + + Actions know to take parameters have been modified to use + this method. + + 2) So far configuration resources e.g. for the database + mapping or form validation are only read once, creating the + necessity to stop and restart the servlet container to make + changes. + org.apache.cocoon,acting.AbstractComplimentaryConfigurableAction + has been extended to check the modification date of a + resource and reload it if necessary like + + org.apache.cocoon.components.language.markup.xsp.XSPFormValidatorHelper. + This can be disabled by a parameter. + + It would be nice to have this configurable from + cocoon.xconf. If someone hints me at where to put the code + for this, I will provide a patch for that. + + To make the reloading configurable per instance, the + signature of the getConfiguration mathod needed to be + extended by a reloadable indicator. All know subclasses are + modified to take advantage of configurable + reloading. Anyway, for backwards compatibility a method + with the original signature is provided that uses the + system default setting or a global parameter to control the + realoding. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, email: [EMAIL PROTECTED]
