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.
Problem/Motivation
The abstract class PathListenerBase
is completely dead code. Neither of the two contained methods (setPath()
and extractPath()
) are used anywhere throughout the code base. As far as I can tell this is a left over from the refactoring done in #1910318: Make path resolution the responsibility of a series of PathProcessor classes, rather than one PathSubscriber class.
As a bonus, removal of this class also kills two occurrences of _system_path
.
Proposed resolution
Remove it.
Remaining tasks
User interface changes
API changes
Comment | File | Size | Author |
---|---|---|---|
#1 | 2329653-remove-path-listener-base.patch | 1.43 KB | znerol |
Comments
Comment #1
znerol CreditAttribution: znerol commentedComment #2
dawehnerIndeed the PathSubscriber has all the logic, so there is no need for a base class. Having one place also makes it easier to remove it in the future.
Nice work!
Comment #3
webchickCommitted and pushed to 8.x. Thanks!