Removed the public decorator from all modules. __all__ is now explicitly set in each module. Did not remove the public decorator itself from the public module in case others wish to still use it.

Changed tick.repeat.Status class to tick.repeat.RepeatStatus since Status is not very descriptive.

Author: satoon101

addons/source-python/packages/source-python/__init__.py

@@ -38,6 +38,13 @@
 from excepthooks import ExceptHooks
 
 
+# =============================================================================
+# >> ALL DECLARATION
+# =============================================================================
+# Set all to an empty list
+__all__ = []
+
+
 # =============================================================================
 # >> LOGGING SETUP
 # =============================================================================

addons/source-python/packages/source-python/auth/manager.py

@@ -4,7 +4,6 @@
 # >> IMPORTS
 # =============================================================================
 # Source.Python imports
-from public import public
 #   Auth
 from auth import AuthLogger
 from auth.paths import AUTH_PROVIDER_PATH
@@ -14,6 +13,15 @@
 from translations.strings import LangStrings
 
 
+# =============================================================================
+# >> ALL DECLARATION
+# =============================================================================
+# Add all the global variables to __all__
+__all__ = [
+    'AuthManager',
+]
+
+
 # =============================================================================
 # >> GLOBAL VARIABLES
 # =============================================================================
@@ -27,7 +35,6 @@
 # =============================================================================
 # >> CLASSES
 # =============================================================================
-@public
 class _AuthManager(dict):
     '''Class used to store loaded auth providers
         and check if a player is authorized'''

addons/source-python/packages/source-python/auth/paths.py

@@ -4,18 +4,25 @@
 # >> IMPORTS
 # =============================================================================
 # Source.Python imports
-from paths import SP_PACKAGES_PATH as _SP_PACKAGES_PATH
-from paths import CFG_PATH as _CFG_PATH
+from paths import SP_PACKAGES_PATH
+from paths import CFG_PATH
+
+
+# =============================================================================
+# >> ALL DECLARATION
+# =============================================================================
+# Add all the global variables to __all__
+__all__ = [
+    'AUTH_CFG_PATH',
+    'AUTH_PROVIDER_PATH',
+]
 
 
 # =============================================================================
 # >> GLOBAL VARIABLES
 # =============================================================================
 # Store the path to the auth providers
-AUTH_PROVIDER_PATH = _SP_PACKAGES_PATH.joinpath('auth', 'providers')
+AUTH_PROVIDER_PATH = SP_PACKAGES_PATH.joinpath('auth', 'providers')
 
 # Store the path to the auth configurations
-AUTH_CFG_PATH = _CFG_PATH.joinpath('auth_providers')
-
-# Add all paths to __all__
-__all__ = [x for x in globals() if x.isupper() and not x.startswith('_')]
+AUTH_CFG_PATH = CFG_PATH.joinpath('auth_providers')

addons/source-python/packages/source-python/auth/providers/simple.py

@@ -12,11 +12,18 @@
 from auth.paths import AUTH_CFG_PATH
 
 
+# =============================================================================
+# >> ALL DECLARATION
+# =============================================================================
+# Set all to an empty list
+__all__ = []
+
+
 # =============================================================================
 # >> GLOBAL VARIABLES
 # =============================================================================
 # Store the path to the simple.txt file
-SIMPLE_FILE_PATH = AUTH_CFG_PATH.joinpath('simple.txt')
+_SIMPLE_FILE_PATH = AUTH_CFG_PATH.joinpath('simple.txt')
 
 
 # =============================================================================
