Ticket #8978 (closed defect (bug): fixed)

Opened 3 years ago

Last modified 3 years ago

Drafts are published in the past

Reported by: markjaquith Owned by: ryan
Priority: high Milestone: 2.7.1
Component: Administration Version: 2.7
Severity: critical Keywords: has-patch needs-testing
Cc:

Description

  1. Save a draft.
  2. Leave it alone for a few days (or modify its post_date to be a few days in the past).
  3. Go back to the post and publish it (or just view the timestamp). You'll find it's in the past, instead of the current time, even though you've not specified a publish time.

Attachments

8978.001.diff Download (672 bytes) - added by markjaquith 3 years ago.

Change History

markjaquith — 3 years ago

comment:1   markjaquith — 3 years ago

  • Keywords has-patch needs-testing added

I think I know what's going on... touch_time() was expecting post_date to be all zeros for drafts and pending posts. But now it seems that post_date equals the last modified date. 8978.001.diff switches that check to post_date_gmt, which seems to stay all zeros when a date is not specified.

comment:2   markjaquith — 3 years ago

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

(In [10451]) Drafts have post_date populated now, so look for a zeroed out post_date_gmt to determine non-scheduled nature. fixes #8978 for trunk

comment:3   markjaquith — 3 years ago

(In [10452]) Drafts have post_date populated now, so look for a zeroed out post_date_gmt to determine non-scheduled nature. fixes #8978 for 2.7

comment:4   ryan — 3 years ago

  • Milestone changed from 2.8 to 2.7.1
Note: See TracTickets for help on using tickets.