Changeset 8130 for trunk/tests/testcases/blogs/functions.php

Timestamp:

03/14/2014 12:15:41 AM (10 years ago)

Author:

boonebgorges

Message:

Don't trim whitespace from $meta_value param in _delete_meta() functions

The $meta_value parameter of BP's delete_meta() functions allows developers to
limit deletion to rows that match both the $meta_key *and* the specified
$meta_value. However, most of BP's delete_meta() functions have always had a
quirk whereby the $meta_value was run through trim() before matching against
the values in the database. This had the unacceptable side effect that passing
a $meta_value of ' foo ' would delete rows with the value 'foo', but *not*
the value ' foo '!

This changeset reverses the behavior, and modifies the relevant unit tests.

See #5399

File:

• trunk/tests/testcases/blogs/functions.php (modified) (1 diff)

trunk/tests/testcases/blogs/functions.php

@@ -29 +29 @@
      * @group blogmeta
      * @group bp_blogs_delete_blogmeta
+     * @ticket BP5399
      */
     public function test_bp_blogs_delete_blogmeta_trim_meta_value() {
         $this->assertTrue( bp_blogs_update_blogmeta( 1, 'foo', 'bar' ) );
         $this->assertSame( 'bar', bp_blogs_get_blogmeta( 1, 'foo' ) );
-        $this->assertTrue( bp_blogs_delete_blogmeta( 1, 'foo', '   bar  ') );
-        $this->assertSame( '', bp_blogs_get_blogmeta( 1, 'foo' ) );
+        bp_blogs_delete_blogmeta( 1, 'foo', '   bar  ' );
+        $this->assertSame( 'bar', bp_blogs_get_blogmeta( 1, 'foo' ) );
     }