Changeset 10778 for trunk/tests/phpunit/testcases/xprofile/class-bp-xprofile-field.php

Timestamp:

05/19/2016 02:51:45 AM (9 years ago)

Author:

boonebgorges

Message:

XProfile: Only cache field object when query is successful.

Previously, a PHP notice was being thrown when a bad $field_id was passed
to BP_XProfile_Field::get_instance().

Props mmcachran.
Fixes #7073.

File:

• trunk/tests/phpunit/testcases/xprofile/class-bp-xprofile-field.php (modified) (1 diff)

trunk/tests/phpunit/testcases/xprofile/class-bp-xprofile-field.php

@@ -175 +175 @@
         $this->assertSame( $num_queries, $wpdb->num_queries );
     }
+
+    /**
+     * @ticket BP7073
+     */
+    public function test_bad_field_id_should_not_be_cached() {
+        BP_XProfile_Field::get_instance( 12345 );
+
+        $this->assertFalse( wp_cache_get( 12345, 'bp_xprofile_fields' ) );
+    }
 }