Replace LegacyIdInterpreter with EntityIdComposer
Closed, DeclinedPublic
Actions

Description

LegacyIdInterpreter allows ItemId and PropertyId to be constructed from numeric IDs. We however need support for custom entities with numeric IDs too, at least as long as we do not have a replacement for the terms table (see T86530). EntityIdComposer provides an extensible mechanism for instantiating IDs for arbitrary entity types from numeric IDs. It should replace LegacyIdInterpreter everywhere.

This may require EntityIdComposer to be moved to WikiBaseServices.

LegacyIdInterpreter should be deleted when this is done.

Related Objects

Mentioned Here: T86530: Replace wb_terms table with more specialized mechanisms for terms (tracking)

Event Timeline

daniel created this task.Nov 24 2016, 6:15 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 24 2016, 6:15 PM

daniel added a project: Wikidata-Former-Sprint-Board.Nov 24 2016, 6:19 PM

daniel updated the task description. (Show Details)

daniel added subscribers: thiemowmde, WMDE-leszek.Nov 24 2016, 6:23 PM

SamanthaNguyen moved this task from Unsorted to Migrate / Replace on the Technical-Debt board.Jan 3 2017, 10:02 PM

Not sure this is still relevant. LegacyIdInterpreter seems to only be used in backwards-compat code now, which is restricted to Items and Properties. It's used in:

EntityIdValue, for the legacy serialization format (which only exists for items and properties)
LegacyEntityIdDeserializer, which only supports items and properties
EntityContentDataCodec, for decoding the old style representation of entity IDs, which was only ever used for items and properties.

So these all seem fine. Perhaps LegacyIdInterpreter could be re-implemented based on EntityIdComposer, or EntityIdComposer could have a static getLegacyIdComposer method, that would return a EntityIdComposer instance for just items and properties.

WMDE-leszek removed a project: Wikidata-Former-Sprint-Board.Mar 7 2017, 4:05 PM

WMDE-leszek triaged this task as Low priority.Mar 15 2017, 9:09 AM

Ladsgroup merged a task: T167963: Deprecate LegacyIdInterpreter and use EntityIdComposer instead.Jun 28 2017, 1:09 AM

Ladsgroup added subscribers: Ladsgroup, Lydia_Pintscher, gerritbot.

WMDE-leszek added a project: Wikidata-Campsite.Jan 5 2021, 12:28 PM

WMDE-leszek moved this task from Incoming to Prioritized Wikidata Tech Backlog (prioritised from top to bottom) on the Wikidata-Campsite board.

Addshore added a project: [DEPRECATED] wdwb-tech (legacy-backlog).Jan 22 2021, 11:06 PM

Addshore moved this task from Incoming to 3. Fill ins (-I -E) on the [DEPRECATED] wdwb-tech (legacy-backlog) board.Jan 22 2021, 11:12 PM

Addshore edited projects, added [DEPRECATED] wdwb-tech; removed [DEPRECATED] wdwb-tech (legacy-backlog).Mar 18 2021, 12:37 PM

Addshore moved this task from Inbox to To Prioritize on the [DEPRECATED] wdwb-tech board.