ember.js icon indicating copy to clipboard operation
ember.js copied to clipboard
verified publisher icon emberjs

Router service's `urlFor` does not include sticky queryparams in generated url

Open emattias opened this issue 2 years ago • 3 comments

🐞 Describe the Bug

When generating url with urlFor function on router service sticky params are not included in the generated url unlike when doing the same thing with generateURL in the private -routing service

🔬 Minimal Reproduction

  1. Go to Ember twiddle
  2. press Go to sub route with page=3 link
  3. notice how only the last url generated with generateURL includes the sticky param ?page=3 in the url

Also it will only include the sticky param if it is supplied with an empty object as the third argument.

😕 Actual Behavior

Sticky params are not included when generating urls with urlFor

🤔 Expected Behavior

Sticky params to be included when generating urls with urlFor

emattias avatar Mar 02 '23 14:03 emattias

Hello! Is this issue confirmed, or was it solved in further version due to breaking changes? Could you give further explanation in why is urlFor expected to grab sticky params even without specifying so I can help, please?

ginaesps avatar Mar 21 '23 18:03 ginaesps

In which version should I work this bug? 3.18 or the most recent one (4.11)?

ginaesps avatar Mar 23 '23 22:03 ginaesps

Also affecting me via https://github.com/rwjblue/ember-router-helpers/issues/313

amk221 avatar Mar 20 '24 11:03 amk221