Legoktm has submitted this change and it was merged.
Change subject: Few more things for uppercase ids ......................................................................
Few more things for uppercase ids
Change-Id: I9b28ee8c597836d37d6e71eb18474466b4e04033 --- M pywikibot/page.py M tests/wikibase_tests.py 2 files changed, 4 insertions(+), 4 deletions(-)
Approvals: Legoktm: Verified; Looks good to me, approved
diff --git a/pywikibot/page.py b/pywikibot/page.py index 3b250c0..b02879a 100644 --- a/pywikibot/page.py +++ b/pywikibot/page.py @@ -2673,7 +2673,7 @@ """ WikibasePage.__init__(self, source, title, ns=120) self.id = self.title(withNamespace=False).upper() - if not self.id.startswith(u'p'): + if not self.id.startswith(u'P'): raise ValueError(u"'%s' is not a property page!" % self.title())
def get(self, force=False, *args): @@ -2698,7 +2698,7 @@ def __init__(self, site, title): WikibasePage.__init__(self, site, title, ns=122) self.id = self.title(withNamespace=False).upper() - if not self.id.startswith(u'u'): + if not self.id.startswith(u'U'): raise ValueError(u"'%s' is not a query page!" % self.title())
diff --git a/tests/wikibase_tests.py b/tests/wikibase_tests.py index fa3facd..3896392 100644 --- a/tests/wikibase_tests.py +++ b/tests/wikibase_tests.py @@ -36,7 +36,7 @@ self.assertTrue('HomePage' in item.aliases['en']) self.assertEqual(item.namespace(), 0) item2 = pywikibot.ItemPage(repo, 'q5296') - self.assertEqual(item2.getID(), 'q5296') + self.assertEqual(item2.getID(), 'Q5296') self.assertEqual(item.labels['en'], 'Main Page') prop = pywikibot.PropertyPage(repo, 'Property:P21') self.assertEqual(prop.getType(), 'wikibase-item') @@ -60,7 +60,7 @@ def setUp(self): super(TestLinks, self).setUp() self.wdp = pywikibot.ItemPage(wikidata, 'Q60') - self.wdp.id = 'q60' + self.wdp.id = 'Q60' self.wdp._content = json.load(open(os.path.join(os.path.split(__file__)[0], 'pages', 'Q60_only_sitelinks.wd'))) self.wdp.get()
pywikibot-commits@lists.wikimedia.org