Use request_args for suppress_reindex

Author: yuanzhou

src/app.py

@@ -1177,7 +1177,7 @@ def create_entity(entity_type):
     #
     # Check if re-indexing is to be suppressed after entity creation.
     try:
-        supress_reindex = schema_manager.suppress_reindex(request)
+        supress_reindex = schema_manager.suppress_reindex(request.args)
     except Exception as e:
         bad_request_error(e)
 
@@ -1486,7 +1486,7 @@ def update_entity(id):
     #
     # Check if re-indexing is to be suppressed after entity creation.
     try:
-        suppress_reindex = schema_manager.suppress_reindex(request)
+        suppress_reindex = schema_manager.suppress_reindex(request.args)
     except Exception as e:
         bad_request_error(e)
 
@@ -4122,7 +4122,7 @@ def multiple_components():
     #
     # Check if re-indexing is to be suppressed after entity creation.
     try:
-        suppress_reindex = schema_manager.suppress_reindex(request)
+        suppress_reindex = schema_manager.suppress_reindex(request.args)
     except Exception as e:
         bad_request_error(e)
 

src/schema/schema_manager.py

@@ -2363,28 +2363,28 @@ def get_organ_types():
 
 
 """
-Use the Flask request.args MultiDict to see if 'reindex' is a URL parameter passed in with the
-request and if it indicates reindexing should be supressed. Default to reindexing in all other cases.
+See if 'reindex' is a URL parameter passed in with the request and 
+if it indicates reindexing should be supressed. Default to reindexing in all other cases.
 
 Parameters
 ----------
-request: Flask request object
-    The instance of Flask request passed in from application request
+request_args: ImmutableMultiDict
+    The Flask request.args passed in from application request
 
 Returns
 -------
 bool
 """
-def suppress_reindex(request) -> bool:
-    if 'reindex' not in request.args:
+def suppress_reindex(request_args) -> bool:
+    if 'reindex' not in request_args:
         return False
-    reindex_str = request.args.get('reindex').lower()
+    reindex_str = request_args.get('reindex').lower()
     if reindex_str == 'false':
         return True
     elif reindex_str == 'true':
         return False
     raise Exception(f"The value of the 'reindex' parameter must be True or False (case-insensitive)."
-                    f" '{request.args.get('reindex')}' is not recognized.")
+                    f" '{request_args.get('reindex')}' is not recognized.")
 
 
 ####################################################################################################

src/schema/schema_triggers.py

@@ -1737,7 +1737,7 @@ def update_status(property_key, normalized_type, request_args, user_token, exist
             # When the parent dataset status update disables reindex via query string '?reindex=false'
             # We'll also disable the reindex call to search-api upon each subsequent child component dataset update
             reindex = 'followed'
-            if schema_manager.suppress_reindex(request): 
+            if schema_manager.suppress_reindex(request_args): 
                 url += '?reindex=false'
                 reindex = 'suppressed'