Merge branch 'master' into moveGetSnakListHash

Author: bekh6ex

composer.json

@@ -43,7 +43,7 @@
 	},
 	"extra": {
 		"branch-alias": {
-			"dev-master": "6.3.x-dev"
+			"dev-master": "7.0.x-dev"
 		}
 	},
 	"scripts": {

src/Snak/SnakList.php

@@ -3,6 +3,8 @@
 namespace Wikibase\DataModel\Snak;
 
 use Hashable;
+use InvalidArgumentException;
+use Traversable;
 use Wikibase\DataModel\HashArray;
 use Wikibase\DataModel\Internal\MapValueHasher;
 
@@ -18,6 +20,21 @@
  */
 class SnakList extends HashArray implements Hashable {
 
+	/**
+	 * @param Snak[]|Traversable $snaks
+	 *
+	 * @throws InvalidArgumentException
+	 */
+	public function __construct( $snaks = [] ) {
+		if ( !is_array( $snaks ) && !( $snaks instanceof Traversable ) ) {
+			throw new InvalidArgumentException( '$snaks must be an array or an instance of Traversable' );
+		}
+
+		foreach ( $snaks as $index => $snak ) {
+			$this->setElement( $index, $snak );
+		}
+	}
+
 	/**
 	 * @see GenericArrayObject::getObjectType
 	 *

tests/unit/HashArray/HashArrayTest.php

@@ -33,7 +33,7 @@ public function instanceProvider() {
 		$instances = [];
 
 		foreach ( $this->constructorProvider() as $args ) {
-			$instances[] = [ new $class( array_key_exists( 0, $args ) ? $args[0] : null ) ];
+			$instances[] = [ new $class( array_key_exists( 0, $args ) ? $args[0] : [] ) ];
 		}
 
 		return $instances;

tests/unit/Snak/SnakListTest.php

@@ -85,6 +85,7 @@ public function invalidConstructorArgumentsProvider() {
 		$id1 = new PropertyId( 'P1' );
 
 		return [
+			[ null ],
 			[ false ],
 			[ 1 ],
 			[ 0.1 ],