diff options
| author | Xavier Morel <xmo@odoo.com> | 2020-06-16 13:57:04 +0000 |
|---|---|---|
| committer | Xavier Morel <xmo@odoo.com> | 2020-06-17 11:19:27 +0000 |
| commit | 5f310366a099e846066ba3190e2b42daa04e165b (patch) | |
| tree | 2080462ef0648e225b0b1ae423bfe24abdf3b01b /addons/sms/models | |
| parent | b27f995089f9a9a1cf0cd8909527f6aa8a7ef425 (diff) | |
[IMP] base, sms, mail: make server action runners not public
There is no reason to call these directly, in fact it's not really
possible to do so as they expect an `action` object as first
parameter.
* warn against the presence of rpc-public runners
* move runner selection outside of ``run``
* improve doc a bit maybe
Diffstat (limited to 'addons/sms/models')
| -rw-r--r-- | addons/sms/models/ir_actions.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/addons/sms/models/ir_actions.py b/addons/sms/models/ir_actions.py index e5421d4546c..dddaa885d74 100644 --- a/addons/sms/models/ir_actions.py +++ b/addons/sms/models/ir_actions.py @@ -26,8 +26,7 @@ class ServerActions(models.Model): if action.state == 'sms' and not action.model_id.is_mail_thread: raise ValidationError(_("Sending SMS can only be done on a mail.thread model")) - @api.model - def run_action_sms_multi(self, action, eval_context=None): + def _run_action_sms_multi(self, action, eval_context=None): # TDE CLEANME: when going to new api with server action, remove action if not action.sms_template_id or self._is_recompute(action): return False |