From f6367e21b850f582c72b3264e266c9d1f4033c18 Mon Sep 17 00:00:00 2001 From: Brian Gough Date: Wed, 9 Mar 2016 14:33:19 +0000 Subject: [PATCH] give separate error for archive in progress vs completed --- services/track-changes/app/coffee/PackManager.coffee | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/services/track-changes/app/coffee/PackManager.coffee b/services/track-changes/app/coffee/PackManager.coffee index 8e9a928d3b..449e07e110 100644 --- a/services/track-changes/app/coffee/PackManager.coffee +++ b/services/track-changes/app/coffee/PackManager.coffee @@ -407,10 +407,12 @@ module.exports = PackManager = PackManager.getPackFromIndex doc_id, pack_id, (err, result) -> return callback(err) if err? return callback new Error("pack not found in index") if not result? - if result.inS3? - callback new Error("pack archiving already in progress") + if result.inS3 + return callback new Error("pack archiving already done") + else if result.inS3? + return callback new Error("pack archiving already in progress") else - callback() + return callback() markPackAsArchiveInProgress: (project_id, doc_id, pack_id, callback) -> logger.log {project_id, doc_id}, "marking pack as archive in progress status"