Uploaded image for project: 'Wicket'
  1. Wicket
  2. WICKET-5056

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

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 6.6.0
    • Fix Version/s: 6.9.0, 7.0.0-M1
    • 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
          cedricgatay 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
          cedricgatay Cedric Gatay added a comment - I'm taking this one, which target version should it be built against ? Wicket 7.x or 6.x ?
          Hide
          cedricgatay 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
          cedricgatay 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
          mgrigorov Martin Grigorov added a comment -

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

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

          Thank you, Cedric!
          You change is applied.

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

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development