Prioritizing route parameter over query parameter

[juancamunozfp]

We are encountering an issue that arises when a route has a route parameter and the URL query string also contains a parameter with the same name as the route parameter.

This modification ensures that the route parameter value takes precedence over the query parameter with the same name

[ivanvermeyen]

Hi,

Thank you for the PR!

I was able to produce the issue in a test, but your proposed solution would remove the query string parameter, which is probably also undesirable.

Instead, I have split the query string parameters from the route parameters and appended them manually.

See commit: fac6e4a

I tagged this in release 3.1.3.
Let me know if this fixes your issue. 👍

[juancamunozfp]

Hi,

Thank you for the PR!

I was able to produce the issue in a test, but your proposed solution would remove the query string parameter, which is probably also undesirable.

Instead, I have split the query string parameters from the route parameters and appended them manually.

See commit: fac6e4a

I tagged this in release 3.1.3. Let me know if this fixes your issue. +1

Thank you for your reply, yes, It works well!!

I prefer also your implementation, so I'll close the pr now that the issue has been solved.

Once again, thank you very much!