wmde / FundraisingFrontend

src/Infrastructure/ConfigReader.php

Braces +1 added lines, -2 removed lines

@@ -62,8 +62,7 @@
 	private function getFileContents( string $filePath ): string {
 		try {
 			return $this->fileFetcher->fetchFile( $filePath );
-		}
-		catch ( FileFetchingException $ex ) {
+		} catch ( FileFetchingException $ex ) {
 			throw new RuntimeException( 'Cannot read config file at path "' . $filePath . '"', 0, $ex );
 		}
 	}

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace WMDE\Fundraising\Frontend\Infrastructure;
 
@@ -41,7 +41,7 @@ 
 		}
 
 		$configs = array_map(
-			function ( string $path ) {
+			function( string $path ) {
 				return $this->getFileConfig( $path );
 			},
 			$this->configPaths

src/Infrastructure/Cache/AuthorizedCachePurger.php

Braces +1 added lines, -2 removed lines

@@ -29,8 +29,7 @@
 
 		try {
 			$this->cachePurger->purgeCache();
-		}
-		catch ( CachePurgingException $ex ) {
+		} catch ( CachePurgingException $ex ) {
 			return self::RESULT_ERROR;
 		}
 

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace WMDE\Fundraising\Frontend\Infrastructure\Cache;
 

cli/ApplicationConfigValidation/ConfigValidationException.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace WMDE\Fundraising\Frontend\Cli\ApplicationConfigValidation;
 

app/RouteHandlers/PageDisplayHandler.php

Braces +2 added lines, -4 removed lines

@@ -31,15 +31,13 @@
 
 		try {
 			$pageId = $pageSelector->getPageId( $pageName );
-		}
-		catch ( PageNotFoundException $exception ) {
+		} catch ( PageNotFoundException $exception ) {
 			throw new NotFoundHttpException( "Page page name '$pageName' not found." );
 		}
 
 		try {
 			return $this->getPageTemplate( $pageId )->render( [ 'page_id' => $pageId ] );
-		}
-		catch ( \Twig_Error_Runtime $exception ) {
+		} catch ( \Twig_Error_Runtime $exception ) {
 			if ( $exception->getPrevious() instanceof ContentNotFoundException ) {
 				throw new NotFoundHttpException( "Content for page id '$pageId' not found." );
 			}

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace WMDE\Fundraising\Frontend\App\RouteHandlers;
 
@@ -37,7 +37,7 @@ 
 		}
 
 		try {
-			return $this->getPageTemplate( $pageId )->render( [ 'page_id' => $pageId ] );
+			return $this->getPageTemplate( $pageId )->render( ['page_id' => $pageId] );
 		}
 		catch ( \Twig_Error_Runtime $exception ) {
 			if ( $exception->getPrevious() instanceof ContentNotFoundException ) {

src/Factories/UnknownChoiceDefinition.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace WMDE\Fundraising\Frontend\Factories;
 

src/Presentation/ContentPage/PageNotFoundException.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace WMDE\Fundraising\Frontend\Presentation\ContentPage;
 

src/BucketTesting/Validation/LoggingCampaignConfigurationLoader.php

Braces +5 added lines, -10 removed lines

@@ -22,20 +22,15 @@
 	public function loadCampaignConfiguration( string ...$configFiles ): array {
 		try {
 			return $this->actualCampaignConfigurationLoader->loadCampaignConfiguration( ...$configFiles );
-		}
-		catch ( ParseException $e ) {
+		} catch ( ParseException $e ) {
 			$this->errorLogger->addError( 'Failed to parse campaign YAML: ' . $e->getMessage() );
-		}
-		catch ( FileFetchingException $e ) {
+		} catch ( FileFetchingException $e ) {
 			$this->errorLogger->addError( 'YAML configuration file read error: ' . $e->getMessage() );
-		}
-		catch ( InvalidConfigurationException $e ) {
+		} catch ( InvalidConfigurationException $e ) {
 			$this->errorLogger->addError( 'Invalid campaign YAML configuration: ' . $e->getMessage() );
-		}
-		catch ( \RuntimeException $e ) {
+		} catch ( \RuntimeException $e ) {
 			$this->errorLogger->addError( 'Runtime error while parsing campaign YAML: ' . $e->getMessage() );
-		}
-		catch ( \Throwable $e ) {
+		} catch ( \Throwable $e ) {
 			$this->errorLogger->addError( get_class( $e ) . ': ' . $e->getMessage() );
 		}
 		return [];

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace WMDE\Fundraising\Frontend\BucketTesting\Validation;
 

src/Infrastructure/Payment/McpCreditCardService.php

Braces +2 added lines, -4 removed lines

@@ -39,8 +39,7 @@ 
 	public function getExpirationDate( string $customerId ): CreditCardExpiry {
 		try {
 			$customerData = $this->microPaymentDispatcher->creditcardDataGet( $this->accessKey, $this->useTestMode, $customerId );
-		}
-		catch ( \Exception $ex ) {
+		} catch ( \Exception $ex ) {
 			throw new CreditCardExpiryFetchingException( 'MCP-API: Request failed', $ex );
 		}
 
@@ -49,8 +48,7 @@ 
 				(int)$customerData['expiryMonth'],
 				(int)$customerData['expiryYear']
 			);
-		}
-		catch ( \InvalidArgumentException $ex ) {
+		} catch ( \InvalidArgumentException $ex ) {
 			throw new CreditCardExpiryFetchingException( 'Malformed expiry date', $ex );
 		}
 

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace WMDE\Fundraising\Frontend\Infrastructure\Payment;
 

src/Infrastructure/Payment/PayPalPaymentNotificationVerifierException.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types=1);
 
 namespace WMDE\Fundraising\Frontend\Infrastructure\Payment;