From f88de17adf85550b824e4ef2235c31e04ae1cb5d Mon Sep 17 00:00:00 2001 From: Joe Fiorini Date: Wed, 3 Jun 2015 10:21:51 -0400 Subject: [PATCH] Use clone of params object to avoid time traveling --- .../data-services/resolve-versions.factory.js | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/awx/ui/static/js/system-tracking/data-services/resolve-versions.factory.js b/awx/ui/static/js/system-tracking/data-services/resolve-versions.factory.js index bf6898c34a..d52408bd36 100644 --- a/awx/ui/static/js/system-tracking/data-services/resolve-versions.factory.js +++ b/awx/ui/static/js/system-tracking/data-services/resolve-versions.factory.js @@ -10,14 +10,20 @@ function resolveVersions(service, _, results) { } function resolveEmpties(result) { - if (_.isEmpty(result.versions)) { - var originalStartDate = result.dateRange.from; - result.dateRange.from = originalStartDate.clone().subtract(1, 'year'); - result.dateRange.to = originalStartDate; - return [result, service.getVersion(result)]; + + var newResult = _.merge({}, result); + + if (_.isEmpty(newResult.versions)) { + var originalStartDate = result.dateRange.from.clone(); + + newResult.dateRange.from = originalStartDate.clone().subtract(1, 'year'); + newResult.dateRange.to = originalStartDate; + + return [newResult, service.getVersion(newResult)]; } - return [result, _.promise(result.versions)]; + return [newResult, _.promise(newResult.versions)]; + } function resolveDuplicates(nonEmptyResults) {