heckyel/mediagoblin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MountStorage: Better error reporting

Browse Source 
Created a new resolver frontend function, that raises and
error, when no backend is found. Useful for all the
wrappers.
This commit is contained in:
Elrond 2011-08-03 23:13:58 +02:00
parent 926aec1473
commit eea315623f
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
mediagoblin/storage.py
View File

@ -227,7 +227,7 @@ class MountStorage(StorageInterface):
new_ent = clean_listy_filepath(dirpath) new_ent = clean_listy_filepath(dirpath)
print "Mounting:", repr(new_ent) print "Mounting:", repr(new_ent)
already, rem_1, table, rem_2 = self.resolve_to_backend(new_ent, True) already, rem_1, table, rem_2 = self._resolve_to_backend(new_ent, True)
print "===", repr(already), repr(rem_1), repr(rem_2) print "===", repr(already), repr(rem_1), repr(rem_2)
assert (already is None) or (len(rem_2) > 0), "Already mounted" assert (already is None) or (len(rem_2) > 0), "Already mounted"
@ -237,7 +237,7 @@ class MountStorage(StorageInterface):
assert not table.has_key(None), "Huh? Already mounted?!" assert not table.has_key(None), "Huh? Already mounted?!"
table[None] = backend table[None] = backend
def resolve_to_backend(self, filepath, extra_info = False): def _resolve_to_backend(self, filepath, extra_info = False):
""" """
extra_info = True is for internal use! extra_info = True is for internal use!
@ -270,6 +270,12 @@ class MountStorage(StorageInterface):
else: else:
return (res_be, res_fp) return (res_be, res_fp)
def resolve_to_backend(self, filepath):
backend, filepath = self._resolve_to_backend(filepath)
if backend is None:
raise Error("Path not mounted")
return backend, filepath
def __repr__(self, table = None, indent = []): def __repr__(self, table = None, indent = []):
res = [] res = []
if table is None: if table is None: