Wicket
  1. Wicket
  2. WICKET-326

Redundant declaring of mount path twice when mounting UrlCodingStrategy

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.3.0-beta1
    • Fix Version/s: None
    • Component/s: wicket
    • Labels:
      None

      Description

      Currently in 1.3 mounting of a page with a UrlCodingStrategy is like this:

      mount("/item", new IndexedParamUrlCodingStrategy("/item", ItemViewPage.class));

      Here "/item" is repeated twice, which is unnecessary and it will not work if the strings are different.

      Nabble discussion: http://www.nabble.com/URL-when-navigating-away-from-a-bookmarkable-URL-tf3286244.html

        Activity

        Hide
        Eelco Hillenius added a comment -

        This is implemented properly with 2.0. We should backport that if it is not too much trouble.

        Show
        Eelco Hillenius added a comment - This is implemented properly with 2.0. We should backport that if it is not too much trouble.
        Hide
        Eelco Hillenius added a comment -

        Could you look at this Johan?

        Show
        Eelco Hillenius added a comment - Could you look at this Johan?
        Hide
        Peter Thomas added a comment -

        This appears fixed as per #WICKET-410

        Show
        Peter Thomas added a comment - This appears fixed as per # WICKET-410
        Hide
        Peter Thomas added a comment -

        oops, maybe I should not have closed this - that should be only done by committers I suppose. apologies, please reopen if required

        Show
        Peter Thomas added a comment - oops, maybe I should not have closed this - that should be only done by committers I suppose. apologies, please reopen if required

          People

          • Assignee:
            Johan Compagner
            Reporter:
            Peter Thomas
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development