pager_query call in conference_manage_assignment function

bigfetch - October 23, 2007 - 18:15
Project:Conference
Version:6.x-1.2-beta2
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:active
Description

This is a tiny, but critical bug. The pager_query and theme('pager') calls in the conference_manage_assignment function have different values for the $element argument. This leads to the theme_pager not producing page links at the bottom of the list of papers, so if the conference has any more than 25 papers submitted they cannot be assigned to a reviewer. It's a very simple fix in conference.module, just changing the $element argument's value from 0 to 5 in the pager_query call.

old line 805:
  $result = pager_query($sql, 25, 0, NULL, $ctype_paper);

new line 805:
  $result = pager_query($sql, 25, 5, NULL, $ctype_paper);

#1

odisey - November 5, 2009 - 05:04
Version:5.x-1.x-dev» 6.x-1.2-beta2

6.x-1.2-beta2

This is a tiny, but critical bug. The pager_query and theme('pager') calls in the conference_manage_assignment function have different values for the $element argument. This leads to the theme_pager not producing page links at the bottom of the list of papers, so if the conference has any more than 25 papers submitted they cannot be assigned to a reviewer. It's a very simple fix in conference.module, just changing the $element argument's value from 0 to 5 in the pager_query call.

Edit one:

old line 591:
$result = pager_query($sql, 25, 0, NULL, $ctype_paper);

new line 591:
$result = pager_query($sql, 25, 5, NULL, $ctype_paper);

Edit two:

old line 2512:

$result = pager_query($sql, 25, 0, NULL, $ctype_paper);

new line 2512:
$result = pager_query($sql, 25, 5, NULL, $ctype_paper);

 
 

Drupal is a registered trademark of Dries Buytaert.