jenkins-bot has submitted this change and it was merged. Change subject: Fix / use dieException in SetClaim ......................................................................
Fix / use dieException in SetClaim dieException is here because of bug 71479 which is being investigated. Change-Id: I6e402de523cebdd35e68d0ea958720f3e9266cef --- M repo/includes/api/SetClaim.php 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Aude: Looks good to me, approved Thiemo Mättig (WMDE): Looks good to me, but someone else must approve jenkins-bot: Verified diff --git a/repo/includes/api/SetClaim.php b/repo/includes/api/SetClaim.php index 3107534..d3a898d 100644 --- a/repo/includes/api/SetClaim.php +++ b/repo/includes/api/SetClaim.php @@ -88,7 +88,7 @@ wfDebugLog( 'wikibase-debug', "Failed to set claim on entity $prefixedId: " . var_export( $params, true ) ); - $this->dieError( 'Failed to save claim.' ); + $this->dieException( $ex, 'invalid-claim' ); } $this->getResultBuilder()->markSuccess(); -- To view, visit https://gerrit.wikimedia.org/r/164749 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6e402de523cebdd35e68d0ea958720f3e9266cef Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Wikibase Gerrit-Branch: wmf/1.25wmf1 Gerrit-Owner: Aude <aude.w...@gmail.com> Gerrit-Reviewer: Addshore <addshorew...@gmail.com> Gerrit-Reviewer: Aude <aude.w...@gmail.com> Gerrit-Reviewer: Thiemo Mättig (WMDE) <thiemo.maet...@wikimedia.de> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits