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
We want to remove include files (including path.inc) and move from procedural to OO code.
Proposed resolution
Move drupal_is_front_page() to the PathMatcher service.
Remaining tasks
User interface changes
API changes
Comment | File | Size | Author |
---|---|---|---|
#2 | interdiff.txt | 808 bytes | kim.pepper |
#2 | 2301975-front-page-2.patch | 3.53 KB | kim.pepper |
#1 | 2301975-front-page-1.patch | 3.53 KB | kim.pepper |
Comments
Comment #1
kim.pepperAn initial patch to get things going...
Comment #2
kim.pepperSlight flaw in my logic there. Should be checking if property is set, not its value.
Comment #3
dawehnermh. One thing we might should do instead is the following: Change the configuration to internally store a route name + parameters and compare it with the one of the current route match object. Do you have an oppinion?
Comment #4
dawehnerOtherwise we could for sure just to that in a dedicated issue.
Comment #5
ParisLiakos CreditAttribution: ParisLiakos commentedyes #3 is a great followup
rtbc++
Comment #6
alexpottCommitted 3cfff37 and pushed to 8.x. Thanks!