Author: reto
Date: Thu Jun 17 22:47:00 2010
New Revision: 955777

URL: http://svn.apache.org/viewvc?rev=955777&view=rev
Log:
encoding user-name before creating query parameters

Modified:
    
incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager.webinterface/src/main/java/org/apache/clerezza/platform/usermanager/webinterface/UserManagerWeb.java
    
incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager.webinterface/src/main/resources/org/apache/clerezza/platform/usermanager/webinterface/user-overview-template.xhtml

Modified: 
incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager.webinterface/src/main/java/org/apache/clerezza/platform/usermanager/webinterface/UserManagerWeb.java
URL: 
http://svn.apache.org/viewvc/incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager.webinterface/src/main/java/org/apache/clerezza/platform/usermanager/webinterface/UserManagerWeb.java?rev=955777&r1=955776&r2=955777&view=diff
==============================================================================
--- 
incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager.webinterface/src/main/java/org/apache/clerezza/platform/usermanager/webinterface/UserManagerWeb.java
 (original)
+++ 
incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager.webinterface/src/main/java/org/apache/clerezza/platform/usermanager/webinterface/UserManagerWeb.java
 Thu Jun 17 22:47:00 2010
@@ -622,7 +622,7 @@ public class UserManagerWeb implements G
                        return result;
                }
                throw new 
WebApplicationException(Response.status(Status.NOT_FOUND)
-                               .entity("User " + userName + "does not exist in 
our database")
+                               .entity("User " + userName + " does not exist 
in our database")
                                .build());
        }
 

Modified: 
incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager.webinterface/src/main/resources/org/apache/clerezza/platform/usermanager/webinterface/user-overview-template.xhtml
URL: 
http://svn.apache.org/viewvc/incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager.webinterface/src/main/resources/org/apache/clerezza/platform/usermanager/webinterface/user-overview-template.xhtml?rev=955777&r1=955776&r2=955777&view=diff
==============================================================================
--- 
incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager.webinterface/src/main/resources/org/apache/clerezza/platform/usermanager/webinterface/user-overview-template.xhtml
 (original)
+++ 
incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager.webinterface/src/main/resources/org/apache/clerezza/platform/usermanager/webinterface/user-overview-template.xhtml
 Thu Jun 17 22:47:00 2010
@@ -44,17 +44,18 @@ resultDocModifier.addNodes2Elem("tx-cont
                                        </tr>
                                </thead>
                                <tbody>
-                    {for (user <- (res!!)) yield {
+                    {import java.net._
+                                       for (user <- (res!!)) yield {
                                        <tr id={user/platform("userName")*}>
                         <td>
                             <input type="checkbox" 
value={user/platform("userName")*} />
                         </td>
-                        <td><a href={"update-user?userName=" + 
(user/platform("userName")*)}>{user/platform("userName")*}</a></td>
+                        <td><a href={"update-user?userName=" + 
URLEncoder.encode(user/platform("userName")*, 
"utf-8")}>{user/platform("userName")*}</a></td>
                         <td>{user/foaf("mbox")*}</td>
                         <td>{user/osgi("agent_path_prefix")*}</td>
                         <td>{user/permission("passwordSha1")*}</td>
                         <td>{for (title <- (user/sioc("has_function"))) yield 
{  title/dc("title")* }}</td>
-                        <td><a href={"manage-user-permissions?userName=" + 
(user/platform("userName")*)}>show permissions</a></td>
+                        <td><a href={"manage-user-permissions?userName=" + 
URLEncoder.encode(user/platform("userName")*, "utf-8")}>show 
permissions</a></td>
                     </tr>
                     }}
                                </tbody>


Reply via email to