summaryrefslogtreecommitdiff
path: root/doc/_extensions/odoo_ext/translator.py
diff options
context:
space:
mode:
authorxmo-odoo <xmo@odoo.com>2017-05-10 09:39:55 +0200
committerGitHub <noreply@github.com>2017-05-10 09:39:55 +0200
commitfffaf735f58f1ac57781c095eb552de6d1fcdc47 (patch)
tree88ab0ab65430d871862a836524e0ecc890d1674d /doc/_extensions/odoo_ext/translator.py
parent10eee18a047d35305dc46282e610e88936d81ffc (diff)
[FIX] P3: list -> iterable builtins (#16811)
In Python 3: * various builtins and dict methods were changed to return view/iterable objects rather than lists * and the separate Python 2 view/iterable builtins and methods were removed altogether This is problematic when using these items as list (which the happens repeatedly in Odoo), but more viciously when iterating *multiple times* over them (which also happens, which I've messed up multiple times while writing this, and which is a pain to debug even when you've just created the issue). Convert all code using these to semantics-matching cross-version helper functions to get the LCD behaviour between P2 and P3, and forbid the builtins via lint. issue #8530
Diffstat (limited to 'doc/_extensions/odoo_ext/translator.py')
-rw-r--r--doc/_extensions/odoo_ext/translator.py6
1 files changed, 3 insertions, 3 deletions
diff --git a/doc/_extensions/odoo_ext/translator.py b/doc/_extensions/odoo_ext/translator.py
index 48c5e901f28..1f7fba93772 100644
--- a/doc/_extensions/odoo_ext/translator.py
+++ b/doc/_extensions/odoo_ext/translator.py
@@ -71,9 +71,9 @@ class BootstrapTranslator(nodes.NodeVisitor, object):
tagname = unicode(tagname).lower()
# extract generic attributes
- attrs = {name.lower(): value for name, value in attributes.iteritems()}
+ attrs = {name.lower(): value for name, value in attributes.items()}
attrs.update(
- (name, value) for name, value in node.attributes.iteritems()
+ (name, value) for name, value in node.attributes.items()
if name.startswith('data-')
)
@@ -97,7 +97,7 @@ class BootstrapTranslator(nodes.NodeVisitor, object):
prefix=u''.join(prefix),
tag=tagname,
attrs=u' '.join(u'{}="{}"'.format(name, self.attval(value))
- for name, value in attrs.iteritems()),
+ for name, value in attrs.items()),
postfix=u''.join(postfix),
)
# only "space characters" SPACE, CHARACTER TABULATION, LINE FEED,