jenkins-bot has submitted this change and it was merged.

Change subject: login.py: use site.code instead of site.lang
......................................................................


login.py: use site.code instead of site.lang

The code used for the site is not necessary the same as the language -
this is clear for instance for commons (site.code = 'commons',
site.lang = 'en') and wikidata.

Changing this means logging in works for people using
    mysite = 'wikidata'; family = 'wikidata'

where before this resulted in a cryptic
    en.wikidata is not a valid site, please remove it from your config
error message.

Change-Id: I978437feb4862a7f1fc688fcd9ba4dee6d315c67
---
M scripts/login.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Ladsgroup: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/scripts/login.py b/scripts/login.py
index afdc9a4..53638e6 100755
--- a/scripts/login.py
+++ b/scripts/login.py
@@ -81,7 +81,7 @@
             namedict = config.usernames
     else:
         site = pywikibot.getSite()
-        namedict = {site.family.name: {site.lang: None}}
+        namedict = {site.family.name: {site.code: None}}
     for familyName in namedict:
         for lang in namedict[familyName]:
             try:

-- 
To view, visit https://gerrit.wikimedia.org/r/82883
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I978437feb4862a7f1fc688fcd9ba4dee6d315c67
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Merlijn van Deen <[email protected]>
Gerrit-Reviewer: Ladsgroup <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
Pywikibot-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/pywikibot-commits

Reply via email to