Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Looking at UserUserSearchTestCase now. Opening to have a nid.
Comment | File | Size | Author |
---|---|---|---|
#2 | 1557590-kernel-usersearch-2.patch | 553 bytes | Niklas Fiekas |
Comments
Comment #1
Niklas Fiekas CreditAttribution: Niklas Fiekas commentedOk. so the problem is that path components are not urldecoded - random resource: http://markmail.org/message/7urcbz4nl3ypx6e7.
Not sure if we should fix it on a lower level than just urldecoding it in search_view(). Not sure if there are other instances without testcoverage where this would be a problem - most of the time machine names are used in URLs.
If we do not change this on a lower level, we would probably at least need a change notification for contrib.
Pushed what would be the "high-level quick-fix" to 1557590-kernel-usersearch.
Comment #2
Niklas Fiekas CreditAttribution: Niklas Fiekas commentedComment #3
Crell CreditAttribution: Crell commentedDiscussing in #1463656-96: Add a Drupal kernel; leverage HttpFoundation and HttpKernel
Comment #4
Crell CreditAttribution: Crell commentedMarking fixed per #1565084: urldecode()ing of the path. If that doesn't actually resolve this issue, please reopen. If it does, go ahead and remove the branch for this issue. Thanks.
Comment #5
Crell CreditAttribution: Crell commented