Allow token replacement for array URLs on views custom URLs

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.

recursiveReplaceTokens
1. This can be typehinted

Ran tests locally without the fix and failed as expected

The read more link with href "/node/22?date[22]=22&foo=bar" was found.
Failed asserting that '<div class="js-view-dom-id-1c759818c178990c87c1b29b00b46d3cce1e380a5e37ade8d751a600f282635d">\n
  \n
  \n
  \n
\n
  \n
  \n
  \n
\n
    \n
\n
  \n
  <div class="more-link"><a href="/node/22?date%5B%7B%7B%20raw_arguments.age%20%7D%7D%5D=22&amp;foo=bar">more</a></div>\n
\n
\n
  \n
  \n
</div>\n
' contains "/node/22?date%5B22%5D=22&amp;foo=bar".

Just need that one tweak I believe.

Updated patch #8 based on suggestion from #13

When converting to an MR, patches should be hidden to avoid confusion.

Feedback has been addressed though.

This looks fine to me, but it needs a rebase.

As suggested in #24 rebased the issue fork with latest update.

Rebase seems fine.

Committed to 11.x - thanks all!

larowlan → closed merge request !5979

Automatically closed - issue fixed for 2 weeks with no activity.