samwilson / simple-wikidata

src/Properties/Time.php

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

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

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

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

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

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

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

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

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;