MFB: Set an alarm to one minute, if users try to set an alarm without time.

diff --git a/nag/docs/CHANGES b/nag/docs/CHANGES
index f5e8a49..9fa8ef8 100644 (file)
--- a/nag/docs/CHANGES
+++ b/nag/docs/CHANGES
@@ -15,6 +15,7 @@ v3.0-git
 v2.3.5-cvs
 ----------
 
+[jan] Set an alarm to one minute, if users try to set an alarm without time.
 [jan] Add missing preference for not sending update notifications to yourself
       (Joel Smith <js-horde@jk1.net>, Request #8978).
 [gwr] Fix organizer field not saved in tasks (kolab/issue3888).

diff --git a/nag/lib/Forms/task.php b/nag/lib/Forms/task.php
index 6639838..878d3b2 100644 (file)
--- a/nag/lib/Forms/task.php
+++ b/nag/lib/Forms/task.php
@@ -215,8 +215,14 @@ class Horde_Form_Type_nag_alarm extends Horde_Form_Type {
         $info = $var->getValue($vars);
         if (!$info['on']) {
             $info = 0;
+        } else {
+            $value = $info['value'];
+            $unit = $info['unit'];
+            if ($value == 0) {
+                $value = $unit = 1;
+            }
+            $info = $value * $unit;
         }
-        $info = $info['value'] * $info['unit'];
     }
 
     function isValid(&$var, &$vars, $value, &$message)
@@ -226,10 +232,6 @@ class Horde_Form_Type_nag_alarm extends Horde_Form_Type {
                 $message = _("A due date must be set to enable alarms.");
                 return false;
             }
-            if (empty($value['value'])) {
-                $message = _("The alarm value must not be empty.");
-                return false;
-            }
         }
 
         return true;

diff --git a/nag/lib/Ui/VarRenderer/Nag.php b/nag/lib/Ui/VarRenderer/Nag.php
index cc30da3..8d22477 100644 (file)
--- a/nag/lib/Ui/VarRenderer/Nag.php
+++ b/nag/lib/Ui/VarRenderer/Nag.php
@@ -11,7 +11,7 @@
 
 /**
  * The Horde_Ui_VarRenderer_Nag class provides additional methods for
- * rendering Horde_Form_Type_alarm fields.
+ * rendering Nag specific fields.
  *
  * @todo    Clean this hack up with Horde_Form/H4
  * @author  Jan Schneider <jan@horde.org>