SemanticMediaWiki / SemanticCite

src/CitationResourceMatchFinder.php

Spacing +31 added lines, -31 removed lines

@@ -38,7 +38,7 @@ 
 	 *
 	 * @param Store $store
 	 */
-	public function __construct( Store $store ) {
+	public function __construct(Store $store) {
 		$this->store = $store;
 		$this->dataValueFactory = DataValueFactory::getInstance();
 	}
@@ -51,11 +51,11 @@ 
 	 *
 	 * @return array
 	 */
-	public function findCitationResourceLinks( array $subjects, $linkClass = '', $caption = '' ) {
+	public function findCitationResourceLinks(array $subjects, $linkClass = '', $caption = '') {
 
 		$citationResourceLinks = [];
 
-		foreach ( $subjects as $subject ) {
+		foreach ($subjects as $subject) {
 
 			$dataValue = $this->dataValueFactory->newDataItemValue(
 				$subject,
@@ -84,21 +84,21 @@ 
 	 *
 	 * @return array
 	 */
-	public function findMatchForResourceIdentifierTypeToValue( $type, $id = null ) {
+	public function findMatchForResourceIdentifierTypeToValue($type, $id = null) {
 
-		if ( $id === null || $id === '' ) {
+		if ($id === null || $id === '') {
 			return [];
 		}
 
 		$resourceIdentifierFactory = new ResourceIdentifierFactory();
 
-		$resourceIdentifierStringValue = $resourceIdentifierFactory->newResourceIdentifierStringValueForType( $type );
-		$resourceIdentifierStringValue->setUserValue( $id );
+		$resourceIdentifierStringValue = $resourceIdentifierFactory->newResourceIdentifierStringValueForType($type);
+		$resourceIdentifierStringValue->setUserValue($id);
 		$id = $resourceIdentifierStringValue->getWikiValue();
 
 		$description = new SomeProperty(
 			$resourceIdentifierStringValue->getProperty(),
-			new ValueDescription( new DIBlob( $id ) )
+			new ValueDescription(new DIBlob($id))
 		);
 
 		$query = new Query(
@@ -108,17 +108,17 @@ 
 		);
 
 		$query->querymode = Query::MODE_INSTANCES;
-		$query->setLimit( 10 );
+		$query->setLimit(10);
 
-		if ( defined( 'SMWQuery::NO_CACHE' ) ) {
-			$query->setOption( Query::NO_CACHE, true );
+		if (defined('SMWQuery::NO_CACHE')) {
+			$query->setOption(Query::NO_CACHE, true);
 		}
 
-		if ( defined( 'SMWQuery::PROC_CONTEXT' ) ) {
-			$query->setOption( Query::PROC_CONTEXT, 'SCI.CitationResourceMatchFinder' );
+		if (defined('SMWQuery::PROC_CONTEXT')) {
+			$query->setOption(Query::PROC_CONTEXT, 'SCI.CitationResourceMatchFinder');
 		}
 
-		return $this->store->getQueryResult( $query )->getResults();
+		return $this->store->getQueryResult($query)->getResults();
 	}
 
 	/**
@@ -128,7 +128,7 @@ 
 	 *
 	 * @return array
 	 */
-	public function findCitationTextFor( $citationReference ) {
+	public function findCitationTextFor($citationReference) {
 
 		$text = '';
 		$subjects = [];
@@ -137,12 +137,12 @@ 
 			$citationReference
 		);
 
-		if ( !$queryResult instanceof QueryResult ) {
-			return [ $subjects, $text ];
+		if (!$queryResult instanceof QueryResult) {
+			return [$subjects, $text];
 		}
 
-		while ( $resultArray = $queryResult->getNext() ) {
-			foreach ( $resultArray as $result ) {
+		while ($resultArray = $queryResult->getNext()) {
+			foreach ($resultArray as $result) {
 
 				// Collect all subjects for the same reference because it can happen
 				// that the same reference key is used for different citation
@@ -150,13 +150,13 @@ 
 				// text but return all subjects to make it easier to find them later
 				$subjects[] = $result->getResultSubject();
 
-				while ( ( $dataValue = $result->getNextDataValue() ) !== false ) {
+				while (($dataValue = $result->getNextDataValue()) !== false) {
 					$text = $dataValue->getShortWikiText();
 				}
 			}
 		}
 
-		return [ $subjects, $text ];
+		return [$subjects, $text];
 	}
 
 	/**
@@ -168,20 +168,20 @@ 
 	 *
 	 * @return QueryResult
 	 */
-	public function findMatchForCitationReference( $citationReference ) {
+	public function findMatchForCitationReference($citationReference) {
 
 		$description = new SomeProperty(
-			new DIProperty( PropertyRegistry::SCI_CITE_KEY ),
-			new ValueDescription( new DIBlob( $citationReference ) )
+			new DIProperty(PropertyRegistry::SCI_CITE_KEY),
+			new ValueDescription(new DIBlob($citationReference))
 		);
 
-		$propertyValue = $this->dataValueFactory->newDataValueByType( '__pro' );
+		$propertyValue = $this->dataValueFactory->newDataValueByType('__pro');
 		$propertyValue->setDataItem(
-			new DIProperty( PropertyRegistry::SCI_CITE_TEXT )
+			new DIProperty(PropertyRegistry::SCI_CITE_TEXT)
 		);
 
 		$description->addPrintRequest(
-			new PrintRequest( PrintRequest::PRINT_PROP, null, $propertyValue )
+			new PrintRequest(PrintRequest::PRINT_PROP, null, $propertyValue)
 		);
 
 		$query = new Query(
@@ -191,13 +191,13 @@ 
 		);
 
 		$query->querymode = Query::MODE_INSTANCES;
-		$query->setLimit( 10 );
+		$query->setLimit(10);
 
-		if ( defined( 'SMWQuery::PROC_CONTEXT' ) ) {
-			$query->setOption( Query::PROC_CONTEXT, 'SCI.CitationResourceMatchFinder' );
+		if (defined('SMWQuery::PROC_CONTEXT')) {
+			$query->setOption(Query::PROC_CONTEXT, 'SCI.CitationResourceMatchFinder');
 		}
 
-		return $this->store->getQueryResult( $query );
+		return $this->store->getQueryResult($query);
 	}
 
 }

src/CitationTextChangeUpdateJobDispatcher.php

Spacing +21 added lines, -21 removed lines

@@ -41,7 +41,7 @@ 
 	 * @param Store $store
 	 * @param ReferenceBacklinksLookup $referenceBacklinksLookup
 	 */
-	public function __construct( Store $store, ReferenceBacklinksLookup $referenceBacklinksLookup ) {
+	public function __construct(Store $store, ReferenceBacklinksLookup $referenceBacklinksLookup) {
 		$this->store = $store;
 		$this->referenceBacklinksLookup = $referenceBacklinksLookup;
 	}
@@ -51,7 +51,7 @@ 
 	 *
 	 * @param boolean $enabledUpdateJobState
 	 */
-	public function setEnabledUpdateJobState( $enabledUpdateJobState ) {
+	public function setEnabledUpdateJobState($enabledUpdateJobState) {
 		$this->enabledUpdateJobState = $enabledUpdateJobState;
 	}
 
@@ -63,21 +63,21 @@ 
 	 *
 	 * @return boolean
 	 */
-	public function dispatchUpdateJobFor( DIWikiPage $subject, ChangeOp $changeOp ) {
+	public function dispatchUpdateJobFor(DIWikiPage $subject, ChangeOp $changeOp) {
 
-		if ( !$this->enabledUpdateJobState ) {
+		if (!$this->enabledUpdateJobState) {
 			return false;
 		}
 
 		$tableName = $this->store->getPropertyTableInfoFetcher()->findTableIdForProperty(
-			new DIProperty( PropertyRegistry::SCI_CITE_TEXT )
+			new DIProperty(PropertyRegistry::SCI_CITE_TEXT)
 		);
 
 		$subjectIdList = $this->getSubjectListFrom(
-			$changeOp->getOrderedDiffByTable( $tableName )
+			$changeOp->getOrderedDiffByTable($tableName)
 		);
 
-		if ( ( $jobList = $this->getDispatchableTargetList( $subjectIdList ) ) === [] ) {
+		if (($jobList = $this->getDispatchableTargetList($subjectIdList)) === []) {
 			return false;
 		}
 
@@ -93,30 +93,30 @@ 
 		return true;
 	}
 
-	private function getSubjectListFrom( array $orderedDiff ) {
+	private function getSubjectListFrom(array $orderedDiff) {
 
 		$subjectIdList = [];
 
 		// Find out whether a cite text object was altered
-		foreach ( $orderedDiff as $key => $value ) {
+		foreach ($orderedDiff as $key => $value) {
 
-			if ( strpos( $key, 'sci_cite_text' ) === false ) {
+			if (strpos($key, 'sci_cite_text') === false) {
 				continue;
 			}
 
-			if ( !isset( $value['delete'] ) ) {
+			if (!isset($value['delete'])) {
 				$value['delete'] = [];
 			}
 
-			foreach ( $value['delete'] as $delete ) {
+			foreach ($value['delete'] as $delete) {
 				$subjectIdList[] = $delete['s_id'];
 			}
 
-			if ( !isset( $value['insert'] ) ) {
+			if (!isset($value['insert'])) {
 				$value['insert'] = [];
 			}
 
-			foreach ( $value['insert'] as $insert ) {
+			foreach ($value['insert'] as $insert) {
 				$subjectIdList[] = $insert['s_id'];
 			}
 		}
@@ -124,25 +124,25 @@ 
 		return $subjectIdList;
 	}
 
-	private function getDispatchableTargetList( array $subjectIdList ) {
+	private function getDispatchableTargetList(array $subjectIdList) {
 
-		if ( $subjectIdList === [] ) {
+		if ($subjectIdList === []) {
 			return [];
 		}
 
-		$hashList = $this->store->getObjectIds()->getDataItemPoolHashListFor( $subjectIdList );
+		$hashList = $this->store->getObjectIds()->getDataItemPoolHashListFor($subjectIdList);
 		$referenceBacklinks = [];
 
-		foreach ( $hashList as $hash ) {
+		foreach ($hashList as $hash) {
 			$referenceBacklinks += $this->referenceBacklinksLookup->findReferenceBacklinksFor(
-				$this->referenceBacklinksLookup->tryToFindCitationKeyFor( DIWikiPage::doUnserialize( $hash ) )
+				$this->referenceBacklinksLookup->tryToFindCitationKeyFor(DIWikiPage::doUnserialize($hash))
 			);
 		}
 
 		$targetBatch = [];
 
-		foreach ( $referenceBacklinks as $referenceBacklink ) {
-			$targetBatch[HashBuilder::getHashIdForDiWikiPage( $referenceBacklink )] = true;
+		foreach ($referenceBacklinks as $referenceBacklink) {
+			$targetBatch[HashBuilder::getHashIdForDiWikiPage($referenceBacklink)] = true;
 		}
 
 		return $targetBatch;