When the procedural functions were deprecated and moved to services last fall, a static call to \Drupal::time() was left in the code. We should inject the datetime.time service and use that instead.

CommentFileSizeAuthor
time-service.patch2.49 KBTR

Issue fork ip2country-3115021

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TR created an issue. See original summary.

  • TR committed 471965f on 8.x-1.x 
    Issue #3115021 by TR: Inject datetime.time service
TR’s picture

TR CreditAttribution: TR commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

zietbukuel made their first commit to this issue’s fork.