From: helge <helge@d1b88da0-ebda-0310-925b-ed51d893ca5b>
Date: Tue, 19 Jul 2005 07:56:26 +0000 (+0000)
Subject: validate that apt has a title
X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0bf78adbafb7e49cc5bfefa8b5684e13701a50df;p=scalable-opengroupware.org

validate that apt has a title


git-svn-id: http://svn.opengroupware.org/SOGo/trunk@814 d1b88da0-ebda-0310-925b-ed51d893ca5b
---

diff --git a/SOGo/SoObjects/SOGo/SOGoAppointment.m b/SOGo/SoObjects/SOGo/SOGoAppointment.m
index ca48a32c..63710860 100644
--- a/SOGo/SoObjects/SOGo/SOGoAppointment.m
+++ b/SOGo/SoObjects/SOGo/SOGoAppointment.m
@@ -55,7 +55,7 @@ static NGLogger                  *logger = nil;
   sax = [[SaxObjectDecoder alloc] initWithMappingNamed:@"NGiCal"];
   if (sax == nil)
     [logger fatalWithFormat:@"could not create the iCal SAX handler!"];
-
+  
   [parser setContentHandler:sax];
   [parser setErrorHandler:sax];
 }
diff --git a/SOGo/UI/Contacts/English.lproj/Localizable.strings b/SOGo/UI/Contacts/English.lproj/Localizable.strings
index bfe7e055..0ececb30 100644
--- a/SOGo/UI/Contacts/English.lproj/Localizable.strings
+++ b/SOGo/UI/Contacts/English.lproj/Localizable.strings
@@ -37,4 +37,4 @@
 
 /* error */
 
-"invalidemailwarn"  = "Invalid E-Mail Field, Continue?";
+"invalidemailwarn"  = "Invalid email field, continue?";
diff --git a/SOGo/UI/Scheduler/English.lproj/Localizable.strings b/SOGo/UI/Scheduler/English.lproj/Localizable.strings
index e3725154..1db9cf39 100644
--- a/SOGo/UI/Scheduler/English.lproj/Localizable.strings
+++ b/SOGo/UI/Scheduler/English.lproj/Localizable.strings
@@ -210,3 +210,7 @@
 "MEETING"		= "Meeting";
 "HOLIDAY"		= "Holiday";
 "PHONE CALL"		= "Phone";
+
+/* validation errors */
+
+validate_notitle = "No title is set, continue?";
diff --git a/SOGo/UI/Templates/ChangeLog b/SOGo/UI/Templates/ChangeLog
index 602c63c6..397ed582 100644
--- a/SOGo/UI/Templates/ChangeLog
+++ b/SOGo/UI/Templates/ChangeLog
@@ -1,3 +1,8 @@
+2005-07-19  Helge Hess  <helge.hess@opengroupware.org>
+
+	* UIxAppointmentEditor.wox: check whether title is set and raise a
+	  confirm panel if not
+
 2005-07-18  Helge Hess  <helge.hess@opengroupware.org>
 
 	* ContactsUI/UIxContactEditor.wox: validate email field prior form
diff --git a/SOGo/UI/Templates/UIxAppointmentEditor.wox b/SOGo/UI/Templates/UIxAppointmentEditor.wox
index ee4d18b3..a3ce8bec 100644
--- a/SOGo/UI/Templates/UIxAppointmentEditor.wox
+++ b/SOGo/UI/Templates/UIxAppointmentEditor.wox
@@ -1,16 +1,31 @@
 <?xml version='1.0' standalone='yes'?>
-<var:component xmlns="http://www.w3.org/1999/xhtml"
-               xmlns:var="http://www.skyrix.com/od/binding"
-               xmlns:const="http://www.skyrix.com/od/constant"
-               xmlns:uix="OGo:uix"
-               xmlns:label="OGo:label"
-               className="UIxPageFrame"
-               title="name"
+<var:component
+  xmlns="http://www.w3.org/1999/xhtml"
+  xmlns:var="http://www.skyrix.com/od/binding"
+  xmlns:const="http://www.skyrix.com/od/constant"
+  xmlns:uix="OGo:uix"
+  xmlns:label="OGo:label"
+  className="UIxPageFrame"
+  title="name"
 >
