Active
Project:
Drupal core
Version:
main
Component:
base system
Priority:
Normal
Category:
Plan
Assigned:
Unassigned
Reporter:
Created:
28 Sep 2016 at 11:55 UTC
Updated:
22 Sep 2019 at 18:37 UTC
Jump to comment: Most recent
As people write more and more unit tests constants in modules are getting in the way. Another problem with constants are that page cache serializes content when modules are loaded, see #2807501: ResourceResponse::$responseData is serialized, but is never used: unserializing it on every Page Cache hit is a huge performance penalty.
Global constants, living in .module files, were moved to appropriate interfaces and deprecated, in #2807785: Move global constants from *.module files into interfaces. But there are still deprecated usages in core.
Script the automatic replacement of usage of the deprecated constants with the new interface constants.
None.
None.
None.
None.
Comments
Comment #2
dawehnerComment #3
wim leersIsn't this a duplicate of #2807785: Move global constants from *.module files into interfaces?
Comment #4
dawehnerIt is and it isn't.
I'm proposing a proper way to conversion, as in splitting it up in two steps ...
Comment #5
wim leersOk. Then why can't #2807785: Move global constants from *.module files into interfaces be step 1 and this be step two?
Comment #6
mpdonadioComment #8
claudiu.cristeaRetitling and updating the IS based on #5.
Postponing on #2807785: Move global constants from *.module files into interfaces.
Comment #9
klausi#2807785: Move global constants from *.module files into interfaces is in, so this can be worked on now.
Comment #14
berdirAdded #3023015: Replace usages of deprecated constants in file.inc as a child issue, could use a review.