Refactor get_actions methods to simplify return statements and remove…

… commented-out code
mjanez · Oct 23, 2024 · 617db41 · 617db41
1 parent dae2d83
commit 617db41
Showing 1 changed file with 2 additions and 21 deletions.
diff --git a/ckanext/schemingdcat/plugin.py b/ckanext/schemingdcat/plugin.py
@@ -93,9 +93,7 @@ def get_signal_subscriptions(self):
 
     #IActions
     def get_actions(self):
-        action_functions = _get_logic_functions('ckanext.schemingdcat.logic.action')
-
-        return action_functions
+        return _get_logic_functions('ckanext.schemingdcat.logic.action')
 
 class SchemingDCATDatasetsPlugin(SchemingDatasetsPlugin):
     p.implements(p.IConfigurer)
@@ -236,24 +234,9 @@ def about_template(self):
 
 
 class SchemingDCATOpenDataStatisticsPlugin(p.SingletonPlugin):
-    #p.implements(p.IConfigurer)
     p.implements(p.IActions)
     p.implements(p.IConfigurable)
 
-    # # IConfigurer
-    # def update_config(self, config_):
-    #     sdct_config.schemingdcat_open_data_statistics_themes = p.toolkit.asbool(
-    #         config_.get(
-    #             "ckanext.schemingdcat.open_data_statistics_themes", sdct_config.schemingdcat_open_data_statistics_themes
-    #         )
-    #     )
-
-    #     sdct_config.schemingdcat_open_data_statistics = p.toolkit.asbool(
-    #         config_.get(
-    #             "ckanext.schemingdcat.open_data_statistics", sdct_config.schemingdcat_open_data_statistics
-    #         )
-    #     )
-
     #IConfigurable
     def configure(self, config):
         args = sys.argv
@@ -266,9 +249,7 @@ def configure(self, config):
 
     #IActions
     def get_actions(self):
-        action_functions = _get_logic_functions('ckanext.schemingdcat.statistics.logic.action')
-
-        return action_functions
+        return _get_logic_functions('ckanext.schemingdcat.statistics.logic.action')
 
 def _get_logic_functions(module_root, logic_functions=None, decorator=None):
     """