On Sun, May 20, 2012 at 2:17 PM, Platonides Platonides@gmail.com wrote:
On 20/05/12 23:10, Brion Vibber wrote:
On Thu, May 17, 2012 at 8:36 AM, Platonides platonides@gmail.com
wrote:
I don't think it should contain all campaign parameters as a single tag, as parameters. Specially evil is the licensesOwnWork="cc-by-sa-3.0|cc-by-3.0|cc-zero".
The data's stored as a bunch of simple key-value pairs, including that pipe-separated list.
That's horrible! If it went to store it that way, at least it should have provided an accessor which translated it into an array.
That it does; with some tweaking I can make it output nicely as an array for both JSON and XML too. But beyond that, there's not much structure to the UploadWizardCampaign class's configuration array.
-- brion
When the extension is the only consumer, we have some flexibility for later fixing the interface. If there third-party apps are reading from such API, it gets set in stone and we can no longer fix it. I'd at least provide a sane interface through the api. It's not so urgent to fix the extension, but it should eventually be done, too.
Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l