Changeset 2203 for alternc-mailman/trunk/bureau/class/m_mailman.php

Timestamp:

04/24/08 20:43:52 (5 years ago)

Author:

anarcat

Message:

better error checking in list deletion too

See #68
See #1096

File:

• alternc-mailman/trunk/bureau/class/m_mailman.php (modified) (2 diffs)

alternc-mailman/trunk/bureau/class/m_mailman.php

@@ -182 +182 @@
       exec("/usr/lib/alternc/mailman.create \"".escapeshellcmd($login."@".$domain)."\" \"".escapeshellcmd($owner)."\" \"".escapeshellcmd($password)."\"", &$output, &$return);
       if ($return) {
-        $err->raise("mailman", "failed to create mailman list. error: %d, output: %s", array($return, join("\n", $output)));
+        $err->raise("mailman", "failed to create mailman list. error: %d, output: %s", $return, join("\n", $output));
       }
       return !$return;
@@ -202 +202 @@
       return false;
     }
-    exec("/usr/lib/alternc/mailman.delete ".escapeshellarg($db->f("name")));
+    exec("/usr/lib/alternc/mailman.delete ".escapeshellarg($db->f("name")), &$output, &$return);
+    if ($return) {
+      $err->raise("mailman", "failed to delete mailman list. error: %d, output: %s", $return, join("\n", $output));
+      return false;
+    }
     $login=$db->f("list");
     $domain=$db->f("domain");