diff options
author | Will Kahn-Greene <willg@bluesock.org> | 2012-07-17 21:02:12 -0400 |
---|---|---|
committer | Will Kahn-Greene <willg@bluesock.org> | 2012-07-17 21:02:12 -0400 |
commit | 05e007c1dbe7b5b8a092f1a99ed361c4e6b71f26 (patch) | |
tree | 36a0d1ba4174baff5a003871a28e08a512b57a84 /mediagoblin/plugins/sampleplugin/__init__.py | |
parent | 8464bcc3e86e223db0739101c0b5d914eea225af (diff) | |
download | mediagoblin-05e007c1dbe7b5b8a092f1a99ed361c4e6b71f26.tar.lz mediagoblin-05e007c1dbe7b5b8a092f1a99ed361c4e6b71f26.tar.xz mediagoblin-05e007c1dbe7b5b8a092f1a99ed361c4e6b71f26.zip |
Rework plugin infrastructure to nix side-effects
This reworks the plugin infrastructure so as to remove module-loading
side-effects which were making things a pain in the ass to test.
With the new system, there's no auto-registering meta class. Instead
plugins do whatever they want and then specify a hooks dict that maps
hook names to callables for the things they're tying into. The most
common one (and the only one we've implemented so far) is "setup".
This also simplifies the sampleplugin a little by moving the code
to __init__.py.
Diffstat (limited to 'mediagoblin/plugins/sampleplugin/__init__.py')
-rw-r--r-- | mediagoblin/plugins/sampleplugin/__init__.py | 28 |
1 files changed, 25 insertions, 3 deletions
diff --git a/mediagoblin/plugins/sampleplugin/__init__.py b/mediagoblin/plugins/sampleplugin/__init__.py index b87348af..2cd077a2 100644 --- a/mediagoblin/plugins/sampleplugin/__init__.py +++ b/mediagoblin/plugins/sampleplugin/__init__.py @@ -15,6 +15,28 @@ # along with this program. If not, see <http://www.gnu.org/licenses/>. -# This imports the module that has the Plugin subclass in it which -# causes that module to get imported and that class to get registered. -import mediagoblin.plugins.sampleplugin.main +import logging + +from mediagoblin.tools.pluginapi import get_config + + +_log = logging.getLogger(__name__) + + +_setup_plugin_called = 0 + +def setup_plugin(): + global _setup_plugin_called + + _log.info('Sample plugin set up!') + config = get_config('mediagoblin.plugins.sampleplugin') + if config: + _log.info('%r' % config) + else: + _log.info('There is no configuration set.') + _setup_plugin_called += 1 + + +hooks = { + 'setup': setup_plugin + } |