Jeroen De Dauw has uploaded a new change for review. Change subject: Moved label+descrption check out of transaction to avoid deadlocks ......................................................................
Moved label+descrption check out of transaction to avoid deadlocks Also * Moved duplicate detection code into dedicated class * Only make edits fail for conflicts between terms that actually changed in the edit * Added tests on the level of the new class * Seggregated getMatchingTermCombination off from TermCache Change-Id: If3a528e55ccfacd30fbe706f9ef7360f6a6c3075 --- M repo/Wikibase.hooks.php M repo/Wikibase.php M repo/includes/EditEntity.php A repo/includes/LabelDescriptionDuplicateDetector.php M repo/includes/content/ItemContent.php M repo/includes/store/TermCache.php A repo/includes/store/TermCombinationMatchFinder.php A repo/tests/phpunit/includes/LabelDescriptionDuplicateDetectorTest.php M repo/tests/phpunit/includes/content/ItemContentTest.php 9 files changed, 503 insertions(+), 105 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase refs/changes/24/38324/1 -- To view, visit https://gerrit.wikimedia.org/r/38324 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: If3a528e55ccfacd30fbe706f9ef7360f6a6c3075 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Wikibase Gerrit-Branch: master Gerrit-Owner: Jeroen De Dauw <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
