Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.
» Replace usages of the deprecated REQUEST_TIME constant
Version:
8.6.x-dev
» 8.7.x-dev
Component:
user.module
» base system
Status:
Needs review
» Needs work
Hello. Thanks for the patch. We do not usually commit changes like this that are specific to one module. I am re-titling this issue as "Replace usages of the deprecated REQUEST_TIME constant".
Let's replace all of them.
It won't be trivial. To replace in one module is too small of a scope, but to change all of them in one patch may be too "big"—there are ~350 usages in core. We in fact may wish to break this down a bit, perhaps by splitting into an issue to replace usages in tests, another for modules, and another for lib.
Drupal 8.8.0-alpha1 will be released the week of October 14th, 2019, which means new developments and disruptive changes should now be targeted against the 8.9.x-dev branch. (Any changes to 8.9.x will also be committed to 9.0.x in preparation for Drupal 9’s release, but some changes like significant feature additions will be deferred to 9.1.x.). For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.
Comments
Comment #2
satyanarayan reddy commentedThe REQUEST_TIME global constant has also been deprecated.
Comment #3
cilefen commentedHello. Thanks for the patch. We do not usually commit changes like this that are specific to one module. I am re-titling this issue as "Replace usages of the deprecated REQUEST_TIME constant".
Let's replace all of them.
It won't be trivial. To replace in one module is too small of a scope, but to change all of them in one patch may be too "big"—there are ~350 usages in core. We in fact may wish to break this down a bit, perhaps by splitting into an issue to replace usages in tests, another for modules, and another for lib.
Thanks for taking this up!
Comment #4
satyanarayan reddy commentedI was created patch for test and Module
Reference url
1) https://www.drupal.org/project/drupal/issues/3030999
2) https://www.drupal.org/project/drupal/issues/3030983
Comment #5
cilefen commentedComment #9
kristen polSee possibly related issues noted here:
https://www.drupal.org/project/drupal/issues/3112283#comment-13605217
Comment #14
megachrizThere is an other meta issue for this too #2902895: [meta][no patch] Replace uses of REQUEST_TIME and time() with time service.
Comment #17
andypostOther issue is fixed #2902895: [meta][no patch] Replace uses of REQUEST_TIME and time() with time service