sylvain 01/11/28 01:21:02
Modified: src/org/apache/cocoon/acting Tag: cocoon_20_branch
AbstractAction.java AbstractDatabaseAction.java
LocaleAction.java SendmailAction.java
Log:
Removed Configurable and Disposable from AbstractAction since it doesn't do anything
with these contracts.
Also checked implements clause in derived classes where the contract is actually
used.
Revision Changes Path
No revision
No revision
1.2.2.8 +2 -23 xml-cocoon2/src/org/apache/cocoon/acting/AbstractAction.java
Index: AbstractAction.java
===================================================================
RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/acting/AbstractAction.java,v
retrieving revision 1.2.2.7
retrieving revision 1.2.2.8
diff -u -r1.2.2.7 -r1.2.2.8
--- AbstractAction.java 2001/10/11 08:52:03 1.2.2.7
+++ AbstractAction.java 2001/11/28 09:21:01 1.2.2.8
@@ -7,10 +7,6 @@
*****************************************************************************/
package org.apache.cocoon.acting;
-import org.apache.avalon.framework.activity.Disposable;
-import org.apache.avalon.framework.configuration.Configurable;
-import org.apache.avalon.framework.configuration.Configuration;
-import org.apache.avalon.framework.configuration.ConfigurationException;
import org.apache.avalon.framework.logger.AbstractLoggable;
import java.util.Collections;
@@ -23,31 +19,14 @@
*
* @author <a href="mailto:[EMAIL PROTECTED]";>Giacomo Pati</a>
* @author <a href="mailto:[EMAIL PROTECTED]";>Christian Haul</a>
- * @version CVS $Revision: 1.2.2.7 $ $Date: 2001/10/11 08:52:03 $
+ * @version CVS $Revision: 1.2.2.8 $ $Date: 2001/11/28 09:21:01 $
*/
public abstract class AbstractAction extends AbstractLoggable
-implements Action, Configurable, Disposable {
+implements Action {
/**
* Empty unmodifiable map. Replace with Collections.EMPTY_MAP when
* pre-jdk1.3 support is dropped.
*/
protected static final Map EMPTY_MAP = Collections.unmodifiableMap(new
TreeMap());
-
- /**
- * Configures the Action.
- *
- */
- public void configure(Configuration conf) throws ConfigurationException {
- // Purposely empty so that we don't need to implement it in every
- // class.
- }
-
- /**
- * dispose
- */
- public void dispose() {
- // Purposely empty so that we don't need to implement it in every
- // class.
- }
}
1.7.2.9 +1 -2
xml-cocoon2/src/org/apache/cocoon/acting/AbstractDatabaseAction.java
Index: AbstractDatabaseAction.java
===================================================================
RCS file:
/home/cvs/xml-cocoon2/src/org/apache/cocoon/acting/AbstractDatabaseAction.java,v
retrieving revision 1.7.2.8
retrieving revision 1.7.2.9
diff -u -r1.7.2.8 -r1.7.2.9
--- AbstractDatabaseAction.java 2001/10/11 08:52:03 1.7.2.8
+++ AbstractDatabaseAction.java 2001/11/28 09:21:01 1.7.2.9
@@ -147,7 +147,7 @@
*
* @author <a href="mailto:[EMAIL PROTECTED]";>Berin Loritsch</a>
* @author <a href="mailto:[EMAIL PROTECTED]";>Donald Ball</a>
- * @version CVS $Revision: 1.7.2.8 $ $Date: 2001/10/11 08:52:03 $
+ * @version CVS $Revision: 1.7.2.9 $ $Date: 2001/11/28 09:21:01 $
*/
public abstract class AbstractDatabaseAction extends
AbstractComplementaryConfigurableAction implements Configurable, Disposable {
protected Map files = new HashMap();
@@ -638,7 +638,6 @@
*/
public void dispose() {
this.manager.release(dbselector);
- super.dispose();
}
/**
1.1.2.7 +2 -1 xml-cocoon2/src/org/apache/cocoon/acting/LocaleAction.java
Index: LocaleAction.java
===================================================================
RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/acting/LocaleAction.java,v
retrieving revision 1.1.2.6
retrieving revision 1.1.2.7
diff -u -r1.1.2.6 -r1.1.2.7
--- LocaleAction.java 2001/10/11 08:52:03 1.1.2.6
+++ LocaleAction.java 2001/11/28 09:21:01 1.1.2.7
@@ -7,6 +7,7 @@
*****************************************************************************/
package org.apache.cocoon.acting;
+import org.apache.avalon.framework.configuration.Configurable;
import org.apache.avalon.framework.configuration.Configuration;
import org.apache.avalon.framework.configuration.ConfigurationException;
import org.apache.avalon.framework.parameters.Parameters;
@@ -102,7 +103,7 @@
* @author <a href="mailto:[EMAIL PROTECTED]";>Konstantin Piroumian</a>
* @author <a href="mailto:[EMAIL PROTECTED]";>Lassi Immonen</a>
*/
-public class LocaleAction extends ComposerAction implements ThreadSafe {
+public class LocaleAction extends ComposerAction implements ThreadSafe,
Configurable {
/**
* Constant representing the language parameter
1.3.2.9 +2 -1 xml-cocoon2/src/org/apache/cocoon/acting/SendmailAction.java
Index: SendmailAction.java
===================================================================
RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/acting/SendmailAction.java,v
retrieving revision 1.3.2.8
retrieving revision 1.3.2.9
diff -u -r1.3.2.8 -r1.3.2.9
--- SendmailAction.java 2001/10/12 08:11:54 1.3.2.8
+++ SendmailAction.java 2001/11/28 09:21:01 1.3.2.9
@@ -7,6 +7,7 @@
*****************************************************************************/
package org.apache.cocoon.acting;
+import org.apache.avalon.framework.configuration.Configurable;
import org.apache.avalon.framework.configuration.Configuration;
import org.apache.avalon.framework.configuration.ConfigurationException;
import org.apache.avalon.framework.parameters.Parameters;
@@ -48,7 +49,7 @@
* @author <a href="mailto:[EMAIL PROTECTED]";>Donald Ball</a>
* @version CVS $Revision
*/
-public class SendmailAction extends AbstractAction implements ThreadSafe {
+public class SendmailAction extends AbstractAction implements ThreadSafe,
Configurable {
Properties default_properties = null;
----------------------------------------------------------------------
In case of troubles, e-mail: [EMAIL PROTECTED]
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
