refactor: Updating the strategy to control the hidden parameter of each callback function through the hide_all_callbacks parameter

Author: CNFeffery

dash/_callback.py

@@ -41,7 +41,6 @@
 from . import _validate
 from .background_callback.managers import BaseBackgroundCallbackManager
 from ._callback_context import context_value
-from ._get_app import get_app
 from ._no_update import NoUpdate
 
 
@@ -179,13 +178,9 @@ def callback(
 
     background_spec: Any = None
 
-    # Get prevent_initial_callbacks and hide_all_callbacks from get_app().config
-    current_app_config = get_app().config
-
-    config_prevent_initial_callbacks = current_app_config.get(
-        "prevent_initial_callbacks", False
+    config_prevent_initial_callbacks = _kwargs.pop(
+        "config_prevent_initial_callbacks", False
     )
-    config_hide_all_callbacks = current_app_config.get("hide_all_callbacks", False)
     callback_map = _kwargs.pop("callback_map", GLOBAL_CALLBACK_MAP)
     callback_list = _kwargs.pop("callback_list", GLOBAL_CALLBACK_LIST)
 
@@ -227,7 +222,6 @@ def callback(
         callback_list,
         callback_map,
         config_prevent_initial_callbacks,
-        config_hide_all_callbacks,
         *_args,
         **_kwargs,
         background=background_spec,
@@ -259,7 +253,6 @@ def clientside_callback(clientside_function: ClientsideFuncType, *args, **kwargs
         GLOBAL_CALLBACK_LIST,
         GLOBAL_CALLBACK_MAP,
         False,
-        False,
         GLOBAL_INLINE_SCRIPTS,
         clientside_function,
         *args,
@@ -272,7 +265,6 @@ def insert_callback(
     callback_list,
     callback_map,
     config_prevent_initial_callbacks,
-    config_hide_all_callbacks,
     output,
     outputs_indices,
     inputs,
@@ -290,9 +282,6 @@ def insert_callback(
     if prevent_initial_call is None:
         prevent_initial_call = config_prevent_initial_callbacks
 
-    if hidden is None:
-        hidden = config_hide_all_callbacks
-
     _validate.validate_duplicate_output(
         output, prevent_initial_call, config_prevent_initial_callbacks
     )
@@ -611,7 +600,6 @@ def register_callback(
     callback_list,
     callback_map,
     config_prevent_initial_callbacks,
-    config_hide_all_callbacks,
     *_args,
     **_kwargs,
 ):
@@ -651,7 +639,6 @@ def register_callback(
         callback_list,
         callback_map,
         config_prevent_initial_callbacks,
-        config_hide_all_callbacks,
         insert_output,
         output_indices,
         flat_inputs,
@@ -664,7 +651,7 @@ def register_callback(
         running=running,
         no_output=not has_output,
         optional=_kwargs.get("optional", False),
-        hidden=_kwargs.get("hidden"),
+        hidden=_kwargs.get("hidden", None),
     )
 
     # pylint: disable=too-many-locals
@@ -849,7 +836,6 @@ def register_clientside_callback(
     callback_list,
     callback_map,
     config_prevent_initial_callbacks,
-    config_hide_all_callbacks,
     inline_scripts,
     clientside_function: ClientsideFuncType,
     *args,
@@ -861,7 +847,6 @@ def register_clientside_callback(
         callback_list,
         callback_map,
         config_prevent_initial_callbacks,
-        config_hide_all_callbacks,
         output,
         None,
         inputs,

dash/dash.py

@@ -69,7 +69,7 @@
 from . import _watch
 from . import _get_app
 
-from ._get_app import with_app_context, with_app_context_async, with_app_context_factory
+from ._get_app import get_app, with_app_context, with_app_context_async, with_app_context_factory
 from ._grouping import map_grouping, grouping_len, update_args_group
 from ._obsolete import ObsoleteChecker
 
@@ -366,7 +366,7 @@ class Dash(ObsoleteChecker):
         ``True`` here in which case you must explicitly set it ``False`` for
         those callbacks you wish to have an initial call. This setting has no
         effect on triggering callbacks when their inputs change later on.
-
+    
     :param hide_all_callbacks: Default ``False``: Sets the default value of
         ``hidden`` for all callbacks added to the app. Normally all callbacks
         are visible in the devtools callbacks tab. You can set this for
@@ -680,7 +680,6 @@ def _setup_hooks(self):
                 self._callback_list,
                 self.callback_map,
                 self.config.prevent_initial_callbacks,
-                self.config.hide_all_callbacks,
                 self._inline_scripts,
                 clientside_function,
                 *args,
@@ -1443,7 +1442,6 @@ def clientside_callback(self, clientside_function, *args, **kwargs):
             self._callback_list,
             self.callback_map,
             self.config.prevent_initial_callbacks,
-            self.config.hide_all_callbacks,
             self._inline_scripts,
             clientside_function,
             *args,
@@ -1466,6 +1464,7 @@ def callback(self, *_args, **_kwargs) -> Callable[..., Any]:
         """
         return _callback.callback(
             *_args,
+            config_prevent_initial_callbacks=self.config.prevent_initial_callbacks,
             callback_list=self._callback_list,
             callback_map=self.callback_map,
             callback_api_paths=self.callback_api_paths,
@@ -1655,7 +1654,21 @@ def _setup_server(self):
 
             self.callback_map[k] = _callback.GLOBAL_CALLBACK_MAP.pop(k)
 
+        # Get config of current app instance
+        current_app_config = get_app().config
+
         self._callback_list.extend(_callback.GLOBAL_CALLBACK_LIST)
+        # For each callback function, if the hidden parameter uses the default value None, 
+        # replace it with the actual value of the hide_all_callbacks property of the current application instance.
+        self._callback_list = [
+            {
+                **_callback,
+                "hidden": current_app_config.get("hide_all_callbacks", False)
+            }
+            if _callback.get("hidden") is None
+            else _callback
+            for _callback in self._callback_list
+        ]
         _callback.GLOBAL_CALLBACK_LIST.clear()
 
         _validate.validate_background_callbacks(self.callback_map)