Changeset 7268

Timestamp:

06/28/2024 10:02:43 PM (13 months ago)

Author:

johnjamesjacoby

Message:

Merge r7250 through r7267 from branches/2.6.

In trunk, for 2.7.

Location:

trunk

Files:

• . (modified) (1 prop)
• bbpress.php (modified) (1 diff)
• package.json (modified) (1 diff)
• src/bbpress.php (modified) (1 diff)
• src/includes/admin/assets/css/admin.css (modified) (2 diffs)
• src/includes/admin/classes/class-bbp-admin.php (modified) (1 diff)
• src/includes/admin/classes/class-bbp-converter-db.php (modified) (1 diff)
• src/includes/admin/tools/repair.php (modified) (5 diffs)
• src/includes/common/functions.php (modified) (3 diffs)
• src/includes/core/capabilities.php (modified) (1 diff)
• src/includes/extend/akismet.php (modified) (1 diff)
• src/includes/forums/functions.php (modified) (2 diffs)
• src/includes/replies/functions.php (modified) (3 diffs)
• src/includes/replies/template.php (modified) (1 diff)
• src/includes/search/template.php (modified) (1 diff)
• src/includes/topics/functions.php (modified) (2 diffs)
• src/includes/topics/template.php (modified) (1 diff)
• src/readme.txt (modified) (1 diff)
• src/templates/default/bbpress/content-archive-topic.php (modified) (1 diff)
• src/templates/default/bbpress/content-search.php (modified) (2 diffs)
• src/templates/default/bbpress/feedback-no-search.php (modified) (2 diffs)
• src/templates/default/bbpress/form-allowed-tags.php (added)
• src/templates/default/bbpress/form-forum.php (modified) (1 diff)
• src/templates/default/bbpress/form-reply.php (modified) (1 diff)
• src/templates/default/bbpress/form-topic.php (modified) (1 diff)

trunk/bbpress.php

@@ -23 +23 @@
  * Domain Path:       /languages/
  * Requires PHP:      5.6.20
- * Requires at least: 5.0
+ * Requires at least: 6.0
+ * Tested up to:      6.5
  * Version:           2.7.0-alpha-2
  */

trunk/package.json

@@ -1 +1 @@
 {
     "name": "bbpress",
+    "author": "The bbPress Contributors",
     "author": "The bbPress Contributors",
     "description": "Forum software with a twist from the creators of WordPress.",

trunk/src/bbpress.php

@@ -23 +23 @@
  * Domain Path:       /languages/
  * Requires PHP:      5.6.20
- * Requires at least: 5.0
+ * Requires at least: 6.0
+ * Tested up to:      6.5
  * Version:           2.7.0-alpha-2
  */

trunk/src/includes/admin/assets/css/admin.css

@@ -1 +1 @@
 
-/* Kludge for too-wide forums dropdown */
+/* Kludge for too-wide inputs & selects */
 #poststuff #bbp_forum_attributes select#parent_id,
+#poststuff #bbp_forum_attributes input#menu_order,
 #poststuff #bbp_topic_attributes select#parent_id,
 #poststuff #bbp_reply_attributes select#bbp_forum_id,
-#poststuff #bbp_reply_attributes select#bbp_reply_to {
+#poststuff #bbp_reply_attributes select#bbp_reply_to,
+#poststuff #bbp_reply_attributes input#bbp_forum_id,
+#poststuff #bbp_reply_attributes input#bbp_topic_id,
+#poststuff #bbp_author_metabox input#bbp_author_id,
+#poststuff #bbp_author_metabox input#bbp_author_ip_address {
     max-width: 170px;
 }
@@ -316 +321 @@
 }
 
