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

Timestamp:

03/14/2014 12:37:28 AM (10 years ago)

Author:

boonebgorges

Message:

Return an empty string from get_meta() functions when no value is found

BP get_meta() functions have been inconsistent about the type of value returned
when no match is found for the object_id+meta_key combination. In some cases,
we followed WP's get_metadata() and returned an empty string. In others, we
returned false.

This changeset aligns all of our get_meta() functions with WP in this regard,
returning an empty string when no matching value is found. A boolean false is
still returned when invalid arguments are passed to the functions.

Relevant unit tests have been updated, including strict assertSame() checking
where appropriate.

See #5399

File:

• trunk/tests/testcases/blogs/functions.php (modified) (3 diffs)

trunk/tests/testcases/blogs/functions.php

@@ -94 +94 @@
 
         $this->assertTrue( bp_blogs_delete_blogmeta( 1, 'foo', '', true ) );
-        $this->assertEmpty( '', bp_blogs_get_blogmeta( 1, 'foo' ) );
-        $this->assertEmpty( '', bp_blogs_get_blogmeta( 2, 'foo' ) );
+        $this->assertSame( '', bp_blogs_get_blogmeta( 1, 'foo' ) );
+        $this->assertSame( '', bp_blogs_get_blogmeta( 2, 'foo' ) );
         $this->assertSame( 'bar1', bp_blogs_get_blogmeta( 1, 'foo1' ) );
         $this->assertSame( 'bar1', bp_blogs_get_blogmeta( 2, 'foo1' ) );
@@ -117 +117 @@
         bp_blogs_update_blogmeta( 1, 'foo', 'bar' );
         $krazy_key = ' f!@#$%^o *(){}o?+';
-        $this->assertEmpty( bp_blogs_get_blogmeta( 1, $krazy_key ) );
+        $this->assertSame( '', bp_blogs_get_blogmeta( 1, $krazy_key ) );
     }
 
@@ -175 +175 @@
         $krazy_key = ' f!@#$%^o *(){}o?+';
         bp_blogs_update_blogmeta( 1, $krazy_key, 'bar' );
-        $this->assertEmpty( bp_blogs_get_blogmeta( 1, 'foo' ) );
+        $this->assertSame( '', bp_blogs_get_blogmeta( 1, 'foo' ) );
     }