Changeset 2657

Timestamp:

04/28/10 18:06:48 (14 years ago)

Author:

niltonneto

Message:

Ticket #1056 - Corrigido problema que remove todas as regras ao criar um nova.

Location:

trunk/expressoMail1_2

Files:

• inc/class.ScriptS.inc.php (modified) (4 diffs)
• js/jscode/filter.js (modified) (4 diffs)

trunk/expressoMail1_2/inc/class.ScriptS.inc.php

@@ -71 +71 @@
                         $aux = $this->SieveS->putscript($this->scriptfile, $this->createScript());
                         if(!$aux){
-                                // Caso de erro, grava dentro da variável errstr;
-                                $this->errstr = "O arquivo não foi criado";
+                                $this->errstr = "Error: file not created";
+                                return $this->errstr;
                         }
                         // Mata a variavel;
@@ -80 +80 @@
                         
                         if(!$aux){
-                                // Caso de erro, grava dentro da variavel errstr;
-                                $this->errstr = "O arquivo não foi ativado";
+                                $this->errstr = "Error: error to activate file";
+                                return $this->errstr;
                         }
                         else{
@@ -94 +94 @@
                 $this->SieveS->close();
                         
-                if($this->rules){               
+                if($this->rules)
                         return $this->rules;
-                }else{
-                        return $this->errstr;
-                }
                         
         }
@@ -109 +106 @@
         
                 $var_decode = rawurldecode($params['arfilter']);
+                $var_decode = preg_replace('/\n\./', '.', $var_decode);
                 
                 $narray  = explode("_end_",$var_decode);

trunk/expressoMail1_2/js/jscode/filter.js

@@ -11 +11 @@
         function cfilter(){
 
+                this.criticalError = false;
                 this.mode_in   = "";
                 this.rulest              = new Array;
@@ -51 +52 @@
                         var handler_sieve = function(data)
                         {
+                                if (data.indexOf('Error:') == 0)
+                                {
+                                        _this.criticalError = true;
+                                        alert(data);
+                                }
+                                else
+                                try{
                                 if(data.rule.length > 0){
                                         for(var i=0 ; i < data.rule.length; i++)
@@ -64 +72 @@
                                 _this.out_officeR = _this.out_officeR ? trim(_this.out_officeR.toString().replace("\n","")) : "";
                                 if(data.mode.length > 0){_this.mode_in = data.mode[0];}
+                        }
+                                catch(e){
+                                        _this.criticalError = true;
+                                        alert("Expresso:"+e.description);
+                                }
                         }
                         if(Element('form_status') != null)
@@ -745 +758 @@
                 var args   = "expressoMail1_2.ScriptS.rec_rules";
                 var params = "arfilter="+aux_rul;
-                cExecute(args,h_filter,params);
+                if(!_this.criticalError)
+                        cExecute(args,h_filter,params);
         }