Make WordPress Core

Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#23065 closed defect (bug) (fixed)

Post Becomes Unprotected (Password) Without Explicit Permission

Reported by: AKTed Owned by: SergeyBiryukov
Priority: normal Milestone: 3.6
Component: Editor Version: 2.7
Severity: major Keywords: has-patch commit editorial-flow
Cc: Focuses:

Description

When you edit a post that's password protected and click 'Edit' next to 'Visibility: Password Protected', if you click 'Cancel' (still shows as 'Password Protected') then subsequently 'Update' it will change the visibility to 'Public'.

If you look closely, right after you click 'Cancel', while the section is collapsing, you'll see the password disappear. But it still says 'Visibility: Password Protected'.

Attachments (2)

patch_commit_1bf7f3273cb1.patch (773 bytes ) - added by thomasvanderbeek 14 years ago.
Fixed #23065
patch_commit_f941f7f69a79.patch (2.2 KB ) - added by thomasvanderbeek 14 years ago.
This patch is more consistent with the rest of the code conventions.

Download all attachments as: .zip

Change History (14)

#1 @miqrogroove
14 years ago

  • Component GeneralEditor
  • Version 3.53.4.2

Tested and confirmed in both 3.5 and 3.4.2.

#2 @thomasvanderbeek
14 years ago

  • Keywords has-patch added
  • Resolutionfixed
  • Status newclosed

#3 @SergeyBiryukov
14 years ago

  • Milestone Awaiting Review3.6
  • Resolution fixed
  • Status closedreopened

Tickets are marked as fixed when a change is made to the WordPress codebase which fixes the issue.

#4 @MikeHansenMe
14 years ago

  • Cc mdhansen@… added

Tested the patch on 3.6-alpha-23223. Fixes typo, correcting the problem.

#5 @nacin
14 years ago

  • Version 3.4.22.8

I tracked this down to at least 2.8. hidden_post_password is the name, hidden-post-password appears to have always been the ID.

#6 @SergeyBiryukov
14 years ago

  • Keywords commit added
  • Version 2.82.7

Introduced in [9639]. patch_commit_1bf7f3273cb1.patch looks proper.

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

@thomasvanderbeek
14 years ago

This patch is more consistent with the rest of the code conventions.

#7 @kovshenin
14 years ago

  • Cc kovshenin added
  • Keywords editorial-flow added

#8 @ocean90
14 years ago

  • Cc ocean90 added

#9 @SergeyBiryukov
14 years ago

patch_commit_f941f7f69a79.patch would also require a change in CSS:
http://core.trac.wordpress.org/browser/tags/3.5/wp-admin/css/wp-admin.css#L3228

I guess it's better to leave post_password ID as is, for backwards compatibility reasons. A similar change was suggested for Dashboard in #17555.

patch_commit_1bf7f3273cb1.patch is enough to fix the issue.

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

#10 @SergeyBiryukov
14 years ago

  • Owner set to SergeyBiryukov
  • Resolutionfixed
  • Status reopenedclosed

In 23338:

Prevent an unintended visibility change from "Password protected" to "Public" when updating the post.

props thomasvanderbeek. fixes #23065.

#11 @SergeyBiryukov
13 years ago

#23633 was marked as a duplicate.

#12 @helen
13 years ago

#23633 was marked as a duplicate.

Note: See TracTickets for help on using tickets.