tonyair
/
silverstripe-reports
mirror of https://github.com/silverstripe/silverstripe-reports
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

API CHANGE: Added capability for batch actions to indicate failure through red checkboxes (from r94868) (from r96824) 

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/cms/trunk@102701 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
Ingo Schommer 2010-04-13 06:57:12 +00:00
parent a9f8fa2d24
commit a88eb81395
3 changed files with 17 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

BIN
images/tickbox-fail.gif Normal file
View File

Binary file not shown.
Side by Side

After

Width:  |  Height:  |  Size: 64 B

BIN
images/tickbox-greyedout.gif Normal file
View File

Binary file not shown.
Side by Side

After

Width:  |  Height:  |  Size: 64 B

25
javascript/LeftAndMain.BatchActions.js
View File

@ -185,6 +185,9 @@
// write (possibly modified) IDs back into to the hidden field
this.setIDs(ids);
// Reset failure states
$(tree).find('li').removeClass('failed');
var button = this.find(':submit:first');
button.addClass('loading');
@ -216,10 +219,19 @@
if(node && node.parentTreeNode) node.parentTreeNode.removeTreeNode(node);
}
}
if(data.error) {
for(id in data.error) {
var node = tree.getTreeNodeByIdx(id);
$(node).addClass('failed');
}
}
// Deselect all nodes
$(tree).find('li').removeClass('selected');
// reset selection state
// TODO Should unselect all selected nodes as well
jQuery(tree).removeClass('multiselect');
// jQuery(tree).removeClass('multiselect');
// Check if current page still exists, and refresh it.
// Otherwise remove the current form
@ -238,7 +250,7 @@
// close panel
// TODO Coupling with tabs
jQuery('#TreeActions').tabs('select', -1);
// jQuery('#TreeActions').tabs('select', -1);
},
dataType: 'json'
});
@ -256,8 +268,7 @@
// auto-select the current node
var node = this.getTree().firstSelected();
if(node){
node.removeNodeClass('current');
node.addNodeClass('selected');
$(node).removeClass('current').addClass('selected');
node.open();
// Open all existing children, which might trigger further
@ -277,12 +288,10 @@
if(!this._isActive()) return;
if(node.selected) {
node.removeNodeClass('selected');
node.selected = false;
$(node).removeClass('selected').attr('selected', false);
} else {
// Select node
node.addNodeClass('selected');
node.selected = true;
$(node).addClass('selected').attr('selected', true);
// Open node in order to allow proper selection of children
if($(node).hasClass('unexpanded')) {