largersmallernormaltext version of this page
Main | My View | View Issues | Change Log | Roadmap | Docs
  

Viewing Issue Simple Details [ Jump to Notes ] [ View Advanced ] [ Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0006638 [phplist] Bounce Management major always 16-06-06 10:28 18-02-08 14:08
Reporter hola View Status public  
Assigned To
Priority normal Resolution fixed  
Status resolved   Product Version 2.10.2
Summary 0006638: bounce.php
Description Is it me or does the bounce.php file ignore my tick to delete user? I had a look at bounce.php and it seems to get the deleteuser checkbox using a $_POST when on the html form generated on the bounce page the form method has been set to GET??

Plus the $deletebounce var seems to be set with true or false outside of the bounce file because its value is honoured and I can delete or not delete bounce records.

Help!
Additional Information
Tags No tags attached.
Attached Files

- Relationships
duplicate of 0005444resolvedmichiel Features in bounce 
related to 0003721closed phplist 2.10.x 

-  Notes
(0029828)
h2b2 (reporter)
27-07-07 13:38

 Tomwing reported "BUG when trying to delete a BOUNCE" on the forum, http://forums.phplist.com/viewtopic.php?p=33934#33934 [^] and includes the followsing fix:


File: bounces.php

Line Number 6:

// This is wrong, because the delete command is a GET Link:
if (isset($_POST['delete']) && $_POST['delete']) {
# delete the index in delete
print $GLOBALS['I18N']->get('deleting').' '.$_POST['delete']."..\n";
if ($GLOBALS["require_login"] && !isSuperUser()) {
} else {
deleteBounce($_POST['delete']);
}
print $GLOBALS['I18N']->get('done') . "
<hr>
\n";
}

// this is right:
if (isset($_REQUEST['delete']) && $_REQUEST['delete']) {
# delete the index in delete
print $GLOBALS['I18N']->get('deleting').' '.$_POST['delete']."..\n";
if ($GLOBALS["require_login"] && !isSuperUser()) {
} else {
deleteBounce($_REQUEST['delete']);
}
print $GLOBALS['I18N']->get('done') . "
<hr>
\n";
}
(0029835)
hernan (developer)
27-07-07 15:57

 Thanks, will be implemented in 2.10.5
And btw, it was the bounces.php file, no bounce.php

Mantis 1.1.6[^]
Copyright © 2000 - 2008 Mantis Group
Powered by Mantis Bugtracker