Changeset 7883 for trunk/bp-blogs/bp-blogs-functions.php

Timestamp:

02/14/2014 07:02:16 PM (11 years ago)

Author:

boonebgorges

Message:

Add delete_all support for all _delete_meta() functions

The delete_all parameter allows you to delete all meta items matching the
specified meta_key, regardless of the associated object.

Because our meta delete functions also support the deletion of all metadata
associated with an object (by leaving out the meta_key param), a decision had
to be made regarding the behavior when _delete_meta() is called with delete_all

true and meta_key = false. The most logical (and least destructive) strategy

was deemed to be: force delete_all to false when meta_key is also false. This
ensures that you don't accidentally wipe out all metadata for a component.

See #5400

File:

• trunk/bp-blogs/bp-blogs-functions.php (modified) (3 diffs)

trunk/bp-blogs/bp-blogs-functions.php

@@ -835 +835 @@
  * @param string $meta_value Optional. If present, the metadata will only be
  *        deleted if the meta_value matches this parameter.
+ * @param bool $delete_all Optional. If true, delete matching metadata entries
+ *    for all objects, ignoring the specified blog_id. Otherwise, only
+ *    delete matching metadata entries for the specified blog.
+ *    Default: false.
  * @return bool True on success, false on failure.
  */
-function bp_blogs_delete_blogmeta( $blog_id, $meta_key = false, $meta_value = false ) {
+function bp_blogs_delete_blogmeta( $blog_id, $meta_key = false, $meta_value = false, $delete_all = false ) {
     global $wpdb, $bp;
 
@@ -851 +855 @@
     if ( empty( $meta_key ) ) {
         $keys = $wpdb->get_col( $wpdb->prepare( "SELECT meta_key FROM {$wpdb->blogmeta} WHERE blog_id = %d", $blog_id ) );
+        $delete_all = false;
     } else {
         $keys = array( $meta_key );
@@ -861 +866 @@
 
     foreach ( $keys as $key ) {
-        $retval = delete_metadata( 'blog', $blog_id, $key, $meta_value );
+        $retval = delete_metadata( 'blog', $blog_id, $key, $meta_value, $delete_all );
     }