aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorElrond <elrond+mediagoblin.org@samba-tng.org>2011-08-09 13:11:34 +0200
committerElrond <elrond+mediagoblin.org@samba-tng.org>2011-08-09 13:18:22 +0200
commit56fc71865919792159ce4307101ef246488b4194 (patch)
tree93c2c4cb28c4e1f270000c2b2c1e08882d9a58ec
parentf7d73aeb34ed16c7d1aa06d977ffb4c9c7a44051 (diff)
downloadmediagoblin-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__.py5
-rw-r--r--mediagoblin/storage.py14
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']