You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by bf...@apache.org on 2014/06/16 23:12:22 UTC

git commit: updated refs/heads/master to 04b1e31

Repository: cloudstack
Updated Branches:
  refs/heads/master 8cf5192a6 -> 04b1e3168


List view, multiSelect: Fix issue where $listView isn't always referenced


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/04b1e316
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/04b1e316
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/04b1e316

Branch: refs/heads/master
Commit: 04b1e3168220df558f6aad353514b460c91b115b
Parents: 8cf5192
Author: Brian Federle <bf...@gmail.com>
Authored: Mon Jun 16 14:11:56 2014 -0700
Committer: Brian Federle <bf...@gmail.com>
Committed: Mon Jun 16 14:11:56 2014 -0700

----------------------------------------------------------------------
 ui/scripts/ui/widgets/listView.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/04b1e316/ui/scripts/ui/widgets/listView.js
----------------------------------------------------------------------
diff --git a/ui/scripts/ui/widgets/listView.js b/ui/scripts/ui/widgets/listView.js
index a93c240..16138e4 100644
--- a/ui/scripts/ui/widgets/listView.js
+++ b/ui/scripts/ui/widgets/listView.js
@@ -88,7 +88,7 @@
 
                 // Make sure the master checkbox is unselected
                 if (multiSelect) {
-                    $listView.find('input.multiSelectMasterCheckbox').attr('checked', false);
+                    $instanceRow.closest('.list-view').find('input.multiSelectMasterCheckbox').attr('checked', false);
                 }
 
                 var externalLinkAction = action.externalLink;
@@ -1078,7 +1078,7 @@
                         var enabled = checked || (numRowsChecked > 0);
                         toggleMultiSelectActions(enabled);
 
-                        $listView.find('input.multiSelectMasterCheckbox').attr('checked', (numRows === numRowsChecked));
+                        $td.closest('.list-view').find('input.multiSelectMasterCheckbox').attr('checked', (numRows === numRowsChecked));
                     });
 
                 $td.append(