Changeset 4071 in subversion

Timestamp:

Oct 10, 2010 12:01:31 PM (3 years ago)

Author:

alec

Message:

• Added popups with form errors
• Removed nonexisting 'wrongformat' label usage

Location:

trunk/plugins/managesieve

Files:

• Changelog (modified) (1 diff)
• managesieve.js (modified) (2 diffs)
• managesieve.php (modified) (11 diffs)
• skins/default/managesieve.css (modified) (1 diff)

trunk/plugins/managesieve/Changelog

@@ +1 @@
+* version 2.10 [2010-10-10]
+-----------------------------------------------------------
 - Fixed import from Avelsieve
 - Use localized size units (#1486976)
 - Added support for relational operators and i;ascii-numeric comparator
+- Added popups with form errors
 
 * version 2.9 [2010-08-02]

trunk/plugins/managesieve/managesieve.js

@@ -35 +35 @@
       }
 
+      // Create layer for form tips
+      if (!rcmail.env.framed) {
+        rcmail.env.ms_tip_layer = $('<div id="managesieve-tip" class="popupmenu"></div>');
+        rcmail.env.ms_tip_layer.appendTo(document.body);
+      }
+
       if (rcmail.gui_objects.filterslist) {
         var p = rcmail;
@@ -473 +479 @@
 };
 
+// Register onmouse(leave/enter) events for tips on specified form element
+rcube_webmail.prototype.managesieve_tip_register = function(tips)
+{
+  for (var n in tips) {
+    $('#'+tips[n][0])
+      .bind('mouseenter', {str: tips[n][1]},
+        function(e) {
+          var offset = $(this).offset(),
+            tip = rcmail.env.framed ? parent.rcmail.env.ms_tip_layer : rcmail.env.ms_tip_layer,
+            left = offset.left,
+            top = offset.top - 12;
+
+          if (rcmail.env.framed) {
+            offset = $(parent.document.getElementById('filter-box')).offset();
+            top  += offset.top;
+            left += offset.left;
+          }
+
+          tip.html(e.data.str)
+          top -= tip.height();
+          
+          tip.css({left: left, top: top}).show();
+        })
+      .bind('mouseleave',
+        function(e) {
+          var tip = parent.rcmail && parent.rcmail.env.ms_tip_layer ? 
+            parent.rcmail.env.ms_tip_layer : rcmail.env.ms_tip_layer;
+          tip.hide();
+      });
+  }
+};

trunk/plugins/managesieve/managesieve.php

@@ -8 +8 @@
  * with server using managesieve protocol. Adds Filters tab in Settings.
  *
- * @version 2.9
+ * @version 2.10
  * @author Aleksander 'A.L.E.C' Machniak <alec@alec.pl>
  *
@@ -24 +24 @@
     private $errors;
     private $form;
+    private $tips = array();
     private $script = array();
     private $exts = array();
@@ -422 +423 @@
                             $this->form['tests'][$i]['arg']  = $sizetarget.$sizeitem;
 
-                            if (!preg_match('/^[0-9]+(K|M|G)*$/i', $sizetarget))
-                                $this->errors['tests'][$i]['sizetarget'] = $this->gettext('wrongformat');
+                            if ($sizetarget == '')
+                                $this->errors['tests'][$i]['sizetarget'] = $this->gettext('cannotbeempty');
+                            else if (!preg_match('/^[0-9]+(K|M|G)*$/i', $sizetarget))
+                                $this->errors['tests'][$i]['sizetarget'] = $this->gettext('forbiddenchars');
                             break;
                         case '...':
@@ -824 +827 @@
         $out .= "</fieldset>\n";
 