+body.post-type-forum .wp-list-table thead th a,
+body.post-type-forum .wp-list-table tfoot th a,
+body.post-type-topic .wp-list-table thead th a,
+body.post-type-topic .wp-list-table tfoot th a,
+body.post-type-reply .wp-list-table thead th a,
+body.post-type-reply .wp-list-table tfoot th a {
+    display: flex;
+}
+
+body.post-type-forum .wp-list-table thead th:not(.sortable),
+body.post-type-forum .wp-list-table tfoot th:not(.sortable),
+body.post-type-forum .wp-list-table thead th a > span:first-child,
+body.post-type-forum .wp-list-table tfoot th a > span:first-child,
+body.post-type-forum .wp-list-table tbody td,
+body.post-type-topic .wp-list-table thead th:not(.sortable),
+body.post-type-topic .wp-list-table tfoot th:not(.sortable),
+body.post-type-topic .wp-list-table thead th a > span:first-child,
+body.post-type-topic .wp-list-table tfoot th a > span:first-child,
+body.post-type-topic .wp-list-table tbody td,
+body.post-type-reply .wp-list-table thead th:not(.sortable),
+body.post-type-reply .wp-list-table tfoot th:not(.sortable),
+body.post-type-reply .wp-list-table thead th a > span:first-child,
+body.post-type-reply .wp-list-table tfoot th a > span:first-child,
+body.post-type-reply .wp-list-table tbody td {
+    overflow-x: hidden;
+    text-overflow: ellipsis;
+    word-wrap: normal;
+}
+
 .column-bbp_forum_topic_count,
 .column-bbp_forum_reply_count,

trunk/src/includes/admin/classes/class-bbp-admin.php

@@ -82 +82 @@
      */
     public $notices = array();
+
+    /** Components ************************************************************/
+
+    /**
+     * @var BBP_Forums_Admin Forums admin
+     */
+    public $forums = null;
+
+    /**
+     * @var BBP_Topics_Admin Topics admin
+     */
+    public $topics = null;
+
+    /**
+     * @var BBP_Replies_Admin Replies admin
+     */
+    public $replies = null;
+
+    /**
+     * @var BBP_Converter Converter admin
+     */
+    public $converter = null;
 
     /** Functions *************************************************************/

trunk/src/includes/admin/classes/class-bbp-converter-db.php

