From d1da2928e972112a50a5cb5a91a9efa9547fdec2 Mon Sep 17 00:00:00 2001 From: "Eteil Djoumatchoua(etdj)" Date: Thu, 1 Jun 2023 06:21:58 +0000 Subject: [PATCH] [FIX] stock: number field for stock picking record The ``stock.picking`` record doesn't have a 'mobile' or a 'phone' field which can be used by ``sms.composer``,thus we need to return the ``partner_id`` field instead. opw-3286153 closes odoo/odoo#123716 X-original-commit: 35311aecef034402eac76e60acdc2658b9c681cf Signed-off-by: Djoumatchoua Eteil Junior (etdj) Signed-off-by: Thibault Delavallee (tde) --- addons/stock_sms/models/stock_picking.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/addons/stock_sms/models/stock_picking.py b/addons/stock_sms/models/stock_picking.py index 554a160fa56..dee5245f270 100644 --- a/addons/stock_sms/models/stock_picking.py +++ b/addons/stock_sms/models/stock_picking.py @@ -46,9 +46,12 @@ class Picking(models.Model): } def _sms_get_number_fields(self): - """ This method returns the fields to use to find the number to use to - send an SMS on a record. """ - return ['mobile', 'phone'] + """ No phone or mobile field is available on ``stock.picking`` model. Instead SMS will + fallback on partner-based computation using ``_sms_get_partner_fields``. """ + return [] + + def _sms_get_partner_fields(self): + return ['partner_id'] def _send_confirmation_email(self): super(Picking, self)._send_confirmation_email()