Re: svn commit: r499201 - in /geronimo/server/trunk/modules/geronimo-kernel/src: main/java/org/apache/geronimo/gbean/ main/java/org/apache/geronimo/gbean/runtime/ test/java/org/apache/geronimo/gbean/

Tue, 23 Jan 2007 19:45:22 -0800

I think this has broken some stuff. This piece of code in PersistenceUnitReference is no longer working:
entityManagerFactory = kernel.invoke(target, "getEntityManagerFactory"); 


I'll try to code around the problem for this instance, but you may  
want to roll back this commit.


-dain

On Jan 23, 2007, at 4:55 PM, [EMAIL PROTECTED] wrote:


Author: hogstrom
Date: Tue Jan 23 16:55:35 2007
New Revision: 499201

URL: http://svn.apache.org/viewvc?view=rev&rev=499201
Log:
GERONIMO-2607 : Deprecated old addOperation methods, added private
addOperation methods
                updated GBeanInfoTest
                Modified GBeanOperation, this class is not serialized

Modified:

    geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/ 
apache/geronimo/gbean/GBeanInfoBuilder.java
    geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/ 
apache/geronimo/gbean/GOperationInfo.java
    geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/ 
apache/geronimo/gbean/runtime/GBeanOperation.java
    geronimo/server/trunk/modules/geronimo-kernel/src/test/java/org/ 
apache/geronimo/gbean/GBeanInfoTest.java



Modified: geronimo/server/trunk/modules/geronimo-kernel/src/main/ 
java/org/apache/geronimo/gbean/GBeanInfoBuilder.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/modules/ 
geronimo-kernel/src/main/java/org/apache/geronimo/gbean/ 
GBeanInfoBuilder.java?view=diff&rev=499201&r1=499200&r2=499201
====================================================================== 
========
--- geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/ 
apache/geronimo/gbean/GBeanInfoBuilder.java (original)
+++ geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/ 
apache/geronimo/gbean/GBeanInfoBuilder.java Tue Jan 23 16:55:35 2007

@@ -403,23 +403,23 @@

     /**
      * @deprecated
-     */
+     */
     public void addOperation(String name) {
-        addOperation(new GOperationInfo(name, NO_ARGS, ""));
+        //addOperation(new GOperationInfo(name, NO_ARGS, ""));
     }

     /**
      * @deprecated
      */
     public void addOperation(String name, Class[] paramTypes) {
-        addOperation(new GOperationInfo(name, paramTypes, ""));
+        //addOperation(new GOperationInfo(name, paramTypes, ""));
     }

