diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2014-05-14 10:03:24 -0400 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2014-05-14 10:03:24 -0400 |
| commit | aa12fdb40e6641cc35e0228a1c38c469afc828f5 (patch) | |
| tree | ba3db72cb74e4062d187035545ce65d180897b5f | |
| parent | 92ab5a8f5ba9fdefe8219cd23753f6652e3a5cfc (diff) | |
| download | sqlalchemy-aa12fdb40e6641cc35e0228a1c38c469afc828f5.tar.gz | |
- Fixed bug in mutable extension where :class:`.MutableDict` did not
report change events for the ``setdefault()`` dictionary operation.
fixes #3051
| -rw-r--r-- | doc/build/changelog/changelog_08.rst | 8 | ||||
| -rw-r--r-- | lib/sqlalchemy/ext/mutable.py | 5 | ||||
| -rw-r--r-- | test/ext/test_mutable.py | 17 |
3 files changed, 30 insertions, 0 deletions
diff --git a/doc/build/changelog/changelog_08.rst b/doc/build/changelog/changelog_08.rst index 428d2b576..61ef96aff 100644 --- a/doc/build/changelog/changelog_08.rst +++ b/doc/build/changelog/changelog_08.rst @@ -12,6 +12,14 @@ :version: 0.8.7 .. change:: + :tags: bug, ext + :versions: 0.9.5 + :tickets: 3051 + + Fixed bug in mutable extension where :class:`.MutableDict` did not + report change events for the ``setdefault()`` dictionary operation. + + .. change:: :tags: bug, mysql :versions: 0.9.5 :pullreq: bitbucket:15 diff --git a/lib/sqlalchemy/ext/mutable.py b/lib/sqlalchemy/ext/mutable.py index 7869e888c..d2c686048 100644 --- a/lib/sqlalchemy/ext/mutable.py +++ b/lib/sqlalchemy/ext/mutable.py @@ -610,6 +610,11 @@ class MutableDict(Mutable, dict): dict.__setitem__(self, key, value) self.changed() + + def setdefault(self, key, value): + dict.setdefault(self, key, value) + self.changed() + def __delitem__(self, key): """Detect dictionary del events and emit change events.""" dict.__delitem__(self, key) diff --git a/test/ext/test_mutable.py b/test/ext/test_mutable.py index 39cfa7ac5..9a030a4e9 100644 --- a/test/ext/test_mutable.py +++ b/test/ext/test_mutable.py @@ -86,6 +86,23 @@ class _MutableDictTestBase(object): eq_(f1.data, {}) + def test_setdefault(self): + sess = Session() + + f1 = Foo(data={'a': 'b'}) + sess.add(f1) + sess.commit() + + f1.data.setdefault('c', 'd') + sess.commit() + + eq_(f1.data, {'a': 'b', 'c': 'd'}) + + f1.data.setdefault('c', 'q') + sess.commit() + + eq_(f1.data, {'a': 'b', 'c': 'd'}) + def test_replace(self): sess = Session() f1 = Foo(data={'a': 'b'}) |