From aa71acef19ee02c33d0202e96d8d38d57ad9b313 Mon Sep 17 00:00:00 2001
From: Vincent Fournier <v.fournier11@gmail.com>
Date: Tue, 18 Aug 2015 15:59:44 -0400
Subject: [PATCH] Fix checkbox bug

Change-Id: Icf6194c78bf3d124c83afb3e455d9b023775d084
---
 app/components/datasource/datasource.js   | 10 ++++++++++
 app/components/directive/table/table.html |  4 ++--
 app/components/directive/table/table.js   |  7 ++++++-
 3 files changed, 18 insertions(+), 3 deletions(-)

diff --git a/app/components/datasource/datasource.js b/app/components/datasource/datasource.js
index 2acbb2b..fa6f20d 100644
--- a/app/components/datasource/datasource.js
+++ b/app/components/datasource/datasource.js
@@ -83,6 +83,16 @@ angular.module('bansho.datasource', ['bansho.surveil'])
 
                     listeners[datasourceId].push(callback);
                 },
+                isAllCheckedTable: function (datasourceId) {
+                    var isAllChecked = true;
+                    angular.forEach(filteredData[datasourceId], function (entry) {
+                        if (!entry.is_checked) {
+                            isAllChecked = false;
+                        }
+                    });
+
+                    return isAllChecked;
+                },
                 setAllCheckTable: function (datasourceId, isChecked) {
                     config[datasourceId].isCheckAll = isChecked;
                     angular.forEach(filteredData[datasourceId], function (entry) {
diff --git a/app/components/directive/table/table.html b/app/components/directive/table/table.html
index cf1f3e4..ae7662b 100644
--- a/app/components/directive/table/table.html
+++ b/app/components/directive/table/table.html
@@ -3,7 +3,7 @@
     <thead data-ng-repeat="head in ['moving-thead', 'static-thead']" data-ng-class="head">
     <tr>
       <th data-ng-show="checkColumn" class="data-table__checkbox">
-        <md-checkbox aria-label='Check all' data-ng-model="isCheckAll" data-ng-change="onCheckChange()"></md-checkbox>
+        <md-checkbox type="checkbox" aria-label='Check all' data-ng-checked="isCheckAll" data-ng-click="onClick()"></md-checkbox>
       </th>
       <th ng-repeat="column in columns" class="data-table__{{column.attributes.class}}">
         {{column.title}} <i class="ico-up-dir"></i>
@@ -13,7 +13,7 @@
     <tbody data-ng-repeat="(groupByKey, groupByItems) in entries | groupBy:'host_host_name'">
     <tr data-ng-repeat="entry in groupByItems | noRepeat:this | wrappableStyle:this">
       <td data-ng-show="checkColumn">
-        <md-checkbox aria-label='Check' data-ng-model="entry.is_checked"></md-checkbox>
+        <md-checkbox aria-label='Check' data-ng-model="entry.is_checked" data-ng-change="entryOnClick(entry.is_checked)"></md-checkbox>
       </td>
       <td data-ng-repeat="cell in columns"
           data-ng-click="createUrl(entry, cell.attributes)">
diff --git a/app/components/directive/table/table.js b/app/components/directive/table/table.js
index b5bbf1e..ad2cb4e 100644
--- a/app/components/directive/table/table.js
+++ b/app/components/directive/table/table.js
@@ -65,9 +65,14 @@ angular.module('bansho.table', ['bansho.datasource',
                         }
                     };
 
-                    $scope.onCheckChange = function () {
+                    $scope.onClick = function () {
+                        $scope.isCheckAll = !$scope.isCheckAll;
                         datasource.setAllCheckTable($scope.datasourceId, $scope.isCheckAll);
                     };
+
+                    $scope.entryOnClick = function () {
+                        $scope.isCheckAll = datasource.isAllCheckedTable($scope.datasourceId);
+                    };
                 }]
         };
     })