You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by hl...@apache.org on 2011/04/14 00:22:54 UTC

svn commit: r1091940 - /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/PropertyConduitSourceImpl.java

Author: hlship
Date: Wed Apr 13 22:22:54 2011
New Revision: 1091940

URL: http://svn.apache.org/viewvc?rev=1091940&view=rev
Log:
TAP5-853: Fix the code that generates the call to a setter method

Modified:
    tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/PropertyConduitSourceImpl.java

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/PropertyConduitSourceImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/PropertyConduitSourceImpl.java?rev=1091940&r1=1091939&r2=1091940&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/PropertyConduitSourceImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/PropertyConduitSourceImpl.java Wed Apr 13 22:22:54 2011
@@ -961,8 +961,9 @@ public class PropertyConduitSourceImpl i
                     }
                     else
                     {
-                        // TODO: Its not a no-args method. It's one arg.
-                        invokeNoArgsMethod(builder, method);
+                        // Invoke the setter method
+                        builder.invoke(method.getDeclaringClass(), void.class, method.getName(),
+                                method.getParameterTypes());
                     }
 
                     builder.returnResult();