From 7808dd535de9db0e6684e046ec831c7278370b42 Mon Sep 17 00:00:00 2001 From: Andrej Date: Fri, 6 Jun 2025 17:24:30 +0500 Subject: [PATCH] Fix contracted cost item removal logic It was removing checking for cost item index, though container is storing cost item ids. --- src/bonsai/bonsai/tool/cost.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/bonsai/bonsai/tool/cost.py b/src/bonsai/bonsai/tool/cost.py index d9d1df369e..0e80a44630 100644 --- a/src/bonsai/bonsai/tool/cost.py +++ b/src/bonsai/bonsai/tool/cost.py @@ -211,8 +211,8 @@ class Cost(bonsai.core.tool.Cost): cls.contracted_cost_items = json.loads(props.contracted_cost_items) if props.active_cost_item_id == cost_item_id: props.active_cost_item_id = 0 - if props.active_cost_item_index in cls.contracted_cost_items: - cls.contracted_cost_items.remove(props.active_cost_item_index) + if cost_item_id in cls.contracted_cost_items: + cls.contracted_cost_items.remove(cost_item_id) props.contracted_cost_items = json.dumps(cls.contracted_cost_items) cls.enable_editing_cost_items(cost_schedule=tool.Ifc.get().by_id(props.active_cost_schedule_id))