Tapestry 5
  1. Tapestry 5
  2. TAP5-1680

A mixin with a formal parameter will prevent an informal parameter of the same name from being bound on the component

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.3, 5.2
    • Fix Version/s: 5.3
    • Component/s: tapestry-core
    • Labels:

      Description

      <t:actionlink t:id="foo" title="Link Title" t:mixins="tapx/confirm" confirm.title="Oh really?"/>

      this will not work as expected (I need to experiment to see which version of title is bound to the mixin's parameter), but because title is not a formal parameter, there's no way to bind it on ActionLink in the presence of a mixin that has a formal paramater with this name.

      I suspect the solution will end up being actionlink.title="Link Title".

        Activity

        Hide
        Howard M. Lewis Ship added a comment -

        Verified that the simple informal parameter gets ignored, and the mixin formal parameter takes over.

        Show
        Howard M. Lewis Ship added a comment - Verified that the simple informal parameter gets ignored, and the mixin formal parameter takes over.
        Hide
        Hudson added a comment -

        Integrated in tapestry-trunk-freestyle #558 (See https://builds.apache.org/job/tapestry-trunk-freestyle/558/)
        TAP5-1680: A mixin with a formal parameter will prevent an informal parameter of the same name from being bound on the component

        hlship : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1178584
        Files :

        • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/pageload/ComponentAssemblerImpl.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/pageload/EmbeddedComponentAssembler.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/pageload/EmbeddedComponentAssemblerImpl.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/pageload/PageloadMessages.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/internal/pageload/PageloadStrings.properties
        • /tapestry/tapestry5/trunk/tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app1/ParameterTests.groovy
        • /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/mixins/AltTitle.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/pages/Index.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/pages/MixinVsInformalParameter.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/test/resources/org/apache/tapestry5/integration/app1/pages/MixinVsInformalParameter.tml
        Show
        Hudson added a comment - Integrated in tapestry-trunk-freestyle #558 (See https://builds.apache.org/job/tapestry-trunk-freestyle/558/ ) TAP5-1680 : A mixin with a formal parameter will prevent an informal parameter of the same name from being bound on the component hlship : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1178584 Files : /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/pageload/ComponentAssemblerImpl.java /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/pageload/EmbeddedComponentAssembler.java /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/pageload/EmbeddedComponentAssemblerImpl.java /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/pageload/PageloadMessages.java /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/internal/pageload/PageloadStrings.properties /tapestry/tapestry5/trunk/tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app1/ParameterTests.groovy /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/mixins/AltTitle.java /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/pages/Index.java /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/pages/MixinVsInformalParameter.java /tapestry/tapestry5/trunk/tapestry-core/src/test/resources/org/apache/tapestry5/integration/app1/pages/MixinVsInformalParameter.tml

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Howard M. Lewis Ship
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development