You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cocoon.apache.org by bu...@apache.org on 2001/07/02 10:56:34 UTC
[Bug 2413] New: - AbstractAction implements configure method & Configuration resources are reloadable
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: cocoon-dev@xml.apache.org |
+ | Reported By: haul@informatik.tu-darmstadt.de |
+ | 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: cocoon-dev-unsubscribe@xml.apache.org
For additional commands, email: cocoon-dev-help@xml.apache.org