[FIX] shoppingfeed_integration: avoid singleton error on order import

Method _shoppingfeed_format_phone called without order
Temporary sale order lines can compute their invoice status before they are
linked to a sale order.
TT62646

Author: carlosdauden

shoppingfeed_integration/models/sale_order.py

@@ -60,7 +60,7 @@ class SaleOrder(models.Model):
     ]
 
     def _is_shoppingfeed_disable_invoicing(self):
-        return self.shoppingfeed_channel_id.disable_invoicing
+        return bool(self and self.shoppingfeed_channel_id.disable_invoicing)
 
     @api.depends("shoppingfeed_channel_id")
     def _compute_invoice_status(self):
@@ -93,6 +93,8 @@ def _shoppingfeed_fetch_orders(self, store):
 
     @api.model
     def _shoppingfeed_format_phone(self, phone_number, country):
+        if not self or not phone_number:
+            return False
         phone_sanitized = self._phone_format(
             number=phone_number, country=country, force_format="INTERNATIONAL"
         )
@@ -487,6 +489,7 @@ class SaleOrderLine(models.Model):
     def _compute_invoice_status(self):
         res = super()._compute_invoice_status()
         self.filtered(
-            lambda sol: sol.order_id._is_shoppingfeed_disable_invoicing()
+            lambda sol: sol.order_id
+            and sol.order_id._is_shoppingfeed_disable_invoicing()
         ).invoice_status = "no"
         return res

shoppingfeed_integration/tests/test_shoppingfeed_integration.py

@@ -119,6 +119,12 @@ def test_invoice_not_auto_paid_when_disabled(self):
         self.assertNotEqual(invoice.payment_state, "paid")
         self.sf_channel.auto_pay = True
 
+    def test_invoice_status_on_new_line_without_order(self):
+        """Temporary lines can compute invoice status before an order exists."""
+        line = self.env["sale.order.line"].new({"product_id": self.product_a.id})
+        line._compute_invoice_status()
+        self.assertEqual(line.invoice_status, "no")
+
     def test_product_reference_cleaning(self):
         """_shoppingfeed_clean_product_reference strips 2-char country suffixes only."""
         so = self.env["sale.order"]