diff options
| author | xmo-odoo <xmo-odoo@users.noreply.github.com> | 2016-01-29 10:20:08 +0100 |
|---|---|---|
| committer | xmo-odoo <xmo-odoo@users.noreply.github.com> | 2016-01-29 10:20:08 +0100 |
| commit | edeb5a8c0fb5c837364f1d92db731f89824bb28a (patch) | |
| tree | 8982b50677cb3bbbab749c81371c764e76c8ea41 /openerp/modules/module.py | |
| parent | ca7983a40fdadf4f28760a879ffcee342850d112 (diff) | |
[FIX] module finder path parameter should be optional
According to PEP302, the signature of `Finder.find_module` should be
`find_module(fullname, path=None)`.
Ever since it was introduced in 64ec5f36df the addons import hook
defines the second parameter as mandatory, which is an issue for
systems relying on the specified behaviour (and not needing to
provide a path) like the stdlib's `pkgutil.find_loader`.
fixes #10670
Diffstat (limited to 'openerp/modules/module.py')
| -rw-r--r-- | openerp/modules/module.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/openerp/modules/module.py b/openerp/modules/module.py index b9f19293fee..8a72fd6f954 100644 --- a/openerp/modules/module.py +++ b/openerp/modules/module.py @@ -65,7 +65,7 @@ class AddonsImportHook(object): thus `import openerp.addons.module`. """ - def find_module(self, module_name, package_path): + def find_module(self, module_name, package_path=None): module_parts = module_name.split('.') if len(module_parts) == 3 and module_name.startswith('openerp.addons.'): return self # We act as a loader too. |