samwilson / simple-wikidata

src/Properties/Time.php

Indentation +39 added lines, -39 removed lines

@@ -27,48 +27,48 @@
  */
 class Time extends Property {
 
-	/**
-	 * @return DateTime
-	 */
-	public function getDateTime() {
-		return new DateTime( $this->claim['mainsnak']['datavalue']['value']['time'] );
-	}
+    /**
+     * @return DateTime
+     */
+    public function getDateTime() {
+        return new DateTime( $this->claim['mainsnak']['datavalue']['value']['time'] );
+    }
 
-	/**
-	 * @return DateTimeZone
-	 */
-	public function getTimezone() {
-		return new DateTimeZone( $this->claim['mainsnak']['datavalue']['value']['timezone'] );
-	}
+    /**
+     * @return DateTimeZone
+     */
+    public function getTimezone() {
+        return new DateTimeZone( $this->claim['mainsnak']['datavalue']['value']['timezone'] );
+    }
 
-	/**
-	 * @return mixed
-	 */
-	public function getBefore() {
-		return $this->claim['mainsnak']['datavalue']['value']['before'];
-	}
+    /**
+     * @return mixed
+     */
+    public function getBefore() {
+        return $this->claim['mainsnak']['datavalue']['value']['before'];
+    }
 
-	/**
-	 * @return mixed
-	 */
-	public function getAfter() {
-		return $this->claim['mainsnak']['datavalue']['value']['time'];
-	}
+    /**
+     * @return mixed
+     */
+    public function getAfter() {
+        return $this->claim['mainsnak']['datavalue']['value']['time'];
+    }
 
-	/**
-	 * @return int The numbers have the following meanings: 0 - billion years,
-	 * 1 - hundred million years, ...,
-	 * 6 - millennium, 7 - century, 8 - decade, 9 - year, 10 - month, 11 - day, 12 - hour,
-	 * 13 - minute, 14 - second.
-	 */
-	public function getPrecision() {
-		return $this->claim['mainsnak']['datavalue']['value']['precision'];
-	}
+    /**
+     * @return int The numbers have the following meanings: 0 - billion years,
+     * 1 - hundred million years, ...,
+     * 6 - millennium, 7 - century, 8 - decade, 9 - year, 10 - month, 11 - day, 12 - hour,
+     * 13 - minute, 14 - second.
+     */
+    public function getPrecision() {
+        return $this->claim['mainsnak']['datavalue']['value']['precision'];
+    }
 
-	/**
-	 * @return mixed
-	 */
-	public function getCalendarModel() {
-		return $this->claim['mainsnak']['datavalue']['value']['calendarmodel'];
-	}
+    /**
+     * @return mixed
+     */
+    public function getCalendarModel() {
+        return $this->claim['mainsnak']['datavalue']['value']['calendarmodel'];
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -31,14 +31,14 @@
 	 * @return DateTime
 	 */
 	public function getDateTime() {
-		return new DateTime( $this->claim['mainsnak']['datavalue']['value']['time'] );
+		return new DateTime($this->claim['mainsnak']['datavalue']['value']['time']);
 	}
 
 	/**
 	 * @return DateTimeZone
 	 */
 	public function getTimezone() {
-		return new DateTimeZone( $this->claim['mainsnak']['datavalue']['value']['timezone'] );
+		return new DateTimeZone($this->claim['mainsnak']['datavalue']['value']['timezone']);
 	}
 
 	/**

src/Properties/Item.php

Indentation +7 added lines, -7 removed lines

@@ -6,11 +6,11 @@
 
 class Item extends Property {
 
-	/**
-	 * @return \Samwilson\SimpleWikidata\Item
-	 */
-	public function getItem() {
-		$itemId = $this->claim['mainsnak']['datavalue']['value']['id'];
-		return \Samwilson\SimpleWikidata\Item::factory( $itemId, $this->lang, $this->cache );
-	}
+    /**
+     * @return \Samwilson\SimpleWikidata\Item
+     */
+    public function getItem() {
+        $itemId = $this->claim['mainsnak']['datavalue']['value']['id'];
+        return \Samwilson\SimpleWikidata\Item::factory( $itemId, $this->lang, $this->cache );
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -11,6 +11,6 @@
 	 */
 	public function getItem() {
 		$itemId = $this->claim['mainsnak']['datavalue']['value']['id'];
-		return \Samwilson\SimpleWikidata\Item::factory( $itemId, $this->lang, $this->cache );
+		return \Samwilson\SimpleWikidata\Item::factory($itemId, $this->lang, $this->cache);
 	}
 }

src/Query.php

Indentation +67 added lines, -67 removed lines

@@ -8,78 +8,78 @@
 
 class Query {
 
-	/** @var string */
-	protected $query;
+    /** @var string */
+    protected $query;
 
-	/** @var string */
-	protected $lang;
+    /** @var string */
+    protected $lang;
 
-	/** @var CacheItemPoolInterface */
-	protected $cache;
+    /** @var CacheItemPoolInterface */
+    protected $cache;
 
-	/**
-	 * Query constructor.
-	 * @param string $query The Sparql query.
-	 * @param string $lang The language.
-	 * @param CacheItemPoolInterface $cache The cache.
-	 */
-	public function __construct( $query, $lang, CacheItemPoolInterface $cache ) {
-		$this->query = $query;
-		$this->lang = $lang;
-		$this->cache = $cache;
-	}
+    /**
+     * Query constructor.
+     * @param string $query The Sparql query.
+     * @param string $lang The language.
+     * @param CacheItemPoolInterface $cache The cache.
+     */
+    public function __construct( $query, $lang, CacheItemPoolInterface $cache ) {
+        $this->query = $query;
+        $this->lang = $lang;
+        $this->cache = $cache;
+    }
 
-	/**
-	 * Get the items.
-	 * @return Item[] The results.
-	 */
-	public function getItems() {
-		$xml = $this->getXml( $this->query );
-		$results = [];
-		foreach ( $xml->results->result as $res ) {
-			$result = $this->getBindings( $res );
-			$id = substr( $result['item'], strrpos( $result['item'], '/' ) + 1 );
-			$item = Item::factory( $id, $this->lang, $this->cache );
-			$results[] = $item;
-		}
-		return $results;
-	}
+    /**
+     * Get the items.
+     * @return Item[] The results.
+     */
+    public function getItems() {
+        $xml = $this->getXml( $this->query );
+        $results = [];
+        foreach ( $xml->results->result as $res ) {
+            $result = $this->getBindings( $res );
+            $id = substr( $result['item'], strrpos( $result['item'], '/' ) + 1 );
+            $item = Item::factory( $id, $this->lang, $this->cache );
+            $results[] = $item;
+        }
+        return $results;
+    }
 
-	/**
-	 * @param string $query The Sparql query.
-	 * @return SimpleXmlElement
-	 * @throws Exception
-	 */
-	protected function getXml( $query ) {
-		$url = "https://query.wikidata.org/bigdata/namespace/wdq/sparql?query=" . urlencode( $query );
-		try {
-			$result = file_get_contents( $url );
-		} catch ( Exception $e ) {
-			throw new Exception( "Unable to run query: <pre>" . htmlspecialchars( $query ) . "</pre>", 500 );
-		}
-		if ( empty( $result ) ) {
-			$msg = "No result from query: <pre>" . htmlspecialchars( $query ) . "</pre>";
-			throw new Exception( $msg, 500 );
-		}
-		$xml = new SimpleXmlElement( $result );
-		return $xml;
-	}
+    /**
+     * @param string $query The Sparql query.
+     * @return SimpleXmlElement
+     * @throws Exception
+     */
+    protected function getXml( $query ) {
+        $url = "https://query.wikidata.org/bigdata/namespace/wdq/sparql?query=" . urlencode( $query );
+        try {
+            $result = file_get_contents( $url );
+        } catch ( Exception $e ) {
+            throw new Exception( "Unable to run query: <pre>" . htmlspecialchars( $query ) . "</pre>", 500 );
+        }
+        if ( empty( $result ) ) {
+            $msg = "No result from query: <pre>" . htmlspecialchars( $query ) . "</pre>";
+            throw new Exception( $msg, 500 );
+        }
+        $xml = new SimpleXmlElement( $result );
+        return $xml;
+    }
 
-	/**
-	 * @param SimpleXmlElement $xml The query result XML.
-	 * @return array
-	 */
-	protected function getBindings( $xml ) {
-		$out = [];
-		foreach ( $xml->binding as $binding ) {
-			if ( isset( $binding->literal ) ) {
-				$out[(string)$binding['name']] = (string)$binding->literal;
-			}
-			if ( isset( $binding->uri ) ) {
-				$out[(string)$binding['name']] = (string)$binding->uri;
-			}
-		}
-		return $out;
-	}
+    /**
+     * @param SimpleXmlElement $xml The query result XML.
+     * @return array
+     */
+    protected function getBindings( $xml ) {
+        $out = [];
+        foreach ( $xml->binding as $binding ) {
+            if ( isset( $binding->literal ) ) {
+                $out[(string)$binding['name']] = (string)$binding->literal;
+            }
+            if ( isset( $binding->uri ) ) {
+                $out[(string)$binding['name']] = (string)$binding->uri;
+            }
+        }
+        return $out;
+    }
 
 }

Spacing +19 added lines, -19 removed lines

@@ -23,7 +23,7 @@ 
 	 * @param string $lang The language.
 	 * @param CacheItemPoolInterface $cache The cache.
 	 */
-	public function __construct( $query, $lang, CacheItemPoolInterface $cache ) {
+	public function __construct($query, $lang, CacheItemPoolInterface $cache) {
 		$this->query = $query;
 		$this->lang = $lang;
 		$this->cache = $cache;
@@ -34,12 +34,12 @@ 
 	 * @return Item[] The results.
 	 */
 	public function getItems() {
-		$xml = $this->getXml( $this->query );
+		$xml = $this->getXml($this->query);
 		$results = [];
-		foreach ( $xml->results->result as $res ) {
-			$result = $this->getBindings( $res );
-			$id = substr( $result['item'], strrpos( $result['item'], '/' ) + 1 );
-			$item = Item::factory( $id, $this->lang, $this->cache );
+		foreach ($xml->results->result as $res) {
+			$result = $this->getBindings($res);
+			$id = substr($result['item'], strrpos($result['item'], '/') + 1);
+			$item = Item::factory($id, $this->lang, $this->cache);
 			$results[] = $item;
 		}
 		return $results;
@@ -50,18 +50,18 @@ 
 	 * @return SimpleXmlElement
 	 * @throws Exception
 	 */
-	protected function getXml( $query ) {
-		$url = "https://query.wikidata.org/bigdata/namespace/wdq/sparql?query=" . urlencode( $query );
+	protected function getXml($query) {
+		$url = "https://query.wikidata.org/bigdata/namespace/wdq/sparql?query=" . urlencode($query);
 		try {
-			$result = file_get_contents( $url );
-		} catch ( Exception $e ) {
-			throw new Exception( "Unable to run query: <pre>" . htmlspecialchars( $query ) . "</pre>", 500 );
+			$result = file_get_contents($url);
+		} catch (Exception $e) {
+			throw new Exception("Unable to run query: <pre>" . htmlspecialchars($query) . "</pre>", 500);
 		}
-		if ( empty( $result ) ) {
-			$msg = "No result from query: <pre>" . htmlspecialchars( $query ) . "</pre>";
-			throw new Exception( $msg, 500 );
+		if (empty($result)) {
+			$msg = "No result from query: <pre>" . htmlspecialchars($query) . "</pre>";
+			throw new Exception($msg, 500);
 		}
-		$xml = new SimpleXmlElement( $result );
+		$xml = new SimpleXmlElement($result);
 		return $xml;
 	}
 
@@ -69,13 +69,13 @@ 
 	 * @param SimpleXmlElement $xml The query result XML.
 	 * @return array
 	 */
-	protected function getBindings( $xml ) {
+	protected function getBindings($xml) {
 		$out = [];
-		foreach ( $xml->binding as $binding ) {
-			if ( isset( $binding->literal ) ) {
+		foreach ($xml->binding as $binding) {
+			if (isset($binding->literal)) {
 				$out[(string)$binding['name']] = (string)$binding->literal;
 			}
-			if ( isset( $binding->uri ) ) {
+			if (isset($binding->uri)) {
 				$out[(string)$binding['name']] = (string)$binding->uri;
 			}
 		}

src/Reference.php

Indentation +33 added lines, -33 removed lines

@@ -6,37 +6,37 @@
 
 class Reference {
 
-	const STATED_IN = 'P248';
-
-	/** @var array */
-	protected $data;
-
-	/** @var string */
-	protected $lang;
-
-	/** @var CacheItemPoolInterface */
-	protected $cache;
-
-	/**
-	 * @param string[] $data The data.
-	 * @param string $lang ISO639 language code.
-	 * @param CacheItemPoolInterface $cache The cache.
-	 */
-	public function __construct( $data, $lang, $cache ) {
-		$this->data = $data;
-		$this->lang = $lang;
-		$this->cache = $cache;
-	}
-
-	/**
-	 * @return Item|bool The item, or false if there isn't one.
-	 */
-	public function statedIn() {
-		if ( !isset( $this->data['snaks'][self::STATED_IN] ) ) {
-			return false;
-		}
-		foreach ( $this->data['snaks'][self::STATED_IN] as $snak ) {
-			return Item::factory( $snak['datavalue']['value']['id'], $this->lang, $this->cache );
-		}
-	}
+    const STATED_IN = 'P248';
+
+    /** @var array */
+    protected $data;
+
+    /** @var string */
+    protected $lang;
+
+    /** @var CacheItemPoolInterface */
+    protected $cache;
+
+    /**
+     * @param string[] $data The data.
+     * @param string $lang ISO639 language code.
+     * @param CacheItemPoolInterface $cache The cache.
+     */
+    public function __construct( $data, $lang, $cache ) {
+        $this->data = $data;
+        $this->lang = $lang;
+        $this->cache = $cache;
+    }
+
+    /**
+     * @return Item|bool The item, or false if there isn't one.
+     */
+    public function statedIn() {
+        if ( !isset( $this->data['snaks'][self::STATED_IN] ) ) {
+            return false;
+        }
+        foreach ( $this->data['snaks'][self::STATED_IN] as $snak ) {
+            return Item::factory( $snak['datavalue']['value']['id'], $this->lang, $this->cache );
+        }
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -22,7 +22,7 @@ 
 	 * @param string $lang ISO639 language code.
 	 * @param CacheItemPoolInterface $cache The cache.
 	 */
-	public function __construct( $data, $lang, $cache ) {
+	public function __construct($data, $lang, $cache) {
 		$this->data = $data;
 		$this->lang = $lang;
 		$this->cache = $cache;
@@ -32,11 +32,11 @@ 
 	 * @return Item|bool The item, or false if there isn't one.
 	 */
 	public function statedIn() {
-		if ( !isset( $this->data['snaks'][self::STATED_IN] ) ) {
+		if (!isset($this->data['snaks'][self::STATED_IN])) {
 			return false;
 		}
-		foreach ( $this->data['snaks'][self::STATED_IN] as $snak ) {
-			return Item::factory( $snak['datavalue']['value']['id'], $this->lang, $this->cache );
+		foreach ($this->data['snaks'][self::STATED_IN] as $snak) {
+			return Item::factory($snak['datavalue']['value']['id'], $this->lang, $this->cache);
 		}
 	}
 }

src/Items/Human.php

Indentation +44 added lines, -44 removed lines

@@ -9,56 +9,56 @@
  */
 class Human extends Item {
 
-	const PROP_DATE_OF_BIRTH = 'P569';
-	const PROP_PLACE_OF_BIRTH = 'P19';
+    const PROP_DATE_OF_BIRTH = 'P569';
+    const PROP_PLACE_OF_BIRTH = 'P19';
 
-	const PROP_DATE_OF_DEATH = 'P570';
-	const PROP_PLACE_OF_DEATH = 'P20';
+    const PROP_DATE_OF_DEATH = 'P570';
+    const PROP_PLACE_OF_DEATH = 'P20';
 
-	const PROP_FATHER = 'P22';
-	const PROP_MOTHER = 'P25';
-	const PROP_SPOUSE = 'P26';
-	const PROP_CHILD = 'P40';
+    const PROP_FATHER = 'P22';
+    const PROP_MOTHER = 'P25';
+    const PROP_SPOUSE = 'P26';
+    const PROP_CHILD = 'P40';
 
-	/**
-	 * @return bool|\Samwilson\SimpleWikidata\Properties\Time[]
-	 */
-	public function getDatesOfBirth() {
-		return $this->getPropertyOfTypeTime( self::PROP_DATE_OF_BIRTH );
-	}
+    /**
+     * @return bool|\Samwilson\SimpleWikidata\Properties\Time[]
+     */
+    public function getDatesOfBirth() {
+        return $this->getPropertyOfTypeTime( self::PROP_DATE_OF_BIRTH );
+    }
 
-	/**
-	 * @return bool|\Samwilson\SimpleWikidata\Properties\Time[]
-	 */
-	public function getDatesOfDeath() {
-		return $this->getPropertyOfTypeTime( self::PROP_DATE_OF_DEATH );
-	}
+    /**
+     * @return bool|\Samwilson\SimpleWikidata\Properties\Time[]
+     */
+    public function getDatesOfDeath() {
+        return $this->getPropertyOfTypeTime( self::PROP_DATE_OF_DEATH );
+    }
 
-	/**
-	 * @return \Samwilson\SimpleWikidata\Properties\Item[]
-	 */
-	public function fathers() {
-		return $this->getPropertyOfTypeItem( self::PROP_FATHER );
-	}
+    /**
+     * @return \Samwilson\SimpleWikidata\Properties\Item[]
+     */
+    public function fathers() {
+        return $this->getPropertyOfTypeItem( self::PROP_FATHER );
+    }
 
-	/**
-	 * @return \Samwilson\SimpleWikidata\Properties\Item[]
-	 */
-	public function mothers() {
-		return $this->getPropertyOfTypeItem( self::PROP_FATHER );
-	}
+    /**
+     * @return \Samwilson\SimpleWikidata\Properties\Item[]
+     */
+    public function mothers() {
+        return $this->getPropertyOfTypeItem( self::PROP_FATHER );
+    }
 
-	/**
-	 * @return \Samwilson\SimpleWikidata\Properties\Item[]
-	 */
-	public function spouses() {
-		return $this->getPropertyOfTypeItem( self::PROP_SPOUSE );
-	}
+    /**
+     * @return \Samwilson\SimpleWikidata\Properties\Item[]
+     */
+    public function spouses() {
+        return $this->getPropertyOfTypeItem( self::PROP_SPOUSE );
+    }
 
-	/**
-	 * @return \Samwilson\SimpleWikidata\Properties\Item[]
-	 */
-	public function children() {
-		return $this->getPropertyOfTypeItem( self::PROP_CHILD );
-	}
+    /**
+     * @return \Samwilson\SimpleWikidata\Properties\Item[]
+     */
+    public function children() {
+        return $this->getPropertyOfTypeItem( self::PROP_CHILD );
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -24,41 +24,41 @@
 	 * @return bool|\Samwilson\SimpleWikidata\Properties\Time[]
 	 */
 	public function getDatesOfBirth() {
-		return $this->getPropertyOfTypeTime( self::PROP_DATE_OF_BIRTH );
+		return $this->getPropertyOfTypeTime(self::PROP_DATE_OF_BIRTH);
 	}
 
 	/**
 	 * @return bool|\Samwilson\SimpleWikidata\Properties\Time[]
 	 */
 	public function getDatesOfDeath() {
-		return $this->getPropertyOfTypeTime( self::PROP_DATE_OF_DEATH );
+		return $this->getPropertyOfTypeTime(self::PROP_DATE_OF_DEATH);
 	}
 
 	/**
 	 * @return \Samwilson\SimpleWikidata\Properties\Item[]
 	 */
 	public function fathers() {
-		return $this->getPropertyOfTypeItem( self::PROP_FATHER );
+		return $this->getPropertyOfTypeItem(self::PROP_FATHER);
 	}
 
 	/**
 	 * @return \Samwilson\SimpleWikidata\Properties\Item[]
 	 */
 	public function mothers() {
-		return $this->getPropertyOfTypeItem( self::PROP_FATHER );
+		return $this->getPropertyOfTypeItem(self::PROP_FATHER);
 	}
 
 	/**
 	 * @return \Samwilson\SimpleWikidata\Properties\Item[]
 	 */
 	public function spouses() {
-		return $this->getPropertyOfTypeItem( self::PROP_SPOUSE );
+		return $this->getPropertyOfTypeItem(self::PROP_SPOUSE);
 	}
 
 	/**
 	 * @return \Samwilson\SimpleWikidata\Properties\Item[]
 	 */
 	public function children() {
-		return $this->getPropertyOfTypeItem( self::PROP_CHILD );
+		return $this->getPropertyOfTypeItem(self::PROP_CHILD);
 	}
 }

src/Search.php

Indentation +37 added lines, -37 removed lines

@@ -8,44 +8,44 @@
 
 class Search {
 
-	/** @var string */
-	protected $lang;
+    /** @var string */
+    protected $lang;
 
-	/** @var CacheItemPoolInterface */
-	protected $cache;
+    /** @var CacheItemPoolInterface */
+    protected $cache;
 
-	/**
-	 * Search constructor.
-	 * @param string $searchTerm What to search for.
-	 * @param string $lang Language to use for the search results.
-	 * @param CacheItemPoolInterface $cache The cache to use.
-	 */
-	public function __construct( $searchTerm, $lang, CacheItemPoolInterface $cache ) {
-		$this->searchTerm = $searchTerm;
-		$this->lang = $lang;
-		$this->cache = $cache;
-	}
+    /**
+     * Search constructor.
+     * @param string $searchTerm What to search for.
+     * @param string $lang Language to use for the search results.
+     * @param CacheItemPoolInterface $cache The cache to use.
+     */
+    public function __construct( $searchTerm, $lang, CacheItemPoolInterface $cache ) {
+        $this->searchTerm = $searchTerm;
+        $this->lang = $lang;
+        $this->cache = $cache;
+    }
 
-	/**
-	 * @param string $limit The number of search results to return.
-	 * @return Item[]
-	 */
-	public function getItems( $limit = 'max' ) {
-		$api = MediawikiApi::newFromApiEndpoint( 'https://www.wikidata.org/w/api.php' );
-		$req = FluentRequest::factory()
-			->setAction( 'wbsearchentities' )
-			->addParams( [
-				'search' => $this->searchTerm,
-				'type' => 'item',
-				'limit' => $limit,
-				'language' => 'en',
-			] );
-		$results = [];
-		$response = $api->getRequest( $req );
-		foreach ( $response['search'] as $info ) {
-			$item = Item::factory( $info['id'], $this->lang, $this->cache );
-			$results[] = $item;
-		}
-		return $results;
-	}
+    /**
+     * @param string $limit The number of search results to return.
+     * @return Item[]
+     */
+    public function getItems( $limit = 'max' ) {
+        $api = MediawikiApi::newFromApiEndpoint( 'https://www.wikidata.org/w/api.php' );
+        $req = FluentRequest::factory()
+            ->setAction( 'wbsearchentities' )
+            ->addParams( [
+                'search' => $this->searchTerm,
+                'type' => 'item',
+                'limit' => $limit,
+                'language' => 'en',
+            ] );
+        $results = [];
+        $response = $api->getRequest( $req );
+        foreach ( $response['search'] as $info ) {
+            $item = Item::factory( $info['id'], $this->lang, $this->cache );
+            $results[] = $item;
+        }
+        return $results;
+    }
 }

Spacing +9 added lines, -9 removed lines

@@ -20,7 +20,7 @@ 
 	 * @param string $lang Language to use for the search results.
 	 * @param CacheItemPoolInterface $cache The cache to use.
 	 */
-	public function __construct( $searchTerm, $lang, CacheItemPoolInterface $cache ) {
+	public function __construct($searchTerm, $lang, CacheItemPoolInterface $cache) {
 		$this->searchTerm = $searchTerm;
 		$this->lang = $lang;
 		$this->cache = $cache;
@@ -30,20 +30,20 @@ 
 	 * @param string $limit The number of search results to return.
 	 * @return Item[]
 	 */
-	public function getItems( $limit = 'max' ) {
-		$api = MediawikiApi::newFromApiEndpoint( 'https://www.wikidata.org/w/api.php' );
+	public function getItems($limit = 'max') {
+		$api = MediawikiApi::newFromApiEndpoint('https://www.wikidata.org/w/api.php');
 		$req = FluentRequest::factory()
-			->setAction( 'wbsearchentities' )
-			->addParams( [
+			->setAction('wbsearchentities')
+			->addParams([
 				'search' => $this->searchTerm,
 				'type' => 'item',
 				'limit' => $limit,
 				'language' => 'en',
-			] );
+			]);
 		$results = [];
-		$response = $api->getRequest( $req );
-		foreach ( $response['search'] as $info ) {
-			$item = Item::factory( $info['id'], $this->lang, $this->cache );
+		$response = $api->getRequest($req);
+		foreach ($response['search'] as $info) {
+			$item = Item::factory($info['id'], $this->lang, $this->cache);
 			$results[] = $item;
 		}
 		return $results;

src/Item.php

Indentation +466 added lines, -466 removed lines

@@ -12,180 +12,180 @@ 
 
 class Item {
 
-	/** @var string|bool */
-	const INSTANCE_OF = false;
-
-	const PROP_INSTANCE_OF = 'P31';
-	const PROP_TITLE = 'P1476';
-	const PROP_IMAGE = 'P18';
-	const PROP_AUTHOR = 'P50';
-
-	/** @var string[] List of Q-numbers of registered classes. */
-	protected static $registeredClasses = [];
-
-	/** @var string */
-	protected $id;
-
-	/** @var MediawikiApi */
-	protected $wdApi;
-
-	/** @var string */
-	protected $lang;
-
-	/** @var CacheItemPoolInterface */
-	protected $cache;
-
-	/** @var string The base URL of Wikidata, with trailing slash. */
-	protected $wikidataUrlBase = 'https://www.wikidata.org/wiki/';
-
-	private function __construct( $id, $lang, CacheItemPoolInterface $cache ) {
-		if ( !is_string( $id ) || preg_match( '/[QP][0-9]*/i', $id ) !== 1 ) {
-			throw new Exception( "Not a valid ID: " . var_export( $id, true ) );
-		}
-		$this->id = $id;
-		$this->wdApi = new MediawikiApi( 'https://www.wikidata.org/w/api.php' );
-		$this->entities = [];
-		$this->lang = $lang;
-		$this->cache = $cache;
-	}
-
-	/**
-	 * Create a new Item object with class based on the item's 'instance of' statement.
-	 *
-	 * @param string $id The item ID (Q-number).
-	 * @param string $lang The language code.
-	 * @param CacheItemPoolInterface $cache The cache to use.
-	 * @return Item
-	 */
-	public static function factory( $id, $lang, CacheItemPoolInterface $cache ) {
-		$item = new Item( $id, $lang, $cache );
-		foreach ( $item->getPropertyOfTypeItem( self::PROP_INSTANCE_OF ) as $instanceOf ) {
-			// Try to find a class matching the registered 'instance of'.
-			foreach ( static::$registeredClasses as $classId => $className ) {
-				// If this 'instance of' is registered, use it.
-				if ( $classId === $instanceOf->getItem()->getId() ) {
-					// This won't re-request the metadata, because that's cached.
-					return new $className( $id, $lang, $cache );
-				}
-			}
-		}
-
-		// If we're here, just leave it as a basic Item.
-		$item->setCache( $cache );
-		return $item;
-	}
-
-	/**
-	 * Register this class as a candidate for being created by Item::factory.
-	 * Should only be called on subclasses of Item.
-	 * @throws Exception if called on Item or the registering class does not have INSTANCE_OF set.
-	 */
-	public static function register() {
-		if ( static::class === self::class ) {
-			throw new Exception( __METHOD__ . ' should only be called on subclasses of Item' );
-		}
-		if ( !static::INSTANCE_OF ) {
-			throw new Exception( 'Please set INSTANCE_OF for ' . static::class );
-		}
-		static::$registeredClasses[ static::INSTANCE_OF ] = static::class;
-	}
-
-	/**
-	 * @param CacheItemPoolInterface $cache The cache to use.
-	 */
-	public function setCache( CacheItemPoolInterface $cache ) {
-		$this->cache = $cache;
-	}
-
-	/**
-	 * Get the ID (Q-number) of this item.
-	 * @return string|bool The ID or false if it couldn't be determined.
-	 */
-	public function getId() {
-		$entity = $this->getEntity( $this->id );
-		return isset( $entity['id'] ) ? $entity['id'] : false;
-	}
-
-	/**
-	 * Get this item's label.
-	 * @return string
-	 */
-	public function getLabel() {
-		$entity = $this->getEntity( $this->id );
-		if ( !empty( $entity['labels'][ $this->lang ]['value'] ) ) {
-			// Use the label if there is one.
-			return $entity['labels'][ $this->lang ]['value'];
-		}
-		// Or just use the ID.
-		return $entity['id'];
-	}
-
-	/**
-	 * @return string The Wikidata.org URL for this item.
-	 */
-	public function getWikidataUrl() {
-		return $this->wikidataUrlBase . $this->id;
-	}
-
-	/**
-	 * Wikiprojects list their properties like this:
-	 *
-	 *     {{List of properties/Header}}
-	 *     {{List of properties/Row|id=31|example-subject=Q923767|example-object=Q3331189}}
-	 *     </table>
-	 *
-	 * @param string $wikiProject The name of the WikiProject (must exist as a Wikidata page e.g.
-	 * [[Wikidata:$wikiProject]]).
-	 * @param string $type
-	 * @return array
-	 */
-	public function getStandardProperties( $wikiProject = 'WikiProject_Books', $type = 'work' ) {
-		if ( $type !== 'work' ) {
-			$type = 'edition';
-		}
-		$cacheKey = $type . '_item_property_IDs';
-		if ( $this->cache->hasItem( $cacheKey ) ) {
-			$propIds = $this->cache->getItem( $cacheKey )->get();
-		} else {
-			$domCrawler = new Crawler();
-			$wikiProjectUrl = 'https://www.wikidata.org/wiki/Wikidata:' . $wikiProject;
-			$domCrawler->addHtmlContent( file_get_contents( $wikiProjectUrl ) );
-			$propAncors = "//h3/span[@id='" . ucfirst( $type ) . "_item_properties']"
-				. "/../following-sibling::table[1]//td[2]/a";
-			$propCells = $domCrawler->filterXPath( $propAncors );
-			$propIds = [];
-			$propCells->each( function ( Crawler $node, $i ) use ( &$propIds ) {
-				$propId = $node->text();
-				$propIds[] = $propId;
-			} );
-			$cacheItem = $this->cache->getItem( $cacheKey )
-				->expiresAfter( new DateInterval( 'PT1H' ) )
-				->set( $propIds );
-			$this->cache->save( $cacheItem );
-		}
-		$workProperties = [];
-		foreach ( $propIds as $propId ) {
-			$workProperties[] = self::factory( $propId, $this->lang, $this->cache );
-		}
-
-		return $workProperties;
-	}
-
-	/**
-	 * @param string $propertyId
-	 * @return bool|Time[]
-	 */
-	public function getPropertyOfTypeTime( $propertyId ) {
-		$times = [];
-		$entity = $this->getEntity();
-		if ( !isset( $entity['claims'][$propertyId] ) ) {
-			// No statements for this property.
-			return $times;
-		}
-		// print_r($entity['claims'][$propertyId]);exit();
-		foreach ( $entity['claims'][$propertyId] as $claim ) {
-			// print_r($claim);
-			$times[] = new Time( $claim, $this->lang, $this->cache );
+    /** @var string|bool */
+    const INSTANCE_OF = false;
+
+    const PROP_INSTANCE_OF = 'P31';
+    const PROP_TITLE = 'P1476';
+    const PROP_IMAGE = 'P18';
+    const PROP_AUTHOR = 'P50';
+
+    /** @var string[] List of Q-numbers of registered classes. */
+    protected static $registeredClasses = [];
+
+    /** @var string */
+    protected $id;
+
+    /** @var MediawikiApi */
+    protected $wdApi;
+
+    /** @var string */
+    protected $lang;
+
+    /** @var CacheItemPoolInterface */
+    protected $cache;
+
+    /** @var string The base URL of Wikidata, with trailing slash. */
+    protected $wikidataUrlBase = 'https://www.wikidata.org/wiki/';
+
+    private function __construct( $id, $lang, CacheItemPoolInterface $cache ) {
+        if ( !is_string( $id ) || preg_match( '/[QP][0-9]*/i', $id ) !== 1 ) {
+            throw new Exception( "Not a valid ID: " . var_export( $id, true ) );
+        }
+        $this->id = $id;
+        $this->wdApi = new MediawikiApi( 'https://www.wikidata.org/w/api.php' );
+        $this->entities = [];
+        $this->lang = $lang;
+        $this->cache = $cache;
+    }
+
+    /**
+     * Create a new Item object with class based on the item's 'instance of' statement.
+     *
+     * @param string $id The item ID (Q-number).
+     * @param string $lang The language code.
+     * @param CacheItemPoolInterface $cache The cache to use.
+     * @return Item
+     */
+    public static function factory( $id, $lang, CacheItemPoolInterface $cache ) {
+        $item = new Item( $id, $lang, $cache );
+        foreach ( $item->getPropertyOfTypeItem( self::PROP_INSTANCE_OF ) as $instanceOf ) {
+            // Try to find a class matching the registered 'instance of'.
+            foreach ( static::$registeredClasses as $classId => $className ) {
+                // If this 'instance of' is registered, use it.
+                if ( $classId === $instanceOf->getItem()->getId() ) {
+                    // This won't re-request the metadata, because that's cached.
+                    return new $className( $id, $lang, $cache );
+                }
+            }
+        }
+
+        // If we're here, just leave it as a basic Item.
+        $item->setCache( $cache );
+        return $item;
+    }
+
+    /**
+     * Register this class as a candidate for being created by Item::factory.
+     * Should only be called on subclasses of Item.
+     * @throws Exception if called on Item or the registering class does not have INSTANCE_OF set.
+     */
+    public static function register() {
+        if ( static::class === self::class ) {
+            throw new Exception( __METHOD__ . ' should only be called on subclasses of Item' );
+        }
+        if ( !static::INSTANCE_OF ) {
+            throw new Exception( 'Please set INSTANCE_OF for ' . static::class );
+        }
+        static::$registeredClasses[ static::INSTANCE_OF ] = static::class;
+    }
+
+    /**
+     * @param CacheItemPoolInterface $cache The cache to use.
+     */
+    public function setCache( CacheItemPoolInterface $cache ) {
+        $this->cache = $cache;
+    }
+
+    /**
+     * Get the ID (Q-number) of this item.
+     * @return string|bool The ID or false if it couldn't be determined.
+     */
+    public function getId() {
+        $entity = $this->getEntity( $this->id );
+        return isset( $entity['id'] ) ? $entity['id'] : false;
+    }
+
+    /**
+     * Get this item's label.
+     * @return string
+     */
+    public function getLabel() {
+        $entity = $this->getEntity( $this->id );
+        if ( !empty( $entity['labels'][ $this->lang ]['value'] ) ) {
+            // Use the label if there is one.
+            return $entity['labels'][ $this->lang ]['value'];
+        }
+        // Or just use the ID.
+        return $entity['id'];
+    }
+
+    /**
+     * @return string The Wikidata.org URL for this item.
+     */
+    public function getWikidataUrl() {
+        return $this->wikidataUrlBase . $this->id;
+    }
+
+    /**
+     * Wikiprojects list their properties like this:
+     *
+     *     {{List of properties/Header}}
+     *     {{List of properties/Row|id=31|example-subject=Q923767|example-object=Q3331189}}
+     *     </table>
+     *
+     * @param string $wikiProject The name of the WikiProject (must exist as a Wikidata page e.g.
+     * [[Wikidata:$wikiProject]]).
+     * @param string $type
+     * @return array
+     */
+    public function getStandardProperties( $wikiProject = 'WikiProject_Books', $type = 'work' ) {
+        if ( $type !== 'work' ) {
+            $type = 'edition';
+        }
+        $cacheKey = $type . '_item_property_IDs';
+        if ( $this->cache->hasItem( $cacheKey ) ) {
+            $propIds = $this->cache->getItem( $cacheKey )->get();
+        } else {
+            $domCrawler = new Crawler();
+            $wikiProjectUrl = 'https://www.wikidata.org/wiki/Wikidata:' . $wikiProject;
+            $domCrawler->addHtmlContent( file_get_contents( $wikiProjectUrl ) );
+            $propAncors = "//h3/span[@id='" . ucfirst( $type ) . "_item_properties']"
+                . "/../following-sibling::table[1]//td[2]/a";
+            $propCells = $domCrawler->filterXPath( $propAncors );
+            $propIds = [];
+            $propCells->each( function ( Crawler $node, $i ) use ( &$propIds ) {
+                $propId = $node->text();
+                $propIds[] = $propId;
+            } );
+            $cacheItem = $this->cache->getItem( $cacheKey )
+                ->expiresAfter( new DateInterval( 'PT1H' ) )
+                ->set( $propIds );
+            $this->cache->save( $cacheItem );
+        }
+        $workProperties = [];
+        foreach ( $propIds as $propId ) {
+            $workProperties[] = self::factory( $propId, $this->lang, $this->cache );
+        }
+
+        return $workProperties;
+    }
+
+    /**
+     * @param string $propertyId
+     * @return bool|Time[]
+     */
+    public function getPropertyOfTypeTime( $propertyId ) {
+        $times = [];
+        $entity = $this->getEntity();
+        if ( !isset( $entity['claims'][$propertyId] ) ) {
+            // No statements for this property.
+            return $times;
+        }
+        // print_r($entity['claims'][$propertyId]);exit();
+        foreach ( $entity['claims'][$propertyId] as $claim ) {
+            // print_r($claim);
+            $times[] = new Time( $claim, $this->lang, $this->cache );
 //
 // $timeValue = $claim['datavalue']['value']['time'];
 // // Ugly workaround for imprecise dates. :-(
@@ -195,296 +195,296 @@ 
 // }
 // $time = strtotime($timeValue);
 // return date($dateFormat, $time);
-			// }
-		}
-		return $times;
-	}
-
-	/**
-	 * Get the Item that is referred to by the specified item's property.
-	 *
-	 * @param string $propertyId
-	 *
-	 * @return \Samwilson\SimpleWikidata\Properties\Item[]
-	 */
-	public function getPropertyOfTypeItem( $propertyId ) {
-		$entity = $this->getEntity( $this->id );
-		if ( !isset( $entity['claims'][$propertyId] ) ) {
-			return [];
-		}
-		$items = [];
-		foreach ( $entity['claims'][$propertyId] as $claim ) {
-			$items[] = new Properties\Item( $claim, $this->lang, $this->cache );
-		}
-
-		return $items;
-	}
-
-	/**
-	 * Set a claim to the given Item.
-	 * @param string $property Property ID, with 'P'.
-	 * @param string $itemId Item ID with 'Q'.
-	 */
-	public function setPropertyOfTypeItem( $property, $itemId ) {
-		$itemIdNumeric = substr( $itemId, 1 );
-
-		// First see if this property already exists, and that it is different from what's being set.
-		$entity = $this->getEntity( $this->id );
-		if ( !empty( $entity['claims'][$property] ) ) {
-			// Get the first claim, and update it if necessary.
-			$claim = array_shift( $entity['claims'][$property] );
-			if ( $claim['mainsnak']['datavalue']['value']['id'] == $itemId ) {
-				// Already is the required value, no need to change.
-				return;
-			}
-			$claim['mainsnak']['datavalue']['value']['id'] = $itemId;
-			$claim['mainsnak']['datavalue']['value']['numeric-id'] = $itemIdNumeric;
-			$apiParams = [
-				'action' => 'wbsetclaim',
-				'claim' => json_encode( $claim ),
-			];
-		}
-
-		// If no claim was found (and modified) above, create a new claim.
-		if ( !isset( $apiParams ) ) {
-			$apiParams = [
-				'action' => 'wbcreateclaim',
-				'entity' => $this->getId(),
-				'property' => $property,
-				'snaktype' => 'value',
-				'value' => json_encode( [ 'entity-type' => 'item', 'numeric-id' => $itemIdNumeric ] ),
-			];
-		}
-
-		// @TODO Save the property.
-
-		// Clear the cache.
-		$this->cache->deleteItem( $this->getEntityCacheKey( $this->id ) );
-	}
-
-	/**
-	 * @param string $entityId
-	 * @param string $propertyId
-	 * @return array|bool
-	 */
-	public function getPropertyOfTypeUrl( $entityId, $propertyId ) {
-		$entity = $this->getEntity( $entityId );
-		if ( !isset( $entity['claims'][$propertyId] ) ) {
-			return false;
-		}
-		$urls = [];
-		foreach ( $entity['claims'][$propertyId] as $claim ) {
-			$urls[] = $claim['mainsnak']['datavalue']['value'];
-		}
-
-		return $urls;
-	}
-
-	/**
-	 * @param string $entityId
-	 * @param string $propertyId
-	 * @return array|bool
-	 */
-	public function getPropertyOfTypeExternalIdentifier( $entityId, $propertyId ) {
-		$entity = $this->getEntity( $entityId );
-		if ( !isset( $entity['claims'][$propertyId] ) ) {
-			return false;
-		}
-		$idents = [];
-		foreach ( $entity['claims'][$propertyId] as $claim ) {
-			$qualifiers = [];
-			if ( !isset( $claim['qualifiers'] ) ) {
-				continue;
-			}
-			foreach ( $claim['qualifiers'] as $qualsInfo ) {
-				foreach ( $qualsInfo as $qualInfo ) {
-					$qualProp = self::factory( $qualInfo['property'], $this->lang, $this->cache );
-					$propLabel = $qualProp->getLabel();
-					if ( !isset( $qualifiers[$propLabel] ) ) {
-						$qualifiers[$propLabel] = [];
-					}
-					$qualifiers[$propLabel][] = $qualInfo['datavalue']['value'];
-				}
-			}
-			$idents[] = [
-				'qualifiers' => $qualifiers,
-				'value' => $claim['mainsnak']['datavalue']['value'],
-			];
-		}
-
-		return $idents;
-	}
-
-	/**
-	 * Get a single-valued text property.
-	 * @param string $property One of the PROP_* constants.
-	 * @return string|bool The value, or false if it can't be found.
-	 */
-	public function getPropertyOfTypeText( $property ) {
-		$entity = $this->getEntity( $this->id );
-		if ( isset( $entity['claims'][$property] ) ) {
-			// Use the first title.
-			foreach ( $entity['claims'][$property] as $t ) {
-				if ( !isset( $t['mainsnak']['datavalue']['value']['language'] ) ) {
-					var_dump( $t['mainsnak']['datavalue']['value'] );
-					exit();
-				}
-				if ( $t['mainsnak']['datavalue']['value']['language'] == $this->lang
-					&& !empty( $t['mainsnak']['datavalue']['value']['text'] )
-				) {
-					return $t['mainsnak']['datavalue']['value']['text'];
-				}
-			}
-		}
-		return false;
-	}
-
-	/**
-	 * Literal data field for a quantity that relates to some kind of well-defined unit.
-	 * The actual unit goes in the data values that is entered.
-	 *   - amount     – implicit part of the string (mapping of unit prefix is unclear)
-	 *   - unit       – implicit part of the string that defaults to "1" (mapping to standardizing
-	 *                  body is unclear)
-	 *   - upperbound - quantity's upper bound
-	 *   - lowerbound - quantity's lower bound
-	 * @param string $property
-	 * @return mixed[]|bool If it's not false it's an array with 'amount', 'unit', etc.
-	 */
-	public function getPropertyOfTypeQuantity( $property ) {
-		$quantities = [];
-		$entity = $this->getEntity( $this->id );
-		if ( !isset( $entity['claims'][$property] ) ) {
-			return false;
-		}
-		foreach ( $entity['claims'][$property] as $t ) {
-			$quantity = $t['mainsnak']['datavalue']['value'];
-			$unitId = substr( $quantity['unit'], strlen( $this->wikidataUrlBase ) + 1 );
-			$quantity['unit'] = self::factory( $unitId, $this->lang, $this->cache );
-			$quantities[] = $quantity;
-		}
-		return $quantities;
-	}
-
-	/**
-	 * Set a single-valued text property.
-	 * @param string $property One of the PROP_* constants.
-	 * @param string $value The value.
-	 */
-	public function setPropertyOfTypeText( $property, $value ) {
-		// First see if this property already exists, and that it is different from what's being set.
-		$entity = $this->getEntity( $this->id );
-		if ( !empty( $entity['claims'][$property] ) ) {
-			// Find this language's claim (if there is one).
-			foreach ( $entity['claims'][$property] as $claim ) {
-				if ( $claim['mainsnak']['datavalue']['value']['language'] == $this->lang ) {
-					// Modify this claim's text value.
-					$titleClaim = $claim;
-					$titleClaim['mainsnak']['datavalue']['value']['text'] = $value;
-					$setTitleParams = [
-						'action' => 'wbsetclaim',
-						'claim' => \GuzzleHttp\json_encode( $titleClaim ),
-					];
-					continue;
-				}
-			}
-		}
-
-		// If no claim was found (and modified) above, create a new claim.
-		if ( !isset( $setTitleParams ) ) {
-			$setTitleParams = [
-				'action' => 'wbcreateclaim',
-				'entity' => $this->getId(),
-				'property' => $property,
-				'snaktype' => 'value',
-				'value' => \GuzzleHttp\json_encode( [ 'text' => $value, 'language' => $this->lang ] ),
-			];
-		}
-
-		// Save the property.
-		$wdWpOauth = new WdWpOauth();
-		$wdWpOauth->makeCall( $setTitleParams, true );
-
-		// Clear the cache.
-		$this->cache->deleteItem( $this->getEntityCacheKey( $this->id ) );
-	}
-
-	/**
-	 * Does this item exist?
-	 * @return bool
-	 */
-	public function exists() {
-		return $this->getId() !== false;
-	}
-
-	/**
-	 * @return array With 'html' and 'title' keys.
-	 */
-	public function getWikipediaIntro() {
-		$cacheKey = 'wikipedia-intro-' . $this->id . $this->lang;
-		if ( $this->cache->hasItem( $cacheKey ) ) {
-			return $this->cache->getItem( $cacheKey )->get();
-		}
-		$entity = $this->getEntity( $this->id );
-		if ( !isset( $entity['sitelinks'] ) ) {
-			return [];
-		}
-		foreach ( $entity['sitelinks'] as $sitelink ) {
-			if ( $sitelink['site'] == $this->lang . 'wiki' ) {
-				$api = new MediawikiApi( 'https://' . $this->lang . '.wikipedia.org/w/api.php' );
-				$req = new SimpleRequest( 'query', [
-					'prop' => 'extracts',
-					'exintro' => true,
-					'titles' => $sitelink['title'],
-				] );
-				$response = $api->getRequest( $req );
-				$page = array_shift( $response['query']['pages'] );
-				$out = [
-					'title' => $page['title'],
-					'html' => $page['extract'],
-				];
-				$cacheItem = $this->cache->getItem( $cacheKey )
-					->expiresAfter( new DateInterval( 'P1D' ) )
-					->set( $out );
-				$this->cache->save( $cacheItem );
-
-				return $out;
-			}
-		}
-
-		return [];
-	}
-
-	/**
-	 * Get the raw entity data from the 'wbgetentities' API call.
-	 * @param string|null $id The Q-number.
-	 * @param bool $ignoreCache
-	 * @return array|bool
-	 */
-	public function getEntity( $id = null, $ignoreCache = false ) {
-		$idActual = $id ?: $this->id;
-		$cacheKey = $this->getEntityCacheKey( $idActual );
-		if ( !$ignoreCache && $this->cache->hasItem( $cacheKey ) ) {
-			return $this->cache->getItem( $cacheKey )->get();
-		}
-		$metadataRequest = new SimpleRequest( 'wbgetentities', [ 'ids' => $idActual ] );
-		$itemResult = $this->wdApi->getRequest( $metadataRequest );
-		if ( !isset( $itemResult['success'] ) || !isset( $itemResult['entities'][$id] ) ) {
-			return false;
-		}
-		$metadata = $itemResult['entities'][$idActual];
-		$cacheItem = $this->cache->getItem( $cacheKey )
-			->expiresAfter( new DateInterval( 'PT10M' ) )
-			->set( $metadata );
-		$this->cache->save( $cacheItem );
-		return $metadata;
-	}
-
-	/**
-	 * @param string $id
-	 *
-	 * @return string
-	 */
-	protected function getEntityCacheKey( $id ) {
-		return 'entities' . $id;
-	}
+            // }
+        }
+        return $times;
+    }
+
+    /**
+     * Get the Item that is referred to by the specified item's property.
+     *
+     * @param string $propertyId
+     *
+     * @return \Samwilson\SimpleWikidata\Properties\Item[]
+     */
+    public function getPropertyOfTypeItem( $propertyId ) {
+        $entity = $this->getEntity( $this->id );
+        if ( !isset( $entity['claims'][$propertyId] ) ) {
+            return [];
+        }
+        $items = [];
+        foreach ( $entity['claims'][$propertyId] as $claim ) {
+            $items[] = new Properties\Item( $claim, $this->lang, $this->cache );
+        }
+
+        return $items;
+    }
+
+    /**
+     * Set a claim to the given Item.
+     * @param string $property Property ID, with 'P'.
+     * @param string $itemId Item ID with 'Q'.
+     */
+    public function setPropertyOfTypeItem( $property, $itemId ) {
+        $itemIdNumeric = substr( $itemId, 1 );
+
+        // First see if this property already exists, and that it is different from what's being set.
+        $entity = $this->getEntity( $this->id );
+        if ( !empty( $entity['claims'][$property] ) ) {
+            // Get the first claim, and update it if necessary.
+            $claim = array_shift( $entity['claims'][$property] );
+            if ( $claim['mainsnak']['datavalue']['value']['id'] == $itemId ) {
+                // Already is the required value, no need to change.
+                return;
+            }
+            $claim['mainsnak']['datavalue']['value']['id'] = $itemId;
+            $claim['mainsnak']['datavalue']['value']['numeric-id'] = $itemIdNumeric;
+            $apiParams = [
+                'action' => 'wbsetclaim',
+                'claim' => json_encode( $claim ),
+            ];
+        }
+
+        // If no claim was found (and modified) above, create a new claim.
+        if ( !isset( $apiParams ) ) {
+            $apiParams = [
+                'action' => 'wbcreateclaim',
+                'entity' => $this->getId(),
+                'property' => $property,
+                'snaktype' => 'value',
+                'value' => json_encode( [ 'entity-type' => 'item', 'numeric-id' => $itemIdNumeric ] ),
+            ];
+        }
+
+        // @TODO Save the property.
+
+        // Clear the cache.
+        $this->cache->deleteItem( $this->getEntityCacheKey( $this->id ) );
+    }
+
+    /**
+     * @param string $entityId
+     * @param string $propertyId
+     * @return array|bool
+     */
+    public function getPropertyOfTypeUrl( $entityId, $propertyId ) {
+        $entity = $this->getEntity( $entityId );
+        if ( !isset( $entity['claims'][$propertyId] ) ) {
+            return false;
+        }
+        $urls = [];
+        foreach ( $entity['claims'][$propertyId] as $claim ) {
+            $urls[] = $claim['mainsnak']['datavalue']['value'];
+        }
+
+        return $urls;
+    }
+
+    /**
+     * @param string $entityId
+     * @param string $propertyId
+     * @return array|bool
+     */
+    public function getPropertyOfTypeExternalIdentifier( $entityId, $propertyId ) {
+        $entity = $this->getEntity( $entityId );
+        if ( !isset( $entity['claims'][$propertyId] ) ) {
+            return false;
+        }
+        $idents = [];
+        foreach ( $entity['claims'][$propertyId] as $claim ) {
+            $qualifiers = [];
+            if ( !isset( $claim['qualifiers'] ) ) {
+                continue;
+            }
+            foreach ( $claim['qualifiers'] as $qualsInfo ) {
+                foreach ( $qualsInfo as $qualInfo ) {
+                    $qualProp = self::factory( $qualInfo['property'], $this->lang, $this->cache );
+                    $propLabel = $qualProp->getLabel();
+                    if ( !isset( $qualifiers[$propLabel] ) ) {
+                        $qualifiers[$propLabel] = [];
+                    }
+                    $qualifiers[$propLabel][] = $qualInfo['datavalue']['value'];
+                }
+            }
+            $idents[] = [
+                'qualifiers' => $qualifiers,
+                'value' => $claim['mainsnak']['datavalue']['value'],
+            ];
+        }
+
+        return $idents;
+    }
+
+    /**
+     * Get a single-valued text property.
+     * @param string $property One of the PROP_* constants.
+     * @return string|bool The value, or false if it can't be found.
+     */
+    public function getPropertyOfTypeText( $property ) {
+        $entity = $this->getEntity( $this->id );
+        if ( isset( $entity['claims'][$property] ) ) {
+            // Use the first title.
+            foreach ( $entity['claims'][$property] as $t ) {
+                if ( !isset( $t['mainsnak']['datavalue']['value']['language'] ) ) {
+                    var_dump( $t['mainsnak']['datavalue']['value'] );
+                    exit();
+                }
+                if ( $t['mainsnak']['datavalue']['value']['language'] == $this->lang
+                    && !empty( $t['mainsnak']['datavalue']['value']['text'] )
+                ) {
+                    return $t['mainsnak']['datavalue']['value']['text'];
+                }
+            }
+        }
+        return false;
+    }
+
+    /**
+     * Literal data field for a quantity that relates to some kind of well-defined unit.
+     * The actual unit goes in the data values that is entered.
+     *   - amount     – implicit part of the string (mapping of unit prefix is unclear)
+     *   - unit       – implicit part of the string that defaults to "1" (mapping to standardizing
+     *                  body is unclear)
+     *   - upperbound - quantity's upper bound
+     *   - lowerbound - quantity's lower bound
+     * @param string $property
+     * @return mixed[]|bool If it's not false it's an array with 'amount', 'unit', etc.
+     */
+    public function getPropertyOfTypeQuantity( $property ) {
+        $quantities = [];
+        $entity = $this->getEntity( $this->id );
+        if ( !isset( $entity['claims'][$property] ) ) {
+            return false;
+        }
+        foreach ( $entity['claims'][$property] as $t ) {
+            $quantity = $t['mainsnak']['datavalue']['value'];
+            $unitId = substr( $quantity['unit'], strlen( $this->wikidataUrlBase ) + 1 );
+            $quantity['unit'] = self::factory( $unitId, $this->lang, $this->cache );
+            $quantities[] = $quantity;
+        }
+        return $quantities;
+    }
+
+    /**
+     * Set a single-valued text property.
+     * @param string $property One of the PROP_* constants.
+     * @param string $value The value.
+     */
+    public function setPropertyOfTypeText( $property, $value ) {
+        // First see if this property already exists, and that it is different from what's being set.
+        $entity = $this->getEntity( $this->id );
+        if ( !empty( $entity['claims'][$property] ) ) {
+            // Find this language's claim (if there is one).
+            foreach ( $entity['claims'][$property] as $claim ) {
+                if ( $claim['mainsnak']['datavalue']['value']['language'] == $this->lang ) {
+                    // Modify this claim's text value.
+                    $titleClaim = $claim;
+                    $titleClaim['mainsnak']['datavalue']['value']['text'] = $value;
+                    $setTitleParams = [
+                        'action' => 'wbsetclaim',
+                        'claim' => \GuzzleHttp\json_encode( $titleClaim ),
+                    ];
+                    continue;
+                }
+            }
+        }
+
+        // If no claim was found (and modified) above, create a new claim.
+        if ( !isset( $setTitleParams ) ) {
+            $setTitleParams = [
+                'action' => 'wbcreateclaim',
+                'entity' => $this->getId(),
+                'property' => $property,
+                'snaktype' => 'value',
+                'value' => \GuzzleHttp\json_encode( [ 'text' => $value, 'language' => $this->lang ] ),
+            ];
+        }
+
+        // Save the property.
+        $wdWpOauth = new WdWpOauth();
+        $wdWpOauth->makeCall( $setTitleParams, true );
+
+        // Clear the cache.
+        $this->cache->deleteItem( $this->getEntityCacheKey( $this->id ) );
+    }
+
+    /**
+     * Does this item exist?
+     * @return bool
+     */
+    public function exists() {
+        return $this->getId() !== false;
+    }
+
+    /**
+     * @return array With 'html' and 'title' keys.
+     */
+    public function getWikipediaIntro() {
+        $cacheKey = 'wikipedia-intro-' . $this->id . $this->lang;
+        if ( $this->cache->hasItem( $cacheKey ) ) {
+            return $this->cache->getItem( $cacheKey )->get();
+        }
+        $entity = $this->getEntity( $this->id );
+        if ( !isset( $entity['sitelinks'] ) ) {
+            return [];
+        }
+        foreach ( $entity['sitelinks'] as $sitelink ) {
+            if ( $sitelink['site'] == $this->lang . 'wiki' ) {
+                $api = new MediawikiApi( 'https://' . $this->lang . '.wikipedia.org/w/api.php' );
+                $req = new SimpleRequest( 'query', [
+                    'prop' => 'extracts',
+                    'exintro' => true,
+                    'titles' => $sitelink['title'],
+                ] );
+                $response = $api->getRequest( $req );
+                $page = array_shift( $response['query']['pages'] );
+                $out = [
+                    'title' => $page['title'],
+                    'html' => $page['extract'],
+                ];
+                $cacheItem = $this->cache->getItem( $cacheKey )
+                    ->expiresAfter( new DateInterval( 'P1D' ) )
+                    ->set( $out );
+                $this->cache->save( $cacheItem );
+
+                return $out;
+            }
+        }
+
+        return [];
+    }
+
+    /**
+     * Get the raw entity data from the 'wbgetentities' API call.
+     * @param string|null $id The Q-number.
+     * @param bool $ignoreCache
+     * @return array|bool
+     */
+    public function getEntity( $id = null, $ignoreCache = false ) {
+        $idActual = $id ?: $this->id;
+        $cacheKey = $this->getEntityCacheKey( $idActual );
+        if ( !$ignoreCache && $this->cache->hasItem( $cacheKey ) ) {
+            return $this->cache->getItem( $cacheKey )->get();
+        }
+        $metadataRequest = new SimpleRequest( 'wbgetentities', [ 'ids' => $idActual ] );
+        $itemResult = $this->wdApi->getRequest( $metadataRequest );
+        if ( !isset( $itemResult['success'] ) || !isset( $itemResult['entities'][$id] ) ) {
+            return false;
+        }
+        $metadata = $itemResult['entities'][$idActual];
+        $cacheItem = $this->cache->getItem( $cacheKey )
+            ->expiresAfter( new DateInterval( 'PT10M' ) )
+            ->set( $metadata );
+        $this->cache->save( $cacheItem );
+        return $metadata;
+    }
+
+    /**
+     * @param string $id
+     *
+     * @return string
+     */
+    protected function getEntityCacheKey( $id ) {
+        return 'entities' . $id;
+    }
 }

Spacing +119 added lines, -119 removed lines

@@ -38,12 +38,12 @@ 
 	/** @var string The base URL of Wikidata, with trailing slash. */
 	protected $wikidataUrlBase = 'https://www.wikidata.org/wiki/';
 
-	private function __construct( $id, $lang, CacheItemPoolInterface $cache ) {
-		if ( !is_string( $id ) || preg_match( '/[QP][0-9]*/i', $id ) !== 1 ) {
-			throw new Exception( "Not a valid ID: " . var_export( $id, true ) );
+	private function __construct($id, $lang, CacheItemPoolInterface $cache) {
+		if (!is_string($id) || preg_match('/[QP][0-9]*/i', $id) !== 1) {
+			throw new Exception("Not a valid ID: " . var_export($id, true));
 		}
 		$this->id = $id;
-		$this->wdApi = new MediawikiApi( 'https://www.wikidata.org/w/api.php' );
+		$this->wdApi = new MediawikiApi('https://www.wikidata.org/w/api.php');
 		$this->entities = [];
 		$this->lang = $lang;
 		$this->cache = $cache;
@@ -57,21 +57,21 @@ 
 	 * @param CacheItemPoolInterface $cache The cache to use.
 	 * @return Item
 	 */
-	public static function factory( $id, $lang, CacheItemPoolInterface $cache ) {
-		$item = new Item( $id, $lang, $cache );
-		foreach ( $item->getPropertyOfTypeItem( self::PROP_INSTANCE_OF ) as $instanceOf ) {
+	public static function factory($id, $lang, CacheItemPoolInterface $cache) {
+		$item = new Item($id, $lang, $cache);
+		foreach ($item->getPropertyOfTypeItem(self::PROP_INSTANCE_OF) as $instanceOf) {
 			// Try to find a class matching the registered 'instance of'.
-			foreach ( static::$registeredClasses as $classId => $className ) {
+			foreach (static::$registeredClasses as $classId => $className) {
 				// If this 'instance of' is registered, use it.
-				if ( $classId === $instanceOf->getItem()->getId() ) {
+				if ($classId === $instanceOf->getItem()->getId()) {
 					// This won't re-request the metadata, because that's cached.
-					return new $className( $id, $lang, $cache );
+					return new $className($id, $lang, $cache);
 				}
 			}
 		}
 
 		// If we're here, just leave it as a basic Item.
-		$item->setCache( $cache );
+		$item->setCache($cache);
 		return $item;
 	}
 
@@ -81,19 +81,19 @@ 
 	 * @throws Exception if called on Item or the registering class does not have INSTANCE_OF set.
 	 */
 	public static function register() {
-		if ( static::class === self::class ) {
-			throw new Exception( __METHOD__ . ' should only be called on subclasses of Item' );
+		if (static::class === self::class) {
+			throw new Exception(__METHOD__ . ' should only be called on subclasses of Item');
 		}
-		if ( !static::INSTANCE_OF ) {
-			throw new Exception( 'Please set INSTANCE_OF for ' . static::class );
+		if (!static::INSTANCE_OF) {
+			throw new Exception('Please set INSTANCE_OF for ' . static::class);
 		}
-		static::$registeredClasses[ static::INSTANCE_OF ] = static::class;
+		static::$registeredClasses[static::INSTANCE_OF] = static::class;
 	}
 
 	/**
 	 * @param CacheItemPoolInterface $cache The cache to use.
 	 */
-	public function setCache( CacheItemPoolInterface $cache ) {
+	public function setCache(CacheItemPoolInterface $cache) {
 		$this->cache = $cache;
 	}
 
@@ -102,8 +102,8 @@ 
 	 * @return string|bool The ID or false if it couldn't be determined.
 	 */
 	public function getId() {
-		$entity = $this->getEntity( $this->id );
-		return isset( $entity['id'] ) ? $entity['id'] : false;
+		$entity = $this->getEntity($this->id);
+		return isset($entity['id']) ? $entity['id'] : false;
 	}
 
 	/**
@@ -111,10 +111,10 @@ 
 	 * @return string
 	 */
 	public function getLabel() {
-		$entity = $this->getEntity( $this->id );
-		if ( !empty( $entity['labels'][ $this->lang ]['value'] ) ) {
+		$entity = $this->getEntity($this->id);
+		if (!empty($entity['labels'][$this->lang]['value'])) {
 			// Use the label if there is one.
-			return $entity['labels'][ $this->lang ]['value'];
+			return $entity['labels'][$this->lang]['value'];
 		}
 		// Or just use the ID.
 		return $entity['id'];
@@ -139,33 +139,33 @@ 
 	 * @param string $type
 	 * @return array
 	 */
-	public function getStandardProperties( $wikiProject = 'WikiProject_Books', $type = 'work' ) {
-		if ( $type !== 'work' ) {
+	public function getStandardProperties($wikiProject = 'WikiProject_Books', $type = 'work') {
+		if ($type !== 'work') {
 			$type = 'edition';
 		}
 		$cacheKey = $type . '_item_property_IDs';
-		if ( $this->cache->hasItem( $cacheKey ) ) {
-			$propIds = $this->cache->getItem( $cacheKey )->get();
+		if ($this->cache->hasItem($cacheKey)) {
+			$propIds = $this->cache->getItem($cacheKey)->get();
 		} else {
 			$domCrawler = new Crawler();
 			$wikiProjectUrl = 'https://www.wikidata.org/wiki/Wikidata:' . $wikiProject;
-			$domCrawler->addHtmlContent( file_get_contents( $wikiProjectUrl ) );
-			$propAncors = "//h3/span[@id='" . ucfirst( $type ) . "_item_properties']"
+			$domCrawler->addHtmlContent(file_get_contents($wikiProjectUrl));
+			$propAncors = "//h3/span[@id='" . ucfirst($type) . "_item_properties']"
 				. "/../following-sibling::table[1]//td[2]/a";
-			$propCells = $domCrawler->filterXPath( $propAncors );
+			$propCells = $domCrawler->filterXPath($propAncors);
 			$propIds = [];
-			$propCells->each( function ( Crawler $node, $i ) use ( &$propIds ) {
+			$propCells->each(function(Crawler $node, $i) use (&$propIds) {
 				$propId = $node->text();
 				$propIds[] = $propId;
 			} );
-			$cacheItem = $this->cache->getItem( $cacheKey )
-				->expiresAfter( new DateInterval( 'PT1H' ) )
-				->set( $propIds );
-			$this->cache->save( $cacheItem );
+			$cacheItem = $this->cache->getItem($cacheKey)
+				->expiresAfter(new DateInterval('PT1H'))
+				->set($propIds);
+			$this->cache->save($cacheItem);
 		}
 		$workProperties = [];
-		foreach ( $propIds as $propId ) {
-			$workProperties[] = self::factory( $propId, $this->lang, $this->cache );
+		foreach ($propIds as $propId) {
+			$workProperties[] = self::factory($propId, $this->lang, $this->cache);
 		}
 
 		return $workProperties;
@@ -175,17 +175,17 @@ 
 	 * @param string $propertyId
 	 * @return bool|Time[]
 	 */
-	public function getPropertyOfTypeTime( $propertyId ) {
+	public function getPropertyOfTypeTime($propertyId) {
 		$times = [];
 		$entity = $this->getEntity();
-		if ( !isset( $entity['claims'][$propertyId] ) ) {
+		if (!isset($entity['claims'][$propertyId])) {
 			// No statements for this property.
 			return $times;
 		}
 		// print_r($entity['claims'][$propertyId]);exit();
-		foreach ( $entity['claims'][$propertyId] as $claim ) {
+		foreach ($entity['claims'][$propertyId] as $claim) {
 			// print_r($claim);
-			$times[] = new Time( $claim, $this->lang, $this->cache );
+			$times[] = new Time($claim, $this->lang, $this->cache);
 //
 // $timeValue = $claim['datavalue']['value']['time'];
 // // Ugly workaround for imprecise dates. :-(
@@ -207,14 +207,14 @@ 
 	 *
 	 * @return \Samwilson\SimpleWikidata\Properties\Item[]
 	 */
-	public function getPropertyOfTypeItem( $propertyId ) {
-		$entity = $this->getEntity( $this->id );
-		if ( !isset( $entity['claims'][$propertyId] ) ) {
+	public function getPropertyOfTypeItem($propertyId) {
+		$entity = $this->getEntity($this->id);
+		if (!isset($entity['claims'][$propertyId])) {
 			return [];
 		}
 		$items = [];
-		foreach ( $entity['claims'][$propertyId] as $claim ) {
-			$items[] = new Properties\Item( $claim, $this->lang, $this->cache );
+		foreach ($entity['claims'][$propertyId] as $claim) {
+			$items[] = new Properties\Item($claim, $this->lang, $this->cache);
 		}
 
 		return $items;
@@ -225,15 +225,15 @@ 
 	 * @param string $property Property ID, with 'P'.
 	 * @param string $itemId Item ID with 'Q'.
 	 */
-	public function setPropertyOfTypeItem( $property, $itemId ) {
-		$itemIdNumeric = substr( $itemId, 1 );
+	public function setPropertyOfTypeItem($property, $itemId) {
+		$itemIdNumeric = substr($itemId, 1);
 
 		// First see if this property already exists, and that it is different from what's being set.
-		$entity = $this->getEntity( $this->id );
-		if ( !empty( $entity['claims'][$property] ) ) {
+		$entity = $this->getEntity($this->id);
+		if (!empty($entity['claims'][$property])) {
 			// Get the first claim, and update it if necessary.
-			$claim = array_shift( $entity['claims'][$property] );
-			if ( $claim['mainsnak']['datavalue']['value']['id'] == $itemId ) {
+			$claim = array_shift($entity['claims'][$property]);
+			if ($claim['mainsnak']['datavalue']['value']['id'] == $itemId) {
 				// Already is the required value, no need to change.
 				return;
 			}
@@ -241,25 +241,25 @@ 
 			$claim['mainsnak']['datavalue']['value']['numeric-id'] = $itemIdNumeric;
 			$apiParams = [
 				'action' => 'wbsetclaim',
-				'claim' => json_encode( $claim ),
+				'claim' => json_encode($claim),
 			];
 		}
 
 		// If no claim was found (and modified) above, create a new claim.
-		if ( !isset( $apiParams ) ) {
+		if (!isset($apiParams)) {
 			$apiParams = [
 				'action' => 'wbcreateclaim',
 				'entity' => $this->getId(),
 				'property' => $property,
 				'snaktype' => 'value',
-				'value' => json_encode( [ 'entity-type' => 'item', 'numeric-id' => $itemIdNumeric ] ),
+				'value' => json_encode(['entity-type' => 'item', 'numeric-id' => $itemIdNumeric]),
 			];
 		}
 
 		// @TODO Save the property.
 
 		// Clear the cache.
-		$this->cache->deleteItem( $this->getEntityCacheKey( $this->id ) );
+		$this->cache->deleteItem($this->getEntityCacheKey($this->id));
 	}
 
 	/**
@@ -267,13 +267,13 @@ 
 	 * @param string $propertyId
 	 * @return array|bool
 	 */
-	public function getPropertyOfTypeUrl( $entityId, $propertyId ) {
-		$entity = $this->getEntity( $entityId );
-		if ( !isset( $entity['claims'][$propertyId] ) ) {
+	public function getPropertyOfTypeUrl($entityId, $propertyId) {
+		$entity = $this->getEntity($entityId);
+		if (!isset($entity['claims'][$propertyId])) {
 			return false;
 		}
 		$urls = [];
-		foreach ( $entity['claims'][$propertyId] as $claim ) {
+		foreach ($entity['claims'][$propertyId] as $claim) {
 			$urls[] = $claim['mainsnak']['datavalue']['value'];
 		}
 
@@ -285,22 +285,22 @@ 
 	 * @param string $propertyId
 	 * @return array|bool
 	 */
-	public function getPropertyOfTypeExternalIdentifier( $entityId, $propertyId ) {
-		$entity = $this->getEntity( $entityId );
-		if ( !isset( $entity['claims'][$propertyId] ) ) {
+	public function getPropertyOfTypeExternalIdentifier($entityId, $propertyId) {
+		$entity = $this->getEntity($entityId);
+		if (!isset($entity['claims'][$propertyId])) {
 			return false;
 		}
 		$idents = [];
-		foreach ( $entity['claims'][$propertyId] as $claim ) {
+		foreach ($entity['claims'][$propertyId] as $claim) {
 			$qualifiers = [];
-			if ( !isset( $claim['qualifiers'] ) ) {
+			if (!isset($claim['qualifiers'])) {
 				continue;
 			}
-			foreach ( $claim['qualifiers'] as $qualsInfo ) {
-				foreach ( $qualsInfo as $qualInfo ) {
-					$qualProp = self::factory( $qualInfo['property'], $this->lang, $this->cache );
+			foreach ($claim['qualifiers'] as $qualsInfo) {
+				foreach ($qualsInfo as $qualInfo) {
+					$qualProp = self::factory($qualInfo['property'], $this->lang, $this->cache);
 					$propLabel = $qualProp->getLabel();
-					if ( !isset( $qualifiers[$propLabel] ) ) {
+					if (!isset($qualifiers[$propLabel])) {
 						$qualifiers[$propLabel] = [];
 					}
 					$qualifiers[$propLabel][] = $qualInfo['datavalue']['value'];
@@ -320,17 +320,17 @@ 
 	 * @param string $property One of the PROP_* constants.
 	 * @return string|bool The value, or false if it can't be found.
 	 */
-	public function getPropertyOfTypeText( $property ) {
-		$entity = $this->getEntity( $this->id );
-		if ( isset( $entity['claims'][$property] ) ) {
+	public function getPropertyOfTypeText($property) {
+		$entity = $this->getEntity($this->id);
+		if (isset($entity['claims'][$property])) {
 			// Use the first title.
-			foreach ( $entity['claims'][$property] as $t ) {
-				if ( !isset( $t['mainsnak']['datavalue']['value']['language'] ) ) {
-					var_dump( $t['mainsnak']['datavalue']['value'] );
+			foreach ($entity['claims'][$property] as $t) {
+				if (!isset($t['mainsnak']['datavalue']['value']['language'])) {
+					var_dump($t['mainsnak']['datavalue']['value']);
 					exit();
 				}
-				if ( $t['mainsnak']['datavalue']['value']['language'] == $this->lang
-					&& !empty( $t['mainsnak']['datavalue']['value']['text'] )
+				if ($t['mainsnak']['datavalue']['value']['language'] == $this->lang
+					&& !empty($t['mainsnak']['datavalue']['value']['text'])
 				) {
 					return $t['mainsnak']['datavalue']['value']['text'];
 				}
@@ -350,16 +350,16 @@ 
 	 * @param string $property
 	 * @return mixed[]|bool If it's not false it's an array with 'amount', 'unit', etc.
 	 */
-	public function getPropertyOfTypeQuantity( $property ) {
+	public function getPropertyOfTypeQuantity($property) {
 		$quantities = [];
-		$entity = $this->getEntity( $this->id );
-		if ( !isset( $entity['claims'][$property] ) ) {
+		$entity = $this->getEntity($this->id);
+		if (!isset($entity['claims'][$property])) {
 			return false;
 		}
-		foreach ( $entity['claims'][$property] as $t ) {
+		foreach ($entity['claims'][$property] as $t) {
 			$quantity = $t['mainsnak']['datavalue']['value'];
-			$unitId = substr( $quantity['unit'], strlen( $this->wikidataUrlBase ) + 1 );
-			$quantity['unit'] = self::factory( $unitId, $this->lang, $this->cache );
+			$unitId = substr($quantity['unit'], strlen($this->wikidataUrlBase) + 1);
+			$quantity['unit'] = self::factory($unitId, $this->lang, $this->cache);
 			$quantities[] = $quantity;
 		}
 		return $quantities;
@@ -370,19 +370,19 @@ 
 	 * @param string $property One of the PROP_* constants.
 	 * @param string $value The value.
 	 */
-	public function setPropertyOfTypeText( $property, $value ) {
+	public function setPropertyOfTypeText($property, $value) {
 		// First see if this property already exists, and that it is different from what's being set.
-		$entity = $this->getEntity( $this->id );
-		if ( !empty( $entity['claims'][$property] ) ) {
+		$entity = $this->getEntity($this->id);
+		if (!empty($entity['claims'][$property])) {
 			// Find this language's claim (if there is one).
-			foreach ( $entity['claims'][$property] as $claim ) {
-				if ( $claim['mainsnak']['datavalue']['value']['language'] == $this->lang ) {
+			foreach ($entity['claims'][$property] as $claim) {
+				if ($claim['mainsnak']['datavalue']['value']['language'] == $this->lang) {
 					// Modify this claim's text value.
 					$titleClaim = $claim;
 					$titleClaim['mainsnak']['datavalue']['value']['text'] = $value;
 					$setTitleParams = [
 						'action' => 'wbsetclaim',
-						'claim' => \GuzzleHttp\json_encode( $titleClaim ),
+						'claim' => \GuzzleHttp\json_encode($titleClaim),
 					];
 					continue;
 				}
@@ -390,22 +390,22 @@ 
 		}
 
 		// If no claim was found (and modified) above, create a new claim.
-		if ( !isset( $setTitleParams ) ) {
+		if (!isset($setTitleParams)) {
 			$setTitleParams = [
 				'action' => 'wbcreateclaim',
 				'entity' => $this->getId(),
 				'property' => $property,
 				'snaktype' => 'value',
-				'value' => \GuzzleHttp\json_encode( [ 'text' => $value, 'language' => $this->lang ] ),
+				'value' => \GuzzleHttp\json_encode(['text' => $value, 'language' => $this->lang]),
 			];
 		}
 
 		// Save the property.
 		$wdWpOauth = new WdWpOauth();
-		$wdWpOauth->makeCall( $setTitleParams, true );
+		$wdWpOauth->makeCall($setTitleParams, true);
 
 		// Clear the cache.
-		$this->cache->deleteItem( $this->getEntityCacheKey( $this->id ) );
+		$this->cache->deleteItem($this->getEntityCacheKey($this->id));
 	}
 
 	/**
@@ -421,31 +421,31 @@ 
 	 */
 	public function getWikipediaIntro() {
 		$cacheKey = 'wikipedia-intro-' . $this->id . $this->lang;
-		if ( $this->cache->hasItem( $cacheKey ) ) {
-			return $this->cache->getItem( $cacheKey )->get();
+		if ($this->cache->hasItem($cacheKey)) {
+			return $this->cache->getItem($cacheKey)->get();
 		}
-		$entity = $this->getEntity( $this->id );
-		if ( !isset( $entity['sitelinks'] ) ) {
+		$entity = $this->getEntity($this->id);
+		if (!isset($entity['sitelinks'])) {
 			return [];
 		}
-		foreach ( $entity['sitelinks'] as $sitelink ) {
-			if ( $sitelink['site'] == $this->lang . 'wiki' ) {
-				$api = new MediawikiApi( 'https://' . $this->lang . '.wikipedia.org/w/api.php' );
-				$req = new SimpleRequest( 'query', [
+		foreach ($entity['sitelinks'] as $sitelink) {
+			if ($sitelink['site'] == $this->lang . 'wiki') {
+				$api = new MediawikiApi('https://' . $this->lang . '.wikipedia.org/w/api.php');
+				$req = new SimpleRequest('query', [
 					'prop' => 'extracts',
 					'exintro' => true,
 					'titles' => $sitelink['title'],
-				] );
-				$response = $api->getRequest( $req );
-				$page = array_shift( $response['query']['pages'] );
+				]);
+				$response = $api->getRequest($req);
+				$page = array_shift($response['query']['pages']);
 				$out = [
 					'title' => $page['title'],
 					'html' => $page['extract'],
 				];
-				$cacheItem = $this->cache->getItem( $cacheKey )
-					->expiresAfter( new DateInterval( 'P1D' ) )
-					->set( $out );
-				$this->cache->save( $cacheItem );
+				$cacheItem = $this->cache->getItem($cacheKey)
+					->expiresAfter(new DateInterval('P1D'))
+					->set($out);
+				$this->cache->save($cacheItem);
 
 				return $out;
 			}
@@ -460,22 +460,22 @@ 
 	 * @param bool $ignoreCache
 	 * @return array|bool
 	 */
-	public function getEntity( $id = null, $ignoreCache = false ) {
+	public function getEntity($id = null, $ignoreCache = false) {
 		$idActual = $id ?: $this->id;
-		$cacheKey = $this->getEntityCacheKey( $idActual );
-		if ( !$ignoreCache && $this->cache->hasItem( $cacheKey ) ) {
-			return $this->cache->getItem( $cacheKey )->get();
+		$cacheKey = $this->getEntityCacheKey($idActual);
+		if (!$ignoreCache && $this->cache->hasItem($cacheKey)) {
+			return $this->cache->getItem($cacheKey)->get();
 		}
-		$metadataRequest = new SimpleRequest( 'wbgetentities', [ 'ids' => $idActual ] );
-		$itemResult = $this->wdApi->getRequest( $metadataRequest );
-		if ( !isset( $itemResult['success'] ) || !isset( $itemResult['entities'][$id] ) ) {
+		$metadataRequest = new SimpleRequest('wbgetentities', ['ids' => $idActual]);
+		$itemResult = $this->wdApi->getRequest($metadataRequest);
+		if (!isset($itemResult['success']) || !isset($itemResult['entities'][$id])) {
 			return false;
 		}
 		$metadata = $itemResult['entities'][$idActual];
-		$cacheItem = $this->cache->getItem( $cacheKey )
-			->expiresAfter( new DateInterval( 'PT10M' ) )
-			->set( $metadata );
-		$this->cache->save( $cacheItem );
+		$cacheItem = $this->cache->getItem($cacheKey)
+			->expiresAfter(new DateInterval('PT10M'))
+			->set($metadata);
+		$this->cache->save($cacheItem);
 		return $metadata;
 	}
 
@@ -484,7 +484,7 @@ 
 	 *
 	 * @return string
 	 */
-	protected function getEntityCacheKey( $id ) {
+	protected function getEntityCacheKey($id) {
 		return 'entities' . $id;
 	}
 }

src/Property.php

Indentation +33 added lines, -33 removed lines

@@ -6,37 +6,37 @@
 
 abstract class Property {
 
-	/** @var string[] */
-	protected $claim;
-
-	/** @var string */
-	protected $lang;
-
-	/** @var CacheItemPoolInterface */
-	protected $cache;
-
-	/**
-	 * @param string $claim The claim data array.
-	 * @param string $lang The language code.
-	 * @param CacheItemPoolInterface $cache
-	 */
-	public function __construct( array $claim, $lang, CacheItemPoolInterface $cache ) {
-		$this->claim = $claim;
-		$this->lang = $lang;
-		$this->cache = $cache;
-	}
-
-	/**
-	 * @return Reference[]
-	 */
-	public function getReferences() {
-		$references = [];
-		if ( !isset( $this->claim['references'] ) ) {
-			return $references;
-		}
-		foreach ( $this->claim['references'] as $ref ) {
-			$references[] = new Reference( $ref, $this->lang, $this->cache );
-		}
-		return $references;
-	}
+    /** @var string[] */
+    protected $claim;
+
+    /** @var string */
+    protected $lang;
+
+    /** @var CacheItemPoolInterface */
+    protected $cache;
+
+    /**
+     * @param string $claim The claim data array.
+     * @param string $lang The language code.
+     * @param CacheItemPoolInterface $cache
+     */
+    public function __construct( array $claim, $lang, CacheItemPoolInterface $cache ) {
+        $this->claim = $claim;
+        $this->lang = $lang;
+        $this->cache = $cache;
+    }
+
+    /**
+     * @return Reference[]
+     */
+    public function getReferences() {
+        $references = [];
+        if ( !isset( $this->claim['references'] ) ) {
+            return $references;
+        }
+        foreach ( $this->claim['references'] as $ref ) {
+            $references[] = new Reference( $ref, $this->lang, $this->cache );
+        }
+        return $references;
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -20,7 +20,7 @@ 
 	 * @param string $lang The language code.
 	 * @param CacheItemPoolInterface $cache
 	 */
-	public function __construct( array $claim, $lang, CacheItemPoolInterface $cache ) {
+	public function __construct(array $claim, $lang, CacheItemPoolInterface $cache) {
 		$this->claim = $claim;
 		$this->lang = $lang;
 		$this->cache = $cache;
@@ -31,11 +31,11 @@ 
 	 */
 	public function getReferences() {
 		$references = [];
-		if ( !isset( $this->claim['references'] ) ) {
+		if (!isset($this->claim['references'])) {
 			return $references;
 		}
-		foreach ( $this->claim['references'] as $ref ) {
-			$references[] = new Reference( $ref, $this->lang, $this->cache );
+		foreach ($this->claim['references'] as $ref) {
+			$references[] = new Reference($ref, $this->lang, $this->cache);
 		}
 		return $references;
 	}

src/Items/Work.php

Indentation +75 added lines, -75 removed lines

@@ -11,94 +11,94 @@
  */
 class Work extends Item {
 
-	const ITEM_WORK = 'Q386724';
-	const PROP_SUBTITLE = 'P1680';
-	const PROP_GENRE = 'P136';
-	const PROP_SUBJECT = 'P921';
+    const ITEM_WORK = 'Q386724';
+    const PROP_SUBTITLE = 'P1680';
+    const PROP_GENRE = 'P136';
+    const PROP_SUBJECT = 'P921';
 
-	/**
-	 * @param string $lang ISO639 language code.
-	 * @param CacheItemPoolInterface $cache The cache.
-	 * @return array|Item[]
-	 */
-	public static function getBookTypes( $lang = 'en', $cache ) {
-		$sparql = "SELECT ?item WHERE {
+    /**
+     * @param string $lang ISO639 language code.
+     * @param CacheItemPoolInterface $cache The cache.
+     * @return array|Item[]
+     */
+    public static function getBookTypes( $lang = 'en', $cache ) {
+        $sparql = "SELECT ?item WHERE {
 			?item wdt:P279 wd:Q571 .
 			?item rdfs:label ?label .
 			FILTER(LANG(?label) = '$lang') .
 			} ORDER BY ?label ";
-		$query = new Query( $sparql, $lang );
-		$query->setCache( $cache );
-		$bookType = Item::factory( self::ITEM_WORK, $lang, $cache );
-		return [ $bookType ] + $query->getItems();
-	}
+        $query = new Query( $sparql, $lang );
+        $query->setCache( $cache );
+        $bookType = Item::factory( self::ITEM_WORK, $lang, $cache );
+        return [ $bookType ] + $query->getItems();
+    }
 
-	/**
-	 * @return bool|string
-	 */
-	public function getSubtitle() {
-		return $this->getPropertyOfTypeText( self::PROP_SUBTITLE );
-	}
+    /**
+     * @return bool|string
+     */
+    public function getSubtitle() {
+        return $this->getPropertyOfTypeText( self::PROP_SUBTITLE );
+    }
 
-	/**
-	 * @param string $subtitle The new subtitle.
-	 */
-	public function setSubtitle( $subtitle ) {
-		$this->setPropertyOfTypeText( self::PROP_SUBTITLE, $subtitle );
-	}
+    /**
+     * @param string $subtitle The new subtitle.
+     */
+    public function setSubtitle( $subtitle ) {
+        $this->setPropertyOfTypeText( self::PROP_SUBTITLE, $subtitle );
+    }
 
-	/**
-	 * @param string $property A property identifier.
-	 * @return array
-	 */
-	public function getPropertyOfTypeItems( $property ) {
-		$entity = $this->getEntity( $this->id );
-		if ( !isset( $entity['claims'][ $property ] ) ) {
-			return [];
-		}
-		$items = [];
-		foreach ( $entity['claims'][ $property ] as $authorClaim ) {
-			$item_id  = $authorClaim['mainsnak']['datavalue']['value']['id'];
-			$items[] = Item::factory( $item_id, $this->lang, $this->cache );
-		}
+    /**
+     * @param string $property A property identifier.
+     * @return array
+     */
+    public function getPropertyOfTypeItems( $property ) {
+        $entity = $this->getEntity( $this->id );
+        if ( !isset( $entity['claims'][ $property ] ) ) {
+            return [];
+        }
+        $items = [];
+        foreach ( $entity['claims'][ $property ] as $authorClaim ) {
+            $item_id  = $authorClaim['mainsnak']['datavalue']['value']['id'];
+            $items[] = Item::factory( $item_id, $this->lang, $this->cache );
+        }
 
-		return $items;
-	}
+        return $items;
+    }
 
-	/**
-	 * @return array
-	 */
-	public function getAuthors() {
-		return $this->getPropertyOfTypeItems( self::PROP_AUTHOR );
-	}
+    /**
+     * @return array
+     */
+    public function getAuthors() {
+        return $this->getPropertyOfTypeItems( self::PROP_AUTHOR );
+    }
 
-	/**
-	 * @return Item[]
-	 */
-	public function getSubjects() {
-		return $this->getPropertyOfTypeItems( self::PROP_SUBJECT );
-	}
+    /**
+     * @return Item[]
+     */
+    public function getSubjects() {
+        return $this->getPropertyOfTypeItems( self::PROP_SUBJECT );
+    }
 
-	/**
-	 * @return Item[]
-	 */
-	public function getEditions() {
-		$sparql = "SELECT ?item WHERE {"
-			. " ?item wdt:" . Edition::PROP_EDITION_OR_TRANSLATION_OF . " wd:" . $this->getId()
-			. "}";
-		$query = new Query( $sparql, $this->lang, $this->cache );
-		$editions = $query->getItems();
-		usort( $editions, function ( Item $a, Item $b ) {
-			if ( $a instanceof Edition && $b instanceof Edition ) {
-				return $a->getPublicationYear() - $b->getPublicationYear();
-			}
-			return 0;
-		} );
+    /**
+     * @return Item[]
+     */
+    public function getEditions() {
+        $sparql = "SELECT ?item WHERE {"
+            . " ?item wdt:" . Edition::PROP_EDITION_OR_TRANSLATION_OF . " wd:" . $this->getId()
+            . "}";
+        $query = new Query( $sparql, $this->lang, $this->cache );
+        $editions = $query->getItems();
+        usort( $editions, function ( Item $a, Item $b ) {
+            if ( $a instanceof Edition && $b instanceof Edition ) {
+                return $a->getPublicationYear() - $b->getPublicationYear();
+            }
+            return 0;
+        } );
 
-		return $editions;
-	}
+        return $editions;
+    }
 
-	public function newEdition() {
-	}
+    public function newEdition() {
+    }
 
 }

Spacing +19 added lines, -19 removed lines

@@ -21,45 +21,45 @@ 
 	 * @param CacheItemPoolInterface $cache The cache.
 	 * @return array|Item[]
 	 */
-	public static function getBookTypes( $lang = 'en', $cache ) {
+	public static function getBookTypes($lang = 'en', $cache) {
 		$sparql = "SELECT ?item WHERE {
 			?item wdt:P279 wd:Q571 .
 			?item rdfs:label ?label .
 			FILTER(LANG(?label) = '$lang') .
 			} ORDER BY ?label ";
-		$query = new Query( $sparql, $lang );
-		$query->setCache( $cache );
-		$bookType = Item::factory( self::ITEM_WORK, $lang, $cache );
-		return [ $bookType ] + $query->getItems();
+		$query = new Query($sparql, $lang);
+		$query->setCache($cache);
+		$bookType = Item::factory(self::ITEM_WORK, $lang, $cache);
+		return [$bookType] + $query->getItems();
 	}
 
 	/**
 	 * @return bool|string
 	 */
 	public function getSubtitle() {
-		return $this->getPropertyOfTypeText( self::PROP_SUBTITLE );
+		return $this->getPropertyOfTypeText(self::PROP_SUBTITLE);
 	}
 
 	/**
 	 * @param string $subtitle The new subtitle.
 	 */
-	public function setSubtitle( $subtitle ) {
-		$this->setPropertyOfTypeText( self::PROP_SUBTITLE, $subtitle );
+	public function setSubtitle($subtitle) {
+		$this->setPropertyOfTypeText(self::PROP_SUBTITLE, $subtitle);
 	}
 
 	/**
 	 * @param string $property A property identifier.
 	 * @return array
 	 */
-	public function getPropertyOfTypeItems( $property ) {
-		$entity = $this->getEntity( $this->id );
-		if ( !isset( $entity['claims'][ $property ] ) ) {
+	public function getPropertyOfTypeItems($property) {
+		$entity = $this->getEntity($this->id);
+		if (!isset($entity['claims'][$property])) {
 			return [];
 		}
 		$items = [];
-		foreach ( $entity['claims'][ $property ] as $authorClaim ) {
-			$item_id  = $authorClaim['mainsnak']['datavalue']['value']['id'];
-			$items[] = Item::factory( $item_id, $this->lang, $this->cache );
+		foreach ($entity['claims'][$property] as $authorClaim) {
+			$item_id = $authorClaim['mainsnak']['datavalue']['value']['id'];
+			$items[] = Item::factory($item_id, $this->lang, $this->cache);
 		}
 
 		return $items;
@@ -69,14 +69,14 @@ 
 	 * @return array
 	 */
 	public function getAuthors() {
-		return $this->getPropertyOfTypeItems( self::PROP_AUTHOR );
+		return $this->getPropertyOfTypeItems(self::PROP_AUTHOR);
 	}
 
 	/**
 	 * @return Item[]
 	 */
 	public function getSubjects() {
-		return $this->getPropertyOfTypeItems( self::PROP_SUBJECT );
+		return $this->getPropertyOfTypeItems(self::PROP_SUBJECT);
 	}
 
 	/**
@@ -86,10 +86,10 @@ 
 		$sparql = "SELECT ?item WHERE {"
 			. " ?item wdt:" . Edition::PROP_EDITION_OR_TRANSLATION_OF . " wd:" . $this->getId()
 			. "}";
-		$query = new Query( $sparql, $this->lang, $this->cache );
+		$query = new Query($sparql, $this->lang, $this->cache);
 		$editions = $query->getItems();
-		usort( $editions, function ( Item $a, Item $b ) {
-			if ( $a instanceof Edition && $b instanceof Edition ) {
+		usort($editions, function(Item $a, Item $b) {
+			if ($a instanceof Edition && $b instanceof Edition) {
 				return $a->getPublicationYear() - $b->getPublicationYear();
 			}
 			return 0;