Changeset 3008 for branches/plugin/bbp-includes/bbp-forum-template.php

Timestamp:

04/21/2011 08:35:23 PM (15 years ago)

Author:

johnjamesjacoby

Message:

Improvements to forum visibility checking, template tags, and ensuring that template tag filters also pass the $forum_id for proper enhancement.

File:

• branches/plugin/bbp-includes/bbp-forum-template.php (modified) (12 diffs)

branches/plugin/bbp-includes/bbp-forum-template.php

@@ -74 +74 @@
         // Don't show private forums to normal users
         if ( !current_user_can( 'read_private_forums' ) && empty( $r['meta_key'] ) && empty( $r['meta_value'] ) ) {
-                $r['meta_key']   = '_bbp_visibility';
-                $r['meta_value'] = 'public';
+                $r['meta_query'] = array( array(
+                        'key'     => '_bbp_visibility',
+                        'value'   => 'public, private',
+                        'compare' => 'IN'
+                ) );
         }
 
@@ -1186 +1189 @@
                 $forum_id = bbp_get_forum_id( $forum_id );
 
-                return apply_filters( 'bbp_get_forum_status', get_post_meta( $forum_id, '_bbp_status', true ) );
+                return apply_filters( 'bbp_get_forum_status', get_post_meta( $forum_id, '_bbp_status', true ), $forum_id );
         }
 
@@ -1215 +1218 @@
                 $forum_id = bbp_get_forum_id( $forum_id );
 
-                return apply_filters( 'bbp_get_forum_visibility', get_post_meta( $forum_id, '_bbp_visibility', true ) );
+                return apply_filters( 'bbp_get_forum_visibility', get_post_meta( $forum_id, '_bbp_visibility', true ), $forum_id );
         }
 
@@ -1230 +1233 @@
         $forum_id = bbp_get_forum_id( $forum_id );
         $type     = get_post_meta( $forum_id, '_bbp_forum_type', true );
-
-        if ( !empty( $type ) && 'category' == $type )
-                return true;
-
-        return false;
+        $retval   = ( !empty( $type ) && 'category' == $type );
+
+        return apply_filters( 'bbp_is_forum_category', (bool) $retval, $forum_id );
 }
 
@@ -1269 +1270 @@
 
                 $forum_id = bbp_get_forum_id( $forum_id );
-
-                if ( $bbp->closed_status_id == bbp_get_forum_status( $forum_id ) )
-                        return true;
+                $retval    = ( $bbp->closed_status_id == bbp_get_forum_status( $forum_id ) );
 
                 if ( !empty( $check_ancestors ) ) {
@@ -1277 +1276 @@
 
                         foreach ( (array) $ancestors as $ancestor ) {
-                                if ( bbp_is_forum_category( $ancestor, false ) && bbp_is_forum_closed( $ancestor, false ) )
-                                        return true;
+                                if ( bbp_is_forum_category( $ancestor, false ) && bbp_is_forum_closed( $ancestor, false ) ) {
+                                        $retval = true;
+                                }
                         }
                 }
 
-                return false;
+                return apply_filters( 'bbp_is_forum_closed', (bool) $retval, $forum_id, $check_ancestors );
         }
 
@@ -1306 +1306 @@
 
         // If post status is public, return true
-        $retval = ( 'public' == $visibility ) ? true : false;
+        $retval = ( 'public' == $visibility );
 
         // Check ancestors and inherit their privacy setting for display
@@ -1319 +1319 @@
         }
 
-        return apply_filters( 'bbp_is_forum_public', (bool) $retval );
+        return apply_filters( 'bbp_is_forum_public', (bool) $retval, $forum_id, $check_ancestors );
 }
 
@@ -1343 +1343 @@
 
         // If post status is private, return true
-        $retval = ( 'private' == $visibility ) ? true : false;
+        $retval = ( 'private' == $visibility );
 
         // Check ancestors and inherit their privacy setting for display
@@ -1356 +1356 @@
         }
 
-        return apply_filters( 'bbp_is_forum_private', (bool) $retval );
+        return apply_filters( 'bbp_is_forum_private', (bool) $retval, $forum_id, $check_ancestors );
 }
 
@@ -1380 +1380 @@
 
         // If post status is private, return true
-        $retval = ( 'hidden' == $visibility ) ? true : false;
+        $retval = ( 'hidden' == $visibility );
 
         // Check ancestors and inherit their privacy setting for display
@@ -1393 +1393 @@
         }
 
-        return apply_filters( 'bbp_is_forum_hidden', (bool) $retval );
-}
+        return apply_filters( 'bbp_is_forum_hidden', (bool) $retval, $forum_id, $check_ancestors );
+}
+
+function bbp_suppress_private_forum_meta( $retval, $forum_id ) {
+        if ( bbp_is_forum_private( $forum_id, false ) && !current_user_can( 'read_private_forums' ) )
+                return '-';
+
+        return $retval;
+}
+add_filter( 'bbp_get_forum_topic_count',    'bbp_suppress_private_forum_meta', 10, 2 );
+add_filter( 'bbp_get_forum_reply_count',    'bbp_suppress_private_forum_meta', 10, 2 );
+add_filter( 'bbp_get_forum_post_count',     'bbp_suppress_private_forum_meta', 10, 2 );
+add_filter( 'bbp_get_forum_freshness_link', 'bbp_suppress_private_forum_meta', 10, 2 );
+
+function bbp_suppress_private_author_link( $author_link, $args ) {
+        if ( empty( $args['post_id'] ) || current_user_can( 'read_private_forums' ) )
+                return $author_link;
+
+        $post_type = get_post_field( 'post_type', $args['post_id'] );
+
+        switch ( $post_type ) {
+                case bbp_get_topic_post_type() :
+                        if ( bbp_is_forum_private( bbp_get_topic_forum_id( $args['post_id'] ) ) )
+                                return '';
+
+                        break;
+
+                case bbp_get_reply_post_type() :
+                        if ( bbp_is_forum_private( bbp_get_reply_forum_id( $args['post_id'] ) ) )
+                                return '';
+
+                        break;
+
+                default :
+                        if ( bbp_is_forum_private( $args['post_id'] ) )
+                                return '';
+
+                        break;
+        }
+
+        return $author_link;
+}
+add_filter( 'bbp_get_author_link',       'bbp_suppress_private_author_link', 10, 2 );
+add_filter( 'bbp_get_topic_author_link', 'bbp_suppress_private_author_link', 10, 2 );
+add_filter( 'bbp_get_reply_author_link', 'bbp_suppress_private_author_link', 10, 2 );
 
 /**