Odoo Web (MOVED TO GITHUB)

Merge lp:~openerp-dev/openerp-web/6.1-opw-579515-cpa into lp:openerp-web/6.1

  1. 6.1-opw-579515-cpa
  2. Merge into 6.1
Proposed by Chirag Patel (OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openerp-web/6.1-opw-579515-cpa
Merge into: lp:openerp-web/6.1
Diff against target: 11 lines (+1/-0)
1 file modified
addons/web/static/src/js/view_form.js (+1/-0)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/6.1-opw-579515-cpa
Reviewer Review Type Date Requested Status
Jiten (OpenERP) (community) Approve
OpenERP Core Team Pending
Review via email: mp+126222@code.launchpad.net

Description of the change

Hello,

Set title for warning dialog box.

When any onchange return warning.
e.g
             return {'warning' : {
                 'title' : 'Warning',
                 'message' : '%s is not a valid email address. Please
                        correct the email and save again.' % (email,),
                 }
             }

Message display properly but title not set in dialog box header.

Thanks.

To post a comment you must log in.
Revision history for this message
Jiten (OpenERP) (jiten-openerp) :
review: Approve
Reply

Unmerged revisions

2461. By Chirag Patel (OpenERP)

[FIX] Fixed set title for warning dialog box.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/web/static/src/js/view_form.js'
2--- addons/web/static/src/js/view_form.js 2012-09-24 15:26:28 +0000
3+++ addons/web/static/src/js/view_form.js 2012-09-25 11:04:00 +0000
4@@ -452,6 +452,7 @@
5 }
6 if (!_.isEmpty(result.warning)) {
7 $(QWeb.render("CrashManagerWarning", result.warning)).dialog({
8+ title: result.warning.title,
9 modal: true,
10 buttons: [
11 {text: _t("Ok"), click: function() { $(this).dialog("close"); }}