Make WordPress Core

Opened 10 years ago

Closed 2 years ago

Last modified 2 years ago

#31040 closed defect (bug) (fixed)

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

Reported by: tyxla's profile tyxla Owned by: joedolson's profile joedolson
Milestone: 6.3 Priority: normal
Severity: minor Version: 3.0
Component: Editor Keywords: has-patch has-screenshots commit
Focuses: ui, accessibility, 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 (3)

31040.patch (608 bytes) - added by tyxla 10 years ago.
Fixing the issue by removing the unnecessary post status check.
31040.jpg (36.7 KB) - added by tyxla 9 years ago.
Screenshot of the issue
31040.1.diff (672 bytes) - added by joedolson 2 years ago.
Refreshed patch

Download all attachments as: .zip

Change History (20)

#1 @tyxla
10 years ago

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

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

#2 @tyxla
10 years ago

  • Keywords needs-patch added

@tyxla
10 years ago

Fixing the issue by removing the unnecessary post status check.

#3 @tyxla
10 years ago

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

#5 in reply to: ↑ description @sdavis2702
9 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
9 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
9 years ago

Screenshot of the issue

#7 @tyxla
9 years ago

  • Keywords has-screenshots added

#8 @sdavis2702
9 years ago

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

#9 @tyxla
9 years ago

  • Keywords needs-testing removed

Great, thanks for testing :)

#10 @joedolson
2 years ago

  • Milestone set to 6.3
  • Owner set to joedolson
  • Status changed from new to accepted

@joedolson
2 years ago

Refreshed patch

#11 @joedolson
2 years ago

  • Keywords commit added

For such a small change, this has been lingering way too long! Patch looks good.

#12 @joedolson
2 years ago

  • Component changed from Administration to Editor

#13 @joedolson
2 years ago

  • Focuses accessibility added

Marking this as accessibility, as clarity and consistency of instructions is important for cognitive impairments.

This ticket was mentioned in PR #4249 on WordPress/wordpress-develop by @joedolson.


2 years ago
#14

In the classic editor, fix the publish status change text shown when a future post's date is changed to a different future date. Show 'scheduled' statement instead of 'publish' statement.

Trac ticket: https://core.trac.wordpress.org/ticket/31040

#15 @joedolson
2 years ago

  • Severity changed from normal to minor

#16 @joedolson
2 years ago

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

In 55561:

Editor: Show scheduled text when changing to new future dates.

In the classic editor, fix the publish status change text shown when a future post's date is changed to a different future date. Show 'scheduled' statement instead of 'publish' statement.

Props tyxla, sdavis2702, joedolson.
Fixes #31040.

@joedolson commented on PR #4249:


2 years ago
#17

Fixed in r55561

Note: See TracTickets for help on using tickets.