Problem/Motivation

A time service was added to 8.3.x in #2717207: Add a time service to provide consistent interaction with time()/microtime() and superglobals.

Core code should be updated to remove deprecated uses of REQUEST_TIME and time() and others.

For more informations, see the change record.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

AndrewsizZ created an issue. See original summary.

AndrewsizZ’s picture

AndrewsizZ’s picture

AndrewsizZ’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 3: Replace-REQUEST_TIME-with-service-3187653-1.patch, failed testing. View results

swentel’s picture

Status: Needs work » Needs review
FileSize
2.23 KB

should be better

Status: Needs review » Needs work

The last submitted patch, 6: 3187653-6.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

swentel’s picture

Status: Needs work » Needs review
FileSize
3.74 KB

Further fixes

  • swentel committed b0277ad on 8.x-3.x
    Issue #3187653 by swentel, AndrewsizZ: Replace REQUEST_TIME in plugins...
swentel’s picture

Status: Needs review » Fixed

committed and pushed.

swentel’s picture

Status: Fixed » Closed (fixed)