Merge lp:~openerp-dev/openobject-client/calendar_fix into lp:~openerp-dev/openobject-client/trunk-dev-client

Proposed by Ravi Gadhia (OpenERP)
Status: Merged
Merged at revision: 1451
Proposed branch: lp:~openerp-dev/openobject-client/calendar_fix
Merge into: lp:~openerp-dev/openobject-client/trunk-dev-client
Diff against target: 38 lines (+4/-3)
2 files modified
bin/modules/gui/window/form.py (+2/-0)
bin/widget/view/calendar_gtk/parser.py (+2/-3)
To merge this branch: bzr merge lp:~openerp-dev/openobject-client/calendar_fix
Reviewer Review Type Date Requested Status
Naresh(OpenERP) Pending
Review via email: mp+47376@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bin/modules/gui/window/form.py'
2--- bin/modules/gui/window/form.py 2011-01-13 05:47:45 +0000
3+++ bin/modules/gui/window/form.py 2011-01-25 10:21:15 +0000
4@@ -299,6 +299,8 @@
5 if autosave:
6 if not self.modified_save():
7 return
8+ if self.screen.current_view.view_type in ['calendar','graph']:
9+ return
10 self.screen.create_new = True
11 self.screen.new()
12 self.message_state('')
13
14=== modified file 'bin/widget/view/calendar_gtk/parser.py'
15--- bin/widget/view/calendar_gtk/parser.py 2011-01-17 06:43:44 +0000
16+++ bin/widget/view/calendar_gtk/parser.py 2011-01-25 10:21:15 +0000
17@@ -280,11 +280,10 @@
18
19 for model in models:
20 name = value = key = model.value[color_field]
21-
22 if isinstance(key, (tuple, list)):
23 value, name = key
24 key = tuple(key)
25-
26+
27 if key in colors:
28 # already present skip
29 continue
30@@ -293,7 +292,7 @@
31 # 'color' field
32 field_color = None
33 field_widget = model.mgroup.mfields.get(color_field, False)
34- if field_widget and field_widget.attrs['type'] == 'many2one':
35+ if value and field_widget and field_widget.attrs['type'] == 'many2one':
36 fproxy = RPCProxy(field_widget.attrs['relation'])
37 try:
38 fdata = fproxy.read(value, [color_field_custom])

Subscribers

People subscribed via source and target branches