Code

Implemented multiserver to macros
authorhickert <hickert@594d385d-05f5-0310-b6e9-bd551577e9d8>
Wed, 10 Jan 2007 10:26:37 +0000 (10:26 +0000)
committerhickert <hickert@594d385d-05f5-0310-b6e9-bd551577e9d8>
Wed, 10 Jan 2007 10:26:37 +0000 (10:26 +0000)
git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@5511 594d385d-05f5-0310-b6e9-bd551577e9d8

plugins/gofon/macro/class_divListMacros.inc
plugins/gofon/macro/class_gofonMacro.inc
plugins/gofon/phoneaccount/class_phoneAccount.inc

index c62a3e1fc53798985e817478dd8118ae8b5bd8e2..cb88a7e33f20bf4892e4b89c2da45d77a1e2db7a 100755 (executable)
@@ -155,13 +155,24 @@ class divListMacro extends MultiSelectWindow
           name='goFonMacro_del_%KEY%' title='"._("Delete macro")."'>";
       }
 
-      $display= $val["displayName"][0]."&nbsp;(".$val["cn"][0].")";
       if(isset($val['goFonMacroVisible']['0'])&&($val['goFonMacroVisible']['0'] == "1")){
         $pic1 = $visible;
       }else{
         $pic1 = $invisible;
       }
 
+      $display= $val["displayName"][0]."&nbsp;(".$val["cn"][0].")";
+      if(isset($val['description'][0])){
+
+        $desc = $val['description'][0];
+        if(strlen($desc) > 55){
+          $desc = substr($desc,0,52)."...";
+        }
+
+        $display .= "&nbsp;[".$desc."]";
+      }
+
+
       $field1 = array("string" => sprintf($macroimg,$val['dn']), "attach" => "style='text-align:center;width:20px;'");
       $field2 = array("string" => sprintf($editlink,$key,$display), "attach" => "style='' title='".preg_replace('/ /', '&nbsp;', @LDAP::fix($val['dn']))."'");
       $field3 = array("string" => $pic1, "attach" => "style='width:50px;'");
index 3d73cec22e526f239e5f9e1957bbd789c010856f..155c43a64550198d50bfc67a3f808a5820bf91f4 100755 (executable)
@@ -43,12 +43,11 @@ class macro extends plugin
   /*! attribute list for save action */
   var $attributes     = array("cn","base", "description","displayName","goFonMacroContent","goFonMacroVisible");
 
-  var $is_new=false;
   var $orig_cn = ""; 
   /*! Objectclasses that this calls handles */
   var $objectclasses  = array("top", "goFonMacro");
 
-  var $ui ;
+  var $goFonHomeServers = array(); // Contains all available asterisk database server 
 
   //! The Konstructor   
   /*!  Konstructor, load class with  attributes of the given dn*/
@@ -56,9 +55,6 @@ class macro extends plugin
   {
     plugin::plugin ($config, $dn, $parent);
 
-    $ldap= $config->get_ldap_link();
-    $this->dn = $dn;
-
     /* This is always an account */
     $this->is_account= TRUE;
 
@@ -66,20 +62,35 @@ class macro extends plugin
     if ($this->dn == "new"){
       if(isset($_SESSION['CurrentMainBase'])){
         $this->base = $_SESSION['CurrentMainBase'];
-        $this->is_new = true;
       }else{
-        $this->is_new = true;
         $ui= get_userinfo();
         $this->base= dn2base($ui->dn);
       }
     } else {
-      $this->is_new = false;
       $this->orig_cn=$this->cn;
-      $this->base= preg_replace("/ou=macros,ou=asterisk,ou=configs,ou=systems,/","",dn2base($this->dn));
+      $this->base= preg_replace ("/^[^,]+,[^,]+,[^,]+,[^,]+,[^,]+,/", "", $this->dn);
+    }
+
+    /* Check server configurations
+     * Load all server configuration in $this->goFonHomeServers if available
+     */
+    $a_SETUP= array();
+    if(array_key_exists('config',$_SESSION) &&
+       array_key_exists('SERVERS',$_SESSION['config']->data) &&
+       count($_SESSION['config']->data['SERVERS']['FON']) && 
+       array_key_exists('FON',$_SESSION['config']->data['SERVERS'])) {
+
+      /* Set available server */
+      $this->goFonHomeServers = $_SESSION['config']->data['SERVERS']['FON'];
+  
+      /* Remove default entry, not necessary here */
+      if(isset($this->goFonHomeServers[0])){
+        unset($this->goFonHomeServers[0]);  
+      }
     }
-    $this->ui = get_userinfo();
   }
 
