You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@clerezza.apache.org by re...@apache.org on 2010/06/18 00:47:01 UTC

svn commit: r955777 - in /incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager.webinterface/src/main: java/org/apache/clerezza/platform/usermanager/webinterface/ resources/org/apache/clerezza/platform/usermanager...

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>