diff options
author | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2011-08-09 13:11:34 +0200 |
---|---|---|
committer | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2011-08-09 13:18:22 +0200 |
commit | 56fc71865919792159ce4307101ef246488b4194 (patch) | |
tree | 93c2c4cb28c4e1f270000c2b2c1e08882d9a58ec | |
parent | f7d73aeb34ed16c7d1aa06d977ffb4c9c7a44051 (diff) | |
download | mediagoblin-56fc71865919792159ce4307101ef246488b4194.tar.lz mediagoblin-56fc71865919792159ce4307101ef246488b4194.tar.xz mediagoblin-56fc71865919792159ce4307101ef246488b4194.zip |
Storage config: Drop all Backward Compatibility
Chris Webber says not to care about backward compatibility
at this stage. So drop the last bits.
-rw-r--r-- | mediagoblin/init/__init__.py | 5 | ||||
-rw-r--r-- | mediagoblin/storage.py | 14 |
2 files changed, 6 insertions, 13 deletions
diff --git a/mediagoblin/init/__init__.py b/mediagoblin/init/__init__.py index 32f0194c..44f604b1 100644 --- a/mediagoblin/init/__init__.py +++ b/mediagoblin/init/__init__.py @@ -112,16 +112,15 @@ def get_staticdirector(app_config): def setup_storage(): - app_config = mg_globals.app_config global_config = mg_globals.global_config key_short = 'publicstore' key_long = "storage:" + key_short - public_store = storage_system_from_config(global_config[key_long], None) + public_store = storage_system_from_config(global_config[key_long]) key_short = 'queuestore' key_long = "storage:" + key_short - queue_store = storage_system_from_config(global_config[key_long], None) + queue_store = storage_system_from_config(global_config[key_long]) setup_globals( public_store = public_store, diff --git a/mediagoblin/storage.py b/mediagoblin/storage.py index 46a0c040..66acc1d4 100644 --- a/mediagoblin/storage.py +++ b/mediagoblin/storage.py @@ -443,7 +443,7 @@ def clean_listy_filepath(listy_filepath): return cleaned_filepath -def storage_system_from_config(paste_config, storage_prefix): +def storage_system_from_config(config): """ Utility for setting up a storage system from the paste app config. @@ -472,15 +472,9 @@ def storage_system_from_config(paste_config, storage_prefix): base_url='/media/', base_dir='/var/whatever/media') """ - if storage_prefix is not None: - prefix_re = re.compile('^%s_(.+)$' % re.escape(storage_prefix)) - else: - prefix_re = re.compile('^(.+)$') - - config_params = dict( - [(prefix_re.match(key).groups()[0], value) - for key, value in paste_config.iteritems() - if prefix_re.match(key)]) + # This construct is needed, because dict(config) does + # not replace the variables in the config items. + config_params = dict(config.iteritems()) if 'storage_class' in config_params: storage_class = config_params['storage_class'] |