jenkins-bot has submitted this change and it was merged.
Change subject: Revert "use collections.OrderedDict instead of built-in dict
for params"
......................................................................
Revert "use collections.OrderedDict instead of built-in dict for params"
orderedDict is only availlable with py2.7+
This reverts commit 1ef546632aefb944b4afce980f85553ff33b7306.
Change-Id: Ib96935a4fb52073bc545df86e9e25ac70ed79e8c
---
M pywikibot/textlib.py
M tests/textlib_tests.py
2 files changed, 10 insertions(+), 14 deletions(-)
Approvals:
Xqt: Looks good to me, approved
jenkins-bot: Verified
diff --git a/pywikibot/textlib.py b/pywikibot/textlib.py
index 80942ec..4535b4d 100644
--- a/pywikibot/textlib.py
+++ b/pywikibot/textlib.py
@@ -20,7 +20,6 @@
mwparserfromhell = False
import pywikibot
import datetime
-from collections import OrderedDict
import re
import sys
if sys.version_info[0] == 2:
@@ -920,7 +919,7 @@
code = mwparserfromhell.parse(text)
result = []
for template in code.filter_templates(recursive=True):
- params = OrderedDict()
+ params = {}
for param in template.params:
params[unicode(param.name)] = unicode(param.value)
result.append((unicode(template.name.strip()), params))
@@ -1042,7 +1041,7 @@
# Parameters
paramString = m.group('params')
- params = OrderedDict()
+ params = {}
numbered_param = 1
if paramString:
# Replace wikilinks with markers
diff --git a/tests/textlib_tests.py b/tests/textlib_tests.py
index bbe7f5b..06770d2 100644
--- a/tests/textlib_tests.py
+++ b/tests/textlib_tests.py
@@ -12,7 +12,6 @@
mwparserfromhell = False
import codecs
import os
-from collections import OrderedDict
import pywikibot
import pywikibot.textlib as textlib
@@ -44,19 +43,17 @@
if not (pywikibot.config.use_mwparserfromhell and mwparserfromhell):
return # We'll test the regex function in the test below
func = textlib.extract_templates_and_params # It's really long.
- self.assertEqual(func('{{a}}'), [('a', OrderedDict())])
- self.assertEqual(func('{{a|b=c}}'), [('a', OrderedDict((('b', 'c'),
)))])
- self.assertEqual(func('{{a|b|c=d}}'), [('a', OrderedDict((('1', 'b'),
('c', 'd'))))])
- self.assertEqual(func('{{a|b={{c}}}}'), [('c', {}), ('a',
OrderedDict((('b', '{{c}}'), )))])
- self.assertEqual(func('{{a|b=c|f=g|d=e|1=}}'), [('a',
OrderedDict((('b', 'c'), ('f', 'g'), ('d', 'e'), ('1', ''))))])
+ self.assertEqual(func('{{a}}'), [('a', {})])
+ self.assertEqual(func('{{a|b=c}}'), [('a', {'b': 'c'})])
+ self.assertEqual(func('{{a|b|c=d}}'), [('a', {u'1': 'b', 'c': 'd'})])
+ self.assertEqual(func('{{a|b={{c}}}}'), [(u'a', {u'b': u'{{c}}'}),
('c', {})])
def testExtractTemplatesRegex(self):
func = textlib.extract_templates_and_params_regex # It's really long.
- self.assertEqual(func('{{a}}'), [('a', OrderedDict())])
- self.assertEqual(func('{{a|b=c}}'), [('a', OrderedDict((('b', 'c'),
)))])
- self.assertEqual(func('{{a|b|c=d}}'), [('a', OrderedDict((('1', 'b'),
('c', 'd'))))])
- self.assertEqual(func('{{a|b={{c}}}}'), [('c', {}), ('a',
OrderedDict((('b', '{{c}}'), )))])
- self.assertEqual(func('{{a|b=c|f=g|d=e|1=}}'), [('a',
OrderedDict((('b', 'c'), ('f', 'g'), ('d', 'e'), ('1', ''))))])
+ self.assertEqual(func('{{a}}'), [('a', {})])
+ self.assertEqual(func('{{a|b=c}}'), [('a', {'b': 'c'})])
+ self.assertEqual(func('{{a|b|c=d}}'), [('a', {u'1': 'b', 'c': 'd'})])
+ self.assertEqual(func('{{a|b={{c}}}}'), [('c', {}), (u'a', {u'b':
u'{{c}}'})])
def testSpacesInSection(self):
self.assertContains("enwiki_help_editing", u"Minor_edits")
--
To view, visit https://gerrit.wikimedia.org/r/127467
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib96935a4fb52073bc545df86e9e25ac70ed79e8c
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Xqt <[email protected]>
Gerrit-Reviewer: Ladsgroup <[email protected]>
Gerrit-Reviewer: Merlijn van Deen <[email protected]>
Gerrit-Reviewer: Xqt <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
Pywikibot-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/pywikibot-commits
