client: Fix settings upgrade missing some steps
authorShane Synan <digitalcircuit36939@gmail.com>
Fri, 22 Jun 2018 05:18:09 +0000 (00:18 -0500)
committerManuel Nickschas <sputnick@quassel-irc.org>
Tue, 10 Jul 2018 20:34:39 +0000 (22:34 +0200)
commitebe555951f043ac230149436fb15627120da945e
tree77af2b55b3ce77e717ed427ab10b54fd5f2a4672
parentda1a6f40b28d65e5b6a5a03e62f9b2604c3f00a7
client: Fix settings upgrade missing some steps

Remove cache check in Settings::localKeyExists().  This fixes
settings version upgrade wrongly skipping some steps due to earlier
migration steps causing default values (not stored in .conf) to be
loaded into settings cache.  Most notably, creating/writing the
QtUiStyle settings Qt StyleSheet.

The symptoms are only visible when migrating multiple steps at once,
hence not catching this earlier.

Bug originally introduced in d2ac8f78a0e050d2efa397c434b249d6b3391576
src/common/settings.cpp