From 1216e598f91b18787f2395dfdce706104941949b Mon Sep 17 00:00:00 2001
From: chrisb <chrisb@netflix.com>
Date: Tue, 1 Dec 2015 09:59:11 -0800
Subject: [PATCH] move notificationHours binding to ctrl on canary stage

---
 .../modules/netflix/pipeline/stage/canary/canaryStage.html  | 2 +-
 .../modules/netflix/pipeline/stage/canary/canaryStage.js    | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/scripts/modules/netflix/pipeline/stage/canary/canaryStage.html b/app/scripts/modules/netflix/pipeline/stage/canary/canaryStage.html
index 06b8459a3ec..63cf29224cb 100644
--- a/app/scripts/modules/netflix/pipeline/stage/canary/canaryStage.html
+++ b/app/scripts/modules/netflix/pipeline/stage/canary/canaryStage.html
@@ -76,7 +76,7 @@ <h5>Analysis Config</h5>
     </span>
   </stage-config-field>
   <stage-config-field label="Notification Hours" help-key="pipeline.config.canary.notificationHours" field-columns="3">
-    <input type="text" ng-model="notificationHours"
+    <input type="text" ng-model="canaryStageCtrl.notificationHours"
            class="form-control input-sm" ng-change="canaryStageCtrl.splitNotificationHours()" />
   </stage-config-field>
   <stage-config-field label="Frequency" help-key="pipeline.config.canary.canaryInterval" field-columns="3">
diff --git a/app/scripts/modules/netflix/pipeline/stage/canary/canaryStage.js b/app/scripts/modules/netflix/pipeline/stage/canary/canaryStage.js
index 45dc36c75ab..ef120231cf2 100644
--- a/app/scripts/modules/netflix/pipeline/stage/canary/canaryStage.js
+++ b/app/scripts/modules/netflix/pipeline/stage/canary/canaryStage.js
@@ -49,10 +49,10 @@ module.exports = angular.module('spinnaker.netflix.pipeline.stage.canaryStage',
       $scope.accounts = accounts;
     });
 
-    $scope.notificationHours = $scope.stage.canary.canaryConfig.canaryAnalysisConfig.notificationHours.join(',');
+    this.notificationHours = $scope.stage.canary.canaryConfig.canaryAnalysisConfig.notificationHours.join(',');
 
-    this.splitNotificationHours = function() {
-      var hoursField = $scope.notificationHours || '';
+    this.splitNotificationHours = () => {
+      var hoursField = this.notificationHours || '';
       $scope.stage.canary.canaryConfig.canaryAnalysisConfig.notificationHours = _.map(hoursField.split(','), function(str) {
         if (!parseInt(str.trim()).isNaN) {
           return parseInt(str.trim());