Mohammed Shekha(Open ERP) has proposed merging
lp:~openerp-dev/openerp-web/6.1-opw-574907-msh into lp:openerp-web/6.1.
Requested reviews:
OpenERP Core Team (openerp)
For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-574907-msh/+merge/106762
Hello,
Fixed the issue of database creation with the same name which is already exist,
currently this is giving traceback instead of giving a user friendly message.
Thanks.
--
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-574907-msh/+merge/106762
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openerp-web/6.1-opw-574907-msh.
=== modified file 'addons/web/static/src/js/chrome.js'
--- addons/web/static/src/js/chrome.js 2012-05-16 14:53:54 +0000
+++ addons/web/static/src/js/chrome.js 2012-05-22 09:18:18 +0000
@@ -386,6 +386,13 @@
self.$option_id.html(QWeb.render("Database.CreateDB", self));
self.$option_id.find("form[name=create_db_form]").validate({
submitHandler: function (form) {
+ if(jQuery.inArray(jQuery('input[name=db_name]').val(), self.db_list)){
+ self.display_error({
+ title: 'Create Database',
+ error: 'Database '+ jQuery('input[name=db_name]').val() +' Already Exist'
+ })
+ return;
+ }
var fields = $(form).serializeArray();
self.rpc("/web/database/create", {'fields': fields}, function(result) {
if (self.db_list) {
_______________________________________________
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
