Looking at UserUserSearchTestCase now. Opening to have a nid.

CommentFileSizeAuthor
#2 1557590-kernel-usersearch-2.patch553 bytesNiklas Fiekas
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Niklas Fiekas’s picture

Assigned: Niklas Fiekas » Unassigned
Status: Active » Needs review

Ok. 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.

Niklas Fiekas’s picture

Crell’s picture

Crell’s picture

Marking 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.

Crell’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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