Closed (outdated)
Project:
Drupal core
Version:
11.x-dev
Component:
base system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
6 Feb 2019 at 11:46 UTC
Updated:
27 Feb 2024 at 21:18 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
satyanarayan reddy commentedReplace usages of the deprecated REQUEST_TIME for modules
Comment #3
cilefen commentedComment #5
berdirthis one could be injected
Those in this file too.
Same.
Same.
This in theory too, but it's tricky, through a factory service and there are 17 existing \Drupal:: calls, so I don't think we need to push that here.
Comment #6
anmolgoyal74 commentedComment #9
kristen polSee possibly related issues noted here:
https://www.drupal.org/project/drupal/issues/3112283#comment-13605217
Comment #14
megachrizThis looks like a duplicate of #3112298: Replace REQUEST_TIME in classes with direct container access, although that other issue only addresses classes and this one handles the following other files:
Comment #16
needs-review-queue-bot commentedThe Needs Review Queue Bot tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.
Comment #18
andypost