Jignesh Rathod(OpenERP) has proposed merging 
lp:~openerp-dev/openobject-addons/trunk-bug-961094-jir into 
lp:openobject-addons.

Requested reviews:
  OpenERP Core Team (openerp)
Related bugs:
  Bug #961094 in OpenERP Addons: "Entering repeating event with no ending 
brings down server"
  https://bugs.launchpad.net/openobject-addons/+bug/961094

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-961094-jir/+merge/99883

Hello Marcel van der Boom ,

Field " Recurrence termination " should not blank. So 
I have set " Recurrence termination " field  required. 

Thanks for reporting.  
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-961094-jir/+merge/99883
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-bug-961094-jir.
=== modified file 'base_calendar/base_calendar.py'
--- base_calendar/base_calendar.py	2012-03-05 14:35:54 +0000
+++ base_calendar/base_calendar.py	2012-03-29 07:21:20 +0000
@@ -1002,8 +1002,8 @@
             event = datas['id']
             if datas.get('interval', 0) < 0:
                 raise osv.except_osv(_('Warning!'), _('Interval cannot be negative'))
-            if datas.get('count', 0) < 0:
-                raise osv.except_osv(_('Warning!'), _('Count cannot be negative'))
+            if datas.get('count', 0) <= 0:
+                raise osv.except_osv(_('Warning!'), _('Count cannot be negative or 0'))
             if datas['recurrency']:
                 result[event] = self.compute_rule_string(datas)
             else:

=== modified file 'crm/crm_meeting_view.xml'
--- crm/crm_meeting_view.xml	2012-03-19 10:09:54 +0000
+++ crm/crm_meeting_view.xml	2012-03-29 07:21:20 +0000
@@ -171,7 +171,7 @@
 
 
                                 <separator string="End of recurrency" colspan="4"/>
-                                <field name="end_type" />
+                                <field name="end_type" required="1" />
                                 <label string=" " colspan="2" />
                                 <newline />
                                 <field name="count" attrs="{'invisible' : [('end_type', '!=', 'count')] }"/>

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to