Addshore has uploaded a new change for review.

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


Change subject: Removing redundant parameter check
......................................................................

Removing redundant parameter check

This is never used as validateParameters() is always
called before createEntity can be which means this is
already checked..

Change-Id: I31743373c15c0b88693ec8585573b42b6bc3c538
---
M repo/includes/api/EditEntity.php
1 file changed, 0 insertions(+), 4 deletions(-)


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

diff --git a/repo/includes/api/EditEntity.php b/repo/includes/api/EditEntity.php
index 9cef820..3702832 100644
--- a/repo/includes/api/EditEntity.php
+++ b/repo/includes/api/EditEntity.php
@@ -66,10 +66,6 @@
         * @see ApiModifyEntity::createEntity()
         */
        protected function createEntity( array $params ) {
-               if ( !isset( $params['new'] ) ) {
-                       $this->dieUsage( "Either 'id' or 'new' parameter has to 
be set", 'no-such-entity' );
-               }
-
                $type = $params['new'];
                $this->flags |= EDIT_NEW;
                $entityContentFactory = EntityContentFactory::singleton();

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

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

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

Reply via email to