GDPR Privacy Page setting allows for Draft Pages

[garrett-eclipse]

Hello,

Currently the Privacy Policy page select provides both published and draft pages. If a draft is selected it would be nice to indicate it's status to the user from this page. Was thinking possibly just by having (draft) appended to the select option for any pages in draft, or (private) for private pages to avoid users selecting non-published pages and overlooking.

For myself when I clicked 'Create New Page' it updated the setting pointing it to the new draft page which resulted in the Privacy Policy link no longer being present on the confirmation pages. Would be nice to either not allow drafts be selected or indicate they're draft so you're aware you have to finalize and publish.

Hope that makes sense,
Thanks

[desrosj]

Moving to the new Privacy component.

[desrosj]

Marking Privacy change as introduced in 4.9.6.

[desrosj]

I think this makes sense. I like including the status in parenthesis next to the page in the dropdown.

I am thinking a bit more about the second part, though. I think that a standard admin notice when editing the Privacy Policy page may be sufficient. We could consider a dismissible notice everywhere in the admin for users that are allowed to edit the Privacy Policy informing them the site's privacy policy is unpublished as well.

[garrett-eclipse]

Thanks @desrosj, I do like that idea of the global dismissable notice about the privacy policy being unpublished I think that'll help admins.

[allendav]

To test:

• publish one or more pages and leave one or more pages as draft
• ensure draft pages show "(Draft)" next to their titles in the dropdown on Settings > Privacy
• select or create a draft privacy policy page
• ensure you are prompted throughout wp-admin to publish that page or choose a different one
• on the privacy page edit page itself, note the wording and links changes slightly
• on the settings > privacy page, also note the wording and links change slightly

Are we really sure we want this throughout wp-admin? Seems excessive. May I recommend we consider limiting this a bit?

[allendav]

Added is-dismissible

[desrosj]

Thought about this more when addressing #44131 and opened #44320 to consider the various notifications related to privacy policy pages as a whole. I am not opposed to merging the notice in with this ticket, but we should at least get some UX feedback.

[azaozz]

As discussed in Slack: ​https://wordpress.slack.com/archives/C9695RJBW/p1529335496000432 perhaps better to only add the - (Draft) in this ticket, and move the new notification to a more general UI/UX ticket where all privacy notifications can be looked at as a whole.

[desrosj]

44100.3.diff​ removes the notification and only adds '(Draft)` to the end of draft page titles in the dropdown. Adding a notification should be explored in #44320 so it can get proper UI/UX attention.

Other improvements:

• Renamed _wp_privacy_settings_screen_filter_pages() to _wp_privacy_settings_filter_draft_page_titles() to better convey what the function does.
• Adds @access, @param, and @return tags to the _wp_privacy_settings_filter_draft_page_titles() function.

One thing I am not sure of is adding a filter and then immediately removing it. I don't know if this is done anywhere else in core.

[azaozz]

Replying to desrosj:

One thing I am not sure of is adding a filter and then immediately removing it.

That's fine, it is a "contextual filtering" for wp_dropdown_pages() that is needed only when outputting the drop-down on the Privacy screen.

Edit: actually, maybe better to move that and use get_current_screen()->id. Cleaner to keep all add_filter() in admin-filters.php.

[azaozz]

In 43376:

Privacy: append (Draft) to draft page titles in the page drop-down on the Privacy Settings screen.

Props allendav, desrosj.
Fixes #44100.

[azaozz]

Reopen for 4.9.7.

[ocean90]

4.9.7 has been released, moving to next milestone.

[SergeyBiryukov]

In 43454:

Privacy: Change @since entry for _wp_privacy_settings_filter_draft_page_titles() added in [43376] to 4.9.8.

See #44100.

[SergeyBiryukov]

In 43455:

Privacy: append (Draft) to draft page titles in the page drop-down on the Privacy Settings screen.

Props allendav, desrosj.
Merges [43376] and [43454] to the 4.9 branch.
Fixes #44100.