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.
This issue is for fixing TaxonomyTermTestCase.
Comment | File | Size | Author |
---|---|---|---|
#1 | 1558346.patch | 1.26 KB | sheise |
Comments
Comment #1
sheise CreditAttribution: sheise commentedThere were 2 problems here:
- The autocomplete string was not being decoded properly and the spaces were breaking it (just like in this issue http://drupal.org/node/1558166).
- There was a typo in the test.
Patch attached.
Comment #2
sheise CreditAttribution: sheise commentedComment #3
Crell CreditAttribution: Crell commentedWhy was this not breaking before, then? I'm reluctant to change the test unless I understand why the test was right before and wrong now.
Comment #4
Crell CreditAttribution: Crell commentedActually, this looks like the same issue as #1557590: Fix UserUserSearchTestCase. Hm.
Comment #5
Crell CreditAttribution: Crell commentedDiscussing in #1463656-96: Add a Drupal kernel; leverage HttpFoundation and HttpKernel
Comment #6
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 #7
Crell CreditAttribution: Crell commented