Changeset 54563 for branches/5.2/src/wp-includes/post.php

Timestamp:

10/17/2022 06:08:00 PM (2 years ago)

Author:

audrasjb

Message:

Grouped backports to the 5.2 branch.

• Editor: Bump @wordpress packages for the branch,
• Media: Refactor search by filename within the admin,
• REST API: Lockdown post parameter of the terms endpoint,
• Customize: Escape blogname option in underscores templates,
• Query: Validate relation in WP_Date_Query,
• Posts, Post types: Apply KSES to post-by-email content,
• General: Validate host on "Are you sure?" screen,
• Posts, Post types: Remove emails from post-by-email logs,
• Pings/trackbacks: Apply KSES to all trackbacks,
• Mail: Reset PHPMailer properties between use,
• Comments: Apply kses when editing comments,
• Widgets: Escape RSS error messages for display.

Merges [54521-54530] to the 5.2 branch.
Props audrasjb, costdev, cu121, dd32, davidbaumwald, ehtis, johnbillion, johnjamesjacoby, martinkrcho, matveb, oztaser, paulkevan, peterwilsoncc, ravipatel, SergeyBiryukov, talldanwp, timothyblynjacobs, tykoted, voldemortensen, vortfu, xknown.

Location:

branches/5.2

Files:

• . (modified) (1 prop)
• src/wp-includes/post.php (modified) (4 diffs)

branches/5.2/src/wp-includes/post.php

@@ -1884 +1884 @@
  * @since 4.5.0 Added the ability to pass a post type name in addition to object.
  * @since 4.6.0 Converted the `$post_type` parameter to accept a `WP_Post_Type` object.
+ * @since 5.9.0 Added `is_post_type_viewable` hook to filter the result.
  *
  * @param string|WP_Post_Type $post_type Post type name or object.
@@ -1891 +1892 @@
     if ( is_scalar( $post_type ) ) {
         $post_type = get_post_type_object( $post_type );
+
         if ( ! $post_type ) {
             return false;
@@ -1896 +1898 @@
     }
 
-    return $post_type->publicly_queryable || ( $post_type->_builtin && $post_type->public );
+    if ( ! is_object( $post_type ) ) {
+        return false;
+    }
+
+    $is_viewable = $post_type->publicly_queryable || ( $post_type->_builtin && $post_type->public );
+
+    /**
+     * Filters whether a post type is considered "viewable".
+     *
+     * The returned filtered value must be a boolean type to ensure
+     * `is_post_type_viewable()` only returns a boolean. This strictness
+     * is by design to maintain backwards-compatibility and guard against
+     * potential type errors in PHP 8.1+. Non-boolean values (even falsey
+     * and truthy values) will result in the function returning false.
+     *
+     * @since 5.9.0
+     *
+     * @param bool         $is_viewable Whether the post type is "viewable" (strict type).
+     * @param WP_Post_Type $post_type   Post type object.
+     */
+    return true === apply_filters( 'is_post_type_viewable', $is_viewable, $post_type );
+}
+
+/**
+ * Determines whether a post status is considered "viewable".
+ *
+ * For built-in post statuses such as publish and private, the 'public' value will be evaluated.
+ * For all others, the 'publicly_queryable' value will be used.
+ *
+ * @since 5.7.0
+ * @since 5.9.0 Added `is_post_status_viewable` hook to filter the result.
+ *
+ * @param string|stdClass $post_status Post status name or object.
+ * @return bool Whether the post status should be considered viewable.
+ */
+function is_post_status_viewable( $post_status ) {
+    if ( is_scalar( $post_status ) ) {
+        $post_status = get_post_status_object( $post_status );
+
+        if ( ! $post_status ) {
+            return false;
+        }
+    }
+
+    if (
+        ! is_object( $post_status ) ||
+        $post_status->internal ||
+        $post_status->protected
+    ) {
+        return false;
+    }
+
+    $is_viewable = $post_status->publicly_queryable || ( $post_status->_builtin && $post_status->public );
+
+    /**
+     * Filters whether a post status is considered "viewable".
+     *
+     * The returned filtered value must be a boolean type to ensure
+     * `is_post_status_viewable()` only returns a boolean. This strictness
+     * is by design to maintain backwards-compatibility and guard against
+     * potential type errors in PHP 8.1+. Non-boolean values (even falsey
+     * and truthy values) will result in the function returning false.
+     *
+     * @since 5.9.0
+     *
+     * @param bool     $is_viewable Whether the post status is "viewable" (strict type).
+     * @param stdClass $post_status Post status object.
+     */
+    return true === apply_filters( 'is_post_status_viewable', $is_viewable, $post_status );
+}
+
+/**
+ * Determines whether a post is publicly viewable.
+ *
+ * Posts are considered publicly viewable if both the post status and post type
+ * are viewable.
+ *
+ * @since 5.7.0
+ *
+ * @param int|WP_Post|null $post Optional. Post ID or post object. Defaults to global $post.
+ * @return bool Whether the post is publicly viewable.
+ */
+function is_post_publicly_viewable( $post = null ) {
+    $post = get_post( $post );
+
+    if ( ! $post ) {
+        return false;
+    }
+
+    $post_type   = get_post_type( $post );
+    $post_status = get_post_status( $post );
+
+    return is_post_type_viewable( $post_type ) && is_post_status_viewable( $post_status );
 }
 
@@ -6864 +6958 @@
 
 /**
- * Filter the SQL clauses of an attachment query to include filenames.
- *
- * @since 4.7.0
- * @access private
- *
- * @global wpdb $wpdb WordPress database abstraction object.
- *
- * @param array $clauses An array including WHERE, GROUP BY, JOIN, ORDER BY,
- *                       DISTINCT, fields (SELECT), and LIMITS clauses.
- * @return array The modified clauses.
- */
-function _filter_query_attachment_filenames( $clauses ) {
-    global $wpdb;
-    remove_filter( 'posts_clauses', __FUNCTION__ );
-
-    // Add a LEFT JOIN of the postmeta table so we don't trample existing JOINs.
-    $clauses['join'] .= " LEFT JOIN {$wpdb->postmeta} AS sq1 ON ( {$wpdb->posts}.ID = sq1.post_id AND sq1.meta_key = '_wp_attached_file' )";
-
-    $clauses['groupby'] = "{$wpdb->posts}.ID";
-
-    $clauses['where'] = preg_replace(
-        "/\({$wpdb->posts}.post_content (NOT LIKE|LIKE) (\'[^']+\')\)/",
-        '$0 OR ( sq1.meta_value $1 $2 )',
-        $clauses['where']
-    );
-
-    return $clauses;
-}
-
-/**
  * Sets the last changed time for the 'posts' cache group.
  *