[IMP] runbot: simplify regex check

xmo-odoo · Xavier-Do · commit 1a661a05d8b0 · 2025-11-14T11:13:22.000+01:00
diff --git a/runbot/models/build_error.py b/runbot/models/build_error.py
@@ -1232,7 +1232,7 @@ def _validate(self):
             except re.error as e:
                 raise ValidationError("Unable to compile regular expression: %s" % e)
             # verify that a named group exist in the pattern
-            if not re.search(r'\(\?P<\w+>.+\)', r.pattern):
+            if not r.groupindex:
                 raise ValidationError(
                     "The regular expresion should contain at least one named group pattern e.g: '(?P<module>.+)'"
                 )

Original file line number	Diff line number	Diff line change
`@@ -1232,7 +1232,7 @@ def _validate(self):`
`1232`	`1232`	`except re.error as e:`
`1233`	`1233`	`raise ValidationError("Unable to compile regular expression: %s" % e)`
`1234`	`1234`	`# verify that a named group exist in the pattern`
`1235`		`- if not re.search(r'\(\?P<\w+>.+\)', r.pattern):`
	`1235`	`+ if not r.groupindex:`
`1236`	`1236`	`raise ValidationError(`
`1237`	`1237`	`"The regular expresion should contain at least one named group pattern e.g: '(?P<module>.+)'"`
`1238`	`1238`	`)`