Page MenuHomePhabricator

Update pywikibot to reflect the anexo namespace removal from the Portuguese Wikipedia
Closed, ResolvedPublic

Description

Looks like the Portuguese are killing the list namespace. This needs to be updated in several places in the family file if I recall correctly.


Version: core-(2.0)
Severity: normal

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:59 AM
bzimport set Reference to bz73170.
bzimport added a subscriber: Unknown Object (????).

Is it a normal namespace like 'Talk:' or 'File:'? If so core shouldn't require an update as it fetches the namespace data from the server.

No, it's not a normal namespace. It got some special provisions so you are allowed to connect it with namespace 0. self.crossnamespace in wikipedia_family.py needs to be updated.

If you look in https://git.wikimedia.org/blob/pywikibot%2Fcore.git/227e2be0f124f1310874cf69630aa01b6e7c98fb/pywikibot%2Ffamilies%2Fwikipedia_family.py for 'pt' you'll see 102 (Anexo) and 103 (talk page for Anexo).

Nemo_bis added a subscriber: Nemo_bis.

This looks good for GCI, can I add it as a task?

I dont see any benefit in getting a newbie to do this change. The minimal fix is a 30 mins code change, max, and they don't learn a generic skill. Verifying that their change works correctly is a much harder problem, and I think it is very unlikely that a newbie will be able to find all the places that need to be tested to verify that this is working correctly, and/or actually test them.

We also need to triage this more.

Family.crossnamespace is only used by interwiki.py , and why do we need to support interwiki.py for Wikipedia? Is there interwiki.py functionality _for_Wikipedia_ which isnt replaced by Wikidata?

But thinking out loud ... we should have a unit test which checks that all Family namespaces are actually namespaces on the site. i.e. when site config changes conflict with our Family classes, we shouldnt need a bug report - the unit tests should yell out that something is wrong.

valhallasw added a subscriber: valhallasw.

Change 230369 had a related patch set uploaded (by Ladsgroup):
Remove list and list talk namespace from Portuguese Wikipedia

https://gerrit.wikimedia.org/r/230369

I don't see, why this task has high priority. crossnamespace is used on Wikipedia project and for interwiki.py only (and by wikisource for their authornamespaces). But Wikipedia has wikibase support and don't need it anymore. Therefore crossnamespace might be removed completely imho.

Change 230377 had a related patch set uploaded (by Xqt):
[FIX] Remove crossnamespaces out of the scope

https://gerrit.wikimedia.org/r/230377

Xqt lowered the priority of this task from High to Low.Aug 9 2015, 5:31 AM
Xqt removed a project: Pywikibot-General.

Change 230369 abandoned by Ladsgroup:
Remove list and list talk namespace from Portuguese Wikipedia

Reason:
In favor of I931e0ff6

https://gerrit.wikimedia.org/r/230369

Change 230377 merged by jenkins-bot:
[FIX] Remove crossnamespaces out of the scope

https://gerrit.wikimedia.org/r/230377

jayvdb claimed this task.

'Fixed' in core; probably similar problems exist in compat, but the impact is likely very low for the same reasons this functionality was removed from core -- interwiki bot is not needed/used on Wikipedia any longer.