diff options
| author | Raphael Collet <rco@openerp.com> | 2016-11-22 12:04:09 +0100 |
|---|---|---|
| committer | Raphael Collet <rco@openerp.com> | 2017-01-03 16:52:49 +0100 |
| commit | d0ca2d115e8b40308e1ff83bf9e7cc874762fbff (patch) | |
| tree | bdad5e8c9019f14a0416c27612969ae4932519b1 /addons/google_account/models/google_service.py | |
| parent | 7543afb0704d624f7abe6a2450a9d7e4a75e158b (diff) | |
[REF] ir_config_parameter: remove `group_ids` and simplify
Add `sudo()` to call `get_param` where necessary, and make the web client use a
controller instead of directly accessing parameters.
Diffstat (limited to 'addons/google_account/models/google_service.py')
| -rw-r--r-- | addons/google_account/models/google_service.py | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/addons/google_account/models/google_service.py b/addons/google_account/models/google_service.py index d707ee8d188..07fa2d12cf5 100644 --- a/addons/google_account/models/google_service.py +++ b/addons/google_account/models/google_service.py @@ -57,11 +57,11 @@ class GoogleService(models.TransientModel): @api.model def _get_google_token_uri(self, service, scope): - Parameters = self.env['ir.config_parameter'].sudo() + get_param = self.env['ir.config_parameter'].sudo().get_param encoded_params = werkzeug.url_encode({ 'scope': scope, - 'redirect_uri': Parameters.get_param('google_redirect_uri'), - 'client_id': Parameters.get_param('google_%s_client_id' % service), + 'redirect_uri': get_param('google_redirect_uri'), + 'client_id': get_param('google_%s_client_id' % service), 'response_type': 'code', }) return '%s?%s' % (GOOGLE_AUTH_ENDPOINT, encoded_params) @@ -77,9 +77,9 @@ class GoogleService(models.TransientModel): 'f': from_url } - Parameters = self.env['ir.config_parameter'] - base_url = Parameters.get_param('web.base.url', default='http://www.odoo.com?NoBaseUrl') - client_id = Parameters.sudo().get_param('google_%s_client_id' % (service,), default=False) + get_param = self.env['ir.config_parameter'].sudo().get_param + base_url = get_param('web.base.url', default='http://www.odoo.com?NoBaseUrl') + client_id = get_param('google_%s_client_id' % (service,), default=False) encoded_params = werkzeug.url_encode({ 'response_type': 'code', @@ -97,10 +97,10 @@ class GoogleService(models.TransientModel): """ Call Google API to exchange authorization code against token, with POST request, to not be redirected. """ - Parameters = self.env['ir.config_parameter'] - base_url = Parameters.get_param('web.base.url', default='http://www.odoo.com?NoBaseUrl') - client_id = Parameters.sudo().get_param('google_%s_client_id' % (service,), default=False) - client_secret = Parameters.sudo().get_param('google_%s_client_secret' % (service,), default=False) + get_param = self.env['ir.config_parameter'].sudo().get_param + base_url = get_param('web.base.url', default='http://www.odoo.com?NoBaseUrl') + client_id = get_param('google_%s_client_id' % (service,), default=False) + client_secret = get_param('google_%s_client_secret' % (service,), default=False) headers = {"content-type": "application/x-www-form-urlencoded"} data = werkzeug.url_encode({ @@ -120,9 +120,9 @@ class GoogleService(models.TransientModel): # FIXME : this method update a field defined in google_calendar module. Since it is used only in that module, maybe it should be moved. @api.model def _refresh_google_token_json(self, refresh_token, service): # exchange_AUTHORIZATION vs Token (service = calendar) - Parameters = self.env['ir.config_parameter'].sudo() - client_id = Parameters.get_param('google_%s_client_id' % (service,), default=False) - client_secret = Parameters.get_param('google_%s_client_secret' % (service,), default=False) + get_param = self.env['ir.config_parameter'].sudo().get_param + client_id = get_param('google_%s_client_id' % (service,), default=False) + client_secret = get_param('google_%s_client_secret' % (service,), default=False) headers = {"content-type": "application/x-www-form-urlencoded"} data = werkzeug.url_encode({ |
