You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by je...@apache.org on 2014/02/14 21:56:36 UTC

git commit: updated refs/heads/master to e1e16a2

Updated Branches:
  refs/heads/master 443acac92 -> e1e16a230


CLOUDSTACK-6120: UI > listView widget > sorting order > fire only one sorting API call(updateXXXXXXX&sortKey=n&id=UUID) for items who have the same UUID.
e.g. An Template/ISO of multiple zones have the same UUID.


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

Branch: refs/heads/master
Commit: e1e16a2301a75c6746bdb69774a6d9fb996455ca
Parents: 443acac
Author: Jessica Wang <je...@apache.org>
Authored: Fri Feb 14 12:55:53 2014 -0800
Committer: Jessica Wang <je...@apache.org>
Committed: Fri Feb 14 12:55:53 2014 -0800

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


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/e1e16a23/ui/scripts/ui/widgets/listView.js
----------------------------------------------------------------------
diff --git a/ui/scripts/ui/widgets/listView.js b/ui/scripts/ui/widgets/listView.js
index cb4544c..daf86ff 100644
--- a/ui/scripts/ui/widgets/listView.js
+++ b/ui/scripts/ui/widgets/listView.js
@@ -1180,8 +1180,17 @@
                                 if (actionName == 'moveDrag') return false;
 
                                 rowActions[actionName]($tr);
+                                var map1 = {};
                                 $tr.closest('tbody').find('tr').each(function() {
+                                	/* 
+                                	 * fire only one sorting API call(updateXXXXXXX&sortKey=n&id=UUID) for items who have the same UUID. 
+                                	 * e.g. An Template/ISO of multiple zones have the same UUID.
+                                	 */
+                                	var objId = $(this).data('json-obj').id;
+                                	if(!(objId in map1)) { 
                                     sort($(this), action);
+                                		map1[objId] = 1;
+                                	}                                       
                                 });
                                 $tr.closest('.data-table').dataTable('selectRow', $tr.index());
 
@@ -1203,7 +1212,15 @@
                             rowActions._std($tr, function() {});
 
                             $tr.closest('tbody').find('tr').each(function() {
+                            	/* 
+                            	 * fire only one sorting API call(updateXXXXXXX&sortKey=n&id=UUID) for items who have the same UUID. 
+                            	 * e.g. An Template/ISO of multiple zones have the same UUID.
+                            	 */
+                            	var objId = $(this).data('json-obj').id;
+                            	if(!(objId in map1)) { 
                                 sort($(this), reorder.moveDrag);
+                            	    map1[objId] = 1;
+                            	}
                             });
                         }
                     });