Page MenuHomePhabricator

Label not shown for lexical category and/or language edit diff of a Lexeme
Closed, ResolvedPublic5 Estimated Story Points

Description

When viewing a diff of a change of the lexical category and/or language of a Lexeme the label for the item is not shown and instead only the Q-ID.
The label should be shown in the user's ULS language (or a fallback when not available - same "Fallback Chain" as everywhere else for item labels).

Example diff:

lexcat-diff1.png (211×1 px, 24 KB)

Example diff with item labels shown:

lexcat-diff2.png (234×1 px, 23 KB)

The same diff in qqx language with fallback:
lexcat-diff3.png (95×1 px, 9 KB)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Lydia_Pintscher renamed this task from label not shown for lexical category edit of a Lexeme to label not shown for lexical category edit diff of a Lexeme.Mar 8 2018, 9:34 AM
WMDE-leszek renamed this task from label not shown for lexical category edit diff of a Lexeme to Label not shown for lexical category edit diff of a Lexeme.Mar 12 2018, 12:27 PM
WMDE-leszek updated the task description. (Show Details)

Some technical signposts in WikibaseLexeme:

  • LexemeDiffVisualizer::visualizeEntityDiff
  • LexemeDiff::getLexicalCategoryDiff
  • LexemeDiff::lexicalCategory ?

And some in Wikibase:

  • WikibaseRepo::getEntityDiffVisualizerFactory
  • which makes a DifferencesSnakVisualizer passing in a $entityIdFormatter from WikibaseRepo::getEntityIdHtmlLinkFormatterFactory
  • this is most likely the formater that formats the item ID in html for output in a diff with the language fallback.
  • You can seethis during the construction when a LanguageFallbackLabelDescriptionLookupFactory is passed in
WMDE-leszek renamed this task from Label not shown for lexical category edit diff of a Lexeme to Label not shown for lexical category and/or language edit diff of a Lexeme.Mar 12 2018, 4:43 PM
WMDE-leszek updated the task description. (Show Details)
WMDE-leszek subscribed.

AFAICT this also concerns the diffing edits which touched the language field of the lexeme. This could potentially be seen as the same bug/story.

WMDE-leszek set the point value for this task to 5.Mar 13 2018, 10:44 AM