move max recursion to __init__.py

Author: nicopop

src/Rules.py

@@ -78,11 +78,11 @@ def checkRequireStringForArea(state: CollectionState, area: dict):
         if requires_list == "":
             return True
 
-        def findAndRecursivelyExecuteFunctions(requires_list: str, maxRecursion: int, recursionDepth: int = 0) -> str:
+        def findAndRecursivelyExecuteFunctions(requires_list: str, recursionDepth: int = 0) -> str:
             found_functions = re.findall(r'\{(\w+)\((.*?)\)\}', requires_list)
             if found_functions:
-                if recursionDepth >= maxRecursion:
-                    raise RecursionError(f'One or more functions in "{area.get("name", f"An area with these parameters: {area}")}"\'s requires looped too many time (maximum recursion is {maxRecursion}) \
+                if recursionDepth >= world.rules_functions_maximum_recursion:
+                    raise RecursionError(f'One or more functions in "{area.get("name", f"An area with these parameters: {area}")}"\'s requires looped too many time (maximum recursion is {world.rules_functions_maximum_recursion}) \
                                          \n    As of this Exception the following function(s) are waiting to run: {[f[0] for f in found_functions]} \
                                          \n    And the currently processed requires look like this: "{requires_list}"')
                 else:
@@ -107,10 +107,10 @@ def findAndRecursivelyExecuteFunctions(requires_list: str, maxRecursion: int, re
                         else:
                             requires_list = requires_list.replace("{" + func_name + "(" + item[1] + ")}", str(result))
 
-                requires_list = findAndRecursivelyExecuteFunctions(requires_list, maxRecursion, recursionDepth + 1)
+                requires_list = findAndRecursivelyExecuteFunctions(requires_list, recursionDepth + 1)
             return requires_list
 
-        requires_list = findAndRecursivelyExecuteFunctions(requires_list, maxRecursion=5)
+        requires_list = findAndRecursivelyExecuteFunctions(requires_list)
 
         # parse user written statement into list of each item
         for item in re.findall(r'\|[^|]+\|', requires_list):

src/__init__.py

@@ -339,21 +339,26 @@ def write_spoiler(self, spoiler_handle):
 
     def extend_hint_information(self, hint_data: dict[int, dict[int, str]]) -> None:
         before_extend_hint_information(hint_data, self, self.multiworld, self.player)
-        
+
         for location in self.multiworld.get_locations(self.player):
             if not location.address:
                 continue
             if "hint_entrance" in self.location_name_to_location[location.name]:
                 if self.player not in hint_data:
                     hint_data.update({self.player: {}})
                 hint_data[self.player][location.address] = self.location_name_to_location[location.name]["hint_entrance"]
-        
+
         after_extend_hint_information(hint_data, self, self.multiworld, self.player)
 
     ###
     # Non-standard AP world methods
     ###
 
+    rules_functions_maximum_recursion: int = 5
+    """Default: 5\n
+    The maximum time a location/region's requirement can loop to check for functions\n
+    One thing to remember is the more you loop the longer generation will take. So probably leave it as is unless you really needs it."""
+
     def add_filler_items(self, item_pool, traps):
         Utils.deprecate("Use adjust_filler_items instead.")
         return self.adjust_filler_items(item_pool, traps)