ProfessionalWiki / ModernTimeline

src/ResultFacade/SubjectCollection.php

Spacing +2 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
 
 	private array $pages;
 
-	public function __construct( Subject ...$subjects ) {
+	public function __construct(Subject ...$subjects) {
 		$this->pages = $subjects;
 	}
 
@@ -14,7 +14,7 @@ 
 	 * @param Subject[] $subjects
 	 * @return self
 	 */
-	public static function newFromArray( array $subjects ): self {
+	public static function newFromArray(array $subjects): self {
 		$instance = new self();
 		$instance->pages = $subjects;
 		return $instance;

src/ResultFacade/ResultFormat.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace ModernTimeline\ResultFacade;
 
@@ -33,7 +33,7 @@ 
 	}
 
 	public function buildPresenter(): ResultPresenter {
-		return call_user_func( $this->constructionFunction );
+		return call_user_func($this->constructionFunction);
 	}
 
 }

src/ResultFacade/SimpleQueryResult.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace ModernTimeline\ResultFacade;
 

src/JsonBuilder.php

Spacing +16 added lines, -16 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace ModernTimeline;
 
@@ -19,32 +19,32 @@ 
 	 * @param Event[] $events
 	 * @return array
 	 */
-	public function eventsToTimelineJson( array $events ): array {
+	public function eventsToTimelineJson(array $events): array {
 		$jsonEvents = [];
 
-		foreach ( $events as $event ) {
-			if ( $event->getSubject()->getWikiPage()->getTitle() !== null ) {
-				$jsonEvents[] = $this->buildEvent( $event );
+		foreach ($events as $event) {
+			if ($event->getSubject()->getWikiPage()->getTitle() !== null) {
+				$jsonEvents[] = $this->buildEvent($event);
 			}
 		}
 
-		return [ 'events' => $jsonEvents ];
+		return ['events' => $jsonEvents];
 	}
 
-	public function buildEvent( Event $event ): array {
+	public function buildEvent(Event $event): array {
 		$jsonEvent = [
 			'text' => [
-				'headline' => $this->newHeadline( $event->getSubject()->getWikiPage()->getTitle() ),
-				'text' => $this->slidePresenter->getText( $event->getSubject() )
+				'headline' => $this->newHeadline($event->getSubject()->getWikiPage()->getTitle()),
+				'text' => $this->slidePresenter->getText($event->getSubject())
 			],
-			'start_date' => $this->timeToJson( $event->getStartDate() ),
+			'start_date' => $this->timeToJson($event->getStartDate()),
 		];
 
-		if ( $event->getEndDate() !== null ) {
-			$jsonEvent['end_date'] = $this->timeToJson( $event->getEndDate() );
+		if ($event->getEndDate() !== null) {
+			$jsonEvent['end_date'] = $this->timeToJson($event->getEndDate());
 		}
 
-		if ( $event->hasImage() ) {
+		if ($event->hasImage()) {
 			$jsonEvent['media'] = [
 				'url' => $event->getImageUrl(),
 				'thumbnail' => $event->getImageUrl()
@@ -54,17 +54,17 @@ 
 		return $jsonEvent;
 	}
 
-	private function newHeadline( \Title $title ): string {
+	private function newHeadline(\Title $title): string {
 		return \Html::element(
 			'a',
-			[ 'href' => $title->getFullURL() ],
+			['href' => $title->getFullURL()],
 			$title->getText()
 		);
 
 //		return DataValueFactory::getInstance()->newDataValueByItem( $subject->getWikiPage() )->getLongHTMLText( smwfGetLinker() );
 	}
 
-	private function timeToJson( SMWDITime $time ): array {
+	private function timeToJson(SMWDITime $time): array {
 		return [
 			'year' => $time->getYear(),
 			'month' => $time->getMonth(),

src/TimelinePresenter.php

Spacing +18 added lines, -18 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace ModernTimeline;
 
@@ -24,43 +24,43 @@ 
 		return 'modern_timeline_' . ++$timelineNumber;
 	}
 
-	public function presentResult( SimpleQueryResult $result ): string {
-		SMWOutputs::requireResource( 'ext.modern.timeline' );
+	public function presentResult(SimpleQueryResult $result): string {
+		SMWOutputs::requireResource('ext.modern.timeline');
 
 		SMWOutputs::requireHeadItem(
 			$this->id,
-			$this->createJs( $this->createJsonString( $result ) )
+			$this->createJs($this->createJsonString($result))
 		);
 
-		return $this->createDiv( $result->getParameters() );
+		return $this->createDiv($result->getParameters());
 	}
 
-	private function createJsonString( SimpleQueryResult $result ): string {
-		$preJson = $this->newJsonBuilder( $result )
-			->eventsToTimelineJson( ( new EventExtractor( $result->getParameters() ) )->extractEvents( $result->getSubjects() ) );
+	private function createJsonString(SimpleQueryResult $result): string {
+		$preJson = $this->newJsonBuilder($result)
+			->eventsToTimelineJson((new EventExtractor($result->getParameters()))->extractEvents($result->getSubjects()));
 
-		$preJson['options'] = TimelineOptions::processedParamsToJson( $result->getParameters() );
+		$preJson['options'] = TimelineOptions::processedParamsToJson($result->getParameters());
 
-		return json_encode( $preJson );
+		return json_encode($preJson);
 	}
 
-	private function newJsonBuilder( SimpleQueryResult $result ): JsonBuilder {
+	private function newJsonBuilder(SimpleQueryResult $result): JsonBuilder {
 		return new JsonBuilder(
-			$this->getSlidePresenter( $result )
+			$this->getSlidePresenter($result)
 		);
 	}
 
-	private function getSlidePresenter( SimpleQueryResult $result ): SlidePresenter {
+	private function getSlidePresenter(SimpleQueryResult $result): SlidePresenter {
 		$templateName = $result->getParameters()['template'];
 
-		if ( $templateName === '' ) {
-			return new SimpleSlidePresenter( $result->getProcessingResult()->getParameterArray() );
+		if ($templateName === '') {
+			return new SimpleSlidePresenter($result->getProcessingResult()->getParameterArray());
 		}
 
-		return new TemplateSlidePresenter( $templateName );
+		return new TemplateSlidePresenter($templateName);
 	}
 
-	private function createJs( string $json ): string {
+	private function createJs(string $json): string {
 		return \Html::rawElement(
 			'script',
 			[
@@ -71,7 +71,7 @@ 
 		);
 	}
 
-	private function createDiv( array $parameters ): string {
+	private function createDiv(array $parameters): string {
 		$width = $parameters[TimelineOptions::PARAM_WIDTH];
 		$height = $parameters[TimelineOptions::PARAM_HEIGHT];
 

src/SlidePresenter/TemplateSlidePresenter.php

Spacing +13 added lines, -13 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace ModernTimeline\SlidePresenter;
 
@@ -16,11 +16,11 @@ 
 	) {
 	}
 
-	public function getText( Subject $subject ): string {
+	public function getText(Subject $subject): string {
 		$parser = $this->getParser();
 
 		return $parser->recursiveTagParseFully(
-			( new TemplateSlidePresenter( $this->templateName ) )->getTemplateText( $subject )
+			(new TemplateSlidePresenter($this->templateName))->getTemplateText($subject)
 		);
 	}
 
@@ -28,21 +28,21 @@ 
 		return MediaWikiServices::getInstance()->getParser();
 	}
 
-	public function getTemplateText( Subject $subject ): string {
-		return '{{' . implode( '|', $this->getTemplateSegments( $subject ) ) . '}}';
+	public function getTemplateText(Subject $subject): string {
+		return '{{' . implode('|', $this->getTemplateSegments($subject)) . '}}';
 	}
 
-	private function getTemplateSegments( Subject $subject ): array {
+	private function getTemplateSegments(Subject $subject): array {
 		return array_merge(
 			[
 				$this->templateName,
-				$this->parameter( 'title', $subject->getWikiPage()->getTitle()->getFullText() )
+				$this->parameter('title', $subject->getWikiPage()->getTitle()->getFullText())
 			],
 			array_map(
-				function( PropertyValueCollection $pvc ) {
+				function(PropertyValueCollection $pvc) {
 					return $this->parameter(
-						$pvc->getPrintRequest()->getText( null ) ?? '',
-						$pvc->getDataItems() === [] ? '' : $this->dataItemToText( $pvc->getDataItems()[0] )
+						$pvc->getPrintRequest()->getText(null) ?? '',
+						$pvc->getDataItems() === [] ? '' : $this->dataItemToText($pvc->getDataItems()[0])
 					);
 				},
 				$subject->getPropertyValueCollections()->toArray()
@@ -50,11 +50,11 @@ 
 		);
 	}
 
-	private function dataItemToText( \SMWDataItem $dataItem ): string {
-		return DataValueFactory::getInstance()->newDataValueByItem( $dataItem )->getLongHTMLText();
+	private function dataItemToText(\SMWDataItem $dataItem): string {
+		return DataValueFactory::getInstance()->newDataValueByItem($dataItem)->getLongHTMLText();
 	}
 
-	private function parameter( string $name, string $value ): string {
+	private function parameter(string $name, string $value): string {
 		return $name . '=' . $value;
 	}
 }

src/SlidePresenter/SimpleSlidePresenter.php

Spacing +16 added lines, -16 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace ModernTimeline\SlidePresenter;
 
@@ -16,37 +16,37 @@ 
 	) {
 	}
 
-	public function getText( Subject $subject ): string {
-		return implode( '<br>', iterator_to_array( $this->getDisplayValues( $subject ) ) );
+	public function getText(Subject $subject): string {
+		return implode('<br>', iterator_to_array($this->getDisplayValues($subject)));
 	}
 
-	private function getDisplayValues( Subject $subject ): Traversable {
-		foreach ( $subject->getPropertyValueCollections()->toArray() as $propertyValues ) {
-			foreach ( $propertyValues->getDataItems() as $dataItem ) {
-				if ( !$this->isHiddenPrintRequest( $propertyValues->getPrintRequest() ) ) {
-					yield $this->getDisplayValue( $propertyValues->getPrintRequest(), $dataItem );
+	private function getDisplayValues(Subject $subject): Traversable {
+		foreach ($subject->getPropertyValueCollections()->toArray() as $propertyValues) {
+			foreach ($propertyValues->getDataItems() as $dataItem) {
+				if (!$this->isHiddenPrintRequest($propertyValues->getPrintRequest())) {
+					yield $this->getDisplayValue($propertyValues->getPrintRequest(), $dataItem);
 				}
 			}
 		}
 	}
 
-	private function isHiddenPrintRequest( PrintRequest $pr ): bool {
-		return $pr->getText( null ) === $this->parameters['image property'];
+	private function isHiddenPrintRequest(PrintRequest $pr): bool {
+		return $pr->getText(null) === $this->parameters['image property'];
 	}
 
-	private function getDisplayValue( PrintRequest $pr, \SMWDataItem $dataItem ): string {
-		$property = $pr->getText( null );
-		$value = $this->dataItemToText( $dataItem );
+	private function getDisplayValue(PrintRequest $pr, \SMWDataItem $dataItem): string {
+		$property = $pr->getText(null);
+		$value = $this->dataItemToText($dataItem);
 
-		if ( $property === '' ) {
+		if ($property === '') {
 			return $value;
 		}
 
 		return $property . ': ' . $value;
 	}
 
-	private function dataItemToText( \SMWDataItem $dataItem ): string {
-		return DataValueFactory::getInstance()->newDataValueByItem( $dataItem )->getLongHTMLText();
+	private function dataItemToText(\SMWDataItem $dataItem): string {
+		return DataValueFactory::getInstance()->newDataValueByItem($dataItem)->getLongHTMLText();
 	}
 
 }

src/ModernTimelineSetup.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace ModernTimeline;
 
@@ -13,10 +13,10 @@ 
 	}
 
 	private static function doSmwCheck(): void {
-		if ( !defined( 'SMW_VERSION' ) ) {
+		if (!defined('SMW_VERSION')) {
 
-			if ( PHP_SAPI === 'cli' || PHP_SAPI === 'phpdbg' ) {
-				die( "\nThe 'Modern Timeline' extension requires 'Semantic MediaWiki' to be installed and enabled.\n" );
+			if (PHP_SAPI === 'cli' || PHP_SAPI === 'phpdbg') {
+				die("\nThe 'Modern Timeline' extension requires 'Semantic MediaWiki' to be installed and enabled.\n");
 			}
 
 			die(

src/ModernTimelinePrinter.php

Spacing +22 added lines, -22 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace ModernTimeline;
 
@@ -25,23 +25,23 @@ 
 		$registry = new ResultFormatRegistry();
 
 		$registry->newFormat()
-			->withName( 'moderntimeline' )
-			->andMessageKey( 'modern-timeline-format-name' )
-			->andParameterDefinitions( TimelineOptions::getTimelineParameterDefinitions() )
-			->andPresenterBuilder( function() {
+			->withName('moderntimeline')
+			->andMessageKey('modern-timeline-format-name')
+			->andParameterDefinitions(TimelineOptions::getTimelineParameterDefinitions())
+			->andPresenterBuilder(function() {
 				return new TimelinePresenter();
 			} )
 			->register();
 
-		$this->format = $registry->getFormatByName( 'moderntimeline' );
+		$this->format = $registry->getFormatByName('moderntimeline');
 	}
 
 	public function getName(): string {
-		return wfMessage( $this->format->getNameMessageKey() )->text();
+		return wfMessage($this->format->getNameMessageKey())->text();
 	}
 
-	public function getParamDefinitions( array $definitions ) {
-		return array_merge( $definitions, $this->format->getParameterDefinitions() );
+	public function getParamDefinitions(array $definitions) {
+		return array_merge($definitions, $this->format->getParameterDefinitions());
 	}
 
 	/**
@@ -51,36 +51,36 @@ 
 	 *
 	 * @return string
 	 */
-	public function getResult( QueryResult $result, array $parameters, $outputMode ): string {
+	public function getResult(QueryResult $result, array $parameters, $outputMode): string {
 		return $this->format->buildPresenter()->presentResult(
 			new SimpleQueryResult(
-				$this->simplifyResult( $result ),
-				$this->newProcessingResultFromParams( $parameters )
+				$this->simplifyResult($result),
+				$this->newProcessingResultFromParams($parameters)
 			)
 		);
 	}
 
-	private function simplifyResult( QueryResult $result ): SubjectCollection {
-		return ( new ResultSimplifier() )->newSubjectCollection( $result );
+	private function simplifyResult(QueryResult $result): SubjectCollection {
+		return (new ResultSimplifier())->newSubjectCollection($result);
 	}
 
 	/**
 	 * This is code copied over from ParamProcessor to go from the deprecated Param[] to ProcessingResult.
 	 * Once the main ResultPrinter interface has been migrated away from Param this can be removed.
 	 */
-	private function newProcessingResultFromParams( array $params ): ProcessingResult {
+	private function newProcessingResultFromParams(array $params): ProcessingResult {
 		$parameters = [];
 
-		foreach ( $params as $param ) {
+		foreach ($params as $param) {
 			$processedParam = new ProcessedParam(
 				$param->getName(),
 				$param->getValue(),
 				$param->wasSetToDefault()
 			);
 
-			if ( !$param->wasSetToDefault() ) {
-				$processedParam->setOriginalName( $param->getOriginalName() );
-				$processedParam->setOriginalValue( $param->getOriginalValue() );
+			if (!$param->wasSetToDefault()) {
+				$processedParam->setOriginalName($param->getOriginalName());
+				$processedParam->setOriginalValue($param->getOriginalValue());
 			}
 
 			$parameters[$processedParam->getName()] = $processedParam;
@@ -92,11 +92,11 @@ 
 		);
 	}
 
-	public function getQueryMode( $context ): int {
+	public function getQueryMode($context): int {
 		return SMWQuery::MODE_INSTANCES;
 	}
 
-	public function setShowErrors( $show ): void {
+	public function setShowErrors($show): void {
 	}
 
 	public function isExportFormat(): bool {
@@ -115,6 +115,6 @@ 
 		return false;
 	}
 
-	public function setRecursiveTextProcessor( RecursiveTextProcessor $recursiveTextProcessor ): void {
+	public function setRecursiveTextProcessor(RecursiveTextProcessor $recursiveTextProcessor): void {
 	}
 }