Merge pull request #756 from wmde/setidDocs

Fix forgotten Item/Property::setId documentation and exception

Author: manicki

src/Entity/Item.php

@@ -83,21 +83,18 @@ public function getId() {
 	}
 
 	/**
-	 * Can be integer since 0.1.
-	 * Can be ItemId since 0.5.
-	 * Can be null since 1.0.
+	 * @since 0.5, can be null since 1.0
 	 *
 	 * @param ItemId|null $id
 	 *
 	 * @throws InvalidArgumentException
 	 */
 	public function setId( $id ) {
-		if ( $id === null || $id instanceof ItemId ) {
-			$this->id = $id;
-		} else {
-			throw new InvalidArgumentException( '$id must be an instance of ItemId, an integer,'
-				. ' or null' );
+		if ( !( $id instanceof ItemId ) && $id !== null ) {
+			throw new InvalidArgumentException( '$id must be an ItemId or null' );
 		}
+
+		$this->id = $id;
 	}
 
 	/**

src/Entity/Property.php

@@ -81,21 +81,18 @@ public function getId() {
 	}
 
 	/**
-	 * Can be integer since 0.1.
-	 * Can be PropertyId since 0.5.
-	 * Can be null since 1.0.
+	 * @since 0.5, can be null since 1.0
 	 *
 	 * @param PropertyId|null $id
 	 *
 	 * @throws InvalidArgumentException
 	 */
 	public function setId( $id ) {
-		if ( $id === null || $id instanceof PropertyId ) {
-			$this->id = $id;
-		} else {
-			throw new InvalidArgumentException( '$id must be an instance of PropertyId, an integer,'
-				. ' or null' );
+		if ( !( $id instanceof PropertyId ) && $id !== null ) {
+			throw new InvalidArgumentException( '$id must be a PropertyId or null' );
 		}
+
+		$this->id = $id;
 	}
 
 	/**