@@ -31 +31 @@
      */
     public function __construct( $dbuser, $dbpassword, $dbname, $dbhost ) {
-        register_shutdown_function( array( $this, '__destruct' ) );
 
         if ( WP_DEBUG && WP_DEBUG_DISPLAY ) {

trunk/src/includes/admin/tools/repair.php

@@ -498 +498 @@
     $result      = esc_html__( 'Failed!', 'bbpress' );
 
-    $sql_select  = "SELECT `post_author`, COUNT(DISTINCT `ID`) as `_count` FROM `{$bbp_db->posts}` WHERE `post_type` = '" . bbp_get_topic_post_type() . "' AND `post_status` = '" . bbp_get_public_status_id() . "' GROUP BY `post_author`";
+    $sql_type    = bbp_get_topic_post_type();
+    $sql_status  = "'" . implode( "','", bbp_get_public_topic_statuses() ) . "'";
+    $sql_select  = "SELECT `post_author`, COUNT(DISTINCT `ID`) as `_count` FROM `{$bbp_db->posts}` WHERE `post_type` = '{$sql_type}' AND `post_status` IN ({$sql_status}) GROUP BY `post_author`";
     $insert_rows = $bbp_db->get_results( $sql_select );
 
@@ -542 +544 @@
 
     // Define variables
-    $bbp_db    = bbp_db();
+    $bbp_db      = bbp_db();
     $statement   = esc_html__( 'Counting the number of topics to which each user has replied… %s', 'bbpress' );
     $result      = esc_html__( 'Failed!', 'bbpress' );
 
-    $sql_select  = "SELECT `post_author`, COUNT(DISTINCT `ID`) as `_count` FROM `{$bbp_db->posts}` WHERE `post_type` = '" . bbp_get_reply_post_type() . "' AND `post_status` = '" . bbp_get_public_status_id() . "' GROUP BY `post_author`";
+    $sql_type    = bbp_get_reply_post_type();
+    $sql_status  = "'" . implode( "','", bbp_get_public_reply_statuses() ) . "'";
+    $sql_select  = "SELECT `post_author`, COUNT(DISTINCT `ID`) as `_count` FROM `{$bbp_db->posts}` WHERE `post_type` = '{$sql_type}' AND `post_status` IN ({$sql_status}) GROUP BY `post_author`";
     $insert_rows = $bbp_db->get_results( $sql_select );
 
@@ -602 +606 @@
     }
 
-    $topics = $bbp_db->get_col( "SELECT `ID` FROM `{$bbp_db->posts}` WHERE `post_type` = '" . bbp_get_topic_post_type() . "' AND `post_status` = '" . bbp_get_public_status_id() . "'" );
-
+    $sql_type   = bbp_get_topic_post_type();
+    $sql_status = "'" . implode( "','", bbp_get_public_topic_statuses() ) . "'";
+    $sql_select = "SELECT `ID` FROM `{$bbp_db->posts}` WHERE `post_type` = '{$sql_type}' AND `post_status` IN ({$sql_status})";
+
+    $topics = $bbp_db->get_col( $sql_select );
     if ( is_wp_error( $topics ) ) {
         return array( 2, sprintf( $statement, $result ) );
@@ -668 +675 @@
     }
 
-    $topics = $bbp_db->get_col( "SELECT `ID` FROM `{$bbp_db->posts}` WHERE `post_type` = '" . bbp_get_topic_post_type() . "' AND `post_status` = '" . bbp_get_public_status_id() . "'" );
+    $sql_type    = bbp_get_topic_post_type();
+    $sql_status  = "'" . implode( "','", bbp_get_public_topic_statuses() ) . "'";
+    $sql_select  = "SELECT `ID` FROM `{$bbp_db->posts}` WHERE `post_type` = '{$sql_type}' AND `post_status` IN ({$sql_status})";
+
+    $topics = $bbp_db->get_col( $sql_select );
     if ( is_wp_error( $topics ) ) {
         return array( 2, sprintf( $statement, $result ) );
@@ -733 +744 @@
     }
 
-    $forums = $bbp_db->get_col( "SELECT `ID` FROM `{$bbp_db->posts}` WHERE `post_type` = '" . bbp_get_forum_post_type() . "' AND `post_status` = '" . bbp_get_public_status_id() . "'" );
+    $sql_type    = bbp_get_forum_post_type();
+    $sql_status  = "'" . implode( "','", bbp_get_public_forum_statuses() ) . "'";
+    $sql_select  = "SELECT `ID` FROM `{$bbp_db->posts}` WHERE `post_type` = '{$sql_type}' AND `post_status` IN ({$sql_status})";
+
+    $forums = $bbp_db->get_col( $sql_select );
     if ( is_wp_error( $forums ) ) {
         return array( 2, sprintf( $statement, $result ) );

trunk/src/includes/common/functions.php

@@ -484 +484 @@
 
         // Declare empty arrays
-        $topics = $topic_titles = array_fill_keys( bbp_get_non_public_reply_statuses(), '' );
+        $replies = $reply_titles = array_fill_keys( bbp_get_non_public_reply_statuses(), '' );
 
         // Pending
@@ -517 +517 @@
 
         // Total hidden (pending, private, hidden, spam, trash)
-        $reply_count_hidden = array_sum( $replies );
+        $reply_count_hidden = array_sum( array_filter( $replies ) );
 
         // Compile the hidden replies title
@@ -2520 +2520 @@
                 $the_query = bbp_get_view_query_args( $view );
 
-                // Output the feed
-                bbp_display_topics_feed_rss2( $the_query );
+                // Output the feed if view exists
+                if ( ! empty( $the_query ) ) {
+                    bbp_display_topics_feed_rss2( $the_query );
+                }
             }
         }

trunk/src/includes/core/capabilities.php

