Implement PSR-20 in datetime.time service

kim.pepper → made their first commit to this issue’s fork.

Need to add the composer dependency.

I'm wondering if we don't just add symfony/clock and replace usages of datetime.time service? 🤔

I'm not sure symfony/clock could replace getRequestTime().

If we are adding Psr/Clock as a dependency, then there should be at least one instance where we swap out to use it.

For #9

prudloff → changed the visibility of the branch 3501375-implement-psr-20 to hidden.

prudloff → changed the visibility of the branch 3501375-implement-psr-20 to active.

Do you mean something like this?

Didn't review as it seems @kimpepper already did.

Am I correct to think we can break BC in the LinearHistory constructor because it is marked as internal and experimental?

Added a CR and updated the IS. I think this is ready.

I was creating a follow up issue to replace TimeInterface->getCurrentTime() with ClockInterface->now()->getTimestamp() in the rest of core.

I searched for usages of \Drupal::time()->getCurrentTime() and found 28 instances.

\Drupal::time() returns \Drupal\Component\Datetime\TimeInterface but this doesn't extend Psr\ClockInterface, so it's not a straight method swap.

I think we need a new \Drupal::clock() method that returns Psr\ClockInterface. This would make it easier to swap out usage of \Drupal::time()->getCurrentTime() with \Drupal::clock()->now()->getTimestamp().

We might want to do #17 and deprecate \Drupal::time()->getCurrentTime() in a follow up.

I created a follow-up 📌 [PP-1] Deprecate TimeInterface::getCurrentTime() and replace with ClockInterface::now()->getTimestamp() Postponed so back to RTBC.

I also squashed the commits and rebased on the latest 11.x. This will help if we need to continually rebase on 11.x until it gets committed.

kim.pepper → changed the visibility of the branch 3501375-symfony-clock to hidden.

Left one nitpick.

I'm trying an alternative approach in https://git.drupalcode.org/project/drupal/-/merge_requests/11533 however I can't tell if test fails are related or not? 🤔

Instead of adding onto the Time class, I'm creating a dedicated Clock implementation.

Unrelated, see 🐛 [random test failure] ImageUrlProviderTest::testResize Active