Changeset 8131 for trunk/tests/testcases/xprofile/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/xprofile/functions.php (modified) (4 diffs)

trunk/tests/testcases/xprofile/functions.php

@@ -221 +221 @@
         $this->assertSame( 'bar', bp_xprofile_get_meta( $g, 'group', 'foo' ) );
         $this->assertTrue( bp_xprofile_delete_meta( $g, 'group', 'foo', 'bar' ) );
-        $this->assertEquals( '', bp_xprofile_get_meta( $g, 'group', 'foo' ) );
+        $this->assertSame( '', bp_xprofile_get_meta( $g, 'group', 'foo' ) );
     }
 
@@ -237 +237 @@
         $this->assertTrue( bp_xprofile_delete_meta( $g, 'group' ) );
 
-        $this->assertEquals( '', bp_xprofile_get_meta( $g, 'group', 'foo' ) );
-        $this->assertEquals( '', bp_xprofile_get_meta( $g, 'group', 'foo2' ) );
+        $this->assertSame( '', bp_xprofile_get_meta( $g, 'group', 'foo' ) );
+        $this->assertSame( '', bp_xprofile_get_meta( $g, 'group', 'foo2' ) );
     }
 
@@ -276 +276 @@
 
         $this->assertTrue( bp_xprofile_delete_meta( $g1, 'group', 'foo', '', true ) );
-        $this->assertEmpty( '', bp_xprofile_get_meta( $g1, 'group', 'foo' ) );
-        $this->assertEmpty( '', bp_xprofile_get_meta( $g2, 'group', 'foo' ) );
+        $this->assertSame( '', bp_xprofile_get_meta( $g1, 'group', 'foo' ) );
+        $this->assertSame( '', bp_xprofile_get_meta( $g2, 'group', 'foo' ) );
         $this->assertSame( 'bar1', bp_xprofile_get_meta( $g1, 'group', 'foo1' ) );
         $this->assertSame( 'bar1', bp_xprofile_get_meta( $g2, 'group', 'foo1' ) );
@@ -386 +386 @@
         $krazy_key = ' f!@#$%^o *(){}o?+';
         bp_xprofile_update_meta( $g, 'group', $krazy_key, 'bar' );
-        $this->assertEmpty( bp_xprofile_get_meta( $g, 'group', 'foo' ) );
+        $this->assertSame( '', bp_xprofile_get_meta( $g, 'group', 'foo' ) );
     }