From 995ff7b94ac12ec7c25b81655ebab99095fac529 Mon Sep 17 00:00:00 2001
From: "Allan I. A" <76656700+Allan-I@users.noreply.github.com>
Date: Thu, 31 Mar 2022 18:59:44 +0300
Subject: [PATCH] Updates match_aov_pattern() logic to handle empty regex

Using `is not None` to simplify code and handle empty regex cases.

Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
---
 openpype/pipeline/farm/patterning.py | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/openpype/pipeline/farm/patterning.py b/openpype/pipeline/farm/patterning.py
index 6d3eb3e5ab..5ba7a8df4b 100644
--- a/openpype/pipeline/farm/patterning.py
+++ b/openpype/pipeline/farm/patterning.py
@@ -19,9 +19,6 @@ def match_aov_pattern(host_name, aov_patterns, render_file_name):
         bool: Review state for rendered file (render_file_name).
     """
     aov_pattern = aov_patterns.get(host_name, [])
-    if aov_pattern:
-        if re.match(aov_pattern, render_file_name):
-            preview = True
-            return preview
-        else:
-            return False
+    if not aov_pattern:
+        return False
+    return re.match(aov_pattern, render_file_name) is not None