OlegK on I think it's a small bug for pager!

Fri, 03 Feb 2012 12:51:38 +0200

You are right. The bug exist in the two lines of code inside of $('.ui-pg-selbox',"#"+pgcnt).bind('change',function() {...}.:

if(tp) { $('.ui-pg-selbox',ts.p.pager).val(this.value); }
else if(ts.p.toppager) { $('.ui-pg-selbox',ts.p.toppager).val(this.value); }

The problem is that the variable tp will be either "_t_list_toppager" or "_pager" (if the pager has id="pager"). So the if(tp) will always work. So we have two problem:

The first one if the number of rows are changed in the bottom pager and the additional top pager exist. In the face the else if part will not work and the top pager will have the old valus.

The second problem exist if only the top pager exist and no pager parameter is defined. In the case the ts.p.pager will be "" and the existing code will change the number of rows in all pager of all grids. Which is also wrong.

As the fix I suggest to change the above two lines of the code to

if(ts.p.pager) { $('.ui-pg-selbox',ts.p.pager).val(this.value); }
if(ts.p.toppager) { $('.ui-pg-selbox',ts.p.toppager).val(this.value); }

Best regards
Oleg

yangzy666 on I think it's a small bug for pager!

Fri, 03 Feb 2012 08:54:27 +0200

Hello everyone,

I found that when i show both toppager and bottompager,the rowNum in both of them is not consistent.

for example,when i change rowNum in toppager,the bottompager will follow it,conversely,it won't.

jQuery Grid Plugin - jqGrid - Topic: I think it's a small bug for pager!

OlegK on I think it's a small bug for pager!

yangzy666 on I think it's a small bug for pager!