Changeset 4659

Timestamp:

08/02/07 22:52:23 (1 year ago)

Author:

p_lindheimer

Message:

#1836 - remove mailbox form filesystem after deleting an extension

Files:

• modules/branches/2.3/voicemail/functions.inc.php (modified) (2 diffs)
• modules/branches/2.3/voicemail/module.xml (modified) (1 diff)

modules/branches/2.3/voicemail/functions.inc.php

@@ -231 +231 @@
-                break;
-                case "del":
+                        // call remove before del, it needs to know context info
+                        //
+                        voicemail_mailbox_remove($extdisplay);
-                        voicemail_mailbox_del($extdisplay);
-                        needreload();
@@ -260 +263 @@
-        
-        return null;
+}
+
+function voicemail_mailbox_remove($mbox) {
+        global $amp_conf;
+        $uservm = voicemail_getVoicemail();
+        $vmcontexts = array_keys($uservm);
+
+        $return = true;
+
+        foreach ($vmcontexts as $vmcontext) {
+                if(isset($uservm[$vmcontext][$mbox])){
+
+                        $vm_dir = $amp_conf['ASTSPOOLDIR']."/voicemail/$vmcontext/$mbox";
+                        exec("rm -rf $vm_dir",$output,$ret);
+                        if ($ret) {
+                                $return = false;
+                                $text   = sprintf(_("Failed to delete vmbox: %s@%s"),$mbox, $vmcontext);
+                                $etext  = sprintf(_("failed with retcode %s while removing %s:"),$ret, $vm_dir)."<br>";
+                                $etext .= implode("<br>",$output);
+                                $nt =& notifications::create($db);
+                                $nt->add_error('voicemail', 'MBOXREMOVE', $text, $etext, '', true, true);
+                                //
+                                // TODO: this does not work but we should give some sort of feedback that id did not work
+                                //
+                                // echo "<script>javascript:alert('$text\n"._("See notification panel for details")."')</script>";
+                        }
+                }
+        }
+        return $return; 
-}
-

modules/branches/2.3/voicemail/module.xml

@@ -2 +2 @@
-        <rawname>voicemail</rawname>
-        <name>Voicemail</name>
-2.4
+3
-        <candisable>no</candisable>
-        <canuninstall>no</canuninstall>
-        <changelog>
+                *2.0.3* #1836 - remove mailbox form filesystem after deleting an extension
-                *2.0.2.4* changed install script to return false on astman not there
-                *2.0.2.3* remove setting vmcontext to department, fixed #2153 and #2136