Cache: Make APCUBagOStuff::set return a useful value
authorDaimona Eaytoy <daimona.wiki@gmail.com>
Sun, 14 Apr 2019 17:13:48 +0000 (19:13 +0200)
committerDaimona Eaytoy <daimona.wiki@gmail.com>
Sun, 14 Apr 2019 17:13:48 +0000 (19:13 +0200)
commit7dc4f6c83291b005cce83249564ad8a22da51e50
tree1be4c3b5d63fb9c875d0e0762b38ddbb4dcf8c3c
parente996b5391357ab03d53f0347854ec42a9b091a53
Cache: Make APCUBagOStuff::set return a useful value

It took me lots of time to debug a CI failure in
I9b3bc36b552901bc6ca7609ee51e80be2979a9c4. I was deceived by
$cache->set returning true, which according to the docs means that the
store was successful. But instead, this function just returns true, even
in case of failure. Make it return the result of apcu_store, which is
already in the format true = success, false = failure.

Change-Id: I2619845c12460e1acb5af696d2c11a5a4dee1bd3
includes/libs/objectcache/APCUBagOStuff.php