Changeset 2213940

Timestamp:

12/17/2019 11:31:34 PM (6 years ago)

Author:

pressupinc

Message:

deploy from git

Location:

require-featured-image/trunk

Files:

• readme.txt (modified) (2 diffs)
• require-featured-image.php (modified) (2 diffs)

require-featured-image/trunk/readme.txt

@@ -4 +4 @@
 Tags: featured image, images, edit, post, admin, require featured image, image, media, thumbnail, thumbnails, post thumbnail, photo, pictures
 Requires at least: 3.5
-Tested up to: 5.2.1
-Stable tag: 1.4.1
+Tested up to: 5.3.1
+Stable tag: 1.5.0
 License: MIT
 License URI: http://opensource.org/licenses/MIT
@@ -70 +70 @@
 
 == CHANGELOG ==
+
+= 1.5.0 (2019.12.17) =
+* It was impossible to publish a post (which was never saved as a "Draft") because of the server-side check. It is no longer.
 
 = 1.4.1 (2019.05.29) =

require-featured-image/trunk/require-featured-image.php

@@ -5 +5 @@
 Description: Like it says on the tin: requires posts to have a featured image set before they'll be published.
 Author: Press Up
-Version: 1.4.1
+Version: 1.5.0
 Author URI: http://pressupinc.com
 Text Domain: require-featured-image
@@ -14 +14 @@
 add_action( 'transition_post_status', 'rfi_guard', 10, 3 );
 function rfi_guard( $new_status, $old_status, $post ) {
+    if ( isset($_GET['_locale']) && $_GET['_locale'] == 'user' ) {
+        return;
+        /* EXPLANATION: The Block/Gutenberg editor works differently than classic,
+         *  especially when a user has a a new post they're seeking to see published
+         *  where the Featured Image wasn't already saved to a draft. Best I can tell
+         *  in that condition they'll always have this weird `_locale=user` set on the URL
+         *  so a quick-and-dirty hack on not enforcing on that post transition is going on
+         *  here. Should probably have more expert eyes find a better solution than this.
+         */
+    }
     if ( $new_status === 'publish' && rfi_should_stop_post_publishing( $post ) ) {
         // transition_post_status comes after the post has changed statuses, so we must roll back here