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) commentedThis was introduced in #2474471: Views page displays do not accept an integer as path.
Comment #3
Anonymous (not verified) 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 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 commentedHow about this then?
Comment #8
geertvd 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!