WordPress.org

Make WordPress Core

#37554 closed defect (bug) (fixed)

update last checked time does not match the system time

Reported by: ctienshi Owned by: swissspidy
Milestone: 4.7 Priority: normal
Severity: normal Version: 3.1
Component: Upgrade/Install Keywords: has-screenshots has-patch commit
Focuses: ui Cc:

Description

when checking for a new update it doesn't modify the last checked time correctly. I think it is a bug.

Attachments (4)

Screen Shot 2016-08-03 at 10.42.24 PM.png (70.2 KB) - added by ctienshi 17 months ago.
i checked for updates at 10.42pm last. But it shows as 5.09pm.
37554.patch (995 bytes) - added by PieWP 17 months ago.
possible patch for showing proper 'version check' time
37554.diff (1.0 KB) - added by swissspidy 16 months ago.
37554.2.diff (956 bytes) - added by swissspidy 15 months ago.

Download all attachments as: .zip

Change History (16)

@ctienshi
17 months ago

i checked for updates at 10.42pm last. But it shows as 5.09pm.

#1 in reply to: ↑ description @ctienshi
17 months ago

  • Focuses ui added
  • Keywords needs-patch dev-feedback has-screenshots added

Replying to ctienshi:

when checking for a new update it doesn't modify the last checked time correctly. I think it is a bug.

#2 follow-up: @PieWP
17 months ago

Hey ctienshi welcome to Trac! Thank you for submitting this report.

It is indeed correct that the 'last checked' time is not shown here. The version check result is stored for 60 seconds in the cache of WordPress before it could be checked again, so in theory the timer could be 60 seconds off. This however does not explain the offset shown in your screenshot.

Could you please check wether the same issue rises when you are creating a new post? And can you maybe check your timezone settings in the general settings under the 'Settings' menu item.

Eager to hear from you.

@PieWP
17 months ago

possible patch for showing proper 'version check' time

#3 @SergeyBiryukov
17 months ago

  • Component changed from Date/Time to Upgrade/Install

#4 in reply to: ↑ 2 @ctienshi
17 months ago

@PieWP The timezone was not set properly. It was the issue. Thanks for the information. I think it would be better if WordPress automatically got the system time and accordingly updated the timezone properly.

Last edited 17 months ago by ctienshi (previous) (diff)

#5 @PieWP
17 months ago

Great suggestion ctienshi, I'll start working on a patch

@swissspidy
16 months ago

#6 @swissspidy
16 months ago

  • Keywords has-patch added; needs-patch removed
  • Version changed from 4.5.3 to 3.1

The line in question on update-core.php simply prints the current time and is actually quite useless:

printf( __( 'Last checked on %1$s at %2$s.' ), date_i18n( __( 'F j, Y' ) ), date_i18n( __( 'g:i a' ) ) );

I would rather remove that information instead of fixing it.

37554.patch doesn't work as it doesn't respect the current timezone / GMT offset and passes true to date_i18n instead of false. 37554.diff fixes that. However, it's only a minor improvement as it's basically still the current date, but now can be 60 seconds off. That's probably why simply the current date was displayed (or perhaps the right data hasn't been available, idk).

Introduced in [15996].

#7 follow-up: @swissspidy
16 months ago

Oh, and regarding:

I think it would be better if WordPress automatically got the system time and accordingly updated the timezone properly.

You won't believe on how many systems the server time is horribly wrong or simply UTC. It's not reliable. I would rather use JavaScript to suggest a timezone during install or when changing settings. But that's for another ticket.

#8 @swissspidy
16 months ago

  • Milestone changed from Awaiting Review to Future Release

#9 in reply to: ↑ 7 @PieWP
16 months ago

Replying to swissspidy:

You won't believe on how many systems the server time is horribly wrong or simply UTC. It's not reliable. I would rather use JavaScript to suggest a timezone during install or when changing settings. But that's for another ticket.

True, I had a simular solution in mind. Set the timezone with PHP best effort in the installation process and auto correct it using javascript if enabled. Additionally automatically inheritance of the timezone once creating a new blog (MS installation) would be a nice to have as-well. If acceptable for a future release I'd gladly work out a patch for this.

Last edited 16 months ago by PieWP (previous) (diff)

#10 @swissspidy
15 months ago

  • Keywords dev-feedback removed
  • Milestone changed from Future Release to 4.7

37554.diff looks good for now. Just need to remove the var_dump().

@swissspidy
15 months ago

#11 @swissspidy
15 months ago

  • Keywords commit added

#12 @swissspidy
15 months ago

  • Owner set to swissspidy
  • Resolution set to fixed
  • Status changed from new to closed

In 38743:

Upgrade/Install: Show correct time of last checked update.

Props PieWP for initial patch.
Fixes #37554.

Note: See TracTickets for help on using tickets.