Vishmita Jadeja (openerp) has proposed merging
lp:~openerp-dev/openerp-web/trunk-bug-1096832-vja into lp:openerp-web.
Requested reviews:
OpenERP R&D Web Team (openerp-dev-web)
Related bugs:
Bug #1096832 in OpenERP Web: "[Usability] Field name in the notification
should be consistent with field label in the form"
https://bugs.launchpad.net/openerp-web/+bug/1096832
For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-1096832-vja/+merge/142442
Hello,
Field name in the notification should be consistent with field label in
the form.
Thanks.
--
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-1096832-vja/+merge/142442
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openerp-web/trunk-bug-1096832-vja.
=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js 2012-12-21 14:02:36 +0000
+++ addons/web/static/src/js/view_form.js 2013-01-09 05:27:23 +0000
@@ -864,7 +864,7 @@
.filter(function (f) { return !f.is_valid(); })
.map(function (f) {
return _.str.sprintf('<li>%s</li>',
- _.escape(f.string));
+ _.escape(f.$label.text() || f.string));
}).value();
warnings.unshift('<ul>');
warnings.push('</ul>');
_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to : openerp-dev-gtk@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help : https://help.launchpad.net/ListHelp
