Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
The path admin UI lives at admin/config/search/path, but the filtering uses admin/config/search/path/list/%.
That got lost in the local task conversions, but was not caught by tests, because we don't have any.
Proposed resolution
Fix the routing
Write tests
Remaining tasks
N/A
User interface changes
N/A
API changes
N/A
Comment | File | Size | Author |
---|---|---|---|
#4 | path-2244757-4.patch | 4.63 KB | tim.plunkett |
Comments
Comment #1
tim.plunkett#1987802: Convert path_admin_overview() to a new style controller will port this to OO code, but we need to have a working baseline first.
Comment #2
tim.plunkettThis just splits the filtered and unfiltered routes to separate methods, these will be utilized further in the conversion issue (right now they call the same function with and without $keys).
Comment #3
dawehnerThat test naming is a c&p
Keys: NULL ensures an optional part of the path.
Comment #4
tim.plunkett1. Heh, laziness--
2. This is purposeful.
As discussed on IRC
Comment #5
dawehnerCoool
Comment #7
tim.plunkettLast fail was the tests-only patch.
Comment #8
webchickCommitted and pushed to 8.x. Thanks!