Missing class identifier for password protected posts

[mdgl]

Password protected posts are insufficiently identified for good CSS styling.

Presently, the class name "post-password-required" is added to protected posts if a valid password cannot be found in the cookie. On the other hand, no special class name is added to protected posts if a valid password *can* be found, even though this is just a temporary condition until the cookie expires.

I suggest that both cases are interesting from a CSS styling perspective. Note that the title of protected posts is prefixed by the string "Protected:" in both situations.

This behaviour occurs because get_post_class() just checks post_password_required() [which validates the cookie] but get_the_title() just checks if the $post->post_password field is not empty.

I suggest an additional class name should be added by get_post_class() for the case when the post->post_password field is not empty. Current usage of class names here is not very consistent with the admin tool, so options might be "visibility-protected", "status-protected" [similar to "status-private" which is added for those with visibility "Private" and reflecting the underlying implementation] or perhaps just simply "protected".

Protected posts would thus be identified as follows:

Protected, with correct password: class="protected"
Protected, missing or invalid password: class="protected post-password-required"

See also #13860 and #18729. I can do a patch if needed but this is literally just a couple of lines of code (three if you include a comment).

[mdgl]

Suggested patch using class name "protected" (this is against 3.4.2 not trunk - sorry!)

[F4rkie]

Refreshed patch to trunk

[obenland]

While "interesting from a CSS styling perspective" is not the strongest reason to add it, a case could be made for consistency with status classes.

They should probably entail all three visibility statuses, public, protected, and private. I also like the idea about prefixing them with visibility-.

[wonderboymusic]

21899.2.diff​ refreshed to include all 3

[obenland]

Looking at it a bit more I changed my mind about the three classes. visibility-private and visibility-public would be duplicates of status-private and status-publish which doesn't really add much value.

21899.3.diff​ just adds one class post-password-protected in case the password is set but not required (to go with post-password-required). It also splits out the post thumbnail check to make it easier to grok.

[obenland]

In 34881:

Posts: Add post class for password protected posts.

Up until now password protected posts would only get a identifying class when
the password was still required. This adds a class for the case when that
requirement has been met.

For the other visibility types we already have status-publish
and status-private.

Props mdgl, F4rkie, wonderboymusic for initial patches.
Fixes #21899.