- 🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
Moving to PNMI for steps to reproduce per #8
- Status changed to Needs work
9 months ago 2:13pm 25 April 2024 - 🇳🇱Netherlands Lendude Amsterdam
Steps to reproduce:
* Create a View of nodes
* Show fields
* Show the nid and title field
* Add a contextual filter 'Content: Has taxonomy term ID' and set default value to ' Display all results for the specified field'
* For the title field set rewrite results: Output this field as a custom link to /node/{{ nid }}?term_id={{ arguments.tid }}
* Create a page display
* Go to the page and see that 'term_id' is added to the link URL on the title, bt with no valueDidn't try the patch yet
- First commit to issue fork.
- 🇪🇸Spain vidorado Pamplona (Navarra)
I’ve created an MR but wasn’t able to figure out how to write the tests. The existing tests in
core/modules/views/tests/src/Unit/Plugin/field/FieldPluginBaseTest.phpare a bit complex for me to follow. Additionally, I’m unsure whether this can be unit tested or if it requires functional tests due to its dependency on view arguments.I’ve also updated the steps to reproduce, as the view argument name (
tid) must match the query parameter name set in the rewrite field (which was mistakenly set toterm_id).
