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

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

Description of the change

Hello,

Display error when enter wrong password in backup db.

Demo: Managa Database > Backup
1) Enter wrong password.
2) Click on Backup.

Observed: Display traceback.
Expected: Error message display in dialog(like drop db with wrong password).

Thanks.

To post a comment you must log in.

Unmerged revisions

2509. By Chirag Patel (OpenERP)

[FIX] Display error when enter wrong password in backup db.

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/chrome.js'
2--- addons/web/static/src/js/chrome.js 2012-07-05 10:27:24 +0000
3+++ addons/web/static/src/js/chrome.js 2013-03-14 12:57:21 +0000
4@@ -455,7 +455,14 @@
5 self.do_notify(_t("Backed"),
6 _t("Database backed up successfully"));
7 },
8- error: openerp.webclient.crashmanager.on_rpc_error,
9+ error: function(error){
10+ if(error){
11+ self.display_error({
12+ title: _t("Backup Database"),
13+ error: 'AccessDenied'
14+ });
15+ }
16+ },
17 complete: function() {
18 self.unblockUI();
19 }