diff --git a/addons/anonymous/static/src/js/anonymous.js b/addons/anonymous/static/src/js/anonymous.js index a28c56ac411..7946c913829 100644 --- a/addons/anonymous/static/src/js/anonymous.js +++ b/addons/anonymous/static/src/js/anonymous.js @@ -36,7 +36,7 @@ openerp.anonymous = function(instance) { var p = self.getParent(); var am = p.action_manager; am.do_action({type:'ir.actions.client', tag:'login'}); - am.client_widget.on('login', p, p.show_application); + am.inner_widget.on('login', p, p.show_application); }); } }); diff --git a/addons/signup/__init__.py b/addons/signup/__init__.py index f15ba34bf0a..d70c1894925 100644 --- a/addons/signup/__init__.py +++ b/addons/signup/__init__.py @@ -1,2 +1,2 @@ -import signup_wizard import res_config +import signup diff --git a/addons/signup/__openerp__.py b/addons/signup/__openerp__.py index b927b9526b7..b899d5bc434 100644 --- a/addons/signup/__openerp__.py +++ b/addons/signup/__openerp__.py @@ -1,6 +1,6 @@ { 'name': 'Signup', - 'description': 'Allow users to register', + 'description': 'Allow users to sign up', 'author': 'OpenERP SA', 'version': '1.0', 'category': 'Tools', @@ -8,15 +8,12 @@ 'installable': True, 'depends': ['anonymous', 'base_setup'], 'data': [ - 'signup_wizard.xml', 'res_config.xml', + 'signup.xml', ], 'js': [ 'static/src/js/signup.js', ], - #'css': [ - # 'static/src/css/reset_password.css', - #], 'qweb': [ 'static/src/xml/signup.xml', ], diff --git a/addons/signup/res_config.py b/addons/signup/res_config.py index 0c8ab8023d8..53f0513b234 100644 --- a/addons/signup/res_config.py +++ b/addons/signup/res_config.py @@ -1,19 +1,20 @@ from openerp.osv import osv, fields -class ResConfig(osv.TransientModel): +class base_config_settings(osv.TransientModel): _inherit = 'base.config.settings' _columns = { - 'signup_user_template_id': fields.many2one('res.users', 'Template user for account creation') + 'signup_template_user_id': fields.many2one('res.users', 'Template user for signup') } - def get_default_user_tpl(self, cr, uid, fields, context=None): + def get_default_signup(self, cr, uid, fields, context=None): icp = self.pool.get('ir.config_parameter') return { - 'signup_user_template_id': icp.get_param(cr, uid, 'signup.user_template_id', 0) or False + 'signup_template_user_id': icp.get_param(cr, uid, 'signup.template_user_id', 0) or False } - def set_user_template(self, cr, uid, ids, context=None): + def set_signup(self, cr, uid, ids, context=None): config = self.browse(cr, uid, ids[0], context=context) icp = self.pool.get('ir.config_parameter') - icp.set_param(cr, uid, 'signup.user_template_id', config.signup_user_template_id.id) + icp.set_param(cr, uid, 'signup.template_user_id', config.signup_user_template_id.id) + diff --git a/addons/signup/res_config.xml b/addons/signup/res_config.xml index d2c85a33fd5..d13f7002d17 100644 --- a/addons/signup/res_config.xml +++ b/addons/signup/res_config.xml @@ -1,13 +1,13 @@ - - Inherit General Settings + + base.config.settings.signup base.config.settings form - + diff --git a/addons/signup/signup_wizard.py b/addons/signup/signup.py similarity index 63% rename from addons/signup/signup_wizard.py rename to addons/signup/signup.py index 939e90aa21d..3819a63da85 100644 --- a/addons/signup/signup_wizard.py +++ b/addons/signup/signup.py @@ -1,4 +1,3 @@ -from functools import partial from openerp.osv import osv, fields class res_users(osv.Model): @@ -8,14 +7,13 @@ class res_users(osv.Model): ('email_uniq', 'UNIQUE (user_email)', 'You can not have two users with the same email!') ] - -class signup_wizard(osv.TransientModel): - _name = 'signup.wizard' +class signup_signup(osv.TransientModel): + _name = 'signup.signup' _columns = { 'name': fields.char('Name', size=64), 'email': fields.char('Email', size=64), - 'pw': fields.char('Password', size=64), - 'cpw': fields.char('Confirm Password', size=64), + 'password': fields.char('Password', size=64), + 'password_confirmation': fields.char('Confirm Password', size=64), 'state': fields.selection([(x, x) for x in 'draft done missmatch'.split()], required=True), } _defaults = { @@ -25,33 +23,26 @@ class signup_wizard(osv.TransientModel): def create(self, cr, uid, values, context=None): # NOTE here, invalid values raises exceptions to avoid storing # sensitive data into the database (which then are available to anyone) - - name = values.get('name') - email = values.get('email') - pw = values.get('pw') - cpw = values.get('cpw') - - if pw != cpw: + if values['password'] != values['password_confirmation']: raise osv.except_osv('Error', 'Passwords missmatch') - Users = self.pool.get('res.users') + new_user = { + 'name': values['name'], + 'login': values['email'], + 'user_email': values['email'], + 'password': values['password'], + 'active': True, + } user_template_id = self.pool.get('ir.config_parameter').get_param(cr, uid, 'signup.user_template_id', 0) if user_template_id: - func = partial(Users.copy, cr, 1, user_template_id, context=context) + self.pool.get('res.users').copy(cr, 1, user_template_id, new_user, context=context) else: - func = partial(Users.create, cr, 1, context=context) - - func({ - 'name': name, - 'login': email, - 'user_email': email, - 'password': pw, - 'active': True, - }) + self.pool.get('res.users').create(cr, 1, new_user, context=context) + # Dont store the password values = {'state': 'done'} - return super(signup_wizard, self).create(cr, uid, values, context) + return super(signup_signup, self).create(cr, uid, values, context) def signup(self, cr, uid, ids, context=None): return { @@ -63,3 +54,4 @@ class signup_wizard(osv.TransientModel): if pw != cpw: return {'value': {'state': 'missmatch'}} return {'value': {'state': 'draft'}} + diff --git a/addons/signup/signup_wizard.xml b/addons/signup/signup.xml similarity index 61% rename from addons/signup/signup_wizard.xml rename to addons/signup/signup.xml index ec5095ff474..3dacf85feb4 100644 --- a/addons/signup/signup_wizard.xml +++ b/addons/signup/signup.xml @@ -1,25 +1,25 @@ - - signup.wizard.form - signup.wizard + + signup.signup.form + signup.signup form
- + - - + +
Passwords missmatch
-