Wbl bugfix

This commit is contained in:
Mikko Husari 2008-12-10 22:27:57 +02:00
parent 8f677cca9d
commit 3addb56e91
3 changed files with 10 additions and 3 deletions

View File

@ -295,6 +295,7 @@ function updateMessages($flag, $content_type, $emailaddresses, $mail_id_array, $
} }
/** /**
* @author Mikko Husari <husku@husku.net>
* Update white/blaclist function * Update white/blaclist function
* @param string $flag 'W', 'B' * @param string $flag 'W', 'B'
* @param array $emailaddresses recipient email address(es) * @param array $emailaddresses recipient email address(es)
@ -318,6 +319,8 @@ function updateWblist($flag, $emailaddresses, $mail_id_array, $action) {
// Check if logged in user is admin or logged in user is trying to delete his own messages // Check if logged in user is admin or logged in user is trying to delete his own messages
if ( Auth::isMailAdmin() || in_array($recip_email, $emailaddresses) ) { if ( Auth::isMailAdmin() || in_array($recip_email, $emailaddresses) ) {
switch ($action) { switch ($action) {
case translate('Whitelist'):
case translate('Blacklist'):
case translate('Whitelist by strict addressing'): case translate('Whitelist by strict addressing'):
case translate('Blacklist by strict addressing'): case translate('Blacklist by strict addressing'):
$rid = $db->mail2userid($recip_email); $rid = $db->mail2userid($recip_email);

View File

@ -90,9 +90,11 @@ if ( ! isset($mail_id_array) && $action != translate('Delete All') )
elseif ( isset( $action ) ) { elseif ( isset( $action ) ) {
switch ( $action ) { switch ( $action ) {
case translate('Whitelist'):
updateWblist('W', $_SESSION['sessionMail'], $mail_id_array, $action);
case translate('Release'): case translate('Release'):
case translate('Release/Request release'): case translate('Release/Request release'):
if ($w_action != "" ) { if ($w_action != "" && $action != translate('Whitelist') ) {
updateWblist('W', $_SESSION['sessionMail'], $mail_id_array, $w_action); updateWblist('W', $_SESSION['sessionMail'], $mail_id_array, $w_action);
} }
$failed_array = releaseMessages($_SESSION['sessionMail'], $mail_id_array); $failed_array = releaseMessages($_SESSION['sessionMail'], $mail_id_array);
@ -115,8 +117,10 @@ elseif ( isset( $action ) ) {
} }
break; break;
case translate('Blacklist'):
updateWblist('B', $_SESSION['sessionMail'], $mail_id_array, $action);
case translate('Delete'): case translate('Delete'):
if ($b_action != "" ) { if ($b_action != "" && $action != translate('Blacklist') ) {
updateWblist('B', $_SESSION['sessionMail'], $mail_id_array, $b_action); updateWblist('B', $_SESSION['sessionMail'], $mail_id_array, $b_action);
} }
$failed_array = updateMessages('D', $content_type, $_SESSION['sessionMail'], $mail_id_array); $failed_array = updateMessages('D', $content_type, $_SESSION['sessionMail'], $mail_id_array);

View File

@ -274,7 +274,7 @@ function printActionButtons( $printDeleteAll = true, $wbselect = true ) {
echo '</select>'; echo '</select>';
} }
} else { } else {
echo ( Auth::isMailAdmin() || Auth::isDomainAdmin() ? translate('Release') : translate('Release/Request release') ); echo '<input type="submit" class="button" name="action" value="'.translate('Blacklist').'" />';
} }
echo '<input type="submit" class="button" name="action" value="'.translate('Delete').'" />'; echo '<input type="submit" class="button" name="action" value="'.translate('Delete').'" />';
if ( $printDeleteAll ) if ( $printDeleteAll )