Make sure to delete any nodes that need to be deleted before attempting to associate

This commit is contained in:
Michael Abashian 2017-01-27 11:23:20 -05:00
parent ae057d2bd5
commit 2dbc3abc34

View File

@ -296,7 +296,11 @@ export default ['$scope', 'WorkflowService', 'generateList', 'TemplateList', 'Pr
});
});
$q.all(disassociatePromises.concat(editNodePromises))
let deletePromises = $scope.treeData.data.deletedNodes.map(function(nodeId) {
return TemplatesService.deleteWorkflowJobTemplateNode(nodeId);
});
$q.all(disassociatePromises.concat(editNodePromises, deletePromises))
.then(function() {
let associatePromises = $scope.associateRequests.map(function(request) {
@ -307,11 +311,7 @@ export default ['$scope', 'WorkflowService', 'generateList', 'TemplateList', 'Pr
});
});
let deletePromises = $scope.treeData.data.deletedNodes.map(function(nodeId) {
return TemplatesService.deleteWorkflowJobTemplateNode(nodeId);
});
$q.all(associatePromises.concat(deletePromises))
$q.all(associatePromises)
.then(function() {
$scope.closeDialog();
});