diff options
| author | Xavier Morel <xmo@odoo.com> | 2020-03-11 11:14:48 +0000 |
|---|---|---|
| committer | Xavier Morel <xmo@odoo.com> | 2020-03-11 11:14:48 +0000 |
| commit | b6af34d30ea9a693b8d4cd3e473371f0fca5f013 (patch) | |
| tree | 311cf7e8e00075eee2afc6875f606588cf367599 | |
| parent | 06dee038ddd0e2344c45d4f20e79c28fc78407de (diff) | |
[IMP] core: only check for the unaccent function to enable the feature
Before this the `--unaccent` flag does double duty to specify whether
to create new databases with the extension *and* to try to use the
corresponding function.
The latter is further gated on the function existing at all in the
database.
Given postgresql does not have unaccent installed by default it seems
only the latter is really useful and we can ignore the flag to decide
whether to enable unaccent features, if the extension is installed
consider that it should be enabled and move on.
Merge this in master rather than previous versions as it can change
things like index access / use.
closes odoo/odoo#47377
Signed-off-by: Xavier Morel (xmo) <xmo@odoo.com>
| -rw-r--r-- | odoo/modules/registry.py | 5 | ||||
| -rw-r--r-- | odoo/tools/config.py | 2 |
2 files changed, 2 insertions, 5 deletions
diff --git a/odoo/modules/registry.py b/odoo/modules/registry.py index 9b39758a30a..b42ebc988d5 100644 --- a/odoo/modules/registry.py +++ b/odoo/modules/registry.py @@ -142,10 +142,7 @@ class Registry(Mapping): self.cache_invalidated = False with closing(self.cursor()) as cr: - has_unaccent = odoo.modules.db.has_unaccent(cr) - if odoo.tools.config['unaccent'] and not has_unaccent: - _logger.warning("The option --unaccent was given but no unaccent() function was found in database.") - self.has_unaccent = odoo.tools.config['unaccent'] and has_unaccent + self.has_unaccent = odoo.modules.db.has_unaccent(cr) @classmethod def delete(cls, db_name): diff --git a/odoo/tools/config.py b/odoo/tools/config.py index 0430b7d3dc9..0ac8c40e31b 100644 --- a/odoo/tools/config.py +++ b/odoo/tools/config.py @@ -293,7 +293,7 @@ class configmanager(object): help="Maximum number of threads processing concurrently cron jobs (default 2).", type="int") group.add_option("--unaccent", dest="unaccent", my_default=False, action="store_true", - help="Use the unaccent function provided by the database when available.") + help="Try to enable the unaccent extension when creating new databases.") group.add_option("--geoip-db", dest="geoip_database", my_default='/usr/share/GeoIP/GeoLite2-City.mmdb', help="Absolute path to the GeoIP database file.") parser.add_option_group(group) |