Page MenuHomePhabricator

Include other items with the same value in “Unique value” violation message
Closed, ResolvedPublic

Description

I forgot about this TODO:

$status = CheckResult::STATUS_VIOLATION;
// TODO include the other entities in the message                                                                                                                                           
$message = wfMessage( 'wbqc-violation-message-unique-value' )->escaped();

But now that this is actually live on Wikidata, the need for it becomes pretty obvious:

This property's value must not be present on any other item.

Thanks for nothing, violation message…

Event Timeline

Change 362234 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/WikibaseQualityConstraints@master] Include other items in Unique value violation message

https://gerrit.wikimedia.org/r/362234

Jonas moved this task from Review to Done on the Wikidata-Former-Sprint-Board board.

Change 362234 merged by jenkins-bot:
[mediawiki/extensions/WikibaseQualityConstraints@master] Include other items in Unique value violation message

https://gerrit.wikimedia.org/r/362234