Mohammed Shekha(Open ERP) has proposed merging
lp:~openerp-dev/openobject-client-web/6.0-opw-50938-msh into
lp:openobject-client-web.
Requested reviews:
OpenERP Core Team (openerp)
For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-50938-msh/+merge/84907
Hello,
Fixed the issue of background color of required field in export.
Demo :- Export any record in web, you will see there is no background color in
required field.
Reason :- The code contains condition like if is_import then ans then we will
make the background color of required field.(line no. 224 of impex.py).
I have removed the whole condition because the UI of import is already changed
in web so there is no need of checking is_import.
Due to this condition when we will export data then else condition will be true
so required param will not be passed anyway in export.
So changed the code.
Thanks.
--
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-50938-msh/+merge/84907
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openobject-client-web/6.0-opw-50938-msh.
=== modified file 'addons/openerp/controllers/impex.py'
--- addons/openerp/controllers/impex.py 2011-06-20 11:11:42 +0000
+++ addons/openerp/controllers/impex.py 2011-12-08 09:26:38 +0000
@@ -221,23 +221,12 @@
id = prefix + (prefix and '/' or '') + field
nm = name + (name and '/' or '') + value['string']
- if is_importing and (value.get('type') not in ('reference',)) and (not value.get('readonly') \
- or not dict(value.get('states', {}).get('draft', [('readonly', True)])).get('readonly', True)):
-
- record.update(id=id, items={'name': nm},
- action='javascript: void(0)', target=None,
- icon=None, children=[],
- required=value.get('required', False))
-
- records.append(record)
-
- elif not is_importing:
-
- record.update(id=id, items={'name': nm},
- action='javascript: void(0)', target=None,
- icon=None, children=[])
- records.append(record)
-
+ record.update(id=id, items={'name': nm},
+ action='javascript: void(0)', target=None,
+ icon=None, children=[],
+ required=value.get('required', False))
+
+ records.append(record)
if len(nm.split('/')) < 3 and value.get('relation', False):
_______________________________________________
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
