update_option + PHP5 Object references not storing correctly in cache

[dd32]

This is related to the changes to caching of the options API.

At present, When objects are saved using the option API, the PHP5 object references are being stored in an array in the "object cache", The result is that if you attempt to update an object variable multiple times in 1 page load, you'll end up with only the first update_option succeeding.

This is because on subsequent calls, the object cache references the NEW value as existing within the database..

Whilst this ticket description might be hard to follow, This is the same as [9740].

The solution is to clone the variables if they're objects in the Options API.

[dd32]

(In [14777]) Clone Objects in add_option()/update_option() to prevent storing PHP5 object references in memory cache. See [9740]. Fixes #13480

[dd32]

See also this wp-testers thread which prompted this ticket: ​http://lists.automattic.com/pipermail/wp-testers/2010-May/013020.html

[westi]

An excellent candidate for unit tests :-)

[dd32]

An excellent candidate for unit tests :-)

Quite agree, I'll write some when i've got a free moment if someone else doesnt get to it first.

To reproduce:

• Create an object, store its value in an option, update a field in that object, update the option with it again.
• All going well, the database should now contain the updated object.
• Under PHP5 previously, This would not be the case, The database would only contain the FIRST version of the object
  • Will need to use a query or similar(disable cache temporarily?) to bypass cache and retrieve directly from the database.

[dd32]

Another way to reproduce:

• Create an object, store its value in an option, update a field in that object WITHOUT SAVING
• Retrieve option, It should contain the first copy of the object, Under PHP5 previously, it will return the updated object (directly from the cache which references the original object)

[hakre]

Another way to comment:

If you want a NEW value you should offer the clone as such:

$value = clone get_option($option_name);
$value->var1 = $new_var1;
update_option($option_name, $value);

my 2 cents. don't fix what isn't broken (so I'll remove the cause.../ [chuckles] but not the symptom.)

[hakre]

And if $alloptions was the cache, then add_option was not to be fixed as $alloptions[$option] is set to the serialized value (which is never subject to cloning.... )

[hakre]

Related: #16813

[nacin]

See [17613].

[SergeyBiryukov]

Didn't notice that the tests from 13480-test.diff​ weren't commited yet. Adding the keyword back.