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 2007/04/17 02:43:25 UTC
svn commit: r529458 - in /tapestry/tapestry5/tapestry-core/trunk/src/main/java/org/apache/tapestry/corelib/components: ActionLink.java PageLink.java

Author: hlship
Date: Mon Apr 16 17:43:24 2007
New Revision: 529458

URL: http://svn.apache.org/viewvc?view=rev&rev=529458
Log:
TAPESTRY-1411: PageLink component does not render informal parameters

Modified:
    tapestry/tapestry5/tapestry-core/trunk/src/main/java/org/apache/tapestry/corelib/components/ActionLink.java
    tapestry/tapestry5/tapestry-core/trunk/src/main/java/org/apache/tapestry/corelib/components/PageLink.java

Modified: tapestry/tapestry5/tapestry-core/trunk/src/main/java/org/apache/tapestry/corelib/components/ActionLink.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/tapestry-core/trunk/src/main/java/org/apache/tapestry/corelib/components/ActionLink.java?view=diff&rev=529458&r1=529457&r2=529458
==============================================================================
--- tapestry/tapestry5/tapestry-core/trunk/src/main/java/org/apache/tapestry/corelib/components/ActionLink.java (original)
+++ tapestry/tapestry5/tapestry-core/trunk/src/main/java/org/apache/tapestry/corelib/components/ActionLink.java Mon Apr 16 17:43:24 2007
@@ -26,11 +26,13 @@
 import org.apache.tapestry.annotations.Inject;
 import org.apache.tapestry.annotations.Mixin;
 import org.apache.tapestry.annotations.Parameter;
+import org.apache.tapestry.annotations.SupportsInformalParameters;
 import org.apache.tapestry.corelib.mixins.RenderInformals;
 
 /**
  * Component that triggers an action on the server with a subsequent full page refresh.
  */
+@SupportsInformalParameters
 public class ActionLink
 {
     /**
@@ -60,8 +62,7 @@
 
     void beginRender(MarkupWriter writer)
     {
-        if (_disabled)
-            return;
+        if (_disabled) return;
 
         String clientId = _support.allocateClientId(_resources.getId());
 
@@ -76,8 +77,7 @@
 
     void afterRender(MarkupWriter writer)
     {
-        if (_disabled)
-            return;
+        if (_disabled) return;
 
         writer.end(); // <a>
     }

Modified: tapestry/tapestry5/tapestry-core/trunk/src/main/java/org/apache/tapestry/corelib/components/PageLink.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/tapestry-core/trunk/src/main/java/org/apache/tapestry/corelib/components/PageLink.java?view=diff&rev=529458&r1=529457&r2=529458
==============================================================================
--- tapestry/tapestry5/tapestry-core/trunk/src/main/java/org/apache/tapestry/corelib/components/PageLink.java (original)
+++ tapestry/tapestry5/tapestry-core/trunk/src/main/java/org/apache/tapestry/corelib/components/PageLink.java Mon Apr 16 17:43:24 2007
@@ -23,6 +23,7 @@
 import org.apache.tapestry.annotations.Environmental;
 import org.apache.tapestry.annotations.Inject;
 import org.apache.tapestry.annotations.Parameter;
+import org.apache.tapestry.annotations.SupportsInformalParameters;
 
 /**
  * Generates a render request link to some other page in the application. If an activation context
@@ -34,6 +35,7 @@
  * context, the value typically represents the identity of some object displayed or otherwise
  * manipulated by the page.
  */
+@SupportsInformalParameters
 public class PageLink
 {
     /** The logical name of the page to link to. */