[F] System mail log: Wrong multipage links [C-StefanT]
#1
If you search your system mail logs and the result is on several pages there's a problem with the links in the pagination:
index.php?module=tools/mailerrors&page=2subject=xyz

The & is missing and because of that it shows all logs if you switch between the pages.

admin/modules/tools/mailerrors.php
PHP Code:
$additional_criteria implode("&"$additional_criteria);
...
echo 
"<br />".draw_admin_pagination($mybb->input['page'], $per_page$total_rows"index.php?module=tools/mailerrors&amp;page={page}{$additional_criteria}"); 
Greets,
Michael
-------------
[Image: donation_drive_sig.png]
#2
Can't $additional_criteria just be changed to:

PHP Code:
$additional_criteria "&amp;".implode("&amp;"$additional_criteria); 

...or is that just me being lazy?... Toungue
#3
(07-15-2009, 09:24 AM)Tomm M Wrote:  Can't $additional_criteria just be changed to:

PHP Code:
$additional_criteria "&amp;".implode("&amp;"$additional_criteria); 

...or is that just me being lazy?... Toungue

It would be cleaner to have it in the echo line imho.
#4
But if there is no additional criteria, the page URL would be "index.php?module=tools/mailerrors&page=2&"...
#5
It still would be with your solution.
#6
Touché... Toungue
#7
PHP Code:
if($additional_criteria)
{
$additional_criteria "&amp;".implode("&amp;"$additional_criteria);


?
#8
Thank you for your bug report.

This bug has been fixed in our internal code repository. Please note that the problem will not be fixed here until these forums are updated.

With regards,
MyBB Group


Forum Jump:


Users browsing this thread: 1 Guest(s)