Changeset 4596 in subversion

Timestamp:

Mar 7, 2011 9:22:11 AM (2 years ago)

Author:

alec

Message:

• Fix saving of a script using flags extension on servers with imap4flags support (#1487825)

Location:

trunk/plugins/managesieve

Files:

• Changelog (modified) (2 diffs)
• lib/rcube_sieve.php (modified) (4 diffs)
• lib/rcube_sieve_script.php (modified) (3 diffs)
• managesieve.php (modified) (2 diffs)

trunk/plugins/managesieve/Changelog

@@ +1 @@
+* version 4.1 [2011-03-07]
+-----------------------------------------------------------
 - Fix fileinto target is always INBOX (#1487776)
 - Fix escaping of backslash character in quoted strings (#1487780)
@@ -4 +6 @@
   or UTF-8 characters (dovecot's implementation bug workaround)
   in script names
+- Fix saving of a script using flags extension on servers with imap4flags support (#1487825)
 
 * version 4.0 [2011-02-10]

trunk/plugins/managesieve/lib/rcube_sieve.php

@@ -32 +32 @@
     public $current;                // name of currently loaded script
     private $disabled;              // array of disabled extensions
+    private $exts;                  // array of supported extensions
 
 
@@ -74 +75 @@
         }
 
+        $this->exts     = $this->get_extensions();
         $this->disabled = $disabled;
     }
@@ -192 +194 @@
     public function get_extensions()
     {
+        if ($this->exts)
+            return $this->exts;
+    
         if (!$this->sieve)
             return $this->_set_error(SIEVE_ERROR_INTERNAL);
@@ -281 +286 @@
     {
         // try to parse from Roundcube format
-        $script = new rcube_sieve_script($txt, $this->disabled);
+        $script = new rcube_sieve_script($txt, $this->disabled, $this->exts);
 
         // ... else try to import from different formats
         if (empty($script->content)) {
             $script = $this->_import_rules($txt);
-            $script = new rcube_sieve_script($script, $this->disabled);
+            $script = new rcube_sieve_script($script, $this->disabled, $this->exts);
         }
 

trunk/plugins/managesieve/lib/rcube_sieve_script.php

@@ -27 +27 @@
     );
 
+    private $capabilities;
+
     /**
      * Object constructor
      *
      * @param  string  Script's text content
-     * @param  array   Disabled extensions
-     */
-    public function __construct($script, $disabled=null)
+     * @param  array   List of disabled extensions
+     * @param  array   List of capabilities supported by server
+     */
+    public function __construct($script, $disabled=null, $capabilities=null)
     {
         if (!empty($disabled)) {
@@ -45 +48 @@
         }
 
-        $this->content = $this->_parse_text($script);
+        $this->capabilities = $capabilities;
+        $this->content      = $this->_parse_text($script);
     }
 
@@ -217 +221 @@
                 case 'setflag':
                 case 'removeflag':
-                    if (!empty($action['mode']) && strtolower($action['mode']) == 'imap4flags')
+                    if (is_array($this->capabilities) && in_array('imap4flags', $this->capabilities))
                         array_push($exts, 'imap4flags');
                     else

trunk/plugins/managesieve/managesieve.php

@@ -535 +535 @@
                     }
                     $this->form['actions'][$i]['target'] = $_target;
-                    if (in_array('imap4flags', $this->exts)) {
-                        $this->form['actions'][$i]['mode'] = 'imap4flags';
-                    }
                     break;
 
@@ -1086 +1083 @@
             . $this->error_class($id, 'action', 'flags', 'action_flags') . '>';
         foreach ($flags as $fidx => $flag) {
-            $out .= '<nobr><input type="checkbox" name="_action_flags[' .$id .'][]" value="' . $flag . '"'
+            $out .= '<input type="checkbox" name="_action_flags[' .$id .'][]" value="' . $flag . '"'
                 . (in_array_nocase($flag, $flags_target) ? 'checked="checked"' : '') . ' />'
-                . Q($this->gettext('flag'.$fidx)) .'</nobr> ';
+                . Q($this->gettext('flag'.$fidx)) .'<br>';
         }
         $out .= '</div>';