[Bug 55371] API action=queryprop=revisions doesn't return diff in xml
https://bugzilla.wikimedia.org/show_bug.cgi?id=55371 Bill Traynor btray...@gmail.com changed: What|Removed |Added CC||btray...@gmail.com --- Comment #9 from Bill Traynor btray...@gmail.com --- As indicated on https://www.mediawiki.org/wiki/API:Main_page#The_format there are plans to remove all output formats except for JSON, should this Bug be moved to a WON'T FIX status? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 55371] API action=queryprop=revisions doesn't return diff in xml
https://bugzilla.wikimedia.org/show_bug.cgi?id=55371 --- Comment #10 from Brad Jorsch bjor...@wikimedia.org --- No. Chances are I'm going to do something along the lines of option 2 as part of https://www.mediawiki.org/wiki/API/Architecture_work/Planning#Changes_to_XML_output_format -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 55371] API action=queryprop=revisions doesn't return diff in xml
https://bugzilla.wikimedia.org/show_bug.cgi?id=55371 --- Comment #11 from Bill Traynor btray...@gmail.com --- Is the statement about moving to JSON only output formats incorrect, or just far in the future? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 55371] API action=queryprop=revisions doesn't return diff in xml
https://bugzilla.wikimedia.org/show_bug.cgi?id=55371 --- Comment #12 from Brad Jorsch bjor...@wikimedia.org --- Far in the future, if we ever get to it. But killing formats other than json, xml, and php is in progress now. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 55371] API action=queryprop=revisions doesn't return diff in xml
https://bugzilla.wikimedia.org/show_bug.cgi?id=55371 --- Comment #8 from Brad Jorsch bjor...@wikimedia.org --- *** Bug 66054 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 55371] API action=queryprop=revisions doesn't return diff in xml
https://bugzilla.wikimedia.org/show_bug.cgi?id=55371 --- Comment #7 from Peter Bena benap...@gmail.com --- But you could still make some other way how to ask for a result which would look like this, so that it wouldn't break any tools that expect the current format. In fact, even if it did revtext/rev in case you don't ask for a diff and rev content=text diff.../diff /rev in case you do ask for a diff it would still hardly break anything, since as you said, the later should have thrown exception (which it doesn't because of some bug as you said: There's supposed to be an error thrown in this situation, but the test at ApiFormatXml.php line 166 is broken so it won't actually get thrown except in certain limited cases.) so if it produced a different kind of output instead of exception, how could it break anything? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 55371] API action=queryprop=revisions doesn't return diff in xml
https://bugzilla.wikimedia.org/show_bug.cgi?id=55371 Brad Jorsch bjor...@wikimedia.org changed: What|Removed |Added CC||yu...@wikimedia.org --- Comment #2 from Brad Jorsch bjor...@wikimedia.org --- Yet another bug with the XML format. Basically, it can't handle having both a text content (i.e. an item with key '*', from ApiResult::setContent()) and non-scalar sub-items at the same time. There's supposed to be an error thrown in this situation, but the test at ApiFormatXml.php line 166 is broken so it won't actually get thrown except in certain limited cases. I can see three possible solutions here: 1. Use a key such as 'content' instead of '*'. But that would break BC for all clients, even those not trying to use diff and content at the same time, so I don't much like that idea. 2. Have the XML formatter shove '*' into a particular named subelement when other subelements are needed. Whether this subelement's name should be hardcoded or if we should allow/require something like setIndexedTagName() I'm not sure. 3. WONTFIX it. I don't like #1 because it would break BC for all clients, even those not trying to use diff and content at the same time. #2 feels like a hack, but it's the better option if we don't want to WONTFIX this. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 55371] API action=queryprop=revisions doesn't return diff in xml
https://bugzilla.wikimedia.org/show_bug.cgi?id=55371 --- Comment #3 from Brad Jorsch bjor...@wikimedia.org --- FYI: I don't see any other situation in core that would be affected by this. As for WMF-deployed extensions, CodeReview's list=coderevisions has the issue if 'message' is requested along with 'tags', 'followups', or 'followedup'. I don't see any other extensions that look like they'd be affected. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 55371] API action=queryprop=revisions doesn't return diff in xml
https://bugzilla.wikimedia.org/show_bug.cgi?id=55371 --- Comment #4 from Peter Bena benap...@gmail.com --- I don't really see what is so hard on fixing this, you can easily combine rev ...content/rev and diffcontent of diff/diff what's the problem? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 55371] API action=queryprop=revisions doesn't return diff in xml
https://bugzilla.wikimedia.org/show_bug.cgi?id=55371 --- Comment #5 from Peter Bena benap...@gmail.com --- what's the point of whole star thingie? Why not just name every property with some solid name instead of some weird characters? It wouldn't look so hackish but it would surely work just as fine... -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 55371] API action=queryprop=revisions doesn't return diff in xml
https://bugzilla.wikimedia.org/show_bug.cgi?id=55371 --- Comment #6 from Brad Jorsch bjor...@wikimedia.org --- (In reply to comment #4) I don't really see what is so hard on fixing this, you can easily combine rev ...content/rev and diffcontent of diff/diff what's the problem? How do you combine revtext.../rev and revdiff.../diff/rev? revtext...diff.../diff/rev or revdiff.../difftext.../rev makes no sense in XML. My option 1 is changing it to rev content=text.../, so you could combine them easily as rev content=text...diff.../diff/rev. My option 2 is doing basically the same (or revsomethingtext.../somethingdiff.../diff/rev) only when the XML formatter runs into this situation. (In reply to comment #5) what's the point of whole star thingie? Why not just name every property with some solid name instead of some weird characters? It wouldn't look so hackish but it would surely work just as fine... The star thingie is how you get revtext.../rev instead of rev content=text.../ in the XML format. And while it would probably have been a good idea to avoid it back in 2006, changing it now would break huge numbers of existing clients which is not something we'd like to do. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 55371] API action=queryprop=revisions doesn't return diff in xml
https://bugzilla.wikimedia.org/show_bug.cgi?id=55371 Peter Bena benap...@gmail.com changed: What|Removed |Added Priority|Unprioritized |High --- Comment #1 from Peter Bena benap...@gmail.com --- this was tested on english wiki -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l