You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by jo...@apache.org on 2005/07/07 19:33:22 UTC

svn commit: r209619 - /cocoon/trunk/src/java/org/apache/cocoon/acting/SetterAction.java

Author: joerg
Date: Thu Jul  7 10:33:21 2005
New Revision: 209619

URL: http://svn.apache.org/viewcvs?rev=209619&view=rev
Log:
fixing bug 35651

Modified:
    cocoon/trunk/src/java/org/apache/cocoon/acting/SetterAction.java

Modified: cocoon/trunk/src/java/org/apache/cocoon/acting/SetterAction.java
URL: http://svn.apache.org/viewcvs/cocoon/trunk/src/java/org/apache/cocoon/acting/SetterAction.java?rev=209619&r1=209618&r2=209619&view=diff
==============================================================================
--- cocoon/trunk/src/java/org/apache/cocoon/acting/SetterAction.java (original)
+++ cocoon/trunk/src/java/org/apache/cocoon/acting/SetterAction.java Thu Jul  7 10:33:21 2005
@@ -31,7 +31,7 @@
  * whereas the parameter name is the key and the value of the parameter
  * will be set as a string value for this key.
  *
- * @version SVN $Id:$
+ * @version SVN $Id$
  */
 public class SetterAction
     extends AbstractAction
@@ -62,7 +62,7 @@
             } else if ( MODEDEF_SESSION_ATTR.equals(modeDef) ) {
                 this.mode = MODE_SESSION_ATTR;
             } else {
-                throw new ParameterException("Unknown mode: " + mode);
+                throw new ParameterException("Unknown mode: " + this.mode);
             }
         }
     }
@@ -83,7 +83,7 @@
                 objectModel.put(name, parameters.getParameter(name));                
             } else if ( this.mode == MODE_REQUEST_ATTR ) {
                 ObjectModelHelper.getRequest(objectModel).setAttribute(name, parameters.getParameter(name));                
-            } else if ( this.mode == MODE_REQUEST_ATTR ) {
+            } else if ( this.mode == MODE_SESSION_ATTR ) {
                 ObjectModelHelper.getRequest(objectModel).getSession().setAttribute(name, parameters.getParameter(name));                
             }
         }