aude added a comment.

What I see is that LabelUniquenessValidator requests all conflicting labels (in 
all languages), including self-conflicts on the same item, then filters out 
self-conflicts.

TermSqlIndex is what detects conflicts and it returns a maximum of 10 
conflicts. If these are all self-conflicts (then cutting off results before 
getting to non-self conflicts), then they all get filtered out afterwards and 
LabelUniquenessValidator finds no conflict.

Easiest solution is to increase max conflicts, to say 500.

also, is there a need at all for the $ignoreEntityId option in 
LabelDescriptionDuplicateDetector::detectTermConflicts? if we always end up 
filtering them, then is it possible to have this done as part of the query and 
get rid of the option + post filtering.


TASK DETAIL
  https://phabricator.wikimedia.org/T102148

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: aude
Cc: aude, Bene, daniel, Aklapper, Lydia_Pintscher, Wikidata-bugs



_______________________________________________
Wikidata-bugs mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs

Reply via email to