Page MenuHomePhabricator

Adapt http://www.mediawiki.org/xml/sitelist-1.0/ for the new XSD file
Open, Needs TriagePublic

Description

Following the modification to "SiteExporter.php" and "SiteImporter.php" to handle "LanguageCode" property, the XSD file was modified as well.
The new XSD file took version 1.1 and replaces the docs/sitelist-1.0.xsd with docs/sitelist-1.1.xsd: http://www.mediawiki.org/xml/sitelist-1.0/ reference must be updated accordingly.

A new HTML page must be created with similar text as the existing one, but with reference to the 1.1 version XSD file as mentioned

Event Timeline

lucamauri created this task.May 4 2019, 6:52 PM
Reedy added a subscriber: Reedy.May 4 2019, 10:22 PM

I note we don't "replace" anything, but we do add to it. These things are in the wild, so need to exist basically indefinitely

We need to add the new file to the links on https://www.mediawiki.org/xml/, add the xsd to the directory, create a new directory for sitelist-1.1, and add an index.html inside that directory

See https://github.com/wikimedia/operations-mediawiki-config/tree/master/docroot/mediawiki.org/xml and https://github.com/wikimedia/operations-mediawiki-config/tree/master/docroot/mediawiki.org/xml/sitelist-1.0 (noting, it still goes via Gerrit, but we often use GitHub as it's better for browsing)

Change 508130 had a related patch set uploaded (by Luca Mauri; owner: Luca Mauri):
[operations/mediawiki-config@master] Create new http://www.mediawiki.org/xml/sitelist-1.1/ to reference sitelist-1.1.xsd

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

I note we don't "replace" anything, but we do add to it. These things are in the wild, so need to exist basically indefinitely

Thanks for specifying that. In my original message I wrote "replace" with the meaning of "replace as a current standard", but I see this might not completely clear to all readers.

We need to add the new file to the links on https://www.mediawiki.org/xml/, add the xsd to the directory, create a new directory for sitelist-1.1, and add an index.html inside that directory

I just did so, hopefully in a proper way.

lucamauri added a comment.EditedMay 5 2019, 7:53 PM

@Reedy please allow me a question: why you made this task a parent of T221149 instead of a sub?
Doesn't this task exist because of and as a consequence of that one?

I am fairly new here and I assume my original reasoning was wrong, but can you explain why?

Thanks

Reedy added a comment.May 5 2019, 8:58 PM

I note we don't "replace" anything, but we do add to it. These things are in the wild, so need to exist basically indefinitely

Thanks for specifying that. In my original message I wrote "replace" with the meaning of "replace as a current standard", but I see this might not completely clear to all readers.

Yeah, that's fine. Just clarifying :)

@Reedy please allow me a question: why you made this task a parent of T221149 instead of a sub?
Doesn't this task exist because of and as a consequence of that one?

I am fairly new here and I assume my original reasoning was wrong, but can you explain why?

So, the easiest way to think about it... T222516 (this task) requires T221149: Modify "SiteImporter.php" and "SiteExporter.php" to handle "LanguageCode" as well to be complete. The code/updates in the latter are needed to be able to do the first. We can't do T222516 without T221149. But, we can do T221149 without T222516 Therefore this way around is how it is