I don't think we actually can iterate over > 1000

pull/47/merge
David Trail 9 years ago
parent 0457f07dbb
commit 38de541513

@ -127,7 +127,6 @@ def get_things(after=None):
def remove_things(things): def remove_things(things):
removal_count = 0
for thing in things: for thing in things:
# Seems to be in users's timezone. Unclear. # Seems to be in users's timezone. Unclear.
thing_time = datetime.fromtimestamp(thing.created_utc) thing_time = datetime.fromtimestamp(thing.created_utc)
@ -192,11 +191,5 @@ def remove_things(things):
thing.edit(replacement_text) thing.edit(replacement_text)
if not edit_only: if not edit_only:
thing.delete() thing.delete()
removal_count += 1
return removal_count remove_things(get_things())
things = get_things()
count = remove_things(things)
while count != 0:
things = get_things()
count = remove_things(things)

Loading…
Cancel
Save