Changeset 10198 for trunk/tests/phpunit/testcases/xprofile/cache.php

Timestamp:

10/07/2015 02:30:31 PM (9 years ago)

Author:

boonebgorges

Message:

Introduce cache support for xprofile fields.

xprofile_get_field() will now look in the cache before returning a
BP_XProfile_Field object (via the new BP_XProfile_Field::get_instance()),
and will populate the cache if necessary. It's strongly recommended to use
xprofile_get_field() when fetching individual existing fields, instead of
creating new BP_XProfile_Field objects directly.

BP_XProfile_Group::get() now leverages the field cache too. Instead of a
SELECT * query, it fetches a list of matching field IDs SELECT id, and
populates the full field objects from the cache, if available. The fields
property of BP_XProfile_Group objects is now an array of BP_XProfile_Field
objects, not stdClass. See #6358.

Props boonebgorges, r-a-y.
See #6638.

File:

• trunk/tests/phpunit/testcases/xprofile/cache.php (modified) (1 diff)

trunk/tests/phpunit/testcases/xprofile/cache.php

@@ -145 +145 @@
         $this->assertFalse( wp_cache_get( $d->id, 'xprofile_data_meta' ) );
     }
+
+    /**
+     * @ticket BP6638
+     */
+    public function test_field_cache_should_be_invalidated_on_save() {
+        $g = $this->factory->xprofile_group->create();
+        $f = $this->factory->xprofile_field->create( array(
+            'field_group_id' => $g,
+            'name' => 'Foo',
+        ) );
+
+        $field = xprofile_get_field( $f );
+        $this->assertSame( 'Foo', $field->name );
+
+        $field->name = 'Bar';
+        $this->assertNotEmpty( $field->save() );
+
+        $field_2 = xprofile_get_field( $f );
+        $this->assertSame( 'Bar', $field_2->name );
+    }
 }