An overeager patch replaced time() with $_SERVER['REQUEST_TIME']. I have replaced that mistake with a call into the timer() API which was made exactly for this use case.

CommentFileSizeAuthor
#1 mw.patch892 bytesmoshe weitzman
mw.patch890 bytesmoshe weitzman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
FileSize
mw.patch892 bytes

Just remembered about format_interval(). Now with even more Drupal.

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
Status: Needs review » Reviewed & tested by the community 
Test run duration: 1 min 17 sec

Works perfectly. Thanks Moshe.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Looks good. I asked about the \n that got removed but DamZ and moshe said it was ok.

Committed. Thanks!

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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