tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 14:05:37 +02:00
Code Issues Projects Releases Wiki Activity

BUGFIX: fix the bug that add some rules for summary columns even when they are not there.

Browse Source 
git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@64049 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
Normann Lou 2008-10-10 03:21:30 +00:00
parent 2b15a2d5b5
commit a32b42fe19
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
javascript/TableListField.js
View File

@ -41,7 +41,10 @@ TableListField.prototype = {
// TODO Breaks with nested divs // TODO Breaks with nested divs
var summaryCols = $$('tfoot tr.summary td', this); var summaryCols = $$('tfoot tr.summary td', this);
this._summaryDefs = []; this._summaryDefs = [];
if(summaryCols) {
//if(summaryCols) {
//should check summaryCols.length, cos summaryCols will be alway an array, though its length could be 0.
if(summaryCols.length) {
rules['#'+this.id+' table.data tbody input'] = { rules['#'+this.id+' table.data tbody input'] = {
onchange: function(e) { onchange: function(e) {
if (!e) e = window.event; // stupid IE if (!e) e = window.event; // stupid IE
@ -63,7 +66,10 @@ TableListField.prototype = {
} }
Behaviour.register('TableListField_'+this.id,rules); Behaviour.register('TableListField_'+this.id,rules);
if(summaryCols) {
//if(summaryCols) {
//should check summaryCols.length, cos summaryCols will be alway an array, though its length could be 0.
if(summaryCols.length) {
//this._getSummaryDefs(summaryCols); //this._getSummaryDefs(summaryCols);
} }
}, },