fix(app):fix additional not save warning EE-799 (#5161)

* fix(app):fix additional not save warning EE-799

* fix additional warning when user leave page

* fix additional warning when user leave page in buildImageController.js

* fix docker build controller additional warning message

* fix changes required from reviews

* - refactored ondestroy hook function to align it closer to (below) oninit
- removed duplicated hook func duplication in configurationController

Co-authored-by: zees-dev <dev.786zshan@gmail.com>

app/portainer/views/custom-templates/create-custom-template-view/createCustomTemplateViewController.js

@@ -189,6 +189,10 @@ class CreateCustomTemplateViewController {
     };
   }
 
+  $onDestroy() {
+    this.state.isEditorDirty = false;
+  }
+
   async uiCanExit() {
     if (this.state.Method === 'editor' && this.formValues.FileContent && this.state.isEditorDirty) {
       return this.ModalService.confirmWebEditorDiscard();

app/portainer/views/custom-templates/edit-custom-template-view/editCustomTemplateViewController.js

@@ -123,6 +123,10 @@ class EditCustomTemplateViewController {
       }
     };
   }
+
+  $onDestroy() {
+    this.state.isEditorDirty = false;
+  }
 }
 
 export default EditCustomTemplateViewController;

app/portainer/views/stacks/create/createStackController.js

@@ -53,6 +53,10 @@ angular
       }
     };
 
+    $scope.$on('$destroy', function() {
+      $scope.state.isEditorDirty = false;
+    })
+
     $scope.onChangeFormValues = onChangeFormValues;
 
     $scope.addEnvironmentVariable = function () {

app/portainer/views/stacks/edit/stackController.js

@@ -70,6 +70,10 @@ angular.module('portainer.app').controller('StackController', [
       }
     };
 
+    $scope.$on('$destroy', function() {
+      $scope.state.isEditorDirty = false;
+    })
+
     $scope.handleEnvVarChange = handleEnvVarChange;
     function handleEnvVarChange(value) {
       $scope.formValues.Env = value;