@@ -286 +286 @@
 function bbp_add_forums_roles( $wp_roles = null ) {
 
+    // Maybe initialize WP_Roles just-in-time, but 
+    if ( empty( $wp_roles ) && ! doing_action( 'wp_roles_init' ) ) {
+        $wp_roles = wp_roles();
+    }
+
+    // Bail if unexpected param type
+    if ( ! is_a( $wp_roles, 'WP_Roles' ) ) {
+        return;
+    }
+
     // Get the dynamic roles
     $bbp_roles = bbp_get_dynamic_roles();

trunk/src/includes/extend/akismet.php

@@ -21 +21 @@
  */
 class BBP_Akismet {
+
+    /**
+     * The last post checked by Akismet.
+     *
+     * @since 2.0.0 bbPress (r3277)
+     *
+     * @var array $last_post Default empty array.
+     */
+    protected $last_post = array();
 
     /**

trunk/src/includes/forums/functions.php

@@ -2302 +2302 @@
     }
 
-    // Get query post types array .
-    $post_types = (array) $posts_query->get( 'post_type' );
+    // Bail to prevent unintended wp-admin post_row overrides
+    if ( is_admin() && isset( $_REQUEST['post_status'] ) ) {
+        return;
+    }
+
+    // Get query post types as an array.
+    $post_types = array_filter( (array) $posts_query->get( 'post_type' ) );
 
     // Forums
-    if ( bbp_get_forum_post_type() === implode( '', $post_types ) ) {
-
-        // Prevent accidental wp-admin post_row override
-        if ( is_admin() && isset( $_REQUEST['post_status'] ) ) {
-            return;
-        }
+    if ( in_array( bbp_get_forum_post_type(), $post_types, true ) ) {
 
         /** Default ***********************************************************/
@@ -2319 +2319 @@
 
         // Get forums to exclude
-        $hidden_ids = bbp_exclude_forum_ids( 'array' );
-
-        // Bail if no forums to exclude
-        if ( empty( $hidden_ids ) ) {
-            return;
-        }
-
-        // Get any existing meta queries
-        $not_in = $posts_query->get( 'post__not_in', array() );
-
-        // Add our meta query to existing
-        $not_in = array_unique( array_merge( $not_in, $hidden_ids ) );
-
-        // Set the meta_query var
-        $posts_query->set( 'post__not_in', $not_in );
+        $forum_ids = bbp_exclude_forum_ids( 'array' );
+
+        // Excluding some forums
+        if ( ! empty( $forum_ids ) ) {
+
+            // Get any existing not-in queries
+            $not_in = $posts_query->get( 'post__not_in', array() );
+
+            // Add our not-in to existing
+            $not_in = array_unique( array_merge( $not_in, $forum_ids ) );
+
+            // Set the new not-in val
+            $posts_query->set( 'post__not_in', $not_in );
+        }
+    }
 
     // Some other post type besides Forums, Topics, or Replies
-    } elseif ( ! array_diff( $post_types, bbp_get_post_types() ) ) {
+    if ( ! array_diff( $post_types, bbp_get_post_types() ) ) {
 
         // Get forums to exclude
         $forum_ids = bbp_exclude_forum_ids( 'meta_query' );
 
-        // Bail if no forums to exclude
-        if ( empty( $forum_ids ) ) {
-            return;
-        }
-
-        // Get any existing meta queries
-        $meta_query   = (array) $posts_query->get( 'meta_query', array() );
-
-        // Add our meta query to existing
-        $meta_query[] = $forum_ids;
-
-        // Set the meta_query var
-        $posts_query->set( 'meta_query', $meta_query );
+        // Excluding some forums
+        if ( ! empty( $forum_ids ) ) {
+
+            // Get any existing meta queries
+            $meta_query   = (array) $posts_query->get( 'meta_query', array() );
+
+            // Add our meta query to existing
+            $meta_query[] = $forum_ids;
+
+            // Set the new meta_query val
+            $posts_query->set( 'meta_query', $meta_query );
+        }
     }
 }

trunk/src/includes/replies/functions.php

@@ -641 +641 @@
     /** Reply Status **********************************************************/
 
+    // Get available reply statuses
+    $reply_statuses = bbp_get_reply_statuses( $reply_id );
+
     // Use existing post_status
     $reply_status = $reply->post_status;
