Daniel Kinzler has uploaded a new change for review.

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


Change subject: Fix validation of snakhash parameter
......................................................................

Fix validation of snakhash parameter

Change-Id: Ib57c03fe8522f5cfb637f371423e1f93b331fa3a
---
M repo/includes/api/SetQualifier.php
1 file changed, 3 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase 
refs/changes/88/85188/1

diff --git a/repo/includes/api/SetQualifier.php 
b/repo/includes/api/SetQualifier.php
index bc0bbe1..1614dfd 100644
--- a/repo/includes/api/SetQualifier.php
+++ b/repo/includes/api/SetQualifier.php
@@ -3,7 +3,6 @@
 namespace Wikibase\Api;
 
 use ApiBase;
-use Wikibase\Entity;
 use Wikibase\Claim;
 use Wikibase\Repo\WikibaseRepo;
 use Wikibase\ChangeOpQualifier;
@@ -44,7 +43,7 @@
                $claim = $this->claimModificationHelper->getClaimFromEntity( 
$params['claim'], $entity );
 
                if ( isset( $params['snakhash'] ) ) {
-                       $this->validateReferenceHash( $claim, 
$params['snakhash'] );
+                       $this->validateQualifierHash( $claim, 
$params['snakhash'] );
                }
 
                $changeOp = $this->getChangeOp();
@@ -95,8 +94,8 @@
         * @param string $qualifierHash
         */
        protected function validateQualifierHash( Claim $claim, $qualifierHash 
) {
-               if ( !$claim->getReferences()->hasReferenceHash( $qualifierHash 
) ) {
-                       $this->dieUsage( "Claim does not have a qualifier with 
the given hash" , 'no-such-reference' );
+               if ( !$claim->getQualifiers()->hasSnakHash( $qualifierHash ) ) {
+                       $this->dieUsage( "Claim does not have a qualifier with 
the given hash" , 'no-such-qualifier' );
                }
        }
 

-- 
To view, visit https://gerrit.wikimedia.org/r/85188
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib57c03fe8522f5cfb637f371423e1f93b331fa3a
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Daniel Kinzler <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to