Make WordPress Core

Opened 13 years ago

Last modified 5 years ago

#18264 new defect (bug)

Future private posts listed as already published with future date specified

Reported by: chrisrudzki's profile chrisrudzki Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Posts, Post Types Keywords: needs-patch editorial-flow
Focuses: ui, administration Cc:

Description

Setting a post to private, and scheduling it for some future date or time, results in the stamp that the post was "Published on (future date)" in the publish module.

The discussion on tickets #5608 and #9136 suggests that all private posts are published immediately (and privately), and that this is intentional. So, it's misleading and confusing to report that it already was published on some future date.

In source:trunk/wp-admin/includes/meta-boxes.php#L163, I suggest we replace:

else if ( 'publish' == $post->post_status || 'private' == $post->post_status ) { // already published
		$stamp = __('Published on: <b>%1$s</b>'); }

with:

else if ( 'publish' == $post->post_status ) { // already publicly published
		$stamp = __('Published on: <b>%1$s</b>');
	} else if (  'private' == $post->post_status ) { // published privately
		$stamp = __('Published privately'); }

Attachments (1)

18264.png (71.1 KB) - added by batcatforever 6 years ago.
Current User Flow

Download all attachments as: .zip

Change History (22)

#1 @chrisrudzki
13 years ago

  • Keywords needs-patch added

#2 @nacin
13 years ago

  • Keywords ux-feedback ui-feedback added

Yeah, this is a UI issue. A post cannot be both future-scheduled and privately published.

#3 @jane
13 years ago

I think the whole concept of Privately Published is wrong, and we should re-evaluate it completely. The way that language is used on wordpress.com (private to the group of your choosing) makes a lot more sense. Saving content so it's viewable only to author/admin is more like a private save. If no one else can view it, it's not really published in the normal sense, right?

#4 @scribu
13 years ago

I just ran into this exact issue myself. Ideally, we would deprecate the "private" status altogether and leave it to plugins to implement "publish to group" or whatever else they need using taxonomies.

#5 @scribu
13 years ago

My quick fix for not showing future private posts on the front-end:

function fix_future_private( $sql, $wp_query ) {
	global $wpdb;

	$sql .= $wpdb->prepare( " AND $wpdb->posts.post_date <= %s", current_time('mysql') );

	return $sql;
}

if ( !is_admin() )
	add_filter( 'posts_where', 'fix_future_private', 10, 2 );
Last edited 13 years ago by scribu (previous) (diff)

#7 @kovshenin
12 years ago

  • Cc kovshenin added

#8 @danielbachhuber
12 years ago

  • Keywords editorial-flow added

#9 follow-up: @danielbachhuber
12 years ago

  • Cc danielbachhuber added

@kovshenin maybe we can assess this as a part of editorial flow user testing and make a decision what should be done. Personally, I think "Privately Published" is confusing but I don't know that deprecating it is the right answer.

It's also odd that the "View Page" link doesn't show:

https://www.evernote.com/shard/s2/sh/19ee21df-43b5-44d0-962c-5b6bb5c0cf47/e85c6956632df3e9806fc488c97247e0

#10 in reply to: ↑ 9 @kovshenin
12 years ago

@danielbachhuber I agree, though I don't think deprecating the post status is the way to go. I think it can live, but perhaps it should become like any other post status, and not a UI of its own. I also think that scheduling a transition from one post status to another should work, regardless what the two statuses are, so the new statuses API (as discussed) should be flexible enough to support this.

#11 @jeremyfelt
11 years ago

  • Component changed from Administration to Posts, Post Types
  • Focuses admin added

#12 @chriscct7
9 years ago

  • Focuses ui added

This ticket was mentioned in Slack in #design by karmatosed. View the logs.


7 years ago

#14 @karmatosed
7 years ago

  • Keywords needs-design added

I agree this should be fixed, we discussed as a team in queue triage today. It would be great to have a visual for this so adding tag for 'needs-design'.

This ticket was mentioned in Slack in #design by boemedia. View the logs.


6 years ago

This ticket was mentioned in Slack in #design by joshuawold. View the logs.


6 years ago

This ticket was mentioned in Slack in #design by michelemiz. View the logs.


6 years ago

@batcatforever
6 years ago

Current User Flow

This ticket was mentioned in Slack in #design by joshuawold. View the logs.


6 years ago

This ticket was mentioned in Slack in #design by boemedia. View the logs.


5 years ago

#20 @ibdz
5 years ago

This was discussed during design triage.

The flow for private and public posts should be the same. After updating, if it’s not published yet, it should remain "Scheduled", whether it's a private post or not.

So the status has two part: public or private, and scheduled or published or draft.

This seems like it needs a patch, not more design.

#21 @karmatosed
5 years ago

  • Keywords ux-feedback ui-feedback needs-design removed

I am going to remove the design keywords as yes I agree with you @ibdz so let's make sure this has the right keywords. Thanks for triaging!

Note: See TracTickets for help on using tickets.