[qdox-scm] [821] trunk/qdox/src/test/com/thoughtworks/qdox/model: Let ModelBuilder bind JavaMethod and JavaParameter to each other

[rfscholte]

Title: [821] trunk/qdox/src/test/com/thoughtworks/qdox/model: Let ModelBuilder bind JavaMethod and JavaParameter to each other

Revision

821

Author

rfscholte

Date

2010-10-25 16:18:39 -0500 (Mon, 25 Oct 2010)

Log Message

Let ModelBuilder bind JavaMethod and JavaParameter to each other

Modified Paths

• trunk/qdox/src/java/com/thoughtworks/qdox/model/JavaMethod.java
• trunk/qdox/src/java/com/thoughtworks/qdox/model/ModelBuilder.java
• trunk/qdox/src/test/com/thoughtworks/qdox/model/DefaultJavaClassTest.java
• trunk/qdox/src/test/com/thoughtworks/qdox/model/DefaultJavaMethodTest.java
• trunk/qdox/src/test/com/thoughtworks/qdox/model/DefaultJavaParameterTest.java

Diff

Modified: trunk/qdox/src/java/com/thoughtworks/qdox/model/JavaMethod.java (820 => 821)

--- trunk/qdox/src/java/com/thoughtworks/qdox/model/JavaMethod.java	2010-10-24 15:59:16 UTC (rev 820)
+++ trunk/qdox/src/java/com/thoughtworks/qdox/model/JavaMethod.java	2010-10-25 21:18:39 UTC (rev 821)
@@ -181,7 +181,6 @@
     }
 
     public void addParameter(JavaParameter javaParameter) {
-        javaParameter.setParentMethod( this );
         parameters.add( javaParameter );
         parametersArray = null;
         this.varArgs = javaParameter.isVarArgs();

Modified: trunk/qdox/src/java/com/thoughtworks/qdox/model/ModelBuilder.java (820 => 821)

--- trunk/qdox/src/java/com/thoughtworks/qdox/model/ModelBuilder.java	2010-10-24 15:59:16 UTC (rev 820)
+++ trunk/qdox/src/java/com/thoughtworks/qdox/model/ModelBuilder.java	2010-10-25 21:18:39 UTC (rev 821)
@@ -305,6 +305,7 @@
 	
 	public void addParameter(FieldDef fieldDef) {
         JavaParameter jParam = new JavaParameter(createType(fieldDef.type, fieldDef.dimensions), fieldDef.name, fieldDef.isVarArgs);
+        jParam.setParentMethod( currentMethod );
         setAnnotations( jParam );
         currentMethod.addParameter( jParam );
 	}

Modified: trunk/qdox/src/test/com/thoughtworks/qdox/model/DefaultJavaClassTest.java (820 => 821)

--- trunk/qdox/src/test/com/thoughtworks/qdox/model/DefaultJavaClassTest.java	2010-10-24 15:59:16 UTC (rev 820)
+++ trunk/qdox/src/test/com/thoughtworks/qdox/model/DefaultJavaClassTest.java	2010-10-25 21:18:39 UTC (rev 821)
@@ -94,6 +94,7 @@
     public void addParameter( JavaMethod method, JavaParameter parameter )
     {
         method.addParameter( parameter );
+        parameter.setParentMethod( method );
     }
 
     // Set-methods

Modified: trunk/qdox/src/test/com/thoughtworks/qdox/model/DefaultJavaMethodTest.java (820 => 821)

--- trunk/qdox/src/test/com/thoughtworks/qdox/model/DefaultJavaMethodTest.java	2010-10-24 15:59:16 UTC (rev 820)
+++ trunk/qdox/src/test/com/thoughtworks/qdox/model/DefaultJavaMethodTest.java	2010-10-25 21:18:39 UTC (rev 821)
@@ -98,6 +98,7 @@
     public void addParameter( JavaMethod method, JavaParameter parameter )
     {
         method.addParameter( parameter );
+        parameter.setParentMethod( method );
     }
 
     public void setSourceCode( JavaMethod method, String code )

Modified: trunk/qdox/src/test/com/thoughtworks/qdox/model/DefaultJavaParameterTest.java (820 => 821)

--- trunk/qdox/src/test/com/thoughtworks/qdox/model/DefaultJavaParameterTest.java	2010-10-24 15:59:16 UTC (rev 820)
+++ trunk/qdox/src/test/com/thoughtworks/qdox/model/DefaultJavaParameterTest.java	2010-10-25 21:18:39 UTC (rev 821)
@@ -26,6 +26,7 @@
     public void addParameter( JavaMethod method, JavaParameter parameter )
     {
         method.addParameter( parameter );
+        parameter.setParentMethod( method );
     }
 
 }

---

To unsubscribe from this list please visit:

http://xircles.codehaus.org/manage_email