+
   /*!  Execute this plugin */
   function execute()
   {
@@ -91,23 +102,12 @@ class macro extends plugin
     $tmp       = array();
     $number = 0; 
 
-    if(!isset($_SESSION['config']->data['SERVERS']['FON'][0])){
-      print_red(_("There is currently no asterisk server defined. Possibly you are missing a server that handles the asterisk management (goFonServer). Your settings can't be saved to asterisk database."));
-    }
-
-    /* Do we represent a valid group? */
-    if (!$this->is_account && $this->parent == NULL){
-      $display= "<img alt=\"\" src=\"images/stop.png\" align=\"middle\">&nbsp;<b>".
-        _("This 'dn' is no phone macro.")."</b>";
-      return ($display);
-    }
-    
     /* Base select dialog */
     $once = true;
     foreach($_POST as $name => $value){
       if(preg_match("/^chooseBase/",$name) && $once){
         $once = false;
-        $this->dialog = new baseSelectDialog($this->config, $this,$this->get_allowed_bases());
+        $this->dialog = new baseSelectDialog($this->config,$this,$this->allowedBasesToMoveTo());
         $this->dialog->setCurrentBase($this->base);
       }
     }
@@ -126,6 +126,7 @@ class macro extends plugin
         if(isset($tmp[$this->dialog->isSelected()])){
           $this->base = $this->dialog->isSelected();
         }
+
         $this->dialog= false;
       }else{
         return($this->dialog->execute());
@@ -135,7 +136,7 @@ class macro extends plugin
     /* Fill templating stuff */
     $smarty= get_smarty();
     $smarty->assign("bases", $this->get_allowed_bases());
-  
+
     $tmp = $this->plInfo();
     foreach($tmp['plProvidedAcls'] as $name => $translation){
       $smarty->assign($name."ACL",$this->getacl($name));
@@ -147,6 +148,7 @@ class macro extends plugin
       $smarty->assign("baseSelect",false);
     }
 
+
     /* Assign all vars to Smarty */
     foreach($this->attributes as $ar){
       $smarty->assign($ar, $this->$ar);
@@ -161,205 +163,197 @@ class macro extends plugin
       $smarty->assign("goFonMacroVisibleChecked","");
     }
 
-    $smarty->assign("cnACL",$this->getacl("cn",!$this->is_new));
+    $smarty->assign("cnACL",$this->getacl("cn",$this->initially_was_account));
     $smarty->assign("cn",$this->cn);
-    $this->generate_mysql_entension_entries();
+
+    /* Ensure that macro content is displayed correctly encoded */
+    $smarty->assign("goFonMacroContent",htmlentities(utf8_decode ($this->goFonMacroContent)));
+
     /* Show main page */
     return($smarty->fetch (get_template_path('generic.tpl', TRUE)));
   }
 