@@ -647 +650 @@
     if ( bbp_is_reply_public( $reply_id ) && ! bbp_check_for_moderation( $anonymous_data, $reply_author, $reply_title, $reply_content ) ) {
         $reply_status = bbp_get_pending_status_id();
+
+    // Check for possible posted reply status
+    } elseif ( ! empty( $_POST['bbp_reply_status'] ) && in_array( $_POST['bbp_reply_status'], array_keys( $reply_statuses ), true ) ) {
+
+        // Allow capable users to explicitly override the status
+        if ( current_user_can( 'moderate', $reply_id ) ) {
+            $reply_status = sanitize_key( $_POST['bbp_reply_status'] );
+
+        // Not capable
+        } else {
+            bbp_add_error( 'bbp_edit_reply_status', __( '<strong>Error</strong>: You do not have permission to do that.', 'bbpress' ) );
+        }
     }
 
@@ -2060 +2075 @@
  *
  * @param string $where
+ * @param WP_Query $query
  * @return string
  */

trunk/src/includes/replies/template.php

@@ -800 +800 @@
  *
  * @param int $reply_id Optional. Reply id
- * @return string reply revisions
+ * @return WP_Post[]|int[] reply revisions
  */
 function bbp_get_reply_revisions( $reply_id = 0 ) {

trunk/src/includes/search/template.php

@@ -47 +47 @@
     }
 
-    // What are the default allowed statuses (based on user caps)
-    if ( bbp_get_view_all() ) {
-
-        // Default view=all statuses
-        $post_statuses = array_keys( bbp_get_topic_statuses() );
-
-        // Add support for private status
-        if ( current_user_can( 'read_private_topics' ) ) {
-            $post_statuses[] = bbp_get_private_status_id();
-        }
-
-        // Join post statuses together
-        $default['post_status'] = $post_statuses;
-
-    // Lean on the 'perm' query var value of 'readable' to provide statuses
-    } else {
-        $default['perm'] = 'readable';
-    }
+    // Default public statuses (topics coincidentally cover all post types)
+    $post_statuses = array_keys( bbp_get_public_topic_statuses() );
+
+    // Add support for private status
+    if ( current_user_can( 'read_private_topics' ) ) {
+        $post_statuses[] = bbp_get_private_status_id();
+    }
+
+    // Add support for hidden status
+    if ( current_user_can( 'read_hidden_topics' ) ) {
+        $post_statuses[] = bbp_get_hidden_status_id();
+    }
+
+    // Join post statuses together
+    $default['post_status'] = $post_statuses;
 
     /** Setup *****************************************************************/

trunk/src/includes/topics/functions.php

@@ -580 +580 @@
 
     // Maybe force into pending
