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