Wicket
  1. Wicket
  2. WICKET-5056

Page mount with an optional named parameter overtakes a mount with more specific path

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 6.6.0
    • Fix Version/s: 6.9.0, 7.0.0
    • Component/s: wicket
    • Labels:
      None

      Description

      See the discussion in http://markmail.org/thread/sgpiku27ah2tmcim

      Having:
      mountPage("/all/sindex",Page1.class);
      mountPage("/all/#

      {exp}

      ", Page2.class);

      Request to /all/sindex will be handled by Page2.

      Compatibility score for optional parameters should be lower than mandatory parameters which should be lower than exact value.

        Issue Links

          Activity

          Hide
          Martin Grigorov added a comment -

          Thank you, Cedric!
          You change is applied.

          Show
          Martin Grigorov added a comment - Thank you, Cedric! You change is applied.
          Hide
          Martin Grigorov added a comment -

          Thank you, Cedric!
          I'll try to review and merge it this week.

          Show
          Martin Grigorov added a comment - Thank you, Cedric! I'll try to review and merge it this week.
          Hide
          Cedric Gatay added a comment -

          Simple implementation is available for code review at the following URL, not doing a Pull Request until someone reviewed this

          https://github.com/CedricGatay/wicket/commit/6ac453529095380c4153edd462ff0d66a2cdb563

          Thanks

          Show
          Cedric Gatay added a comment - Simple implementation is available for code review at the following URL, not doing a Pull Request until someone reviewed this https://github.com/CedricGatay/wicket/commit/6ac453529095380c4153edd462ff0d66a2cdb563 Thanks
          Hide
          Cedric Gatay added a comment -

          I'm taking this one, which target version should it be built against ? Wicket 7.x or 6.x ?

          Show
          Cedric Gatay added a comment - I'm taking this one, which target version should it be built against ? Wicket 7.x or 6.x ?

            People

            • Assignee:
              Martin Grigorov
              Reporter:
              Martin Grigorov
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development