-  /*!
-  Remove this Object
-  */
-  function remove_from_parent()
-  {
-    $ldap= $this->config->get_ldap_link();
-    
-    $res = $ldap->search("(&(objectClass=goFonAccount)(objectClass=gosaAccount))", array("goFonMacro"));
-
-    while ($val = $ldap->fetch()){ 
-      if(isset($val['goFonMacro'])){
-        if(strstr($val['goFonMacro'][0],$this->dn)){ 
-          print_red(_("This macro ist still in use. To delete this Macro ensure that nobody has selected this Macro."));
-          return false;
-        }
-      }
-    }
-  
-    $ldap->rmDir($this->dn); 
-    show_ldap_error($ldap->get_error(), sprintf(_("Removing of goFonMacro/generic account with dn '%s' failed."),$this->dn));
 
-    if(isset($this->orig_cn)){
-      $this->generate_mysql_entension_entries(false,true,$this->orig_cn);
-    }else{
-      $this->generate_mysql_entension_entries(false,true);
+  /* This method check if all databases are reachable.  
+   *  Returns with error message or an empty string on success.
+   * 
+   * - Is mysql extension available  
+   * - Is every server reachable 
+   * - Does the database exists/is accessible
+   */
+  function check_database_accessibility()
+  {
+    /* Check if mysql extension is available */
+    if(!is_callable("mysql_pconnect")){
+      return(_("Can't save any changes to asterisk database, there is currently no mysql extension available in your php setup."));
     }
 
-    /* Delete references to object groups */
-    $ldap->cd ($this->config->current['BASE']);
-    $ldap->search ("(&(objectClass=gosaGroupOfNames)(member=".$this->dn."))", array("cn"));
-    while ($ldap->fetch()){
-        $og= new ogroup($this->config, $ldap->getDN());
-        unset($og->member[$this->dn]);
-        $og->save ();
-        show_ldap_error($ldap->get_error(), sprintf(_("Removing of goFonMacro/generic account with dn '%s' failed."),$this->dn));
+    /********************
+     * Check all home server
+     ********************/
+    foreach($this->goFonHomeServers as $goFonHomeServer => $cfg_Current){
+      $r_current    =  @mysql_pconnect($cfg_Current['SERVER'],$cfg_Current['LOGIN'],$cfg_Current['PASSWORD']);
+      if(!$r_current){
+        gosa_log(@mysql_error($r_current));
+        return(sprintf(_("The MySQL home server '%s' isn't reachable as user '%s', check GOsa log for mysql error."),
+              $cfg_Current['SERVER'],$cfg_Current['LOGIN']));
+      }
+      $db_current  =  @mysql_select_db($cfg_Current['DB'],$r_current);
+      if(!$db_current){
+        gosa_log(@mysql_error($r_current));
+        mysql_close($r_current);
+        return( sprintf(_("Can't select database '%s' on home server '%s'."),$cfg_Current['DB'],$cfg_Current['SERVER']));
+      }
     }
   }
 
-  // Generate MySQL Syntax 
-  function generate_mysql_entension_entries($save = false,$delete_only=false,$remove_old_macroname=false){
 
-    if(!isset($_SESSION['config']->data['SERVERS']['FON'][0])){
-      return(true);
+  /* Remove current macro from all asterisk server.
+   * First of all check if we have access to all databases. 
+   * - Remove old entries 
+   */ 
+  function remove_from_database($save)
+  {
+    /* Check if all databases are reachable */
+    $str = $this->check_database_accessibility();
+    if($str){
+      return($str);
     }
 
-    if(!is_callable("mysql_connect")){
-      if((!$this->error_shown)&&($save)){
-        print_red(_("There is no mysql extension available in your php setup, can't save any changes to asterisk db."));
-        $this->error_shown = true;
+    /* Create query string */
+    $context  = addslashes("macro-".$this->cn);
+
+    /* Remove current macro from each server available */ 
+    if($save){
+      foreach($this->goFonHomeServers as $dn => $Server){
+        $query      = "DELETE FROM ".$Server['EXT_TABLE']." WHERE context='".$context."';";
+        $r_current  =  @mysql_pconnect($Server['SERVER'],$Server['LOGIN'],$Server['PASSWORD']);
+        $db_current =  @mysql_select_db($Server['DB'],$r_current);
+        $res = @mysql_query($query,$r_current);
+        if(!$res){
+          gosa_log(@mysql_error($r_current));
+          return(sprintf(_("Removing marco from '%s' failed. Check GOsa log for mysql error."),$Server['SERVER']));
+        }
+        @mysql_close($r_current);
       }
-      return(true);
     }
+  }
 
-    // Get Configuration for Mysql database Server  
-    $a_SETUP = $_SESSION['config']->data['SERVERS']['FON'][0];
-
-    // Connect to DB server
-    $r_con =  @mysql_pconnect($a_SETUP['SERVER'],$a_SETUP['LOGIN'],$a_SETUP['PASSWORD']); 
-  
-    // Check if we are  connected correctly 
-    if(!$r_con){
-      $this->generate_error = sprintf(_("The MySQL Server '%s' isn't reachable as user '%s', check GOsa log for mysql error."),
-          $a_SETUP['SERVER'],$a_SETUP['LOGIN']);
-      gosa_log(mysql_error());
-      return false;
+  /* Add current macro to all asterisk server.
+   * First of all check if we have access to all databases. 
+   * - Remove old entries 
+   * - Add new entries 
+   */ 
+  function add_to_database($save)
+  {
+    /* Check if all databases are reachable */
+    $str = $this->check_database_accessibility();
+    if($str){
+      return($str);
     }
 
-    // Select database for Extensions 
-    $db  =  @mysql_select_db($a_SETUP['DB'],$r_con);
-
-    // Test if we have the database selected correctly
-    if(!$db){
-      $this->generate_error = sprintf(_("Can't select database %s on %s."),$a_SETUP['DB'],$a_SETUP['SERVER']);
-      gosa_log(mysql_error());
-      return false;
+    /* Remove old entries first. Else we got duplicated entries */
+    $str = $this->remove_from_database($save);
+    if($str){
+      return($str);
     }
 
+    /* Create query string */
+    $context      = "macro-".$this->cn;
 
-    // Context def
-    $context  = "macro-".$this->cn;
-
-    // Parse Content if we connected correctly
-    if($db &&  $r_con ){
-
-      // Split Content into lines 
-      $a_contentLines = split ("\n",$this->goFonMacroContent);
-
-      // Foreach single line ...
-      foreach($a_contentLines as $i_linenum => $s_linestr){
-    
-        // Remove unwanted exten => tag 
-        $s_linestr= preg_replace ("/^.*\> /","",$s_linestr);
+    /************
+     * Parse Macro content
+     ************/
+    $sql = 
+      "INSERT INTO %TABLENAME% ".
+      " (context,exten,priority,app,appdata) ".
+      " VALUES ";
       
-        // Remove  spaces 
-        $s_linestr = trim ( $s_linestr);
-
-        // Skip empty lines
-        if ($s_linestr == ""){
-          continue;
-        }
-
-        // If not empty or linebreak at [0]
-        if((!empty($s_linestr))&&($s_linestr[0]!=";")&&(ord($s_linestr[0]) !=13)){
-  
-          // Set general SQL statement
-          $SQL[$i_linenum] =  
-            "INSERT INTO ".$a_SETUP['EXT_TABLE'].
-            " (context,exten,priority,app,appdata) ".
-            " VALUES ".
-            " (";
-      
-          // Parse linestr to entry data
-          $linecontents = preg_replace("/;.*$/","",$s_linestr) ;
+    $a_contentLines = split("\n",$this->goFonMacroContent);
+    foreach($a_contentLines as $i_linenum => $s_linestr){
+
+      /* Remove the 'exten => ' string in front of the macro content line 
+       *  example line  'exten => s,2,GotoIf(${ARG3}?3:5)'
+       * Remove comments introduced by ;
+       * Skip empty lines 
+       */ 
+      $s_linestr = preg_replace ("/^.*=\> /","",$s_linestr);
+      $s_linestr = preg_replace("/;.*$/","",$s_linestr) ;
+      $s_linestr = trim($s_linestr);
+      if(empty($s_linestr)){
+        continue;
+      }
 
-          $tmp  = split(",", preg_replace("/\(.*$/","",$linecontents));
-  
-          if(!isset($tmp[2])){
-            $this->generate_error = sprintf(_("Unable to parse macro contents on line: %s"),$i_linenum);
-            return false;
-          }    
-            $exten = $tmp[0];
-            $prio = $tmp[1];
-            $app  = $tmp[2];
-            $para = $linecontents; 
-            $para = preg_replace("/^.*\(/","",$para);
-            $para = preg_replace("/\)$/","",$para);
-
-            // Append SQL syntax
-            $SQL[$i_linenum].="'".$context."','".$exten."','".$prio."','".$app."','".$para."');";
-        }
-     }
+      /* A line that passes the check above should look like this 
+       *  s,1,SetLanguage(de)
+       * 3 parts seperated by , 
+       * If there are more or less parts, abort.
+       * The preg_replace exclude parameters from split .. 
+       */
+      $tmp  = split(",", $s_linestr,3);
+
+      /* Check if there are exactly 2 , */ 
+#      if(substr_count($s_linestr,",") !=2){
+#        return(sprintf(_("More than two ',' given in line : '%s'. Remember that parameters are seperated by '|'."),$i_linenum));
+#      }
+      /* Multiple () are not supproted currently ... */  
+      if(substr_count($s_linestr,"(") >1 ){
+        return(sprintf(_("More than one '(' is currently not supported. Line : '%s'."),$i_linenum));
+      }
+      if(substr_count($s_linestr,")") >1 ){
+        return(sprintf(_("More than one ')' is currently not supported. Line : '%s'."),$i_linenum));
+      }
+      /* Check if there is an application given */
+      if(empty($tmp[1])){
+        return(sprintf(_("There is no application given in line : '%s'."),$i_linenum));
+      } 
+      /* Check if there is an extension given */
+      if(empty($tmp[0])){
+        return(sprintf(_("There is no extension type given in line : '%s'."),$i_linenum));
+      } 
+
+      /* Create extension entry for current line 
+       *  and add this line to an array that will be inserted 
+       *  to each database.
+       */
+      $exten  = addslashes($tmp[0]);
+      $prio   = addslashes($tmp[1]);
+      $app    = addslashes(preg_replace("/\(.*\).*$/","",$tmp[2]));
+      $para   = addslashes(preg_replace("/^.*\(/","",$tmp[2]));
+      $para   = preg_replace("/\).*$/","",$para);
+      $sql.= " ('".$context."','".$exten."','".$prio."','".$app."','".$para."'),";
+    }
+    
+    /* Remove last , from query string */
+    $sql = preg_replace("/,$/","",$sql);
 
-      if(($save)||($delete_only)){
-        $res = mysql_query("SELECT count(*) FROM ".$a_SETUP['EXT_TABLE']." WHERE context= '".$context."'");
+    /* Save current changes to the database */
+    if($save){
+    
+      /* Macro are spread to each asterisk server */
+      foreach($this->goFonHomeServers as $dn => $cfg){
+        $r_con  = @mysql_pconnect($cfg['SERVER'],$cfg['LOGIN'],$cfg['PASSWORD']); 
+        $db     = @mysql_select_db($cfg['DB'],$r_con);
+        $query  = preg_replace("/%TABLENAME%/",$cfg['EXT_TABLE'],$sql);
+        $res    = @mysql_query($query,$r_con);
         if(!$res){
-          $this->generate_error = sprintf(_("Can't perform SELECT query in DB '%s'"),$a_SETUP['DB']);
-          gosa_log(mysql_error());
-          return false;
-        }
-        $cnt = mysql_fetch_row($res);
-        $cnt = $cnt[0];
-
-        if($cnt != 0) {
-          if(!mysql_query("DELETE FROM ".$a_SETUP['EXT_TABLE']." WHERE context= '".$context."'")){
-            $this->generate_error = sprintf(_("Can't perform DELETE query in DB '%s'"),$a_SETUP['DB']);
-            gosa_log(mysql_error());
-            return false;
-          }
+          gosa_log(@mysql_error($r_con));
+          return(sprintf(_("Insert of new macro failed for server '%s'."),$cfg['SERVER']));
         }
-
-        if($remove_old_macroname!="false"){
-          if(!mysql_query("DELETE FROM ".$a_SETUP['EXT_TABLE']." WHERE context= 'Makro-".$remove_old_macroname."'")){
-            $this->generate_error = sprintf(_("Can't perform DELETE query in DB '%s'"),$a_SETUP['DB']);
-            gosa_log(mysql_error());
-            return false;
-          }
-        }
-
-
-
+        @mysql_close($r_con);
       }
-
-      // We want to save this 
-      if(($save)&&(isset($SQL))){
-        foreach($SQL as $entry){
-          if(!mysql_query($entry)){
-            $this->generate_error = sprintf(_("Can't perform INSERT query in DB '%s'"),$a_SETUP['DB']);
-            gosa_log(mysql_error());
-            return false;
-          }
-        } 
-      }    
     }
-    if((isset($r_con))&&($r_con)){
-      @mysql_close($r_con);
-    }
-    return true;
   }
 
 
-
-
-  /*!
-  Save data to object 
-  */
   function save_object()
   {
     if (isset($_POST['gofonMacroGenericPosted'])){
 
       $old_cn       = $this->cn;
       $old_visible  = $this->goFonMacroVisible;
-      
+
       /* Create a base backup and reset the
          base directly after calling plugin::save_object();
          Base will be set seperatly a few lines below */
@@ -376,13 +370,13 @@ class macro extends plugin
       }
 
       /* Restore old cn if we have insuficient acls to change cn ... */
-      if(!$this->acl_is_writeable("cn",!$this->is_new)){
+      if(!$this->acl_is_writeable("cn",$this->initially_was_account)){
         $this->cn = $old_cn;
       }
 
       /* check if we are allowed to toggle visibility */
       if($this->acl_is_writeable("goFonMacroVisible")) {
-    
+
         /* Checkbox selected ? */
         if(isset($_POST['goFonMacroVisible'])) {
           $this->goFonMacroVisible= 1 ;
@@ -398,18 +392,23 @@ class macro extends plugin
   }
 
 
-  /*! 
-  Check values 
-  */
+  /*! Check values */
   function check()
   {
     /* Call common method to give check the hook */
     $message= plugin::check();
 
-    if(!$this->generate_mysql_entension_entries()){
-      $message[] = $this->generate_error;
+    if(!count($this->goFonHomeServers)){
+      $message[] = _("There must be at least one server with an asterisk database to save this phone macro.");
+    }
+
+    /* Check if insert/replace is possible and all servers are available */
+    $str = $this->add_to_database(false);
+    if($str){
+      $message[] = $str;
     }
 
+    /* Check if cn is already used  */
     if(($this->dn=="new")||($this->orig_cn!=$this->cn)){
       $ldap = $this->config->get_ldap_link();
       $ldap->search("(&(objectClass=goFonMacro)(cn=".$this->cn."))",array("cn"));
@@ -417,51 +416,97 @@ class macro extends plugin
         $message[]=sprintf(_("The given cn '%s' already exists."),$this->cn);
       }
     }
-
+  
+    /* Check if display name is set */
     if(empty($this->displayName)){
       $message[] = _("You must specify the 'Display Name' in order to save this macro");
     }  
+    /* CN is restricted to 20 chars */
     if(strlen("Makro-".$this->cn)>20 ){
       $message[]=_("The given cn is too long, to create a Makro entry, maximum 20 chars.");
     }
+  
+    /* If this macro is still in use we should not change the visible for user flag to invisible */
+    if(!$this->goFonMacroVisible){
+      $ldap = $this->config->get_ldap_link();
+      $res = $ldap->search("(&(objectClass=goFonAccount)(objectClass=gosaAccount)(goFonMacro=*))", array("goFonMacro"));
+      while ($val = $ldap->fetch()){
+        if(strstr($val['goFonMacro'][0],$this->dn)){
+          $message[] = _("This macro is still in use. It is necessary to mark this macro as visible for users.");
+          return($message);
+        }
+      }
+    }
+
+    /* Macro content must be smaller than 100 lines */
     if(count(split("\n",$this->goFonMacroContent))>100){
       $message[] = _("Makro length must be lower than 100 lines");
     }
-   
-    /*Some stupid IE fixes again*/
-    if(empty ($this->base)) {
-      $message[] = _("Please choose a valid  base.");
-    }
+
     return $message;
   }
 
 
-  /*! 
-  Save to LDAP 
-  */
-  function save()
+  /*! Remove makro from all given databases 
+   *   and ldap too.
+   */
+  function remove_from_parent()
   {
-    /* Post checks */
-    $this->execute();
+    $ldap= $this->config->get_ldap_link();
+
+    /* Skip remove if this macro is still in use */
+    $res = $ldap->search("(&(objectClass=goFonAccount)(objectClass=gosaAccount)(goFonMacro=*))", array("goFonMacro"));
+    while ($val = $ldap->fetch()){ 
+      if(strstr($val['goFonMacro'][0],$this->dn)){ 
+        print_red(_("This macro is still in use. To delete this Macro ensure that nobody has selected it."));
+        return false;
+      }
+    }
+
+    /* Try to remove from database */
+    if(count($this->goFonHomeServers)){
+      $str = $this->remove_from_database(true);
+      if($str){ 
+        print_red($str);
+        return false;
+      }
+    }else{
+      print_red(_("Could not remove the macro entry from asterisk databases. Please check your asterisk database configurations."));
+      return false;
+    }
 
+    /* Remove phone macro */ 
+    $ldap->rmDir($this->dn); 
+    show_ldap_error($ldap->get_error(), sprintf(_("Removing of goFonMacro/generic account with dn '%s' failed."),$this->dn));
+
+    /* Delete references to object groups */
+    $ldap->cd ($this->config->current['BASE']);
+    $ldap->search ("(&(objectClass=gosaGroupOfNames)(member=".$this->dn."))", array("cn"));
+    while ($ldap->fetch()){
+      $og= new ogroup($this->config, $ldap->getDN());
+      unset($og->member[$this->dn]);
+      $og->save ();
+      show_ldap_error($ldap->get_error(), sprintf(_("Removing of goFonMacro/generic account with dn '%s' failed."),$this->dn));
+    }
+  }
+
+
+  /*! Save to LDAP */
+  function save()
+  {
     plugin::save();
     unset($this->attrs['base']);
 
-    /* Write back to ldap */
-    $ldap= $this->config->get_ldap_link();
-    $ldap->cat($this->dn, array('dn'));
-    $a= $ldap->fetch();
-    
-    if(isset($this->orig_cn)){
-      $this-> generate_mysql_entension_entries(true,false,$this->orig_cn);
+    /* Try to add entries to databases */
+    $str = $this->add_to_database(true);
+    if($str){
+      print_red($str);
     }else{
-      $this-> generate_mysql_entension_entries(true);
-    }
+      /* Write back to ldap */
+      $ldap= $this->config->get_ldap_link();
+      $ldap->cat($this->dn, array('dn'));
+      $a= $ldap->fetch();
 
-    if($this->generate_mysql_entension_entries()){
       if (count($a)){
         $ldap->cd($this->dn);
         $this->cleanup();
@@ -483,23 +528,23 @@ class macro extends plugin
   function plInfo()
   {
     return (array(
-        "plShortName"   => _("Generic"),
-        "plDescription" => _("Asterisk macro management"),
-        "plSelfModify"  => FALSE,
-        "plDepends"     => array(),
-        "plPriority"    => 0,
-        "plSection"     => array("administration"),
-        "plCategory"    => array("gofonmacro" => array("description" => _("GOfon macro"),
-                                                  "objectClass" => "gofonMacro")),
-
-        "plProvidedAcls" => array(
-          "cn"                            => _("Macro name"),
-          "base"                          => _("Base"), 
-          "description"                   => _("Description"), 
-          "displayName"                   => _("Display name"),
-          "goFonMacroContent"             => _("Macro content and parameter"),
-          "goFonMacroVisible"             => _("Visibility flag"))
-        ));
+          "plShortName"   => _("Generic"),
+          "plDescription" => _("Asterisk macro management"),
+          "plSelfModify"  => FALSE,
+          "plDepends"     => array(),
+          "plPriority"    => 0,
+          "plSection"     => array("administration"),
+          "plCategory"    => array("gofonmacro" => array("description" => _("GOfon macro"),
+              "objectClass" => "gofonMacro")),
+
+          "plProvidedAcls" => array(
+            "cn"                            => _("Macro name"),
+            "base"                          => _("Base"),
+            "description"                   => _("Description"),
+            "displayName"                   => _("Display name"),
+            "goFonMacroContent"             => _("Macro content and parameter"),
+            "goFonMacroVisible"             => _("Visibility flag"))
+          ));
   }
 
 }
index 9235dfd4498d90fc3ac75b135cf1132173964a7e..46291a13cf8337d96c74f98b25dc3984ebe51396 100644 (file)
@@ -1178,7 +1178,7 @@ class phoneAccount extends plugin
      *  - the macro saved is up to date on the destination server.
      */
     if(!empty($this->macro) && $this->macro != "none")  {
-      $macro_tab = new macrotabs($this->config,$this->config->data['TABS']['MACROTABS'], $this->macro);
+      $macro_tab= new macrotabs($this->config,$this->config->data['TABS']['MACROTABS'], $this->macro,"gofonmacro");
       $macro_tab -> save();
     }