| Arian_Bozorg | |
| Sat, Apr 6, 4:04 PM |
| F46955787: 2024-04-16-110643_697x586_scrot.png | |
| Tue, Apr 16, 10:11 AM |
In a quick test using the Action API wbformatvalue doesn’t seem to be working for EntitySchema data values and returns the message: “An illegal set of parameters have been used”
We need to investigate if wbformatvalue works for EntitySchema data values or if the test wasn't properly executed.
Acceptance Crieteria
OR
Notes
This could be as a result of the same issue as T362003
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T91505 [Epic] Adding new datatypes to Wikidata (tracking) | |||
| Open | None | T214884 [ES-M2]: [EPIC] Linking EntitySchemas in statements | |||
| Resolved | • karapayneWMDE | T332139 [ES-M2]: Create a new EntitySchema data type | |||
| Open | None | T332157 [ES-M2]: Enable new EntitySchema data type on Wikidata | |||
| Open | None | T339920 [ES-M2]: Return EntitySchema as `wikibase-entityid` through the Action API | |||
| Open | None | T362000 [ES-M2]: [EPIC] Continue development of PoC EntitySchema patch | |||
| Open | None | T362008 [ES-M2]: Investigate if `wbformatvalue` works for EntitySchema data values |
I would expect the datavalue {"value":{"id":"E1"},"type":"wikibase-entityid"} to work, when property is set to the property ID of a property with data type EntitySchema. (For comparison, for item properties, {"value":{"id":"Q1"},"type":"wikibase-entityid"} works.)
Change #1019275 had a related patch set uploaded (by Arthur taylor; author: Arthur taylor):
[mediawiki/extensions/EntitySchema@master] [WIP] Sketch entity-schema entity-id registration
This was / is not working - @Lucas_Werkmeister_WMDE expectations are correct. I've made a sketch of a patch that makes wbformatvalue work with the example that Lucas provides. The JSON returned in this case is:
{ "result": "E1" }
but I need to tidy up the registration and add some tests.
Change #1019755 had a related patch set uploaded (by Arthur taylor; author: Arthur taylor):
[mediawiki/extensions/Wikibase@master] [WIP] Extend the DataType registration to include entity builder
Change #1019758 had a related patch set uploaded (by Arthur taylor; author: Arthur taylor):
[mediawiki/extensions/EntitySchema@master] [WIP] Extend the DataType registration to include entity builder
Change #1019817 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/Wikibase@master] WIP: Use snak deserializer in FormatSnakValue
Change #1019818 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/EntitySchema@master] WIP: Make wbformatvalue work
Change #1019758 abandoned by Arthur taylor:
[mediawiki/extensions/EntitySchema@master] [WIP] Extend the DataType registration to include entity builder
Reason:
Based on obsolete patches
Change #1019755 abandoned by Arthur taylor:
[mediawiki/extensions/Wikibase@master] [WIP] Extend the DataType registration to include entity builder
Reason:
Based on abandoned patches
Change #1019275 abandoned by Arthur taylor:
[mediawiki/extensions/EntitySchema@master] [WIP] Sketch entity-schema entity-id registration
Reason:
Wrong approach here
Got some updated patches from @Jakob_WMDE this morning. Using those, plus the latest changes from @Lucas_Werkmeister_WMDE - 1019818 and 1019817 - we are able to make some wbformatvalue requests work:
Note that the specified property must be of EntitySchema Data Type.
With the current set of patches (also with Lucas' changes), setting datatype=entity-schema does not work and generates the error An illegal set of parameters have been used.
Thanks for spotting this issue! We had wbformatvalue on our radar as well. Getting it to work as long as datatype or property are provided should be straightforward. I'll make a patch!
I guess we'll have to figure out if requiring datatype or property for these kinds of values is an acceptable compromise or if we need to look into making it work without those.
Change #1020193 had a related patch set uploaded (by Jakob; author: Jakob):
[mediawiki/extensions/Wikibase@master] Make FormatValue work with data specific value parsers
Change #1020197 had a related patch set uploaded (by Arthur taylor; author: Arthur taylor):
[mediawiki/extensions/EntitySchema@master] [WIP] Add API test for wbformatvalue
great - thanks @Jakob_WMDE . I added a test in 1020197 for wbformatvalue from the EntitySchema extension, and it seems to pass for both cases.
Change #1019817 abandoned by Lucas Werkmeister (WMDE):
[mediawiki/extensions/Wikibase@master] WIP: Use snak deserializer in FormatSnakValue
Reason:
superseded by I7bcac030d7
Change #1019818 abandoned by Lucas Werkmeister (WMDE):
[mediawiki/extensions/EntitySchema@master] WIP: Make wbformatvalue work
Reason:
already done in PS8 of Ie3d4350ed0
Change #1020193 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Make FormatValue work with data type specific value parsers