Preferences: Remove unwise caching of Preferences::getPreferences()
authorBartosz Dziewoński <matma.rex@gmail.com>
Mon, 13 Nov 2017 16:42:04 +0000 (17:42 +0100)
committerBartosz Dziewoński <matma.rex@gmail.com>
Wed, 15 Nov 2017 11:04:52 +0000 (11:04 +0000)
commiteed3cf7747dd9d183b62f2550c741584093e6373
treeb0a0da11de4312fb705dde9edb840b9152ebb9a7
parentf1ca6b8ca9c2dc65014bc8ae79f02158d2dc52c3
Preferences: Remove unwise caching of Preferences::getPreferences()

The result of this function depends on the $user and $context
parameters (e.g. it includes the username from the user, and
localisation messages the language from the context). However,
both of them would be ignored if the result was cached, even
if calling with a different $user or $context.

Rather than make this more complicated just remove the caching.
This is not a hot code path: this function is not called at all
on normal page views, it's called just once when viewing
preferences, and at most twice when saving them.

Change-Id: I92390120a16448383a25e9ba2dd35a434a2f21bf
includes/Preferences.php
tests/phpunit/includes/PreferencesTest.php