geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From de...@apache.org
Subject svn commit: r800709 - /geronimo/devtools/eclipse-plugin/trunk/plugins/org.apache.geronimo.st.v21.core/src/main/java/org/apache/geronimo/st/v21/core/jaxb/JAXBModelUtils.java
Date Tue, 04 Aug 2009 09:12:17 GMT
Author: delos
Date: Tue Aug  4 09:12:16 2009
New Revision: 800709

URL: http://svn.apache.org/viewvc?rev=800709&view=rev
Log:
GERONIMODEVTOOLS-573 <sec:security/> was added to plan after add a resource-ref via
GEP UI. Thanks Rodger for this patch!

Modified:
    geronimo/devtools/eclipse-plugin/trunk/plugins/org.apache.geronimo.st.v21.core/src/main/java/org/apache/geronimo/st/v21/core/jaxb/JAXBModelUtils.java

Modified: geronimo/devtools/eclipse-plugin/trunk/plugins/org.apache.geronimo.st.v21.core/src/main/java/org/apache/geronimo/st/v21/core/jaxb/JAXBModelUtils.java
URL: http://svn.apache.org/viewvc/geronimo/devtools/eclipse-plugin/trunk/plugins/org.apache.geronimo.st.v21.core/src/main/java/org/apache/geronimo/st/v21/core/jaxb/JAXBModelUtils.java?rev=800709&r1=800708&r2=800709&view=diff
==============================================================================
--- geronimo/devtools/eclipse-plugin/trunk/plugins/org.apache.geronimo.st.v21.core/src/main/java/org/apache/geronimo/st/v21/core/jaxb/JAXBModelUtils.java
(original)
+++ geronimo/devtools/eclipse-plugin/trunk/plugins/org.apache.geronimo.st.v21.core/src/main/java/org/apache/geronimo/st/v21/core/jaxb/JAXBModelUtils.java
Tue Aug  4 09:12:16 2009
@@ -39,30 +39,21 @@
         Object plan = element.getValue();
         if (WebApp.class.isInstance (plan)) {
             if (((WebApp)plan).getSecurity() == null) {
-                setSecurity (element, new org.apache.geronimo.jee.security.ObjectFactory().createSecurity());
+                return null;
             }
             Security security = (Security)((WebApp)plan).getSecurity().getValue();
-            if (security.getRoleMappings() == null) {
-                security.setRoleMappings (new org.apache.geronimo.jee.security.ObjectFactory().createRoleMappings());
-            }
             return security;
         } else if (Application.class.isInstance (plan)) {
             if (((Application)plan).getSecurity() == null) {
-                setSecurity (element, new org.apache.geronimo.jee.security.ObjectFactory().createSecurity());
+            	return null;
             }
             Security security = (Security)((Application)plan).getSecurity().getValue();
-            if (security.getRoleMappings() == null) {
-                security.setRoleMappings (new org.apache.geronimo.jee.security.ObjectFactory().createRoleMappings());
-            }
             return security;
         } else if (OpenejbJar.class.isInstance (plan)) {
             if (((OpenejbJar)plan).getSecurity() == null) {
-                setSecurity (element, new org.apache.geronimo.jee.security.ObjectFactory().createSecurity());
+            	return null;
             }
             Security security = (Security)((OpenejbJar)plan).getSecurity().getValue();
-            if (security.getRoleMappings() == null) {
-                security.setRoleMappings (new org.apache.geronimo.jee.security.ObjectFactory().createRoleMappings());
-            }
             return security;
         }
         return null;



Mime
View raw message