Skip to content

Commit 63f9208

Browse files
committed
[FIX] fieldservice: Fixed expected singleton on mass edit
1 parent fe5be19 commit 63f9208

File tree

2 files changed

+10
-9
lines changed

2 files changed

+10
-9
lines changed

fieldservice/models/fsm_order.py

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -260,13 +260,14 @@ def create(self, vals):
260260
is_button = fields.Boolean(default=False)
261261

262262
def write(self, vals):
263-
if vals.get("stage_id", False) and vals.get("is_button", False):
264-
vals["is_button"] = False
265-
else:
266-
stage_id = self.env["fsm.stage"].browse(vals.get("stage_id"))
267-
if stage_id == self.env.ref("fieldservice.fsm_stage_completed"):
268-
raise UserError(_("Cannot move to completed from Kanban"))
269-
self._calc_scheduled_dates(vals)
263+
for record in self:
264+
if vals.get("stage_id", False) and vals.get("is_button", False):
265+
vals["is_button"] = False
266+
else:
267+
stage_id = self.env["fsm.stage"].browse(vals.get("stage_id"))
268+
if stage_id == self.env.ref("fieldservice.fsm_stage_completed"):
269+
raise UserError(_("Cannot move to completed from Kanban"))
270+
record._calc_scheduled_dates(vals)
270271
res = super().write(vals)
271272
return res
272273

fieldservice/tests/test_fsm_order.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -256,15 +256,15 @@ def test_fsm_order(self):
256256
)
257257
)
258258
self.assertTrue(data, "It should be able to read group")
259-
self.Order.write(
259+
self.order.write(
260260
{
261261
"location_id": self.test_location.id,
262262
"stage_id": self.stage1.id,
263263
"is_button": True,
264264
}
265265
)
266266
with self.assertRaises(UserError):
267-
self.Order.write(
267+
self.order.write(
268268
{
269269
"location_id": self.test_location.id,
270270
"stage_id": self.stage1.id,

0 commit comments

Comments
 (0)