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.
In the Views UI, if you are looking at a Page display, the actions dropdown in the upper right has a link to "Display page".
This link is broken. For instance if my page path is "number-test", it is taking me instead to the page (my site URL)/base%3Anumber-test
If it makes any difference, my test site is multilingual with 2 languages defined, but I'm currently looking at English.
Comment | File | Size | Author |
---|---|---|---|
#8 | interdiff-2561943-5-7.txt | 1.01 KB | geertvd |
#7 | views_ui_view_page-2561943-7-complete.patch | 2 KB | geertvd |
#7 | views_ui_view_page-2561943-7-test.patch | 906 bytes | geertvd |
#5 | views_ui_view_page-2561943-5-complete.patch | 2.08 KB | geertvd |
#5 | views_ui_view_page-2561943-5-test.patch | 990 bytes | geertvd |
Comments
Comment #2
Anonymous (not verified) CreditAttribution: Anonymous at XIO commentedThis was introduced in #2474471: Views page displays do not accept an integer as path.
Comment #3
Anonymous (not verified) CreditAttribution: Anonymous at XIO commentedComment #4
jhodgdonThat was quick!
This patch fixes the problem for me. Still needs an automated web regression test, so it won't break again, so setting to Needs Work.
Comment #5
geertvd CreditAttribution: geertvd at XIO commentedAdded a test for this
Comment #6
jhodgdonThat test looks overly specific to me, with all the IDs and paths and stuff in there.... Can't we just test that if we click the link on a view that's been saved, it goes to the right page? The way this test is written, it seems like if anything changes in the theme being used for the test, the test will fail.
Comment #7
geertvd CreditAttribution: geertvd at XIO commentedHow about this then?
Comment #8
geertvd CreditAttribution: geertvd at XIO commentedinterdiff
Comment #12
jhodgdonThat looks great to me! The test-only patch failed with the reported bug ("base%03" in the URL), and it passes with the code patch that I had manually tested to fix the problem.
Thanks!
Comment #13
jibran+1 LGTM
Comment #14
alexpottCommitted a8a685a and pushed to 8.0.x. Thanks!