geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From vamsic...@apache.org
Subject svn commit: r474682 - /geronimo/server/branches/1.1/applications/console-standard/src/java/org/apache/geronimo/console/securitymanager/realm/SecurityRealmPortlet.java
Date Tue, 14 Nov 2006 07:25:32 GMT
Author: vamsic007
Date: Mon Nov 13 23:25:31 2006
New Revision: 474682

URL: http://svn.apache.org/viewvc?view=rev&rev=474682
Log:
GERONIMO-2284 Console DB/JMS and Security Realm naming inconsistent.
* Changes groupId from "console" to "console.realm"
* replaces any slashes in the artifactId with "%2F"

Modified:
    geronimo/server/branches/1.1/applications/console-standard/src/java/org/apache/geronimo/console/securitymanager/realm/SecurityRealmPortlet.java

Modified: geronimo/server/branches/1.1/applications/console-standard/src/java/org/apache/geronimo/console/securitymanager/realm/SecurityRealmPortlet.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/1.1/applications/console-standard/src/java/org/apache/geronimo/console/securitymanager/realm/SecurityRealmPortlet.java?view=diff&rev=474682&r1=474681&r2=474682
==============================================================================
--- geronimo/server/branches/1.1/applications/console-standard/src/java/org/apache/geronimo/console/securitymanager/realm/SecurityRealmPortlet.java
(original)
+++ geronimo/server/branches/1.1/applications/console-standard/src/java/org/apache/geronimo/console/securitymanager/realm/SecurityRealmPortlet.java
Mon Nov 13 23:25:31 2006
@@ -337,8 +337,14 @@
         ModuleType root = doc.addNewModule();
         EnvironmentType environment = root.addNewEnvironment();
         ArtifactType configId = environment.addNewModuleId();
-        configId.setGroupId("console");
-        configId.setArtifactId("realm-" + data.getName());
+        configId.setGroupId("console.realm");
+        String artifactId = data.getName();
+        if(artifactId.indexOf('/') != -1) {
+            // slash in artifact-id results in invalid configuration-id and leads to deployment
errors.
+            // Note: 0x002F = '/'
+            artifactId = artifactId.replaceAll("/", "%2F");
+        }
+        configId.setArtifactId(artifactId);
         configId.setVersion("1.0");
         configId.setType("car");
 



Mime
View raw message