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
#2175917: Clean up configuration system events introduced the Drupal\Core\Config\ConfigEvents class which defines configuration event names. RoutingEvents and KernelEvents classes contain similar event definitions, but these classes are defined as 'final' class and ConfigEvents is not.
Proposed resolution
Make ConfigEvents a final class.
Remaining tasks
T.b.d.
User interface changes
none
API changes
none
Comment | File | Size | Author |
---|---|---|---|
#1 | config-final-class-2195407-1.patch | 437 bytes | Sutharsan |
Comments
Comment #1
Sutharsan CreditAttribution: Sutharsan commentedComment #2
alexpottYep makes heaps of sense.
Comment #3
jibranPlease also update this so that it describes the appropriate changes.
Comment #4
jibranWhat I meant was please update the comment why this class is final. I have looked at other final classes in core and all has simple description so we don't have to update the comment. Please ignore #3. RTBC +1.
Comment #5
catchCommitted/pushed to 8.x, thanks!