[wp-trac] [WordPress Trac] #47223: Site Health Check: "last missed cron" test too aggressive

[WordPress Trac]

#47223: Site Health Check: "last missed cron" test too aggressive
-------------------------------------------------+---------------------
 Reporter:  DavidAnderson                        |       Owner:  (none)
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  5.3
Component:  Site Health                          |     Version:  5.2
 Severity:  normal                               |  Resolution:
 Keywords:  site-health needs-testing has-patch  |     Focuses:
-------------------------------------------------+---------------------

Comment (by rockfire):

 Replying to [comment:25 SergeyBiryukov]:
 > Haven't fully reviewed the patch yet, just noted that comment:13
 suggests checking for `has_late_cron()` first and then for
 `has_missed_cron()`, while the latest patch does it vice versa. Is that
 intentional?

 Yes that is intentional. It first checks the worst-case-scenario (a missed
 cron) and if that is the case there is no need to check for the late cron.
 Only if there is no missed cron we check for a late cron.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/47223#comment:26>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform