diff options
| author | xmo-odoo <xmo@odoo.com> | 2017-05-10 09:39:55 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-05-10 09:39:55 +0200 |
| commit | fffaf735f58f1ac57781c095eb552de6d1fcdc47 (patch) | |
| tree | 88ab0ab65430d871862a836524e0ecc890d1674d /doc/_extensions/odoo_ext/translator.py | |
| parent | 10eee18a047d35305dc46282e610e88936d81ffc (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.py | 6 |
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, |