-    if ( bbp_is_topic_public( $topic->ID ) && ! bbp_check_for_moderation( $anonymous_data, $topic_author, $topic_title, $topic_content ) ) {
+    if ( bbp_is_topic_public( $topic_id ) && ! bbp_check_for_moderation( $anonymous_data, $topic_author, $topic_title, $topic_content ) ) {
         $topic_status = bbp_get_pending_status_id();
 
@@ -587 +587 @@
 
         // Allow capable users to explicitly override the status
-        if ( current_user_can( 'moderate', $forum_id ) ) {
+        if ( current_user_can( 'moderate', $topic_id ) ) {
             $topic_status = sanitize_key( $_POST['bbp_topic_status'] );
 

trunk/src/includes/topics/template.php

@@ -964 +964 @@
  *
  * @param int $topic_id Optional. Topic id
- * @return string Topic revisions
+ * @return WP_Post[]|int[] Topic revisions
  */
 function bbp_get_topic_revisions( $topic_id = 0 ) {

trunk/src/readme.txt

@@ -5 +5 @@
 License URI:       https://www.gnu.org/licenses/gpl-2.0.html
 Requires PHP:      5.6.20
-Requires at least: 5.0
-Tested up to:      5.9
+Requires at least: 6.0
+Tested up to:      6.5
 Stable tag:        2.7.0-alpha-2
 

trunk/src/templates/default/bbpress/content-archive-topic.php

@@ -15 +15 @@
 <div id="bbpress-forums" class="bbpress-wrapper">
 
-    <?php if ( bbp_allow_search() ) : ?>
-
-        <div class="bbp-search-form">
-
-            <?php bbp_get_template_part( 'form', 'search' ); ?>
-
-        </div>
-
-    <?php endif; ?>
+    <?php bbp_get_template_part( 'form', 'search' ); ?>
 
     <?php bbp_breadcrumb(); ?>

trunk/src/templates/default/bbpress/content-search.php

@@ -15 +15 @@
 <div id="bbpress-forums" class="bbpress-wrapper">
 
+    <?php bbp_get_template_part( 'form', 'search' ); ?>
+
     <?php bbp_breadcrumb(); ?>
 
@@ -29 +31 @@
         <?php bbp_get_template_part( 'pagination', 'search' ); ?>
 
-    <?php elseif ( bbp_get_search_terms() ) : ?>
+    <?php else : ?>
 
         <?php bbp_get_template_part( 'feedback',   'no-search' ); ?>
-
-    <?php else : ?>
-
-        <?php bbp_get_template_part( 'form', 'search' ); ?>
 
     <?php endif; ?>

trunk/src/templates/default/bbpress/feedback-no-search.php

@@ -11 +11 @@
 defined( 'ABSPATH' ) || exit;
 
-?>
+if ( bbp_get_search_terms() ) : ?>
 
 <div class="bbp-template-notice">
@@ -18 +18 @@
     </ul>
 </div>
+
+<?php else : ?>
+
+<div class="bbp-template-notice">
+    <ul>
+        <li><?php esc_html_e( 'Please enter some search terms.', 'bbpress' ); ?></li>
+    </ul>
+</div>
+
+<?php endif;

trunk/src/templates/default/bbpress/form-forum.php

@@ -85 +85 @@
                     <?php do_action( 'bbp_theme_after_forum_form_content' ); ?>
 
-                    <?php if ( ! ( bbp_use_wp_editor() || current_user_can( 'unfiltered_html' ) ) ) : ?>
-
-                        <p class="form-allowed-tags">
-                            <label><?php esc_html_e( 'You may use these <abbr title="HyperText Markup Language">HTML</abbr> tags and attributes:', 'bbpress' ); ?></label><br />
-                            <code><?php bbp_allowed_tags(); ?></code>
-                        </p>
-
-                    <?php endif; ?>
+                    <?php bbp_get_template_part( 'form', 'allowed-tags' ); ?>
 
                     <?php if ( bbp_allow_forum_mods() && current_user_can( 'assign_moderators' ) ) : ?>

trunk/src/templates/default/bbpress/form-reply.php

@@ -74 +74 @@
                     <?php do_action( 'bbp_theme_after_reply_form_content' ); ?>
 
-                    <?php if ( ! ( bbp_use_wp_editor() || current_user_can( 'unfiltered_html' ) ) ) : ?>
-
-                        <p class="form-allowed-tags">
-                            <label><?php esc_html_e( 'You may use these <abbr title="HyperText Markup Language">HTML</abbr> tags and attributes:', 'bbpress' ); ?></label><br />
-                            <code><?php bbp_allowed_tags(); ?></code>
-                        </p>
-
-                    <?php endif; ?>
+                    <?php bbp_get_template_part( 'form', 'allowed-tags' ); ?>
 
                     <?php if ( bbp_allow_topic_tags() && current_user_can( 'assign_topic_tags', bbp_get_topic_id() ) ) : ?>

trunk/src/templates/default/bbpress/form-topic.php

@@ -95 +95 @@
                     <?php do_action( 'bbp_theme_after_topic_form_content' ); ?>
 
-                    <?php if ( ! ( bbp_use_wp_editor() || current_user_can( 'unfiltered_html' ) ) ) : ?>
-
-                        <p class="form-allowed-tags">
-                            <label><?php printf( esc_html__( 'You may use these %s tags and attributes:', 'bbpress' ), '<abbr title="HyperText Markup Language">HTML</abbr>' ); ?></label><br />
-                            <code><?php bbp_allowed_tags(); ?></code>
-                        </p>
-
-                    <?php endif; ?>
+                    <?php bbp_get_template_part( 'form', 'allowed-tags' ); ?>
 
                     <?php if ( bbp_allow_topic_tags() && current_user_can( 'assign_topic_tags', bbp_get_topic_id() ) ) : ?>