Changeset 8263 for sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-hooks.php

Timestamp:

02/18/2019 12:54:32 AM (8 years ago)

Author:

SergeyBiryukov

Message:

Support Forums: Output rel="next", rel="prev" meta tags for paginated archives.

Fixes #4142.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-hooks.php (modified) (5 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-hooks.php

@@ -23 +23 @@
                 // Output rel="canonical" meta tag. Runs before WP's rel_canonical to unhook that if needed.
                 add_action( 'wp_head', array( $this, 'rel_canonical' ), 9 );
+
+                // Output rel="next", rel="prev" meta tags.
+                add_action( 'wp_head', array( $this, 'rel_next_prev' ), 9 );
 
                 // Link to create new topics atop topic list.
@@ -299 +302 @@
 
         /**
-         * Outputs <link rel="canonical"> tags for various pages.
-         */
-        public function rel_canonical() {
-                $canonical_url = false;
+         * Returns the canonical URL for various bbPress pages.
+         *
+         * @return string The canonical URL.
+         */
+        public function get_canonical_url() {
+                $canonical_url = '';
+
                 if ( bbp_is_topic_tag() ) {
                         $canonical_url = bbp_get_topic_tag_link();
@@ -310 +316 @@
                         $canonical_url = get_post_type_archive_link( 'forum' );
                 } elseif ( bbp_is_single_topic() ) {
-                        remove_action( 'wp_head', 'rel_canonical' ); // Doesn't handle pagination.
                         $canonical_url = bbp_get_topic_permalink();
                 } elseif ( bbp_is_single_forum() ) {
-                        remove_action( 'wp_head', 'rel_canonical' ); // Doesn't handle pagination.
                         $canonical_url = bbp_get_forum_permalink();
                 } elseif ( bbpress()->displayed_user && bbpress()->displayed_user->exists() ) {
@@ -320 +324 @@
                 }
 
+                return $canonical_url;
+        }
+
+        /**
+         * Outputs <link rel="canonical"> tags for various bbPress pages.
+         */
+        public function rel_canonical() {
+                $canonical_url = $this->get_canonical_url();
+
+                if ( bbp_is_single_topic() || bbp_is_single_forum() ) {
+                        remove_action( 'wp_head', 'rel_canonical' ); // Doesn't handle pagination.
+                }
+
                 // Make sure canonical has pagination if needed.
                 $page = get_query_var( 'paged', 0 );
@@ -328 +345 @@
                 if ( $canonical_url ) {
                         echo '<link rel="canonical" href="' . esc_url( $canonical_url ) . '" />' . "\n";
+                }
+        }
+
+        /**
+         * Outputs rel="next", rel="prev" for paginated archives.
+         */
+        public function rel_next_prev() {
+                global $wp_query;
+
+                $canonical_url = $this->get_canonical_url();
+                $max_pages     = $wp_query->max_num_pages;
+
+                if ( bbp_is_single_view() ) {
+                        bbp_view_query();  // Populate bbpress()->topic_query.
+                        $max_pages = bbpress()->topic_query->max_num_pages;
+                } elseif ( bbp_is_single_topic() ) {
+                        bbp_has_replies(); // Populate bbpress()->reply_query.
+                        $max_pages = bbpress()->reply_query->max_num_pages;
+                } elseif ( bbp_is_single_forum() ) {
+                        $topic_count = get_post_meta( get_queried_object_id(), '_bbp_topic_count', true );
+                        if ( $topic_count ) {
+                                $max_pages = $topic_count / bbp_get_topics_per_page();
+                        }
+                }
+
+                if ( ! $canonical_url || ! $max_pages ) {
+                        return;
+                }
+
+                $page      = max( 1, get_query_var( 'paged', 0 ) );
+                $next_page = min( $page + 1, $max_pages );
+                $prev_page = max( $page - 1, 1 );
+
+                if ( $page < $max_pages ) {
+                        $next_page_url = $canonical_url . 'page/' . absint( $next_page ) . '/';
+                        echo '<link rel="next" href="' . esc_url( $next_page_url ) . '" />' . "\n";
+                }
+
+                if ( $page > 1 ) {
+                        $prev_page_url = $canonical_url . ( $prev_page > 1 ? 'page/' . absint( $prev_page ) . '/' : '' );
+                        echo '<link rel="prev" href="' . esc_url( $prev_page_url ) . '" />' . "\n";
                 }
         }