Problem/Motivation

As the getTimeValues use the t() function and this function is a wrapper around creating a new TranslatableMarkup() object, which in turn uses the string_translation service the method should not be in the Time utility class.

Read more about in How to shadow a core function used in two different tests files?

Proposed resolution

Transfer the getTimeValues method to the node_revision_delete service

Remaining tasks

None.

User interface changes

None.

API changes

The getTimeValues method is now available in the node_revision_delete service and will not be more accessible in the Time utility class.

Data model changes

None.

Comments

adriancid created an issue. See original summary.

  • adriancid committed 38ccc90 on 8.x-1.x
    Issue #2992133 by adriancid, Berdir: Transfer the getTimeValues method...

adriancid credited Berdir.

adriancid’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

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