You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lenya.apache.org by an...@apache.org on 2007/02/19 15:26:18 UTC

svn commit: r509212 - /lenya/trunk/src/modules-core/usecase/java/src/org/apache/lenya/cms/usecase/AbstractUsecase.java

Author: andreas
Date: Mon Feb 19 06:26:17 2007
New Revision: 509212

URL: http://svn.apache.org/viewvc?view=rev&rev=509212
Log:
AbstractUsecase.getSourceURL() doesn't use getParameter() to avoid NPEs

Modified:
    lenya/trunk/src/modules-core/usecase/java/src/org/apache/lenya/cms/usecase/AbstractUsecase.java

Modified: lenya/trunk/src/modules-core/usecase/java/src/org/apache/lenya/cms/usecase/AbstractUsecase.java
URL: http://svn.apache.org/viewvc/lenya/trunk/src/modules-core/usecase/java/src/org/apache/lenya/cms/usecase/AbstractUsecase.java?view=diff&rev=509212&r1=509211&r2=509212
==============================================================================
--- lenya/trunk/src/modules-core/usecase/java/src/org/apache/lenya/cms/usecase/AbstractUsecase.java (original)
+++ lenya/trunk/src/modules-core/usecase/java/src/org/apache/lenya/cms/usecase/AbstractUsecase.java Mon Feb 19 06:26:17 2007
@@ -86,9 +86,11 @@
 
     /**
      * @see org.apache.lenya.cms.usecase.Usecase#getSourceURL()
+     * We don't use getParameterAsString() because this will typically
+     * cause stack overflows or NPEs in connection with initParameters().
      */
     public String getSourceURL() {
-        return getParameterAsString(SOURCE_URL);
+        return (String) this.parameters.get(SOURCE_URL);
     }
 
     /**



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@lenya.apache.org
For additional commands, e-mail: commits-help@lenya.apache.org