Merge pull request #1 from w33ble/fix/advanced-listener

unhook the change:config listener on destroy
spalger · Sep 12, 2015 · 075a18e · 075a18e
2 parents 5a4cadb + 34992c0
commit 075a18e
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/plugins/kibana/public/settings/sections/advanced/index.js b/src/plugins/kibana/public/settings/sections/advanced/index.js
@@ -42,8 +42,12 @@ define(function (require) {
           .value();
         }
 
+        // react to changes of the config values
+        var unhook = $rootScope.$on('change:config', readConfigVals);
+        $scope.$on('$destroy', unhook);
+
+        // initial config setup
         readConfigVals();
-        $rootScope.$on('change:config', readConfigVals);
       }
     };
   });