-    public void addOperation(String name, String returnType) {
+    private void addOperation(String name, String returnType) {
         addOperation(new GOperationInfo(name, NO_ARGS, returnType));
     }


-    public void addOperation(String name, Class[] paramTypes,  
String returnType) {
+    private void addOperation(String name, Class[] paramTypes,  
String returnType) {
         addOperation(new GOperationInfo(name, paramTypes,  
returnType));

     }



Modified: geronimo/server/trunk/modules/geronimo-kernel/src/main/ 
java/org/apache/geronimo/gbean/GOperationInfo.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/modules/ 
geronimo-kernel/src/main/java/org/apache/geronimo/gbean/ 
GOperationInfo.java?view=diff&rev=499201&r1=499200&r2=499201
====================================================================== 
========
--- geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/ 
apache/geronimo/gbean/GOperationInfo.java (original)
+++ geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/ 
apache/geronimo/gbean/GOperationInfo.java Tue Jan 23 16:55:35 2007

@@ -38,7 +38,7 @@
     /**
      * The return type of this method.
      */
-    private final String type;
+    private final String returnType;

     /**
      * Parameters of this method.
@@ -54,9 +54,9 @@
         this(name, name, Collections.EMPTY_LIST, type);
     }


-    public GOperationInfo(String name, Class[] paramTypes, String  
type) {
+    public GOperationInfo(String name, Class[] paramTypes, String  
returnType) {

         this.name = this.methodName = name;
-        this.type = type;
+        this.returnType = returnType;
         String[] args = new String[paramTypes.length];
         for (int i = 0; i < args.length; i++) {
             args[i] = paramTypes[i].getName();
@@ -64,17 +64,17 @@

         this.parameters = Collections.unmodifiableList 
(Arrays.asList(args));

     }


-    public GOperationInfo(String name, String[] paramTypes, String  
type) {

-        this(name, name, Arrays.asList(paramTypes), type);

+    public GOperationInfo(String name, String[] paramTypes, String  
returnType) {

+        this(name, name, Arrays.asList(paramTypes), returnType);
     }


-    public GOperationInfo(String name, List parameters, String  
type) {

-        this(name, name, parameters, type);

+    public GOperationInfo(String name, List parameters, String  
returnType) {

+        this(name, name, parameters, returnType);
     }


-    public GOperationInfo(String name, String methodName, List  
parameters, String type) {
+    public GOperationInfo(String name, String methodName, List  
parameters, String returnType) {

         this.name = name;
-        this.type = type;
+        this.returnType = returnType;
         this.methodName = methodName;

         this.parameters = Collections.unmodifiableList(new  
ArrayList(parameters));

     }
@@ -84,7 +84,7 @@
     }

     public String getReturnType() {
-        return type;
+        return returnType;
     }

     public String getMethodName() {
@@ -96,6 +96,6 @@
     }

     public String toString() {

-        return "[GOperationInfo: name=" + name + " parameters=" +  
parameters + " type =" + type + "]";
+        return "[GOperationInfo: name=" + name + " parameters=" +  
parameters + " returnType =" + returnType + "]";

     }
 }


Modified: geronimo/server/trunk/modules/geronimo-kernel/src/main/ 
java/org/apache/geronimo/gbean/runtime/GBeanOperation.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/modules/ 
geronimo-kernel/src/main/java/org/apache/geronimo/gbean/runtime/ 
GBeanOperation.java?view=diff&rev=499201&r1=499200&r2=499201
====================================================================== 
========
--- geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/ 
apache/geronimo/gbean/runtime/GBeanOperation.java (original)
+++ geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/ 
apache/geronimo/gbean/runtime/GBeanOperation.java Tue Jan 23  
16:55:35 2007

@@ -22,7 +22,6 @@
 import java.util.Collections;
 import java.util.Iterator;
 import java.util.List;
-import java.io.Serializable;

 import org.apache.geronimo.gbean.DynamicGBean;
 import org.apache.geronimo.gbean.DynamicGOperationInfo;
@@ -33,9 +32,7 @@
 /**
  * @version $Rev$ $Date$
  */
-public final class GBeanOperation implements Serializable {

-    private static final long serialVersionUID =  
-5185515581104192977L;

-
+public final class GBeanOperation {
     private final GBeanInstance gbeanInstance;
     private final String name;
     private final List parameterTypes;
@@ -43,7 +40,6 @@
     private final boolean framework;
     private final GOperationInfo operationInfo;

-    // TODO - deprecate this and add returnType

     static GBeanOperation createFrameworkOperation(GBeanInstance  
gbeanInstance, String name, List parameterTypes, MethodInvoker  
methodInvoker) {
         return new GBeanOperation(gbeanInstance, name,  
parameterTypes, methodInvoker);

     }
@@ -54,6 +50,7 @@
         this.name = name;

         this.parameterTypes = Collections.unmodifiableList(new  
ArrayList(parameterTypes));

         this.methodInvoker = methodInvoker;
+        // this is not used

         this.operationInfo = new GOperationInfo(this.name,  
this.parameterTypes, "java.lang.Object");

     }



Modified: geronimo/server/trunk/modules/geronimo-kernel/src/test/ 
java/org/apache/geronimo/gbean/GBeanInfoTest.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/modules/ 
geronimo-kernel/src/test/java/org/apache/geronimo/gbean/ 
GBeanInfoTest.java?view=diff&rev=499201&r1=499200&r2=499201
====================================================================== 
========
--- geronimo/server/trunk/modules/geronimo-kernel/src/test/java/org/ 
apache/geronimo/gbean/GBeanInfoTest.java (original)
+++ geronimo/server/trunk/modules/geronimo-kernel/src/test/java/org/ 
apache/geronimo/gbean/GBeanInfoTest.java Tue Jan 23 16:55:35 2007

@@ -85,7 +85,7 @@

     public void testGetOperationsSet() {
         Set gbeanOpSet = gbeanInfo.getOperations();
-        assertEquals(1, gbeanOpSet.size());
+        assertEquals(3, gbeanOpSet.size());
         assertTrue(gbeanOpSet.contains(opInfo));
     }

@@ -155,7 +155,8 @@
             infoFactory.addAttribute(persistentAttrInfo);

             infoFactory.addOperation(opInfo);
-

+            infoFactory.addOperation("addSomething", new Class[] 
{String.class}); // ignored
+            infoFactory.addOperation("removeSomething", new Class[] 
{String.class}); // ignored

             infoFactory.addReference(refInfo);


             infoFactory.addAttribute(CONSTRUCTOR_ARG_0,  
String.class, true);

@@ -175,6 +176,13 @@
         }

         public void setReference(String reference) {
+        }
+
+        public void addSomething(String something){
+        }
+
+        public String removeSomething(String something){
+           return null;
         }
     }
 }


























					Reply via email to