-  <form var:href="clientObject.baseURL">
+  <script language="JavaScript">
+    function validateAptEditor() {
+      var e;
+      
+      e = document.getElementById('summary');
+      if (e.value.length == 0)
+        return confirm('<var:string label:value="validate_notitle"/>');
+      return true;
+    }
+  </script>
+  
+  <form var:href="clientObject.baseURL" name="editform"
+        onsubmit="return validateAptEditor()">
+
       <script language="JavaScript"> <!-- TODO: use a resource -->
         <var:string value="jsCode" const:escapeHTML="NO" />
       </script>
+
       <table cellspacing="0" cellpadding="5" width="100%">
         <tr>
           <td>
@@ -29,12 +44,12 @@
         <tr>
           <td>
             <var:if condition="hasErrorText">
-  	    <div style="background-color: #AA0000;">
-  	      <var:string value="errorText" />
+    	      <div style="background-color: #AA0000;">
+  	        <var:string value="errorText" />
               </div>
               <hr />
             </var:if>
-  
+
             <table border="0" cellpadding="2" cellspacing="0" width="100%"
                    bgcolor="#e8e8e0"
             >
@@ -85,8 +100,8 @@
                 </td>
                 <td align="left" bgcolor="#FFFFF0">
                   <span class="aptview_text">
-                    <input type="text" name="summary" const:size="40"
-                           var:value="title" />
+                    <input type="text" name="summary" id="summary"
+                           const:size="40" var:value="title" />
                   </span>
                 </td>
               </tr>
@@ -111,11 +126,8 @@
                 </td>
                 <td align="left" bgcolor="#FFFFF0">
                   <span class="aptview_text">
-                    <var:popup list="priorities"
-                               item="item"
-                               string="itemPriorityText"
-                               selection="priority"
-                    />
+                    <var:popup list="priorities" item="item"
+                               string="itemPriorityText" selection="priority"/>
                   </span>
                 </td>
               </tr>
@@ -130,8 +142,7 @@
                     <table>
                       <tr>
                         <td>
-                          <var:popup list="cycles"
-                                     item="item"
+                          <var:popup list="cycles" item="item"
                                      label:string="$cycleLabel"
                                      selection="cycle"
                                      const:onChange="selectHasCycle(this);"
@@ -145,10 +156,8 @@
                         <td id="cycle_end_mode"
                             var:style="initialCycleVisibility"
                         >
-                          <var:popup list="cycleEnds"
-                                     item="item"
-                                     label:string="$item"
-                                     value="item"
+                          <var:popup list="cycleEnds" item="item"
+                                     label:string="$item" value="item"
                                      selection="cycleEnd"
                                      const:onChange="selectCycleEnd(this);"
                                      const:id="cycle_end_mode_selection"
@@ -177,8 +186,7 @@
                 </td>
                 <td align="left" bgcolor="#FFFFF0">
                   <span class="aptview_text">
-                    <var:checkbox-list list="categoryItems"
-                                       item="item"
+                    <var:checkbox-list list="categoryItems" item="item"
                                        suffix="itemCategoryText"
                                        selections="categories"
                     />
@@ -193,10 +201,9 @@
                 </td>
                 <td align="left" bgcolor="#FFFFF0">
                   <span class="aptview_text">
-                    <input type="checkbox"
-                           var:selection="isPrivate"
-                           var:checked="isPrivate"
-                    /> <var:string label:value="is private" />
+                    <input type="checkbox" var:selection="isPrivate"
+                           var:checked="isPrivate" />
+                    <var:string label:value="is private" />
                   </span>
                 </td>
               </tr>