refactor: simplify conditions in BaseConnection using Rector

Author: gr8man

system/Database/BaseConnection.php

@@ -1280,10 +1280,8 @@ public function protectIdentifiers($item, bool $prefixSingle = false, ?bool $pro
         //
         // Added exception for single quotes as well, we don't want to alter
         // literal strings.
-        if (strcspn($item, "()'") !== strlen($item)) {
-            if ($this->isIdentifierEscapeExempt($item)) {
-                return $item;
-            }
+        if (strcspn($item, "()'") !== strlen($item) && $this->isIdentifierEscapeExempt($item)) {
+            return $item;
         }
 
         // Do not protect identifiers and do not prefix, no swap prefix, there is nothing to do
@@ -1454,16 +1452,9 @@ private function isIdentifierEscapeExempt(string $item): bool
         if ($this->escapeChar !== '"' && $item[0] === '"' && preg_match('/^"(?:[^"]|"")*"$/s', $item)) {
             return true;
         }
-
         // SQL functions or subqueries (e.g. MAX(id), (SELECT ...)) with an optional alias
-        if (str_contains($item, '(')) {
-            // Regex matching balanced parentheses (from start to end or with a safe alias)
-            if (preg_match('/^(?:[a-zA-Z0-9_.]+\s*)?(?P<parens>\((?:[^()]+|(?&parens))*\))(?:\s+AS\s+(?:[a-zA-Z0-9_.]+|"[^"]*"|\'[^\']*\'|`[^`]*`))?$/is', $item)) {
-                return true;
-            }
-        }
-
-        return false;
+        // Regex matching balanced parentheses (from start to end or with a safe alias)
+        return str_contains($item, '(') && preg_match('/^(?:[a-zA-Z0-9_.]+\s*)?(?P<parens>\((?:[^()]+|(?&parens))*\))(?:\s+AS\s+(?:[a-zA-Z0-9_.]+|"[^"]*"|\'[^\']*\'|`[^`]*`))?$/is', $item);
     }
 
     /**