samwilson / simple-wikidata

src/Item.php

Spacing +118 added lines, -118 removed lines

@@ -33,12 +33,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;
@@ -52,24 +52,24 @@ 
 	 *
 	 * @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 mating the 'instance of' name.
-			$possibleClassName = __NAMESPACE__ . '\\Items\\' . Str::toCamelCase( $instanceOf->getItem()->getLabel() );
-			if ( class_exists( $possibleClassName ) ) {
+			$possibleClassName = __NAMESPACE__ . '\\Items\\' . Str::toCamelCase($instanceOf->getItem()->getLabel());
+			if (class_exists($possibleClassName)) {
 				// This won't re-request the metadata, because that's cached.
-				$specificItem = new $possibleClassName( $id, $lang, $cache );
+				$specificItem = new $possibleClassName($id, $lang, $cache);
 				return $specificItem;
 			}
 		}
 
 		// If we're here, just leave it as a basic Item.
-		$item->setCache( $cache );
+		$item->setCache($cache);
 		return $item;
 	}
 
-	public function setCache( CacheItemPoolInterface $cache_item_pool ) {
+	public function setCache(CacheItemPoolInterface $cache_item_pool) {
 		$this->cache = $cache_item_pool;
 	}
 
@@ -78,8 +78,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;
 	}
 
 	/**
@@ -87,17 +87,17 @@ 
 	 * @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'];
 	}
 
 	public function getWikidataUrl() {
-		return $this->wikidataUrlBase.$this->id;
+		return $this->wikidataUrlBase . $this->id;
 	}
 
 	/**
@@ -111,32 +111,32 @@ 
 	 * @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']/../following-sibling::table[1]//td[2]/a";
-			$propCells = $domCrawler->filterXPath( $propAncors );
+			$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 ) {
+			$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;
@@ -146,17 +146,17 @@ 
 	 * @param string $propertyId
 	 * @return bool|Time[]
 	 */
-	protected function getPropertyOfTypeTime( $propertyId ) {
+	protected 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. :-(
@@ -178,28 +178,28 @@ 
 	 *
 	 * @return \Samwilson\SimpleWikidata\Properties\Item[]
 	 */
-	protected function getPropertyOfTypeItem( $propertyId ) {
-		$entity = $this->getEntity( $this->id );
-		if ( !isset( $entity['claims'][$propertyId] ) ) {
+	protected 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;
 	}
 
-	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;
 			}
@@ -207,58 +207,58 @@ 
 			$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]),
 			];
 		}
 
 		// Save the property.
 		$wdWpOauth = new WdOauth();
-		$wdWpOauth->makeCall( $apiParams, true );
+		$wdWpOauth->makeCall($apiParams, true);
 
 		// Clear the cache.
-		$this->cache->deleteItem( $this->getEntityCacheKey( $this->id ) );
+		$this->cache->deleteItem($this->getEntityCacheKey($this->id));
 	}
 
-	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'];
 		}
 
 		return $urls;
 	}
 
-	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'];
@@ -278,17 +278,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'];
 				}
@@ -306,16 +306,16 @@ 
 	 * @param $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;
@@ -326,19 +326,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;
 				}
@@ -346,27 +346,27 @@ 
 		}
 
 		// 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));
 	}
 
 	public function getInstanceOf() {
-		$instancesOf = $this->getPropertyOfTypeItem( $this->getId(), self::PROP_INSTANCE_OF );
-		return array_shift( $instancesOf );
+		$instancesOf = $this->getPropertyOfTypeItem($this->getId(), self::PROP_INSTANCE_OF);
+		return array_shift($instancesOf);
 	}
 
 	/**
@@ -379,31 +379,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;
 			}
@@ -418,22 +418,22 @@ 
 	 * @param bool $ignoreCache
 	 * @return 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;
 	}
 
@@ -442,7 +442,7 @@ 
 	 *
 	 * @return string
 	 */
-	protected function getEntityCacheKey( $id ) {
+	protected function getEntityCacheKey($id) {
 		return 'entities' . $id;
 	}
 }

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 EditionItem and $b instanceof EditionItem ) {
+		usort($editions, function(Item $a, Item $b) {
+			if ($a instanceof EditionItem and $b instanceof EditionItem) {
 				return $a->getPublicationYear() - $b->getPublicationYear();
 			}
 			return 0;