Skip to content

Commit 7f418ed

Browse files
committed
No need to keep around an extra local variable
1 parent 189d300 commit 7f418ed

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

annotator/annotation.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -95,13 +95,14 @@ def search_raw(cls, query=None, params=None, raw_result=False,
9595
def _build_query(cls, query=None, offset=None, limit=None, sort=None, order=None):
9696
if query is None:
9797
query = {}
98+
else:
99+
query = dict(query) # shallow copy
98100

99101
# Pop 'before' and 'after' parameters out of the query
100-
copied_query = dict(query)
101-
after = copied_query.pop('after', None)
102-
before = copied_query.pop('before', None)
102+
after = query.pop('after', None)
103+
before = query.pop('before', None)
103104

104-
q = super(Annotation, cls)._build_query(copied_query, offset, limit, sort, order)
105+
q = super(Annotation, cls)._build_query(query, offset, limit, sort, order)
105106

106107
# Create range query from before and/or after
107108
if before is not None or after is not None:

0 commit comments

Comments
 (0)