Tapestry
  1. Tapestry
  2. TAPESTRY-1411

PageLink component does not render informal parameters

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major 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
        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
        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.
        Hide
        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
        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.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development