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.

CommentFileSizeAuthor
#8 3187653-8.patch3.74 KBswentel
#6 3187653-6.patch2.23 KBswentel
#3 Replace-REQUEST_TIME-with-service-3187653-1.patch1.74 KBandrewsizz

Comments

AndrewsizZ created an issue. See original summary.

andrewsizz’s picture

andrewsizz commented
Related issues: +#2902895: [meta][no patch] Replace uses of REQUEST_TIME and time() with time service
andrewsizz’s picture

andrewsizz commented
StatusFileSize
new Replace-REQUEST_TIME-with-service-3187653-1.patch1.74 KB
andrewsizz’s picture

andrewsizz commented
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

swentel commented
Status: Needs work » Needs review
StatusFileSize
new 3187653-6.patch2.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

swentel commented
Status: Needs work » Needs review
StatusFileSize
new 3187653-8.patch3.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

swentel commented
Status: Needs review » Fixed

committed and pushed.

swentel’s picture

swentel commented
Status: Fixed » Closed (fixed)