Rename BaseModule to Module and Module to LoadedModule
This commit is contained in:
parent
5c71be36f5
commit
5a1d83b567
166 changed files with 359 additions and 359 deletions
|
|
@ -50,7 +50,7 @@ class BackendsCall(object):
|
|||
def __init__(self, backends, function, *args, **kwargs):
|
||||
"""
|
||||
:param backends: List of backends to call
|
||||
:type backends: list[:class:`BaseModule`]
|
||||
:type backends: list[:class:`Module`]
|
||||
:param function: backends' method name, or callable object.
|
||||
:type function: :class:`str` or :class:`callable`
|
||||
"""
|
||||
|
|
|
|||
|
|
@ -21,11 +21,11 @@ import os
|
|||
import imp
|
||||
import logging
|
||||
|
||||
from weboob.tools.backend import BaseModule
|
||||
from weboob.tools.backend import Module
|
||||
from weboob.tools.log import getLogger
|
||||
|
||||
|
||||
__all__ = ['Module', 'ModulesLoader', 'RepositoryModulesLoader', 'ModuleLoadError']
|
||||
__all__ = ['LoadedModule', 'ModulesLoader', 'RepositoryModulesLoader', 'ModuleLoadError']
|
||||
|
||||
|
||||
class ModuleLoadError(Exception):
|
||||
|
|
@ -34,17 +34,17 @@ class ModuleLoadError(Exception):
|
|||
self.module = module_name
|
||||
|
||||
|
||||
class Module(object):
|
||||
class LoadedModule(object):
|
||||
def __init__(self, package):
|
||||
self.logger = getLogger('backend')
|
||||
self.package = package
|
||||
self.klass = None
|
||||
for attrname in dir(self.package):
|
||||
attr = getattr(self.package, attrname)
|
||||
if isinstance(attr, type) and issubclass(attr, BaseModule) and attr != BaseModule:
|
||||
if isinstance(attr, type) and issubclass(attr, Module) and attr != Module:
|
||||
self.klass = attr
|
||||
if not self.klass:
|
||||
raise ImportError('%s is not a backend (no BaseModule class found)' % package)
|
||||
raise ImportError('%s is not a backend (no Module class found)' % package)
|
||||
|
||||
@property
|
||||
def name(self):
|
||||
|
|
@ -143,7 +143,7 @@ class ModulesLoader(object):
|
|||
try:
|
||||
fp, pathname, description = imp.find_module(module_name, [path])
|
||||
try:
|
||||
module = Module(imp.load_module(module_name, fp, pathname, description))
|
||||
module = LoadedModule(imp.load_module(module_name, fp, pathname, description))
|
||||
finally:
|
||||
if fp:
|
||||
fp.close()
|
||||
|
|
|
|||
|
|
@ -25,7 +25,7 @@ from weboob.core.modules import ModulesLoader, RepositoryModulesLoader, ModuleLo
|
|||
from weboob.core.backendscfg import BackendsConfig
|
||||
from weboob.core.repositories import Repositories, IProgress
|
||||
from weboob.core.scheduler import Scheduler
|
||||
from weboob.tools.backend import BaseModule
|
||||
from weboob.tools.backend import Module
|
||||
from weboob.tools.config.iconfig import ConfigError
|
||||
from weboob.tools.log import getLogger
|
||||
|
||||
|
|
@ -97,7 +97,7 @@ class WebNip(object):
|
|||
:type storage: :class:`weboob.tools.storage.IStorage`
|
||||
:param name: name of backend
|
||||
:type name: :class:`basestring`
|
||||
:rtype: :class:`weboob.tools.backend.BaseModule`
|
||||
:rtype: :class:`weboob.tools.backend.Module`
|
||||
"""
|
||||
module = self.modules_loader.get_or_load_module(module_name)
|
||||
|
||||
|
|
@ -127,7 +127,7 @@ class WebNip(object):
|
|||
:type params: :class:`dict`
|
||||
:param storage: storage to use
|
||||
:type storage: :class:`weboob.tools.storage.IStorage`
|
||||
:rtype: :class:`weboob.tools.backend.BaseModule`
|
||||
:rtype: :class:`weboob.tools.backend.Module`
|
||||
"""
|
||||
if name is None:
|
||||
name = module_name
|
||||
|
|
@ -192,7 +192,7 @@ class WebNip(object):
|
|||
|
||||
:param caps: optional list of capabilities to select backends
|
||||
:type caps: tuple[:class:`weboob.capabilities.base.CapBase`]
|
||||
:rtype: iter[:class:`weboob.tools.backend.BaseModule`]
|
||||
:rtype: iter[:class:`weboob.tools.backend.Module`]
|
||||
"""
|
||||
for _, backend in sorted(self.backend_instances.iteritems()):
|
||||
if caps is None or backend.has_caps(caps):
|
||||
|
|
@ -228,7 +228,7 @@ class WebNip(object):
|
|||
backends = self.backend_instances.values()
|
||||
_backends = kwargs.pop('backends', None)
|
||||
if _backends is not None:
|
||||
if isinstance(_backends, BaseModule):
|
||||
if isinstance(_backends, Module):
|
||||
backends = [_backends]
|
||||
elif isinstance(_backends, basestring):
|
||||
if len(_backends) > 0:
|
||||
|
|
@ -376,7 +376,7 @@ class Weboob(WebNip):
|
|||
:type storage: :class:`weboob.tools.storage.IStorage`
|
||||
:param name: name of backend
|
||||
:type name: :class:`basestring`
|
||||
:rtype: :class:`weboob.tools.backend.BaseModule`
|
||||
:rtype: :class:`weboob.tools.backend.Module`
|
||||
"""
|
||||
minfo = self.repositories.get_module_info(module_name)
|
||||
if minfo is None:
|
||||
|
|
@ -404,7 +404,7 @@ class Weboob(WebNip):
|
|||
:param errors: if specified, store every errors in this list
|
||||
:type errors: list[:class:`LoadError`]
|
||||
:returns: loaded backends
|
||||
:rtype: dict[:class:`str`, :class:`weboob.tools.backend.BaseModule`]
|
||||
:rtype: dict[:class:`str`, :class:`weboob.tools.backend.Module`]
|
||||
"""
|
||||
loaded = {}
|
||||
if storage is None:
|
||||
|
|
@ -446,7 +446,7 @@ class Weboob(WebNip):
|
|||
|
||||
try:
|
||||
backend_instance = module.create_instance(self, instance_name, params, storage)
|
||||
except BaseModule.ConfigError as e:
|
||||
except Module.ConfigError as e:
|
||||
if errors is not None:
|
||||
errors.append(self.LoadError(instance_name, e))
|
||||
else:
|
||||
|
|
|
|||
|
|
@ -32,7 +32,7 @@ from compileall import compile_dir
|
|||
from io import BytesIO
|
||||
|
||||
from weboob.tools.exceptions import BrowserHTTPError, BrowserHTTPNotFound
|
||||
from .modules import Module
|
||||
from .modules import LoadedModule
|
||||
from weboob.tools.log import getLogger
|
||||
from weboob.tools.misc import to_unicode
|
||||
try:
|
||||
|
|
@ -281,7 +281,7 @@ class Repository(object):
|
|||
try:
|
||||
fp, pathname, description = imp.find_module(name, [path])
|
||||
try:
|
||||
module = Module(imp.load_module(name, fp, pathname, description))
|
||||
module = LoadedModule(imp.load_module(name, fp, pathname, description))
|
||||
finally:
|
||||
if fp:
|
||||
fp.close()
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue