From 3d33d0406e46e7545682f0053bc46f3c75ced77d Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@odoo.com>
Date: Wed, 20 Nov 2024 12:33:00 +0100
Subject: [PATCH] [FIX] runbot_merge: tracking author is already a res.partner

Not sure why it didn't break tests in 16...
---
 runbot_merge/models/pull_requests.py | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/runbot_merge/models/pull_requests.py b/runbot_merge/models/pull_requests.py
index 64cd4d94..1833a6f2 100644
--- a/runbot_merge/models/pull_requests.py
+++ b/runbot_merge/models/pull_requests.py
@@ -1460,9 +1460,7 @@ For your own safety I've ignored *everything in your entire comment*.
         if newhead:
             authors = self.env.cr.precommit.data.get(f'mail.tracking.author.{self._name}', {})
             for p in self:
-                if pid := authors.get(p.id):
-                    writer = self.env['res.partner'].browse(pid)
-                else:
+                if not (writer := authors.get(p.id)):
                     writer = self.env.user.partner_id
                 p.unstage("updated by %s", writer.github_login or writer.name)
             # this can be batched