Cutlery/calibre
1
0
Fork 0
mirror of https://github.com/kovidgoyal/calibre.git synced 2025-07-09 03:04:10 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fix change check on preferences not working for dbs because of random serialization order and a typo in the comparison. Also add test for the change check.

Browse Source
This commit is contained in:
Kovid Goyal 2015-02-21 14:45:12 +05:30
parent 3146735962
commit 45bedfc1a9
2 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/calibre/db/backend.py
View File

@ -90,7 +90,9 @@ class DBPrefs(dict): # {{{
return json.loads(raw, object_hook=from_json)
def to_raw(self, val):
return json.dumps(val, indent=2, default=to_json)
# sort_keys=True is required so that the serialization of dictionaries is
# not random, which is needed for the changed check in __setitem__
return json.dumps(val, indent=2, default=to_json, sort_keys=True)
def has_setting(self, key):
return key in self
@ -110,8 +112,11 @@ class DBPrefs(dict): # {{{
raw = self.to_raw(val)
with self.db.conn:
dbraw = self.db.execute('SELECT val FROM preferences WHERE key=?', (key,)).fetchone()
if dbraw != raw:
self.db.execute('INSERT OR REPLACE INTO preferences (key,val) VALUES (?,?)', (key, raw))
if dbraw != (raw,):
if dbraw is None:
self.db.execute('INSERT INTO preferences (key,val) VALUES (?,?)', (key, raw))
else:
self.db.execute('UPDATE preferences SET val=? WHERE key=?', (raw, key))
dict.__setitem__(self, key, val)
def set(self, key, val):

10
src/calibre/db/tests/writing.py
View File

@ -7,7 +7,6 @@ __license__ = 'GPL v3'
__copyright__ = '2013, Kovid Goyal <kovid at kovidgoyal.net>'
__docformat__ = 'restructuredtext en'
import os
from collections import namedtuple
from functools import partial
from io import BytesIO
@ -704,7 +703,10 @@ class WritingTest(BaseTest):
prefs['test mutable'] = a
prefs.load_from_db()
self.assertIn(4, prefs['test mutable'])
before = os.stat(cache.backend.library_path)
prefs['test mutable'] = a
self.assertEqual(before, os.stat(cache.backend.library_path), 'The database was written to despite there being no change in value')
changes = []
cache.backend.conn.setupdatehook(lambda typ, dbname, tblname, rowid: changes.append(rowid))
prefs['test mutable'] = {str(k):k for k in range(4)}
self.assertEqual(len(changes), 1)
prefs['test mutable'] = {str(k):k for k in range(4)}
self.assertEqual(len(changes), 1, 'The database was written to despite there being no change in value')
# }}}