Uploaded image for project: 'Tapestry'
  1. Tapestry
  2. TAPESTRY-1411

PageLink component does not render informal parameters

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 5.0.3
    • Fix Version/s: 5.0.4
    • Component/s: Core Components
    • Labels:
      None

      Description

      When specifying a PageLink, Tapestry 5 will remove extra attributes from the containing link tag.

      Specifically, it will remove the class="CSS_CLASS" attribute.

      Example:

      <a t:type="pagelink" t:page="logoff/Logoff" class="navitab">Logoff</a>

      Will render as:

      <a href="/irm/logoff/logoff" id="pagelink">Logoff</a>

      1. pagelink-patch
        1.0 kB
        Yann Ramin

        Activity

        Hide
        theatrus Yann Ramin added a comment -

        This patch will allow you to specify pagelinks which work around the issue:

        <a t:type="pagelink" t:page="logoff/Logoff" t:class="navitab">Logoff</a>

        (interestingly enough, using just class="xyz" as in actionlink will not work, it must be in the t: namespace)

        This change was done by comparing actionlink and pagelink. Actionlink does not have this issue. The actual problem is probably deeper in the Tapestry core.

        Show
        theatrus Yann Ramin added a comment - This patch will allow you to specify pagelinks which work around the issue: <a t:type="pagelink" t:page="logoff/Logoff" t:class="navitab">Logoff</a> (interestingly enough, using just class="xyz" as in actionlink will not work, it must be in the t: namespace) This change was done by comparing actionlink and pagelink. Actionlink does not have this issue. The actual problem is probably deeper in the Tapestry core.
        Hide
        hlship Howard M. Lewis Ship added a comment -

        The issue for PageLink, and for ActionLink, is that the @SupportsInformalParameters annotation is missing, so the informal parameters aren't available when the components invoke the renderInformalParameters() method.

        Show
        hlship Howard M. Lewis Ship added a comment - The issue for PageLink, and for ActionLink, is that the @SupportsInformalParameters annotation is missing, so the informal parameters aren't available when the components invoke the renderInformalParameters() method.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development