WordPress.org

Make WordPress Core

Opened 5 years ago

Last modified 9 months ago

#31040 new defect (bug)

When changing post date from future date to another future date, status being wrongly changed to "Publish on".

Reported by: tyxla Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.0
Component: Administration Keywords: has-patch has-screenshots
Focuses: ui, javascript Cc:

Description

Steps to reproduce:

  1. Create a new post, make it "Scheduled" for a future date.
  2. Save the post.
  3. Change the date to a different date in the future.
  4. The status text is changed to "Publish on" (or "Published on" on older WordPress versions), while it should remain "Scheduled".

Attachments (2)

31040.patch (608 bytes) - added by tyxla 5 years ago.
Fixing the issue by removing the unnecessary post status check.
31040.jpg (36.7 KB) - added by tyxla 4 years ago.
Screenshot of the issue

Download all attachments as: .zip

Change History (11)

#1 @tyxla
5 years ago

Related ticket: #12751 - applies a similar fix (on the backend side), but for the "publish" post status.

Last edited 5 years ago by SergeyBiryukov (previous) (diff)

#2 @tyxla
5 years ago

  • Keywords needs-patch added

@tyxla
5 years ago

Fixing the issue by removing the unnecessary post status check.

#3 @tyxla
5 years ago

  • Keywords has-patch needs-testing added; needs-patch removed

#5 in reply to: ↑ description @sdavis2702
4 years ago

I'm having trouble replicating this, @tyxla.

The status text is changed to "Publish on" (or "Published on" on older WordPress versions), while it should remain "Scheduled".

Where should I be seeing the text change? I looked all around and don't see it. I'm on 4.4-alpha. A screenshot would be perfect. I'll test the patch but I need to replicate first. Thanks!

#6 @tyxla
4 years ago

@sdavis2702 Thanks for your interest!

I am still able to reproduce the issue.

In order to reproduce, in step 3, after changing the date to a different future date, click the "OK" button.

Then observe the text that appears before the date - it changes from "Scheduled for:" to "Published on:".

I'll attach a screenshot of the issue to help you locate it.

Also, the patch still applies cleanly and fixes the issue for me when I'm testing.

@tyxla
4 years ago

Screenshot of the issue

#7 @tyxla
4 years ago

  • Keywords has-screenshots added

#8 @sdavis2702
4 years ago

Ah! Good catch there. That was hard to notice. The patch works perfectly for me on 4.4-beta3-20151105.

#9 @tyxla
4 years ago

  • Keywords needs-testing removed

Great, thanks for testing :)

Note: See TracTickets for help on using tickets.