From b078275c94466270e82a1ec6399bcab51e168779 Mon Sep 17 00:00:00 2001 From: Xavier-Do Date: Fri, 17 Mar 2023 11:38:06 +0100 Subject: [PATCH] [IMP] runbot: add commit_export_ids --- runbot/models/build.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/runbot/models/build.py b/runbot/models/build.py index 41333dfc..07c30bd7 100644 --- a/runbot/models/build.py +++ b/runbot/models/build.py @@ -233,6 +233,7 @@ class BuildResult(models.Model): killable = fields.Boolean('Killable') database_ids = fields.One2many('runbot.database', 'build_id') + commit_export_ids = fields.One2many('runbot.commit.export', 'build_id') static_run = fields.Char('Static run URL') @@ -343,7 +344,8 @@ class BuildResult(models.Model): # some validation to ensure db consistency if 'local_state' in values: if values['local_state'] == 'done': - self.env['runbot.commit.export'].search([('build_id', 'in', self.ids)]).unlink() + self.filtered(lambda b: b.local_state != 'done').commit_export_ids.unlink() + local_result = values.get('local_result') for build in self: if local_result and local_result != self._get_worst_result([build.local_result, local_result]): # dont write ok on a warn/error build @@ -368,9 +370,6 @@ class BuildResult(models.Model): if not build.parent_id and build.global_state not in ('done', 'running'): build._github_status() - - - return res def _add_child(self, param_values, orphan=False, description=False, additionnal_commit_links=False):