Merge PR #992 into 17.0

Signed-off-by florian-dacosta

Author: OCA-git-bot

delivery_roulier/models/delivery_carrier.py

@@ -21,7 +21,7 @@ def _is_roulier(self):
         return "get_label" in available_carrier_actions.get(self.delivery_type, [])
 
     def cancel_shipment(self, pickings):
-        if self._is_roulier:
+        if self._is_roulier():
             pickings._cancel_shipment()
         else:
             return super().cancel_shipment(pickings)
@@ -44,7 +44,7 @@ def rate_shipment(self, order):
         res = super().rate_shipment(order)
         # for roulier carrier, usually getting the price by carrier webservice
         # is usually not available for now. Avoid failure in that case.
-        if not res and self.is_roulier():
+        if not res and self._is_roulier():
             res = {
                 "success": True,
                 "price": 0.0,