@@ -31,7 +38,7 @@ def _parse_admins(self):
         '''
 
         # Open the simple auth config file
-        with SIMPLE_FILE_PATH.open() as auth_file:
+        with _SIMPLE_FILE_PATH.open() as auth_file:
 
             # Loop through each line in the file
             for line in auth_file.readlines():
@@ -58,9 +65,9 @@ def is_player_authorized(self, uniqueid, level, permission, flag):
         return False
 
 # Get the _SimpleAuth instance
-SimpleAuth = _SimpleAuth()
+_SimpleAuthInstance = _SimpleAuth()
 
-is_player_authorized = SimpleAuth.is_player_authorized
+is_player_authorized = _SimpleAuthInstance.is_player_authorized
 
 
 # =============================================================================
@@ -70,11 +77,11 @@ def load():
     '''Loads the provider by getting all uniqueids that are authorized'''
 
     # Parse the simple auth file
-    SimpleAuth._parse_admins()
+    _SimpleAuthInstance._parse_admins()
 
 
 def unload():
     '''Unloads the provider by clearing the set'''
 
     # Clear the set
-    SimpleAuth.clear()
+    _SimpleAuthInstance.clear()

addons/source-python/packages/source-python/commands/client/__init__.py

@@ -13,5 +13,9 @@
 # =============================================================================
 # >> ALL DECLARATION
 # =============================================================================
-# Add all imported classes to all
-__all__ = list(globals())
+# Add all the global variables to __all__
+__all__ = [
+    'ClientCommand',
+    'ClientCommandFilter',
+    'ClientCommandManager',
+]

addons/source-python/packages/source-python/commands/client/command.py

@@ -4,16 +4,23 @@
 # >> IMPORTS
 # =============================================================================
 # Source.Python Imports
-from public import public
 #   Commands
 from commands.command import _BaseCommand
 from commands.client.manager import ClientCommandManager
 
 
+# =============================================================================
+# >> ALL DECLARATION
+# =============================================================================
+# Add all the global variables to __all__
+__all__ = [
+    'ClientCommand',
+]
+
+
 # =============================================================================
 # >> CLASSES
 # =============================================================================
-@public
 class ClientCommand(_BaseCommand):
     '''Decorator class used to register a client command'''
 

addons/source-python/packages/source-python/commands/client/filter.py

@@ -4,16 +4,23 @@
 # >> IMPORTS
 # =============================================================================
 # Source.Python Imports
-from public import public
 #   Commands
 from commands.filter import _BaseFilter
 from commands.client.manager import ClientCommandManager
 
 
+# =============================================================================
+# >> ALL DECLARATION
+# =============================================================================
+# Add all the global variables to __all__
+__all__ = [
+    'ClientCommandFilter',
+]
+
+
 # =============================================================================
 # >> CLASSES
 # =============================================================================
-@public
 class ClientCommandFilter(_BaseFilter):
     '''Class used to register a client command filter'''
 

addons/source-python/packages/source-python/commands/client/manager.py

@@ -7,15 +7,22 @@
 from command_c import get_client_command
 from command_c import register_client_command_filter
 from command_c import unregister_client_command_filter
-from public import public
 #   Commands
 from commands.player import _PlayerCommandManager
 
 
+# =============================================================================
+# >> ALL DECLARATION
+# =============================================================================
+# Add all the global variables to __all__
+__all__ = [
+    'ClientCommandManager',
+]
+
+
 # =============================================================================
 # >> CLASSES
 # =============================================================================
-@public
 class _ClientCommandManager(_PlayerCommandManager):
     '''Manager class used to register client
         commands and client command filters'''

addons/source-python/packages/source-python/commands/say/__init__.py

@@ -13,5 +13,9 @@
 # =============================================================================
 # >> ALL DECLARATION
 # =============================================================================
-# Add all imported classes to all
-__all__ = list(globals())
+# Add all the global variables to __all__
+__all__ = [
+    'SayCommand',
+    'SayCommandManager',
+    'SayFilter',
+]

addons/source-python/packages/source-python/commands/say/command.py

@@ -4,16 +4,23 @@
 # >> IMPORTS
 # =============================================================================
 # Source.Python Imports
-from public import public
 #   Commands
 from commands.command import _BaseCommand
 from commands.say.manager import SayCommandManager
 
 
+# =============================================================================
+# >> ALL DECLARATION
+# =============================================================================
+# Add all the global variables to __all__
+__all__ = [
+    'SayCommand',
+]
+
+
 # =============================================================================
 # >> CLASSES
 # =============================================================================
-@public
 class SayCommand(_BaseCommand):
     '''Decorator class used to register a say command'''