From 229758122dba321d53ddeef2804aad6c95fc4da6 Mon Sep 17 00:00:00 2001
From: Saurang Suthar <ssu@tinyerp.com>
Date: Wed, 10 Oct 2012 11:52:04 +0530
Subject: [PATCH] [IMP]event:improved code as per suggestion

bzr revid: ssu@tinyerp.com-20121010062204-gna6c83qfhqhr4zy
---
 addons/event/event.py       | 10 ++++------
 addons/event/event_view.xml |  2 +-
 2 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/addons/event/event.py b/addons/event/event.py
index 51e559f9e0c..73836f45b09 100644
--- a/addons/event/event.py
+++ b/addons/event/event.py
@@ -289,16 +289,14 @@ class event_event(osv.osv):
 
         return {'value' : values}
 
-    def onchange_start_date(self, cr, uid, ids, date_end=False, date_begin=False, context=None):
+    def onchange_start_date(self, cr, uid, ids, date_begin=False, date_end=False, context=None):
         res = {'value':{}}
         if date_end:
             return res
-        if isinstance(date_begin, str):
+        if date_begin and isinstance(date_begin, str):
             date_begin = datetime.strptime(date_begin, "%Y-%m-%d %H:%M:%S")
-        if isinstance(date_end, str):
-            date_end = datetime.strptime(date_end, "%Y-%m-%d %H:%M:%S")
-        date_end = date_begin + timedelta(hours=1)
-        res['value'] = {'date_end': date_end.strftime("%Y-%m-%d %H:%M:%S")}
+            date_end = date_begin + timedelta(hours=1)
+            res['value'] = {'date_end': date_end.strftime("%Y-%m-%d %H:%M:%S")}
         return res
 
     # ----------------------------------------
diff --git a/addons/event/event_view.xml b/addons/event/event_view.xml
index 3f936b9ba4d..0a6b24ad3a1 100644
--- a/addons/event/event_view.xml
+++ b/addons/event/event_view.xml
@@ -134,7 +134,7 @@
                                 </group>
                                 <group>
                                     <field name="type" on_change="onchange_event_type(type,context)"/>
-                                    <field name="date_begin" on_change="onchange_start_date(date_end,date_begin)"/>
+                                    <field name="date_begin" on_change="onchange_start_date(date_begin,date_end)"/>
                                     <field name="date_end"/>
                                     <field name="company_id" groups="base.group_multi_company"/>
                                 </group>