From 86c554e3d358d13ffba5da33a918bb14d4d83775 Mon Sep 17 00:00:00 2001
From: Mads Rasmussen <mads@umbraco.com>
Date: Wed, 9 May 2018 09:20:23 +0200
Subject: [PATCH] add option to prevent redirect of succesfull saves to new
 route

---
 .../src/common/services/contenteditinghelper.service.js  | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/Umbraco.Web.UI.Client/src/common/services/contenteditinghelper.service.js b/src/Umbraco.Web.UI.Client/src/common/services/contenteditinghelper.service.js
index e8e10c5d63..6c3f1054f7 100644
--- a/src/Umbraco.Web.UI.Client/src/common/services/contenteditinghelper.service.js
+++ b/src/Umbraco.Web.UI.Client/src/common/services/contenteditinghelper.service.js
@@ -44,6 +44,7 @@ function contentEditingHelper(fileManager, $q, $location, $routeParams, notifica
                 throw "args.saveMethod is not defined";
             }
 
+            var redirectOnSuccess = args.redirectOnSuccess !== undefined ? args.redirectOnSuccess : true;
             var redirectOnFailure = args.redirectOnFailure !== undefined ? args.redirectOnFailure : true;
 
             var self = this;
@@ -63,6 +64,7 @@ function contentEditingHelper(fileManager, $q, $location, $routeParams, notifica
                         self.handleSuccessfulSave({
                             scope: args.scope,
                             savedContent: data,
+                            redirectOnSuccess: redirectOnSuccess,
                             rebindCallback: function() {
                                 rebindCallback.apply(self, [args.content, data]);
                             }
@@ -562,7 +564,10 @@ function contentEditingHelper(fileManager, $q, $location, $routeParams, notifica
                 throw "args.savedContent cannot be null";
             }
 
-            if (!this.redirectToCreatedContent(args.redirectId ? args.redirectId : args.savedContent.id)) {
+            // the default behaviour is to redirect on success. This adds option to prevent when false
+            args.redirectOnSuccess = args.redirectOnSuccess !== undefined ? args.redirectOnSuccess : true;
+            
+            if (!args.redirectOnSuccess || !this.redirectToCreatedContent(args.redirectId ? args.redirectId : args.savedContent.id)) {
 
                 //we are not redirecting because this is not new content, it is existing content. In this case
                 // we need to detect what properties have changed and re-bind them with the server data.
@@ -586,7 +591,7 @@ function contentEditingHelper(fileManager, $q, $location, $routeParams, notifica
          */
         redirectToCreatedContent: function (id, modelState) {
 
-            //only continue if we are currently in create mode and if there is no 'Name' modelstate errors
+            //only continue if we are currently in create mode and not in infinite mode and if there is no 'Name' modelstate errors
             // since we need at least a name to create content.
             if ($routeParams.create && (isValidIdentifier(id) && (!modelState || !modelState["Name"]))) {