adrianheine added a comment.

- The example allows all languages known to ULS plus `zxx` and `und`, but minus 
`en`, `fr` and `de. It's obviously artificial :)
- I don't see why you would need to reuse parts in the serialization. Can you 
give me a use-case for that?
- Merging and filtering are operations with two operands. The tree structure 
makes this structure very explicit. Your proposal seems to be a postfix 
notation, which makes it difficult to follow which operand belongs to which 
operation. Also you're breaking polymorphism by only allowing plain lists as 
second operand. A true postfix notation would in my opinion look like this:

  mw.config.set( 'wbMonolingualTextValueLanguages', [
    { type: 'Uls' },
    { type: 'List', data: [ 'zxx', 'und' ] },
    { type: 'Merging' },
    { type: 'List', data: [ 'en', 'fr', 'de' ] },
    { type: 'Filtering' }
  ] );


TASK DETAIL
  https://phabricator.wikimedia.org/T78007

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: adrianheine
Cc: Ricordisamoa, Tobi_WMDE_SW, JeroenDeDauw, Wikidata-bugs, jeblad, 
thiemowmde, JanZerebecki, adrianheine, Aklapper, Lydia_Pintscher, daniel, aude



_______________________________________________
Wikidata-bugs mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs

Reply via email to