tony on [Bug] definition of clearToolbar - v is not undefined, is not initialized.

Sat, 14 Jan 2012 01:06:16 +0200

Hello,

See here:

https://github.com/tonytomov/jqGrid/commit/36ae4008fbf4d4d47d7993c28d523c5f356e4f20

Thanks.

Kind Regards

Scevra on [Bug] definition of clearToolbar - v is not undefined, is not initialized.

Fri, 13 Jan 2012 17:03:42 +0200

Hi tony,

I saw the corrections you made, but I think I found another bug in the committed file js/grid.custom.js line 199-200:

https://github.com/tonytomov/jqGrid/commit/bd5ac35861c269ffa48f80dbdfd37d6d1be7dcb4#L0L199

Into the each part, in the last version (updated in GitHub) you have removed this line:


v = (this.searchoptions && this.searchoptions.defaultValue) ? this.searchoptions.defaultValue : "";

and you add only this line:


 if(this.searchoptions && this.searchoptions.defaultValue !== undefined) { v = this.searchoptions.defaultValue; }

But doing this, v is undefined the first time of the each (and all works well), but not in the future.
v can't be updated from the previous value if the IF condition fails (and that my case, in my application).
This is a problem, cause there are a lot of if ( v !== undefined ) in the code after that line.

So I think v has to be initialized every time after the $.each($t.p.colModel,function(i,n){.

I changed my code like this, and it works fine to me:

var clearToolbar = function(trigger){
               var sdata={}, v, j=0, nm;
               trigger = (typeof trigger != 'boolean') ? true : trigger;
               $.each($t.p.colModel,function(i,n){
                   v = undefined;
                    if(this.searchoptions && this.searchoptions.defaultValue !== undefined) { v = this.searchoptions.defaultValue; }
                   nm = this.index || this.name;
                   switch (this.stype) {

Is a bug, right?

Bye,
Scevra

jQuery Grid Plugin - jqGrid - Topic: [Bug] definition of clearToolbar - v is not undefined, is not initialized.

tony on [Bug] definition of clearToolbar - v is not undefined, is not initialized.

Scevra on [Bug] definition of clearToolbar - v is not undefined, is not initialized.