Merge lp:~openerp-dev/openobject-client/6.1-opw-575749-rgo into lp:openobject-client/6.1
Proposed by
Ravi Gohil (OpenERP)
Status: | Merged |
---|---|
Approved by: | Naresh(OpenERP) |
Approved revision: | 2066 |
Merged at revision: | 2077 |
Proposed branch: | lp:~openerp-dev/openobject-client/6.1-opw-575749-rgo |
Merge into: | lp:openobject-client/6.1 |
Diff against target: |
32 lines (+3/-3) 2 files modified
bin/SpiffGtkWidgets/Calendar/CanvasEvent.py (+1/-1) bin/widget/view/calendar_gtk/parser.py (+2/-2) |
To merge this branch: | bzr merge lp:~openerp-dev/openobject-client/6.1-opw-575749-rgo |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Naresh(OpenERP) | Pending | ||
Review via email: mp+109836@code.launchpad.net |
Description of the change
Hello,
When the calendar view doesn't get the value for it's fields, it shows `False` as a string and not the blank string.
You can reproduce the issue by defining a meeting and not setting the value for the fields `Sales Team` or `Partner`.
This MP contains fix for this issue. Kindly review it.
Thanks.
To post a comment you must log in.
just modified the patch a bit before final merge
=== modified file 'bin/widget/ view/calendar_ gtk/parser. py' view/calendar_ gtk/parser. py 2011-10-31 09:12:35 +0000 view/calendar_ gtk/parser. py 2012-07-20 06:25:32 +0000
--- bin/widget/
+++ bin/widget/
@@ -440,13 +440,14 @@
if isinstance(s, (tuple, list)): s = s[-1]
- caption = ustr(s) f]['type' ] == 'boolean'): f]['type' ] == 'boolean'):
-
+ if not (isinstance(s,bool) or self.fields[
+ caption = ustr(s)
+
for f in self.axis[1:]:
s = event[f]
if isinstance(s, (tuple, list)): s = s[-1]
-
- description += [ustr(s)]
+ if not (isinstance(s, bool) or self.fields[
+ description += [ustr(s)]
starts = event.get( self.date_ start) self.date_ delay) or 1.0
ends = event.get(