Tapestry
  1. Tapestry
  2. TAPESTRY-1735

PageLink doesn't have a disabled parameter

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 5.0.5
        • Fix Version/s: 5.0.6
        • Component/s: tapestry-core
        • Labels:
          None

          Description

          An obvious omission. Will submit a patch ... perhaps when SVN trunk builds!

          Cheers,
          Nick.

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            Nick Westgate added a comment -

            Here's the trivial patch. No tests are affected or added. (ActionLink doesn't test disabled either.)

            Cheers,
            Nick.

            Show
            Nick Westgate added a comment - Here's the trivial patch. No tests are affected or added. (ActionLink doesn't test disabled either.) Cheers, Nick.
            Hide
            Permalink
            Howard M. Lewis Ship added a comment -

            Just because ActionLink is wrong doesn't make not testing right. Thanks for the patch, I've added tests.

            Show
            Howard M. Lewis Ship added a comment - Just because ActionLink is wrong doesn't make not testing right. Thanks for the patch, I've added tests.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development