From 514bdcb0f1c084e87f6f809c4e893468836cce0c Mon Sep 17 00:00:00 2001 From: Nicolas Martinelli Date: Thu, 4 May 2017 16:48:31 +0200 Subject: [PATCH] [FIX] tools: unnecessary error logging The limit of 64 characters on the XMLID has been removed a long time ago. This error is therefore not necessary anymore. --- odoo/tools/convert.py | 3 --- odoo/tools/yaml_import.py | 2 -- 2 files changed, 5 deletions(-) diff --git a/odoo/tools/convert.py b/odoo/tools/convert.py index 0d0b14ab341..284b1f783ce 100644 --- a/odoo/tools/convert.py +++ b/odoo/tools/convert.py @@ -250,9 +250,6 @@ form: module.record_id""" % (xml_id,) modcnt = self.env['ir.module.module'].search_count([('name', '=', module), ('state', '=', 'installed')]) assert modcnt == 1, """The ID "%s" refers to an uninstalled module""" % (xml_id,) - if len(id) > 64: - _logger.error('id: %s is to long (max: 64)', id) - def _tag_delete(self, rec, data_node=None, mode=None): d_model = rec.get("model") d_search = rec.get("search",'').encode('utf-8') diff --git a/odoo/tools/yaml_import.py b/odoo/tools/yaml_import.py index 93394a48f67..ced0738647a 100644 --- a/odoo/tools/yaml_import.py +++ b/odoo/tools/yaml_import.py @@ -139,8 +139,6 @@ class YamlInterpreter(object): if module != self.module: module_count = self.env['ir.module.module'].search_count([('name', '=', module), ('state', '=', 'installed')]) assert module_count == 1, 'The ID "%s" refers to an uninstalled module.' % (xml_id,) - if len(id) > 64: # TODO where does 64 come from (DB is 128)? should be a constant or loaded form DB - _logger.error('id: %s is to long (max: 64)', id) def get_id(self, xml_id): if xml_id is False or xml_id is None: