diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2013-08-20 13:27:27 -0400 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2013-08-20 13:29:14 -0400 |
| commit | 4bcd7e38e12ca2a74de1644b7d260497a64bc7dd (patch) | |
| tree | f9e099e2a8b27782542d5c06b967c54079984136 | |
| parent | cc031065806c630f337271a26fc722fbc9f2a79a (diff) | |
| download | sqlalchemy-4bcd7e38e12ca2a74de1644b7d260497a64bc7dd.tar.gz | |
Fixed bug where list instrumentation would fail to represent a
setslice of ``[0:0]`` correctly, which in particular could occur
when using ``insert(0, item)`` with the association proxy. Due
to some quirk in Python collections, the issue was much more likely
with Python 3 rather than 2. Also in 0.8.3, 0.7.11.
[ticket:2807]
Conflicts:
doc/build/changelog/changelog_09.rst
Conflicts:
doc/build/changelog/changelog_08.rst
| -rw-r--r-- | doc/build/changelog/changelog_07.rst | 10 | ||||
| -rw-r--r-- | lib/sqlalchemy/orm/collections.py | 5 | ||||
| -rw-r--r-- | test/orm/test_collection.py | 11 |
3 files changed, 22 insertions, 4 deletions
diff --git a/doc/build/changelog/changelog_07.rst b/doc/build/changelog/changelog_07.rst index 738e89746..d7d3019d2 100644 --- a/doc/build/changelog/changelog_07.rst +++ b/doc/build/changelog/changelog_07.rst @@ -7,6 +7,16 @@ :version: 0.7.11 .. change:: + :tags: bug, orm + :tickets: 2807 + + Fixed bug where list instrumentation would fail to represent a + setslice of ``[0:0]`` correctly, which in particular could occur + when using ``insert(0, item)`` with the association proxy. Due + to some quirk in Python collections, the issue was much more likely + with Python 3 rather than 2. + + .. change:: :tags: bug, sql :tickets: 2801 diff --git a/lib/sqlalchemy/orm/collections.py b/lib/sqlalchemy/orm/collections.py index e26a59731..bb2d272c4 100644 --- a/lib/sqlalchemy/orm/collections.py +++ b/lib/sqlalchemy/orm/collections.py @@ -1078,7 +1078,10 @@ def _list_decorators(): start = index.start or 0 if start < 0: start += len(self) - stop = index.stop or len(self) + if index.stop is not None: + stop = index.stop + else: + stop = len(self) if stop < 0: stop += len(self) diff --git a/test/orm/test_collection.py b/test/orm/test_collection.py index b3de03aae..0f54da551 100644 --- a/test/orm/test_collection.py +++ b/test/orm/test_collection.py @@ -127,9 +127,9 @@ class CollectionsTest(fixtures.ORMTest): control = list() def assert_eq(): - self.assert_(set(direct) == canary.data) - self.assert_(set(adapter) == canary.data) - self.assert_(direct == control) + eq_(set(direct), canary.data) + eq_(set(adapter), canary.data) + eq_(direct, control) # assume append() is available for list tests e = creator() @@ -259,6 +259,11 @@ class CollectionsTest(fixtures.ORMTest): control[-2:-1] = values assert_eq() + values = [creator()] + direct[0:0] = values + control[0:0] = values + assert_eq() + if hasattr(direct, '__delitem__') or hasattr(direct, '__delslice__'): for i in range(1, 4): |