Ticket #7112: 7112.01.patch

src/bp-xprofile/classes/class-bp-xprofile-field.php

@@ -943 +943 @@
                         $sql = $wpdb->prepare( "UPDATE {$table_name} SET group_id = %d WHERE parent_id = %d", $field_group_id, $field_id );
                         $wpdb->query( $sql );
 
+                        // Invalidate profile field cache.
+                        wp_cache_delete( $field_id, 'bp_xprofile_fields' );
+
                         return $parent;
                 }
 

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

@@ -183 +183 @@
 
                 $this->assertFalse( wp_cache_get( 12345, 'bp_xprofile_fields' ) );
         }
+
+        /**
+         * @ticket BP7112
+         */
+        public function test_update_position_should_invalidate_cache() {
+                $group = $this->factory->xprofile_group->create();
+                $field = $this->factory->xprofile_field->create( array(
+                        'field_group_id' => $group,
+                ) );
+
+                // Prime cache.
+                xprofile_get_field( $field );
+
+                // Update field position.
+                BP_XProfile_Field::update_position( $field, 12345, 12345 );
+
+                // Fetch cache after position update.
+                $cached = wp_cache_get( $field, 'bp_xprofile_fields' );
+
+                // Cache should be invalidated at this point.
+                $this->assertEmpty( $cached );
+        }
 }