Translate needs a UserMergeAccountFields subscriber to support UserMerge.
Version: unspecified
Severity: enhancement
Translate needs a UserMergeAccountFields subscriber to support UserMerge.
Version: unspecified
Severity: enhancement
Status | Subtype | Assigned | Task | |
---|---|---|---|---|
· · · | ||||
Resolved | Jdforrester-WMF | T69758 Identify extensions that need updates to work with user merging | ||
Resolved | Nikerabbit | T69778 Add UserMerge support to Translate | ||
· · · |
Any table that stores a user's id or username needs a handler for the UserMergeAccountFields hook in the UserMerge extension. It'll be necessary for bug 47918.
If Translate already supports Renameuser, adding support for UserMerge is simple, see I64f178ca186fa72d4163cf28191c45e9bac97d17 for an example in CodeReview.
*shakes head* Can someone please first make the extensions use the same hook or at least the same format for the same data.
(In reply to Niklas Laxström from comment #3)
*shakes head* Can someone please first make the extensions use the same hook
or at least the same format for the same data.
No, because they do different things. Renameuser only needs to change things if you're storing the user's name. UserMerge needs to update rows if user id is stored as well.
Depending on how you're storing the data, you might need to use a different hook point as well (ex: AccountAudit in I4deffe2c9969f9a1a4a679b1ee2a9f902a59fcab).
(In reply to Kunal Mehta (Legoktm) from comment #4)
(In reply to Niklas Laxström from comment #3)
*shakes head* Can someone please first make the extensions use the same hook
or at least the same format for the same data.No, because they do different things. Renameuser only needs to change things
if you're storing the user's name. UserMerge needs to update rows if user id
is stored as well.
But in your example you are providing the same data just in slightly different formats, or what am I not understanding?
This is not possible to fix in Translate unless UserMerge gains capability to deduplicate rows.