From 952cfc6d5fb9b760b7da84f40f3813d6ac46e120 Mon Sep 17 00:00:00 2001 From: Mads Rasmussen Date: Tue, 9 Aug 2016 12:13:56 +0200 Subject: [PATCH] Fixes: U4-8830 Redirect url dashboard - add confirm dialog when removing redirect --- .../developer/redirecturls.controller.js | 20 +++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/dashboard/developer/redirecturls.controller.js b/src/Umbraco.Web.UI.Client/src/views/dashboard/developer/redirecturls.controller.js index b3cedf74f2..129b7a0753 100644 --- a/src/Umbraco.Web.UI.Client/src/views/dashboard/developer/redirecturls.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/dashboard/developer/redirecturls.controller.js @@ -76,19 +76,19 @@ } function removeRedirect(redirectToDelete) { + var toggleConfirm = confirm('Are you sure you want to remove the redirect from ' + '"' + redirectToDelete.originalUrl + '"' + " to " + '"' + redirectToDelete.destinationUrl + '"' + " ?"); - redirectUrlsResource.deleteRedirectUrl(redirectToDelete.redirectId).then(function () { + if (toggleConfirm) { + redirectUrlsResource.deleteRedirectUrl(redirectToDelete.redirectId).then(function () { - var index = vm.redirectUrls.indexOf(redirectToDelete); - vm.redirectUrls.splice(index, 1); - notificationsService.success("Redirect Url Removed!", "Redirect Url " + redirectToDelete.Url + " has been deleted"); - - }, function(error) { - - notificationsService.error("Redirect Url Error!", "Redirect Url " + redirectToDelete.Url + " was not deleted"); - - }); + var index = vm.redirectUrls.indexOf(redirectToDelete); + vm.redirectUrls.splice(index, 1); + notificationsService.success("Redirect Url Removed!", "Redirect Url has been deleted"); + }, function(error) { + notificationsService.error("Redirect Url Error!", "Redirect Url was not deleted"); + }); + } } function disableUrlTracker() {