paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-10-25 00:53:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

populate_db: Rename confusing sub local variable.

Browse Source 
While the code was technically correct, elsewhere in that function sub
a Subscription object, not a tuple of ints.
This commit is contained in:
Tim Abbott
2016-01-27 13:09:40 -08:00
parent 37e987e250
commit c55ac01ae6
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
zilencer/management/commands/populate_db.py
View File

@@ -658,9 +658,9 @@ def restore_saved_messages():
active=pending_subs[pending_sub])
subscriptions_to_add.append(s)
Subscription.objects.bulk_create(subscriptions_to_add)
for (sub, active) in subscriptions_to_change:
current_subs_obj[sub].active = active
current_subs_obj[sub].save(update_fields=["active"])
for (sub_tuple, active) in subscriptions_to_change:
current_subs_obj[sub_tuple].active = active
current_subs_obj[sub_tuple].save(update_fields=["active"])
subs = {}
for sub in Subscription.objects.all():