Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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