You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pluto-scm@portals.apache.org by dd...@apache.org on 2005/09/09 06:06:02 UTC

svn commit: r279697 - /portals/pluto/branches/pluto-1.1/pluto-container/src/main/java/org/apache/pluto/core/impl/RenderResponseImpl.java

Author: ddewolf
Date: Thu Sep  8 21:05:58 2005
New Revision: 279697

URL: http://svn.apache.org/viewcvs?rev=279697&view=rev
Log:
PLUTO-150: Fixing namespace to be javascript usable.

Modified:
    portals/pluto/branches/pluto-1.1/pluto-container/src/main/java/org/apache/pluto/core/impl/RenderResponseImpl.java

Modified: portals/pluto/branches/pluto-1.1/pluto-container/src/main/java/org/apache/pluto/core/impl/RenderResponseImpl.java
URL: http://svn.apache.org/viewcvs/portals/pluto/branches/pluto-1.1/pluto-container/src/main/java/org/apache/pluto/core/impl/RenderResponseImpl.java?rev=279697&r1=279696&r2=279697&view=diff
==============================================================================
--- portals/pluto/branches/pluto-1.1/pluto-container/src/main/java/org/apache/pluto/core/impl/RenderResponseImpl.java (original)
+++ portals/pluto/branches/pluto-1.1/pluto-container/src/main/java/org/apache/pluto/core/impl/RenderResponseImpl.java Thu Sep  8 21:05:58 2005
@@ -67,17 +67,28 @@
     }
 
     public PortletURL createRenderURL() {
-        PortletURL url = createURL(false);
-        return url;
+        return createURL(false);
     }
 
     public PortletURL createActionURL() {
-        PortletURL url = createURL(true);
-        return url;
+        return createURL(true);
     }
 
     public String getNamespace() {
-        return mapper.encode(getInternalPortletWindow().getId(), "");
+         String namespace =
+            mapper.encode(getInternalPortletWindow().getId(), "");
+
+         StringBuffer validNamespace = new StringBuffer();
+         for (int i = 0; i < namespace.length(); i++) {
+         	char ch = namespace.charAt(i);
+         	if (Character.isJavaIdentifierPart(ch)) {
+         		validNamespace.append(ch);
+         	} else {
+         		validNamespace.append('_');
+         	}
+         }
+
+         return validNamespace.toString();
     }
 
     public void setTitle(String title) {
@@ -155,7 +166,6 @@
                                                         java.lang.IllegalStateException {
         if (currentContentType == null) {
             String msg = EXCEPTIONS.getString("error.contenttype.null");
-
             //throw new java.lang.IllegalStateException(msg);
         }
         return getOutputStream();