Running version 1.4.19 of Squirrelmail, users hit the 'list all' button on the Address Book Search page, from compose.php -- the LDAP results were greater than the $maxrows = 250 (in functions/abook_ldap_server.php) in addition to the personal address book. Proceeded to download & install 1.5.2 and noticed the $maxrows variable is still in abook_ldap_server.php & the results from hitting the 'list all' button aren't paginated.
1. Would it make sense to pull out some of the variables from the abook_ldap_server.php and put them into config.php?
2. Since the address book with version 1.5.2 is paginated, would it make sense to paginate the results on the Address Book Search page?