Changeset 2203

Show
Ignore:
Timestamp:
04/24/08 20:43:52 (3 weeks ago)
Author:
anarcat
Message:

better error checking in list deletion too

See #68
See #1096

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

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

    r2197 r2203  
    182182      exec("/usr/lib/alternc/mailman.create \"".escapeshellcmd($login."@".$domain)."\" \"".escapeshellcmd($owner)."\" \"".escapeshellcmd($password)."\"", &$output, &$return); 
    183183      if ($return) { 
    184         $err->raise("mailman", "failed to create mailman list. error: %d, output: %s", array($return, join("\n", $output))); 
     184        $err->raise("mailman", "failed to create mailman list. error: %d, output: %s", $return, join("\n", $output)); 
    185185      } 
    186186      return !$return; 
    …… 
    202202      return false; 
    203203    } 
    204     exec("/usr/lib/alternc/mailman.delete ".escapeshellarg($db->f("name"))); 
     204    exec("/usr/lib/alternc/mailman.delete ".escapeshellarg($db->f("name")), &$output, &$return); 
     205    if ($return) { 
     206      $err->raise("mailman", "failed to delete mailman list. error: %d, output: %s", $return, join("\n", $output)); 
     207      return false; 
     208    } 
    205209    $login=$db->f("list"); 
    206210    $domain=$db->f("domain");