Page MenuHomePhabricator

Add language codes to sense edit summaries
Closed, ResolvedPublic


Where possible (when the edit only affects one gloss), add language codes to edit summaries regarding glosses.

In other words,

/* add-sense:1||L3-S2 */ the second sense

needs to become

/* add-sense:1|en|L3-S2 */ the second sense

Note: some other message already include the language code, apparently, so I’m not sure how many messages remain that don’t have it.

Event Timeline

I think that might actually be the only such message, because it’s a special case – all the other messages (editing a sense instead of adding a new one) start out from minor ChangeOps, which each have their own language code, and the language code is only erased if necessary by the SummaryAggregator.

Change 449502 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/WikibaseLexeme@master] Set language in ChangeOpSenseAdd if possible

Change 449502 merged by jenkins-bot:
[mediawiki/extensions/WikibaseLexeme@master] Set language in ChangeOpSenseAdd if possible

@Lydia_Pintscher, you should be able to test this by creating a new sense with one gloss on beta wikidata and checking that the edit summary mentions the language.