Commit d5cb8826 authored by Nigel Kukard's avatar Nigel Kukard
Browse files

* Check the return codes from removing database entries more carefully

parent 35f93382
......@@ -77,7 +77,7 @@ if ($_POST['frmaction'] == "delete") {
$db->beginTransaction();
$res = $db->exec("DELETE FROM policy_members WHERE PolicyID = ".$db->quote($_POST['policy_id']));
if ($res) {
if ($res !== FALSE) {
?>
<div class="notice">Policy members deleted</div>
<?php
......
......@@ -112,7 +112,7 @@ if ($_POST['frmaction'] == "delete") {
if ($res !== FALSE) {
$res = $db->exec("DELETE FROM mailboxes WHERE ID = ".$db->quote($_POST['postfix_mailbox_id']));
if ($res !== FALSE) {
if ($res) {
?>
<div class="notice">Mailbox deleted</div>
<?php
......
......@@ -142,7 +142,7 @@ if ($_POST['frmaction'] == "delete") {
);
$res = $stmt->execute(array($_POST['quota_id']));
if ($res) {
if ($res !== FALSE) {
?>
<div class="notice">Quota limits deleted</div>
<?php
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment