Changeset 6930

Timestamp:

11/09/2019 06:29:34 PM (5 years ago)

Author:

johnjamesjacoby

Message:

Topic Views: pass view ID into body class.

This commit does a quick audit of Topic View API functions, and ensures their return values are as documented. This allows for safely adding the Topic View ID as a class to the body element of the page when viewing a single Topic View.

Fixes #3278. Props Clorith.

Location:

trunk/src/includes

Files:

• common/template.php (modified) (2 diffs)
• core/functions.php (modified) (1 diff)
• core/template-functions.php (modified) (1 diff)

trunk/src/includes/common/template.php

@@ -970 +970 @@
     } elseif ( bbp_is_single_view() ) {
         $bbp_classes[] = 'bbp-view';
+        $bbp_classes[] = 'bbp-view-' . bbp_get_view_id();
 
     /** User ******************************************************************/
@@ -1980 +1981 @@
         $bbp = bbpress();
 
+        // User supplied string
         if ( ! empty( $view ) ) {
-            $view = sanitize_title( $view );
+            $view_id = sanitize_key( $view );
+
+        // Current view ID
         } elseif ( ! empty( $bbp->current_view_id ) ) {
-            $view = $bbp->current_view_id;
+            $view_id = $bbp->current_view_id;
+
+        // Querying for view
         } else {
-            $view = get_query_var( bbp_get_view_rewrite_id() );
-        }
-
-        if ( array_key_exists( $view, $bbp->views ) ) {
-            return $view;
-        }
-
-        return false;
+            $view_id = get_query_var( bbp_get_view_rewrite_id() );
+        }
+
+        // Filter & return
+        return apply_filters( 'bbp_get_view_id', $view_id, $view );
     }
 

trunk/src/includes/core/functions.php

@@ -268 +268 @@
     $retval = ! empty( $view ) && ! empty( $bbp->views[ $view ] )
         ? $bbp->views[ $view ]['query']
-        : false;
+        : array();
 
     // Filter & return
-    return apply_filters( 'bbp_get_view_query_args', $retval, $view );
+    return (array) apply_filters( 'bbp_get_view_query_args', $retval, $view );
 }
 

trunk/src/includes/core/template-functions.php

@@ -650 +650 @@
         $view_args = bbp_get_view_query_args( $bbp_view );
 
-        // Bail if view args is false (view isn't registered)
-        if ( false === $view_args ) {
+        // Bail if view args are empty
+        if ( empty( $view_args ) ) {
             $posts_query->bbp_is_404 = true;
             return;