Changeset 7883 for trunk/bp-xprofile/bp-xprofile-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-xprofile/bp-xprofile-functions.php (modified) (3 diffs)

trunk/bp-xprofile/bp-xprofile-functions.php

@@ -539 +539 @@
  * @param mixed $meta_value Optional. If provided, only metadata that matches
  *        the value will be permitted.
+ * @param bool $delete_all Optional. If true, delete matching metadata entries
+ *    for all objects, ignoring the specified object_id. Otherwise, only
+ *    delete matching metadata entries for the specified object.
+ *    Default: false.
  * @return bool True on success, false on failure.
  */
-function bp_xprofile_delete_meta( $object_id, $object_type, $meta_key = false, $meta_value = false ) {
+function bp_xprofile_delete_meta( $object_id, $object_type, $meta_key = false, $meta_value = false, $delete_all = false ) {
     global $wpdb;
 
@@ -559 +563 @@
         $table_name = $wpdb->{$table_key};
         $keys = $wpdb->get_col( $wpdb->prepare( "SELECT meta_key FROM {$table_name} WHERE object_type = %s AND object_id = %d", $object_type, $object_id ) );
+
+        // Force delete_all to false if deleting all for object
+        $delete_all = false;
     } else {
         $keys = array( $meta_key );
@@ -570 +577 @@
 
     foreach ( $keys as $key ) {
-        $retval = delete_metadata( 'xprofile_' . $object_type, $object_id, $key, $meta_value );
+        $retval = delete_metadata( 'xprofile_' . $object_type, $object_id, $key, $meta_value, $delete_all );
     }