diff --git a/product_contract/models/sale_order.py b/product_contract/models/sale_order.py
index c34a118eb..57467736c 100644
--- a/product_contract/models/sale_order.py
+++ b/product_contract/models/sale_order.py
@@ -127,7 +127,9 @@ class SaleOrder(models.Model):
     @api.depends("order_line")
     def _compute_contract_count(self):
         for rec in self:
-            rec.contract_count = len(rec.order_line.mapped('contract_id'))
+            rec.contract_count = len(
+                rec.order_line.mapped('contract_id').filtered(
+                    lambda r: r.active))
 
     @api.multi
     def action_show_contracts(self):
diff --git a/product_contract/views/sale_order.xml b/product_contract/views/sale_order.xml
index b1caab5c8..d00f241a8 100644
--- a/product_contract/views/sale_order.xml
+++ b/product_contract/views/sale_order.xml
@@ -23,7 +23,7 @@
                 <button name="action_show_contracts"
                         type="object" icon="fa-book"
                         class="oe_stat_button"
-                        attrs="{'invisible': ['|', ('is_contract', '!=', True), ('state', 'not in', ['sale', 'done'])]}">
+                        attrs="{'invisible': ['|', '|', ('is_contract', '!=', True), ('state', 'not in', ['sale', 'done']), ('contract_count', '=', 0)]}">
                     <field string="Contracts"
                            name="contract_count"
                            widget="statinfo"/>