Fix item_url bad query detection

Author: akariv

budgetkey_api/modules/simpledb.py

@@ -42,13 +42,13 @@ def check_for_common_errors(table, sql):
                 'item multiple times. Use an exact match instead, e.g. "code = 12.34.56" or filter the query using the '
                 '`level` field.'
             )
-        group_by_item_url = re.search(r'group by.+item_url', sql, re.I | re.M | re.S | re.U)
-        if group_by_item_url:
-            ret.append(
-                'Using "GROUP BY" with "item_url" may lead to unwanted results.'
-                'If you are aggregating multiple items, remove the "item_url" from the query '
-                'or use "ARRAY_AGG(item_url)"'
-            )
+    group_by_item_url = re.search(r'group by.+item_url', sql, re.I | re.M | re.S | re.U)
+    if group_by_item_url:
+        ret.append(
+            'Using "GROUP BY" with "item_url" may lead to unwanted results.'
+            'If you are aggregating multiple items, remove the "item_url" from the query '
+            'or use "ARRAY_AGG(item_url)"'
+        )
     return ret