From 78a3fd06c55f990533a3a8b7e6769bc1f3ed9502 Mon Sep 17 00:00:00 2001
From: hickert <hickert@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Wed, 22 Jun 2005 05:18:21 +0000
Subject: [PATCH] Some changes, no execute in save

git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@791 594d385d-05f5-0310-b6e9-bd551577e9d8
---
 .../macro/class_gofonMacroParameters.inc      | 56 ++++++++-----------
 1 file changed, 23 insertions(+), 33 deletions(-)

diff --git a/plugins/gofon/macro/class_gofonMacroParameters.inc b/plugins/gofon/macro/class_gofonMacroParameters.inc
index f8f93a19a..65f6614ca 100755
--- a/plugins/gofon/macro/class_gofonMacroParameters.inc
+++ b/plugins/gofon/macro/class_gofonMacroParameters.inc
@@ -153,33 +153,6 @@ class macroParameter extends plugin
       $this->goFonMacroParameter[]=array("name"=>"new","type"=>"string","default"=>"test");
     }
 
-
-    #FIXME: Der Transfer der momentanen Variablen muss nach save_object(),
-    #       damit kann in save() auf den execute() Aufruf verzichtet werden!
-    
-    /* read out post data, and assign it to the parameters */
-    /* And or delete */
-    foreach($_POST as $name=>$value){	
-
-      /* Test if there is a variable begining with "del" */
-      if(preg_match("/del/",$name)){
-
-        /* Extract entry id to delete */
-        $nr = str_replace("del","",$name)	;
-
-        /* unset entry */			
-        unset($this->goFonMacroParameter[$nr]);
-
-      }elseif(preg_match("/number/",$name)){
-
-        /* Set Post vars */
-        $key = $_POST[$name];
-        $this->goFonMacroParameter[$key]['name']	 = $_POST["varname".$key];		
-        $this->goFonMacroParameter[$key]['type']	 = $_POST["vartype".$key];		
-        $this->goFonMacroParameter[$key]['default']= $_POST["default".$key];		
-      }
-    } 
-
     /*generate Table which shows als parameters */
 
     $FonParas = $this->check_paras($content,$this->goFonMacroParameter); 
@@ -193,9 +166,6 @@ class macroParameter extends plugin
         list($sel1, $sel2, $sel3)= array("", "", "");
       }
 
-      #FIXME: Der Name des Parameters ist nicht gesetzt
-      #       -> Argument (ARG#)    Name [....]    Typ [...^]    Default [....]
-
       /* Assemble output table */
       $vars .="<tr>
         <td><input name=\"number".$key."\" value='".$key."' type='hidden'> </td>
@@ -240,6 +210,29 @@ class macroParameter extends plugin
     if (isset($_POST['phoneparameters'])){
       plugin::save_object();
     }
+        /* read out post data, and assign it to the parameters */
+    /* And or delete */
+    foreach($_POST as $name=>$value){
+
+      /* Test if there is a variable begining with "del" */
+      if(preg_match("/del/",$name)){
+
+        /* Extract entry id to delete */
+        $nr = str_replace("del","",$name) ;
+
+        /* unset entry */
+        unset($this->goFonMacroParameter[$nr]);
+
+      }elseif(preg_match("/number/",$name)){
+
+        /* Set Post vars */
+        $key = $_POST[$name];
+        $this->goFonMacroParameter[$key]['name']   = $_POST["varname".$key];
+        $this->goFonMacroParameter[$key]['type']   = $_POST["vartype".$key];
+        $this->goFonMacroParameter[$key]['default']= $_POST["default".$key];
+      }
+    }
+
   }
 
 
@@ -263,9 +256,6 @@ class macroParameter extends plugin
   {
     /* Post checks */
 
-    # FIXME: das hat hier nix zu suchen...
-    $this->execute();
-
     plugin::save();
 
     $this->attrs['goFonMacroParameter']=array();
-- 
2.30.2