+        $this->print_tips();
+
         if ($scr['disabled']) {
             $this->rc->output->set_env('rule_disabled', true);
@@ -878 +883 @@
 
         $out .= '<div id="custom_header' .$id. '" style="display:' .(isset($custom) ? 'inline' : 'none'). '">
-            <input type="text" name="_custom_header[]" '. $this->error_class($id, 'test', 'header')
+            <input type="text" name="_custom_header[]" id="custom_header_i'.$id.'" '
+            . $this->error_class($id, 'test', 'header', 'custom_header_i')
             .' value="' .Q($custom). '" size="20" />&nbsp;</div>' . "\n";
 
@@ -927 +933 @@
 
         $out .= '<input type="text" name="_rule_target[]" id="rule_target' .$id. '"
-            value="' .Q($target). '" size="20" ' . $this->error_class($id, 'test', 'target')
+            value="' .Q($target). '" size="20" ' . $this->error_class($id, 'test', 'target', 'rule_target')
             . ' style="display:' . ($rule['test']!='size' && $rule['test'] != 'exists' ? 'inline' : 'none') . '" />'."\n";
 
@@ -936 +942 @@
         $out .= '<div id="rule_size' .$id. '" style="display:' . ($rule['test']=='size' ? 'inline' : 'none') .'">';
         $out .= $select_size_op->show($rule['test']=='size' ? $rule['type'] : '');
-        $out .= '<input type="text" name="_rule_size_target[]" value="'.$sizetarget.'" size="10" ' 
-            . $this->error_class($id, 'test', 'sizetarget') .' />
+        $out .= '<input type="text" name="_rule_size_target[]" id="rule_size_i'.$id.'" value="'.$sizetarget.'" size="10" ' 
+            . $this->error_class($id, 'test', 'sizetarget', 'rule_size_i') .' />
             <input type="radio" name="_rule_size_item['.$id.']" value=""'
                 . (!$sizeitem ? ' checked="checked"' : '') .' class="radio" />'.rcube_label('B').'
@@ -1005 +1011 @@
             .'value="' .($action['type']=='redirect' ? Q($action['target'], 'strict', false) : ''). '" size="40" '
             .'style="display:' .($action['type']=='redirect' ? 'inline' : 'none') .'" '
-            . $this->error_class($id, 'action', 'target') .' />';
+            . $this->error_class($id, 'action', 'target', 'action_target') .' />';
         $out .= '<textarea name="_action_target_area[]" id="action_target_area' .$id. '" '
-            .'rows="3" cols="40" '. $this->error_class($id, 'action', 'targetarea')
+            .'rows="3" cols="40" '. $this->error_class($id, 'action', 'targetarea', 'action_target_area')
             .'style="display:' .(in_array($action['type'], array('reject', 'ereject')) ? 'inline' : 'none') .'">'
             . (in_array($action['type'], array('reject', 'ereject')) ? Q($action['target'], 'strict', false) : '')
@@ -1016 +1022 @@
         $out .= '<span class="label">'. Q($this->gettext('vacationreason')) .'</span><br />'
             .'<textarea name="_action_reason[]" id="action_reason' .$id. '" '
-            .'rows="3" cols="40" '. $this->error_class($id, 'action', 'reason') . '>'
+            .'rows="3" cols="40" '. $this->error_class($id, 'action', 'reason', 'action_reason') . '>'
             . Q($action['reason'], 'strict', false) . "</textarea>\n";
         $out .= '<br /><span class="label">' .Q($this->gettext('vacationaddresses')) . '</span><br />'
-            .'<input type="text" name="_action_addresses[]" '
+            .'<input type="text" name="_action_addresses[]" id="action_addr'.$id.'" '
             .'value="' . (is_array($action['addresses']) ? Q(implode(', ', $action['addresses']), 'strict', false) : $action['addresses']) . '" size="40" '
-            . $this->error_class($id, 'action', 'addresses') .' />';
+            . $this->error_class($id, 'action', 'addresses', 'action_addr') .' />';
         $out .= '<br /><span class="label">' . Q($this->gettext('vacationdays')) . '</span><br />'
-            .'<input type="text" name="_action_days[]" '
+            .'<input type="text" name="_action_days[]" id="action_days'.$id.'" '
             .'value="' .Q($action['days'], 'strict', false) . '" size="2" '
-            . $this->error_class($id, 'action', 'days') .' />';
+            . $this->error_class($id, 'action', 'days', 'action_days') .' />';
         $out .= '</div>';
 
@@ -1100 +1106 @@
     }
 
-    private function error_class($id, $type, $target, $name_only=false)
+    private function error_class($id, $type, $target, $elem_prefix='')
     {
         // TODO: tooltips
-        if ($type == 'test' && isset($this->errors['tests'][$id][$target]))
-            return ($name_only ? 'error' : ' class="error"');
-        else if ($type == 'action' && isset($this->errors['actions'][$id][$target]))
-            return ($name_only ? 'error' : ' class="error"');
+        if (($type == 'test' && ($str = $this->errors['tests'][$id][$target])) ||
+            ($type == 'action' && ($str = $this->errors['actions'][$id][$target]))
+        ) {
+            $this->add_tip($elem_prefix.$id, $str, true);
+            return ' class="error"';
+        }
 
         return '';
@@ -1115 +1123 @@
         return rcube_charset_convert($text, 'UTF7-IMAP', $encoding);
     }
+
+    private function add_tip($id, $str, $error=false)
+    {
+        if ($error)
+            $str = html::span('sieve error', $str);
+
+        $this->tips[] = array($id, $str);
+    }
+
+    private function print_tips()
+    {
+        if (empty($this->tips))
+            return;
+
+        $script = JS_OBJECT_NAME.'.managesieve_tip_register('.json_encode($this->tips).');';
+        $this->rc->output->add_script($script, 'foot');
+    }
+
 }

trunk/plugins/managesieve/skins/default/managesieve.css

@@ -290 +290 @@
   vertical-align: middle;
 }
+
+span.sieve.error
+{
+  color: red;
+}
+
+#managesieve-tip
+{
+  width: 200px;
+}