Changeset 6930 for trunk/src/includes/common/template.php

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.

File:

• trunk/src/includes/common/template.php (modified) (2 diffs)

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 );
     }