Changeset 3852 for branches/plugin/bbp-includes/bbp-forum-functions.php

Timestamp:

04/17/2012 05:37:00 PM (14 years ago)

Author:

johnjamesjacoby

Message:

More aggressively check private and hidden forums when changing forum status. (plugin branch)

File:

• branches/plugin/bbp-includes/bbp-forum-functions.php (modified) (8 diffs)

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

@@ -803 +803 @@
     if ( in_array( $forum_id, $private ) ) {
 
-        $offset = array_search( $forum_id, (array) $private );
+        $offset = array_search( $forum_id, $private );
 
         // Splice around it
@@ -809 +809 @@
 
         // Update private forums minus this one
-        update_option( '_bbp_private_forums', array_unique( array_values( $private ) ) );
+        update_option( '_bbp_private_forums', array_unique( array_filter( array_values( $private ) ) ) );
     }
 
@@ -818 +818 @@
     if ( in_array( $forum_id, $hidden ) ) {
 
-        $offset = array_search( $forum_id, (array) $hidden );
+        $offset = array_search( $forum_id, $hidden );
 
         // Splice around it
@@ -824 +824 @@
 
         // Update hidden forums minus this one
-        update_option( '_bbp_hidden_forums', array_unique( array_values( $hidden ) ) );
+        update_option( '_bbp_hidden_forums', array_unique( array_filter( array_values( $hidden ) ) ) );
     }
 
@@ -859 +859 @@
     if ( bbp_get_private_status_id() != $current_visibility ) {
 
-        // Remove from _bbp_hidden_forums site option
-        if ( bbp_get_hidden_status_id() == $current_visibility ) {
-
-            // Get hidden forums
-            $hidden = bbp_get_hidden_forum_ids();
-
-            // Find this forum in the array
-            if ( in_array( $forum_id, $hidden ) ) {
-
-                $offset = array_search( $forum_id, (array) $hidden );
-
-                // Splice around it
-                array_splice( $hidden, $offset, 1 );
-
-                // Update hidden forums minus this one
-                update_option( '_bbp_hidden_forums', array_unique( array_values( $hidden ) ) );
-            }
+        // Get hidden forums
+        $hidden = bbp_get_hidden_forum_ids();
+
+        // Find this forum in the array
+        if ( in_array( $forum_id, $hidden ) ) {
+
+            $offset = array_search( $forum_id, $hidden );
+
+            // Splice around it
+            array_splice( $hidden, $offset, 1 );
+
+            // Update hidden forums minus this one
+            update_option( '_bbp_hidden_forums', array_unique( array_filter( array_values( $hidden ) ) ) );
         }
 
@@ -881 +877 @@
         $private   = bbp_get_private_forum_ids();
         $private[] = $forum_id;
-        update_option( '_bbp_private_forums', array_unique( array_values( $private ) ) );
+        update_option( '_bbp_private_forums', array_unique( array_filter( array_values( $private ) ) ) );
 
         // Update forums visibility setting
@@ -912 +908 @@
     if ( bbp_get_hidden_status_id() != $current_visibility ) {
 
-        // Remove from _bbp_private_forums site option
-        if ( bbp_get_private_status_id() == $current_visibility ) {
-
-            // Get private forums
-            $private = bbp_get_private_forum_ids();
-
-            // Find this forum in the array
-            if ( in_array( $forum_id, $private ) ) {
-
-                $offset = array_search( $forum_id, (array) $private );
-
-                // Splice around it
-                array_splice( $private, $offset, 1 );
-
-                // Update private forums minus this one
-                update_option( '_bbp_private_forums', array_unique( array_values( $private ) ) );
-            }
+        // Get private forums
+        $private = bbp_get_private_forum_ids();
+
+        // Find this forum in the array
+        if ( in_array( $forum_id, $private ) ) {
+
+            $offset = array_search( $forum_id, $private );
+
+            // Splice around it
+            array_splice( $private, $offset, 1 );
+
+            // Update private forums minus this one
+            update_option( '_bbp_private_forums', array_unique( array_filter( array_values( $private ) ) ) );
         }
 
@@ -934 +926 @@
         $hidden   = bbp_get_hidden_forum_ids();
         $hidden[] = $forum_id;
-        update_option( '_bbp_hidden_forums', array_unique( array_values( $hidden ) ) );
+        update_option( '_bbp_hidden_forums', array_unique( array_filter( array_values( $hidden ) ) ) );
 
         // Update forums visibility setting