jQuery Grid Plugin - jqGridForum

Topic RSS

I think it's a small bug for pager!

03/02/2012
08:54

yangzy666

New Member

Members

Forum Posts: 1

Member Since:
03/02/2012

Offline

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.

03/02/2012
12:51

OlegK

Germany

Member

Members

Forum Posts: 1255

Member Since:
10/08/2009

Offline

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

All RSS

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
45 Guest(s)

Currently Browsing this Page:
1 Guest(s)