From 1e26ea1f56dc891dbedb4fd1f5c5a142925c13de Mon Sep 17 00:00:00 2001 From: Rob Hudson Date: Wed, 21 Nov 2012 15:07:44 -0800 Subject: [PATCH] Moved current_version clearing to top before deleting versions --- mkt/zadmin/management/commands/collapse_versions.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/mkt/zadmin/management/commands/collapse_versions.py b/mkt/zadmin/management/commands/collapse_versions.py index 2cc77ed3747..70415ccf3c7 100644 --- a/mkt/zadmin/management/commands/collapse_versions.py +++ b/mkt/zadmin/management/commands/collapse_versions.py @@ -48,6 +48,12 @@ def do_collapsing(): print 'No older versions found.' continue + # Clear the current_version. It gets re-set when we're done. + cursor.execute(''' + UPDATE addons SET current_version=NULL + WHERE id=%s''', (app.id,)) + print 'Cleared current_version for app [%s]' % app.id + for old_version in old_versions: # Hosted app reviews' version column is always NULL. @@ -185,7 +191,6 @@ def do_collapsing(): file_.update(reviewed=version.reviewed) # Call `update_version` to set the current_version properly. - app.update(_current_version=None) updated = app.update_version() # Call app.save to invalidate and re-index, etc.