From cc660779438723a2d3b12e35459a9bbba3c9246b Mon Sep 17 00:00:00 2001
From: Mark McDowall <markus.mcd5@gmail.com>
Date: Wed, 29 Jan 2014 21:52:50 -0800
Subject: [PATCH] Cancelling adding an indexer will stop listening to save
 event

Fixed: Cancelling adding indexer will not break saving
---
 src/UI/Settings/Indexers/EditTemplate.html |  4 ++--
 src/UI/Settings/Indexers/EditView.js       | 10 +++++++++-
 src/UI/Settings/SettingsModelBase.js       |  6 +++++-
 3 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/src/UI/Settings/Indexers/EditTemplate.html b/src/UI/Settings/Indexers/EditTemplate.html
index f9c3779ca..0bf5bdc06 100644
--- a/src/UI/Settings/Indexers/EditTemplate.html
+++ b/src/UI/Settings/Indexers/EditTemplate.html
@@ -1,5 +1,5 @@
 <div class="modal-header">
-    <button type="button" class="close" data-dismiss="modal" aria-hidden="true">&times;</button>
+    <button type="button" class="close x-cancel"aria-hidden="true">&times;</button>
     {{#if id}}
         <h3>Edit</h3>
     {{else}}
@@ -42,7 +42,7 @@
 
     <span class="x-activity"></span>
 
-    <button class="btn" data-dismiss="modal">cancel</button>
+    <button class="btn x-cancel">cancel</button>
 
     <div class="btn-group">
         <button class="btn btn-primary x-save">save</button>
diff --git a/src/UI/Settings/Indexers/EditView.js b/src/UI/Settings/Indexers/EditView.js
index 05793bf1c..dca99a2ac 100644
--- a/src/UI/Settings/Indexers/EditView.js
+++ b/src/UI/Settings/Indexers/EditView.js
@@ -18,7 +18,8 @@ define(
 
             events: {
                 'click .x-save'        : '_save',
-                'click .x-save-and-add': '_saveAndAdd'
+                'click .x-save-and-add': '_saveAndAdd',
+                'click .x-cancel'      : '_cancel'
             },
 
             initialize: function (options) {
@@ -68,6 +69,13 @@ define(
                         self.ui.activity.empty();
                     });
                 }
+            },
+
+            _cancel: function () {
+                if (this.model.isNew()) {
+                    this.model.destroy();
+                    vent.trigger(vent.Commands.CloseModalCommand);
+                }
             }
         });
 
diff --git a/src/UI/Settings/SettingsModelBase.js b/src/UI/Settings/SettingsModelBase.js
index 5b2b5bec3..834f10469 100644
--- a/src/UI/Settings/SettingsModelBase.js
+++ b/src/UI/Settings/SettingsModelBase.js
@@ -10,6 +10,7 @@ define(
 
             initialize: function () {
                 this.listenTo(vent, vent.Commands.SaveSettings, this.saveSettings);
+                this.listenTo(this, 'destroy', this._stopListening);
             },
 
             saveSettings: function () {
@@ -28,8 +29,11 @@ define(
                 }
 
                 return undefined;
-            }
+            },
 
+            _stopListening: function () {
+                this.stopListening(vent, vent.Commands.SaveSettings);
+            }
         });
 
         return AsChangeTrackingModel.call(model);