Tapestry
  1. Tapestry
  2. TAPESTRY-1166

If and Else components ignore element parameter when renderTag is false

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 4.1
    • Fix Version/s: 4.1.1
    • Component/s: tapestry-core
    • Labels:
      None
    • Environment:
      N/A

      Description

      If an element is specified, it should be rendered regardless of the value of renderTag. This is preferable for the sake of backwards compatibility and, arguably, is the correct logic as long as the element and renderTag parameters co-exist.

      This issue does not affect the For component, which always renders a tag if the element parameter is specified.

        Activity

        Ryan Holmes created issue -
        Hide
        Ryan Holmes added a comment -

        Simple patch to make If and Else always render a tag when element is specified.

        Show
        Ryan Holmes added a comment - Simple patch to make If and Else always render a tag when element is specified.
        Ryan Holmes made changes -
        Field Original Value New Value
        Attachment If-Else-element-compat.patch [ 12345799 ]
        Hide
        Jesse Kuhnert added a comment -

        Applied, thanks.

        Show
        Jesse Kuhnert added a comment - Applied, thanks.
        Jesse Kuhnert made changes -
        Resolution Fixed [ 1 ]
        Fix Version/s 4.1.1 [ 12312021 ]
        Assignee Jesse Kuhnert [ jkuhnert ]
        Status Open [ 1 ] Resolved [ 5 ]
        Mark Thomas made changes -
        Workflow jira [ 12390760 ] Default workflow, editable Closed status [ 12567970 ]
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12567970 ] jira [ 12591083 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        1d 16h 21m 1 Jesse Kuhnert 28/Nov/06 21:53

          People

          • Assignee:
            Jesse Kuhnert
            Reporter:
            Ryan Holmes
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development