Yannick Tivisse
100f320217
Currently, all the methods that start with `get_default_` and `set_` are called on a res.config.settings loading or saving.
This commit purpose is to replace all the occurences of `get_default_foo` and `set_foo`. As these methods won't be called anymore, a warning is logged to notice its deprecation.
The generic method `get_default_fields` and `set_fields` could be improved too. The method names and API are not so good:
Why "default" fields? What you ask for is the current value of the stuff, shown as fields in the model. The values are fed as default values in the wizard, but that's an implementation trick.
Why "fields"? What you ask for are configuration parameters, and such things.
Why passing a list of fields? Its value is never used.
We should further simplify the API of both methods to something like
def get_values(self):
return {}
def set_values(self):
pass
36 lines
1.6 KiB
Python
36 lines
1.6 KiB
Python
# -*- coding: utf-8 -*-
|
|
# Part of Odoo. See LICENSE file for full copyright and licensing details.
|
|
|
|
from odoo import api, fields, models
|
|
|
|
|
|
class BaseConfigSettings(models.TransientModel):
|
|
_inherit = "base.config.settings"
|
|
|
|
google_drive_authorization_code = fields.Char(string='Authorization Code')
|
|
google_drive_uri = fields.Char(compute='_compute_drive_uri', string='URI', help="The URL to generate the authorization code from Google")
|
|
|
|
@api.depends('google_drive_authorization_code')
|
|
def _compute_drive_uri(self):
|
|
google_drive_uri = self.env['google.service']._get_google_token_uri('drive', scope=self.env['google.drive.config'].get_google_scope())
|
|
for config in self:
|
|
config.google_drive_uri = google_drive_uri
|
|
|
|
@api.model
|
|
def get_values(self):
|
|
res = super(BaseConfigSettings, self).get_values()
|
|
res.update(
|
|
google_drive_authorization_code=self.env['ir.config_parameter'].sudo().get_param('google_drive_authorization_code'),
|
|
)
|
|
return res
|
|
|
|
def set_values(self):
|
|
super(BaseConfigSettings, self).set_values()
|
|
params = self.env['ir.config_parameter'].sudo()
|
|
authorization_code = self.google_drive_authorization_code
|
|
refresh_token = False
|
|
if authorization_code and authorization_code != params.get_param('google_drive_authorization_code'):
|
|
refresh_token = self.env['google.service'].generate_refresh_token('drive', authorization_code)
|
|
params.set_param('google_drive_authorization_code', authorization_code)
|
|
params.set_param('google_drive_refresh_token', refresh_token)
|