wmde / FundraisingFrontend

src/Infrastructure/DonationTokenFetchingException.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\Infrastructure;
 
@@ -10,8 +10,8 @@ 
  */
 class CreditCardExpiryFetchingException extends \RuntimeException {
 
-	public function __construct( string $message, \Exception $previous = null ) {
-		parent::__construct( $message, 0, $previous );
+	public function __construct(string $message, \Exception $previous = null) {
+		parent::__construct($message, 0, $previous);
 	}
 
 }

src/Infrastructure/DonationTokens.php

Spacing +6 added lines, -6 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\Infrastructure;
 
@@ -13,13 +13,13 @@ 
 	private $accessToken;
 	private $updateToken;
 
-	public function __construct( string $accessToken, string $updateToken ) {
-		if ( $accessToken === '' ) {
-			throw new \InvalidArgumentException( 'Access token cannot be empty' );
+	public function __construct(string $accessToken, string $updateToken) {
+		if ($accessToken === '') {
+			throw new \InvalidArgumentException('Access token cannot be empty');
 		}
 
-		if ( $updateToken === '' ) {
-			throw new \InvalidArgumentException( 'Update token cannot be empty' );
+		if ($updateToken === '') {
+			throw new \InvalidArgumentException('Update token cannot be empty');
 		}
 
 		$this->accessToken = $accessToken;

src/Infrastructure/DonationTokenFetcher.php

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;
 
@@ -16,6 +16,6 @@ 
 	 * @return DonationTokens
 	 * @throws DonationTokenFetchingException
 	 */
-	public function getTokens( int $donationId ): DonationTokens;
+	public function getTokens(int $donationId): DonationTokens;
 
 }

src/DataAccess/DoctrineDonationTokenFetcher.php

Spacing +6 added lines, -6 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\DataAccess;
 
@@ -18,7 +18,7 @@ 
 
 	private $entityManager;
 
-	public function __construct( EntityManager $entityManager ) {
+	public function __construct(EntityManager $entityManager) {
 		$this->entityManager = $entityManager;
 	}
 
@@ -28,8 +28,8 @@ 
 	 * @return DonationTokens
 	 * @throws DonationTokenFetchingException
 	 */
-	public function getTokens( int $donationId ): DonationTokens {
-		$donation = $this->getDonationById( $donationId );
+	public function getTokens(int $donationId): DonationTokens {
+		$donation = $this->getDonationById($donationId);
 
 		return new DonationTokens(
 			$donation->getDataObject()->getAccessToken(),
@@ -42,8 +42,8 @@ 
 	 *
 	 * @return Donation|null
 	 */
-	private function getDonationById( int $donationId ) {
-		return $this->entityManager->find( Donation::class, $donationId );
+	private function getDonationById(int $donationId) {
+		return $this->entityManager->find(Donation::class, $donationId);
 	}
 
 }

src/DataAccess/DoctrineDonationPrePersistSubscriber.php

Spacing +13 added lines, -13 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\DataAccess;
 
@@ -24,37 +24,37 @@ 
 	private $updateTokenGenerator;
 	private $accessTokenGenerator;
 
-	public function __construct( TokenGenerator $updateTokenGenerator, TokenGenerator $accessTokenGenerator ) {
+	public function __construct(TokenGenerator $updateTokenGenerator, TokenGenerator $accessTokenGenerator) {
 		$this->updateTokenGenerator = $updateTokenGenerator;
 		$this->accessTokenGenerator = $accessTokenGenerator;
 	}
 
 	public function getSubscribedEvents(): array {
-		return [ Events::prePersist ];
+		return [Events::prePersist];
 	}
 
-	public function prePersist( LifecycleEventArgs $args ) {
+	public function prePersist(LifecycleEventArgs $args) {
 		$entity = $args->getObject();
 
-		if ( $entity instanceof Donation ) {
-			$entity->modifyDataObject( function ( DonationData $data ) {
-				if ( $this->isEmpty( $data->getAccessToken() ) ) {
-					$data->setAccessToken( $this->accessTokenGenerator->generateToken() );
+		if ($entity instanceof Donation) {
+			$entity->modifyDataObject(function(DonationData $data) {
+				if ($this->isEmpty($data->getAccessToken())) {
+					$data->setAccessToken($this->accessTokenGenerator->generateToken());
 				}
 
-				if ( $this->isEmpty( $data->getUpdateToken() ) ) {
-					$data->setUpdateToken( $this->updateTokenGenerator->generateToken() );
+				if ($this->isEmpty($data->getUpdateToken())) {
+					$data->setUpdateToken($this->updateTokenGenerator->generateToken());
 				}
 
-				if ( $this->isEmpty( $data->getUpdateTokenExpiry() ) ) {
+				if ($this->isEmpty($data->getUpdateTokenExpiry())) {
 					$expiry = $this->updateTokenGenerator->generateTokenExpiry();
-					$data->setUpdateTokenExpiry( $expiry->format( self::DATE_TIME_FORMAT ) );
+					$data->setUpdateTokenExpiry($expiry->format(self::DATE_TIME_FORMAT));
 				}
 			} );
 		}
 	}
 
-	private function isEmpty( $stringOrNull ): bool {
+	private function isEmpty($stringOrNull): bool {
 		return $stringOrNull === null || $stringOrNull === '';
 	}
 

src/DataAccess/DoctrineDonationAuthorizer.php

Spacing +21 added lines, -21 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\DataAccess;
 
@@ -19,7 +19,7 @@ 
 	private $updateToken;
 	private $accessToken;
 
-	public function __construct( EntityManager $entityManager, string $updateToken = null, string $accessToken = null ) {
+	public function __construct(EntityManager $entityManager, string $updateToken = null, string $accessToken = null) {
 		$this->entityManager = $entityManager;
 		$this->updateToken = $updateToken;
 		$this->accessToken = $accessToken;
@@ -30,18 +30,18 @@ 
 	 * @param int $donationId
 	 * @return bool
 	 */
-	public function userCanModifyDonation( int $donationId ): bool {
+	public function userCanModifyDonation(int $donationId): bool {
 		try {
-			$donation = $this->getDonationById( $donationId );
+			$donation = $this->getDonationById($donationId);
 		}
-		catch ( ORMException $ex ) {
+		catch (ORMException $ex) {
 			// TODO: might want to log failure here
 			return false;
 		}
 
 		return $donation !== null
-			&& $this->updateTokenMatches( $donation )
-			&& $this->tokenHasNotExpired( $donation );
+			&& $this->updateTokenMatches($donation)
+			&& $this->tokenHasNotExpired($donation);
 	}
 
 	/**
@@ -49,17 +49,17 @@ 
 	 * @param int $donationId
 	 * @return bool
 	 */
-	public function systemCanModifyDonation( int $donationId ): bool {
+	public function systemCanModifyDonation(int $donationId): bool {
 		try {
-			$donation = $this->getDonationById( $donationId );
+			$donation = $this->getDonationById($donationId);
 		}
-		catch ( ORMException $ex ) {
+		catch (ORMException $ex) {
 			// TODO: might want to log failure here
 			return false;
 		}
 
 		return $donation !== null
-			&& $this->updateTokenMatches( $donation );
+			&& $this->updateTokenMatches($donation);
 	}
 
 	/**
@@ -67,33 +67,33 @@ 
 	 *
 	 * @return Donation|null
 	 */
-	private function getDonationById( int $donationId ) {
-		return $this->entityManager->find( Donation::class, $donationId );
+	private function getDonationById(int $donationId) {
+		return $this->entityManager->find(Donation::class, $donationId);
 	}
 
-	private function updateTokenMatches( Donation $donation ): bool {
+	private function updateTokenMatches(Donation $donation): bool {
 		return $donation->getDataObject()->getUpdateToken() === $this->updateToken;
 	}
 
-	private function tokenHasNotExpired( Donation $donation ): bool {
+	private function tokenHasNotExpired(Donation $donation): bool {
 		$expiryTime = $donation->getDataObject()->getUpdateTokenExpiry();
-		return $expiryTime !== null && strtotime( $expiryTime ) >= time();
+		return $expiryTime !== null && strtotime($expiryTime) >= time();
 	}
 
-	public function canAccessDonation( int $donationId ): bool {
+	public function canAccessDonation(int $donationId): bool {
 		try {
-			$donation = $this->getDonationById( $donationId );
+			$donation = $this->getDonationById($donationId);
 		}
-		catch ( ORMException $ex ) {
+		catch (ORMException $ex) {
 			// TODO: might want to log failure here
 			return false;
 		}
 
 		return $donation !== null
-			&& $this->accessTokenMatches( $donation );
+			&& $this->accessTokenMatches($donation);
 	}
 
-	private function accessTokenMatches( Donation $donation ): bool {
+	private function accessTokenMatches(Donation $donation): bool {
 		return $donation->getDataObject()->getAccessToken() === $this->accessToken;
 	}
 

src/UseCases/AddDonation/AddDonationUseCase.php

Spacing +55 added lines, -55 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\UseCases\AddDonation;
 
@@ -39,10 +39,10 @@ 
 	private $bankDataConverter;
 	private $tokenFetcher;
 
-	public function __construct( DonationRepository $donationRepository, DonationValidator $donationValidator,
+	public function __construct(DonationRepository $donationRepository, DonationValidator $donationValidator,
 								 ReferrerGeneralizer $referrerGeneralizer, DonationConfirmationMailer $mailer,
 								 TransferCodeGenerator $transferCodeGenerator, BankDataConverter $bankDataConverter,
-								 DonationTokenFetcher $tokenFetcher ) {
+								 DonationTokenFetcher $tokenFetcher) {
 
 		$this->donationRepository = $donationRepository;
 		$this->donationValidator = $donationValidator;
@@ -53,28 +53,28 @@ 
 		$this->tokenFetcher = $tokenFetcher;
 	}
 
-	public function addDonation( AddDonationRequest $donationRequest ): AddDonationResponse {
-		$donation = $this->newDonationFromRequest( $donationRequest );
+	public function addDonation(AddDonationRequest $donationRequest): AddDonationResponse {
+		$donation = $this->newDonationFromRequest($donationRequest);
 
-		$validationResult = $this->donationValidator->validate( $donation );
+		$validationResult = $this->donationValidator->validate($donation);
 
-		if ( $validationResult->hasViolations() ) {
-			return AddDonationResponse::newFailureResponse( $validationResult->getViolations() );
+		if ($validationResult->hasViolations()) {
+			return AddDonationResponse::newFailureResponse($validationResult->getViolations());
 		}
 
-		$needsModeration = $this->donationValidator->needsModeration( $donation );
+		$needsModeration = $this->donationValidator->needsModeration($donation);
 
-		if ( $needsModeration ) {
+		if ($needsModeration) {
 			$donation->markForModeration();
 		}
 
 		// TODO: handle exceptions
-		$this->donationRepository->storeDonation( $donation );
+		$this->donationRepository->storeDonation($donation);
 
 		// TODO: handle exceptions
-		$tokens = $this->tokenFetcher->getTokens( $donation->getId() );
+		$tokens = $this->tokenFetcher->getTokens($donation->getId());
 
-		$this->sendDonationConfirmationEmail( $donation );
+		$this->sendDonationConfirmationEmail($donation);
 
 		return AddDonationResponse::newSuccessResponse(
 			$donation,
@@ -83,93 +83,93 @@ 
 		);
 	}
 
-	private function newDonationFromRequest( AddDonationRequest $donationRequest ): Donation {
+	private function newDonationFromRequest(AddDonationRequest $donationRequest): Donation {
 		return new Donation(
 			null,
-			$this->getInitialDonationStatus( $donationRequest->getPaymentType() ),
+			$this->getInitialDonationStatus($donationRequest->getPaymentType()),
 			$donationRequest->getDonor(),
-			$this->getPaymentFromRequest( $donationRequest ),
+			$this->getPaymentFromRequest($donationRequest),
 			$donationRequest->getOptIn() === '1',
-			$this->newTrackingInfoFromRequest( $donationRequest )
+			$this->newTrackingInfoFromRequest($donationRequest)
 		);
 	}
 
-	private function getInitialDonationStatus( string $paymentType ): string {
-		if ( $paymentType === PaymentType::DIRECT_DEBIT ) {
+	private function getInitialDonationStatus(string $paymentType): string {
+		if ($paymentType === PaymentType::DIRECT_DEBIT) {
 			return Donation::STATUS_NEW;
 		}
 
-		if ( $paymentType === PaymentType::BANK_TRANSFER ) {
+		if ($paymentType === PaymentType::BANK_TRANSFER) {
 			return Donation::STATUS_PROMISE;
 		}
 
 		return Donation::STATUS_EXTERNAL_INCOMPLETE;
 	}
 
-	private function getPaymentFromRequest( AddDonationRequest $donationRequest ): DonationPayment {
+	private function getPaymentFromRequest(AddDonationRequest $donationRequest): DonationPayment {
 		return new DonationPayment(
 			$donationRequest->getAmount(),
 			$donationRequest->getInterval(),
-			$this->getPaymentMethodFromRequest( $donationRequest )
+			$this->getPaymentMethodFromRequest($donationRequest)
 		);
 	}
 
-	private function getPaymentMethodFromRequest( AddDonationRequest $donationRequest ): PaymentMethod {
-		if ( $donationRequest->getPaymentType() === PaymentType::BANK_TRANSFER ) {
-			return new BankTransferPayment( $this->transferCodeGenerator->generateTransferCode() );
+	private function getPaymentMethodFromRequest(AddDonationRequest $donationRequest): PaymentMethod {
+		if ($donationRequest->getPaymentType() === PaymentType::BANK_TRANSFER) {
+			return new BankTransferPayment($this->transferCodeGenerator->generateTransferCode());
 		}
 
-		if ( $donationRequest->getPaymentType() === PaymentType::DIRECT_DEBIT ) {
-			return new DirectDebitPayment( $this->newBankDataFromRequest( $donationRequest ) );
+		if ($donationRequest->getPaymentType() === PaymentType::DIRECT_DEBIT) {
+			return new DirectDebitPayment($this->newBankDataFromRequest($donationRequest));
 		}
 
-		if ( $donationRequest->getPaymentType() === PaymentType::PAYPAL ) {
-			return new PayPalPayment( new PayPalData() );
+		if ($donationRequest->getPaymentType() === PaymentType::PAYPAL) {
+			return new PayPalPayment(new PayPalData());
 		}
 
-		return new PaymentWithoutAssociatedData( $donationRequest->getPaymentType() );
+		return new PaymentWithoutAssociatedData($donationRequest->getPaymentType());
 	}
 
-	private function newBankDataFromRequest( AddDonationRequest $request ): BankData {
+	private function newBankDataFromRequest(AddDonationRequest $request): BankData {
 		$bankData = new BankData();
 
-		$bankData->setIban( new Iban( $request->getIban() ) )
-			->setBic( $request->getBic() )
-			->setAccount( $request->getBankAccount() )
-			->setBankCode( $request->getBankCode() )
-			->setBankName( $request->getBankName() );
+		$bankData->setIban(new Iban($request->getIban()))
+			->setBic($request->getBic())
+			->setAccount($request->getBankAccount())
+			->setBankCode($request->getBankCode())
+			->setBankName($request->getBankName());
 
-		if ( $bankData->hasIban() && !$bankData->hasCompleteLegacyBankData() ) {
-			$bankData = $this->newBankDataFromIban( $bankData->getIban() );
+		if ($bankData->hasIban() && !$bankData->hasCompleteLegacyBankData()) {
+			$bankData = $this->newBankDataFromIban($bankData->getIban());
 		}
 
-		if ( $bankData->hasCompleteLegacyBankData() && !$bankData->hasIban() ) {
-			$bankData = $this->newBankDataFromAccountAndBankCode( $bankData->getAccount(), $bankData->getBankCode() );
+		if ($bankData->hasCompleteLegacyBankData() && !$bankData->hasIban()) {
+			$bankData = $this->newBankDataFromAccountAndBankCode($bankData->getAccount(), $bankData->getBankCode());
 		}
 
 		return $bankData->freeze()->assertNoNullFields();
 	}
 
-	private function newBankDataFromIban( Iban $iban ): BankData {
-		$bankData = $this->bankDataConverter->getBankDataFromIban( $iban );
+	private function newBankDataFromIban(Iban $iban): BankData {
+		$bankData = $this->bankDataConverter->getBankDataFromIban($iban);
 		return $bankData->freeze()->assertNoNullFields();
 	}
 
-	private function newBankDataFromAccountAndBankCode( string $account, string $bankCode ): BankData {
-		$bankData = $this->bankDataConverter->getBankDataFromAccountData( $account, $bankCode );
+	private function newBankDataFromAccountAndBankCode(string $account, string $bankCode): BankData {
+		$bankData = $this->bankDataConverter->getBankDataFromAccountData($account, $bankCode);
 		return $bankData->freeze()->assertNoNullFields();
 	}
 
-	private function newTrackingInfoFromRequest( AddDonationRequest $request ): DonationTrackingInfo {
+	private function newTrackingInfoFromRequest(AddDonationRequest $request): DonationTrackingInfo {
 		$trackingInfo = new DonationTrackingInfo();
 
-		$trackingInfo->setTracking( $request->getTracking() );
-		$trackingInfo->setSource( $this->referrerGeneralizer->generalize( $request->getSource() ) );
-		$trackingInfo->setTotalImpressionCount( $request->getTotalImpressionCount() );
-		$trackingInfo->setSingleBannerImpressionCount( $request->getSingleBannerImpressionCount() );
-		$trackingInfo->setColor( $request->getColor() );
-		$trackingInfo->setSkin( $request->getSkin() );
-		$trackingInfo->setLayout( $request->getLayout() );
+		$trackingInfo->setTracking($request->getTracking());
+		$trackingInfo->setSource($this->referrerGeneralizer->generalize($request->getSource()));
+		$trackingInfo->setTotalImpressionCount($request->getTotalImpressionCount());
+		$trackingInfo->setSingleBannerImpressionCount($request->getSingleBannerImpressionCount());
+		$trackingInfo->setColor($request->getColor());
+		$trackingInfo->setSkin($request->getSkin());
+		$trackingInfo->setLayout($request->getLayout());
 
 		return $trackingInfo->freeze()->assertNoNullFields();
 	}
@@ -180,9 +180,9 @@ 
 	 * @throws \RuntimeException
 	 * TODO: handle exception
 	 */
-	private function sendDonationConfirmationEmail( Donation $donation ) {
-		if ( $donation->getDonor() !== null && !$donation->hasExternalPayment() ) {
-			$this->mailer->sendConfirmationMailFor( $donation );
+	private function sendDonationConfirmationEmail(Donation $donation) {
+		if ($donation->getDonor() !== null && !$donation->hasExternalPayment()) {
+			$this->mailer->sendConfirmationMailFor($donation);
 		}
 	}
 }
\ No newline at end of file

src/Factories/FunFunFactory.php

Spacing +154 added lines, -154 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\Factories;
 
@@ -153,7 +153,7 @@ 
 	private $tokenGenerator;
 	private $addDoctrineSubscribers = true;
 
-	public function __construct( array $config ) {
+	public function __construct(array $config) {
 		$this->config = $config;
 		$this->pimple = $this->newPimple();
 	}
@@ -161,101 +161,101 @@ 
 	private function newPimple(): \Pimple {
 		$pimple = new \Pimple();
 
-		$pimple['dbal_connection'] = $pimple->share( function() {
-			return DriverManager::getConnection( $this->config['db'] );
+		$pimple['dbal_connection'] = $pimple->share(function() {
+			return DriverManager::getConnection($this->config['db']);
 		} );
 
-		$pimple['entity_manager'] = $pimple->share( function() {
-			$entityManager = ( new StoreFactory( $this->getConnection() ) )->getEntityManager();
-			if ( $this->addDoctrineSubscribers ) {
-				$entityManager->getEventManager()->addEventSubscriber( $this->newDoctrineDonationPrePersistSubscriber() );
+		$pimple['entity_manager'] = $pimple->share(function() {
+			$entityManager = (new StoreFactory($this->getConnection()))->getEntityManager();
+			if ($this->addDoctrineSubscribers) {
+				$entityManager->getEventManager()->addEventSubscriber($this->newDoctrineDonationPrePersistSubscriber());
 			}
 
 			return $entityManager;
 		} );
 
-		$pimple['subscription_repository'] = $pimple->share( function() {
+		$pimple['subscription_repository'] = $pimple->share(function() {
 			return new LoggingSubscriptionRepository(
-				new DoctrineSubscriptionRepository( $this->getEntityManager() ),
+				new DoctrineSubscriptionRepository($this->getEntityManager()),
 				$this->getLogger()
 			);
 		} );
 
-		$pimple['donation_repository'] = $pimple->share( function() {
+		$pimple['donation_repository'] = $pimple->share(function() {
 			return new LoggingDonationRepository(
-				new DoctrineDonationRepository( $this->getEntityManager() ),
+				new DoctrineDonationRepository($this->getEntityManager()),
 				$this->getLogger()
 			);
 		} );
 
-		$pimple['membership_application_repository'] = $pimple->share( function() {
+		$pimple['membership_application_repository'] = $pimple->share(function() {
 			return new LoggingMembershipApplicationRepository(
-				new DoctrineMembershipApplicationRepository( $this->getEntityManager() ),
+				new DoctrineMembershipApplicationRepository($this->getEntityManager()),
 				$this->getLogger()
 			);
 		} );
 
-		$pimple['comment_repository'] = $pimple->share( function() {
+		$pimple['comment_repository'] = $pimple->share(function() {
 			return new LoggingCommentFinder(
-				new DoctrineCommentFinder( $this->getEntityManager() ),
+				new DoctrineCommentFinder($this->getEntityManager()),
 				$this->getLogger()
 			);
 		} );
 
-		$pimple['mail_validator'] = $pimple->share( function() {
-			return new EmailValidator( new InternetDomainNameValidator() );
+		$pimple['mail_validator'] = $pimple->share(function() {
+			return new EmailValidator(new InternetDomainNameValidator());
 		} );
 
-		$pimple['subscription_validator'] = $pimple->share( function() {
+		$pimple['subscription_validator'] = $pimple->share(function() {
 			return new SubscriptionValidator(
 				$this->getEmailValidator(),
-				$this->getTextPolicyValidator( 'fields' ),
+				$this->getTextPolicyValidator('fields'),
 				$this->newSubscriptionDuplicateValidator(),
 				$this->newHonorificValidator()
 			);
 		} );
 
-		$pimple['template_name_validator'] = $pimple->share( function() {
-			return new TemplateNameValidator( $this->getTwig() );
+		$pimple['template_name_validator'] = $pimple->share(function() {
+			return new TemplateNameValidator($this->getTwig());
 		} );
 
-		$pimple['contact_validator'] = $pimple->share( function() {
-			return new GetInTouchValidator( $this->getEmailValidator() );
+		$pimple['contact_validator'] = $pimple->share(function() {
+			return new GetInTouchValidator($this->getEmailValidator());
 		} );
 
-		$pimple['greeting_generator'] = $pimple->share( function() {
+		$pimple['greeting_generator'] = $pimple->share(function() {
 			return new GreetingGenerator();
 		} );
 
-		$pimple['mw_api'] = $pimple->share( function() {
+		$pimple['mw_api'] = $pimple->share(function() {
 			return new MediawikiApi(
 				$this->config['cms-wiki-api-url'],
 				$this->getGuzzleClient()
 			);
 		} );
 
-		$pimple['guzzle_client'] = $pimple->share( function() {
+		$pimple['guzzle_client'] = $pimple->share(function() {
 			$middlewareFactory = new MiddlewareFactory();
-			$middlewareFactory->setLogger( $this->getLogger() );
+			$middlewareFactory->setLogger($this->getLogger());
 
-			$handlerStack = HandlerStack::create( new CurlHandler() );
-			$handlerStack->push( $middlewareFactory->retry() );
+			$handlerStack = HandlerStack::create(new CurlHandler());
+			$handlerStack->push($middlewareFactory->retry());
 
-			return new Client( [
+			return new Client([
 				'cookies' => true,
 				'handler' => $handlerStack,
-				'headers' => [ 'User-Agent' => 'WMDE Fundraising Frontend' ],
-			] );
+				'headers' => ['User-Agent' => 'WMDE Fundraising Frontend'],
+			]);
 		} );
 
-		$pimple['translator'] = $pimple->share( function() {
+		$pimple['translator'] = $pimple->share(function() {
 			$translationFactory = new TranslationFactory();
 			$loaders = [
 				'json' => $translationFactory->newJsonLoader()
 			];
 			$locale = $this->config['locale'];
-			$translator = $translationFactory->create( $loaders, $locale );
-			$translator->addResource( 'json', __DIR__ . '/../../app/translations/messages.' . $locale . '.json', $locale );
+			$translator = $translationFactory->create($loaders, $locale);
+			$translator->addResource('json', __DIR__ . '/../../app/translations/messages.' . $locale . '.json', $locale);
 			$translator->addResource(
 				'json',
 				__DIR__ . '/../../app/translations/paymentTypes.' . $locale . '.json',
@@ -268,55 +268,55 @@ 
 				$locale,
 				'paymentIntervals'
 			);
-			$translator->addResource( 'json', __DIR__ . '/../../app/translations/validations.' . $locale . '.json', $locale,
-				'validations' );
+			$translator->addResource('json', __DIR__ . '/../../app/translations/validations.' . $locale . '.json', $locale,
+				'validations');
 			return $translator;
 		} );
 
 		// In the future, this could be locale-specific or filled from a DB table
-		$pimple['honorifics'] = $pimple->share( function() {
-			return new Honorifics( [
+		$pimple['honorifics'] = $pimple->share(function() {
+			return new Honorifics([
 				'' => 'Kein Titel',
 				'Dr.' => 'Dr.',
 				'Prof.' => 'Prof.',
 				'Prof. Dr.' => 'Prof. Dr.'
-			] );
+			]);
 		} );
 
-		$pimple['twig_factory'] = $pimple->share( function () {
-			return new TwigFactory( $this->config['twig'] );
+		$pimple['twig_factory'] = $pimple->share(function() {
+			return new TwigFactory($this->config['twig']);
 		} );
 
-		$pimple['twig'] = $pimple->share( function() {
+		$pimple['twig'] = $pimple->share(function() {
 			$twigFactory = $this->getTwigFactory();
-			$loaders = array_filter( [
+			$loaders = array_filter([
 				$twigFactory->newFileSystemLoader(),
 				$twigFactory->newArrayLoader(), // This is just a fallback for testing
-				$twigFactory->newWikiPageLoader( $this->newWikiContentProvider() ),
-			] );
+				$twigFactory->newWikiPageLoader($this->newWikiContentProvider()),
+			]);
 			$extensions = [
-				$twigFactory->newTranslationExtension( $this->getTranslator() ),
+				$twigFactory->newTranslationExtension($this->getTranslator()),
 				new Twig_Extensions_Extension_Intl()
 			];
-			return $twigFactory->create( $loaders, $extensions );
+			return $twigFactory->create($loaders, $extensions);
 		} );
 
-		$pimple['logger'] = $pimple->share( function() {
-			$logger = new Logger( 'WMDE Fundraising Frontend logger' );
+		$pimple['logger'] = $pimple->share(function() {
+			$logger = new Logger('WMDE Fundraising Frontend logger');
 
-			$streamHandler = new StreamHandler( $this->newLoggerPath( ( new \DateTime() )->format( 'Y-m-d\TH:i:s\Z' ) ) );
-			$bufferHandler = new BufferHandler( $streamHandler, 500, Logger::DEBUG, true, true );
-			$streamHandler->setFormatter( new LineFormatter( "%message%\n" ) );
-			$logger->pushHandler( $bufferHandler );
+			$streamHandler = new StreamHandler($this->newLoggerPath((new \DateTime())->format('Y-m-d\TH:i:s\Z')));
+			$bufferHandler = new BufferHandler($streamHandler, 500, Logger::DEBUG, true, true);
+			$streamHandler->setFormatter(new LineFormatter("%message%\n"));
+			$logger->pushHandler($bufferHandler);
 
-			$errorHandler = new StreamHandler( $this->newLoggerPath( 'error' ), Logger::ERROR );
-			$errorHandler->setFormatter( new JsonFormatter() );
-			$logger->pushHandler( $errorHandler );
+			$errorHandler = new StreamHandler($this->newLoggerPath('error'), Logger::ERROR);
+			$errorHandler->setFormatter(new JsonFormatter());
+			$logger->pushHandler($errorHandler);
 
 			return $logger;
 		} );
 
-		$pimple['messenger'] = $pimple->share( function() {
+		$pimple['messenger'] = $pimple->share(function() {
 			return new Messenger(
 				new Swift_MailTransport(),
 				$this->getOperatorAddress(),
@@ -324,11 +324,11 @@ 
 			);
 		} );
 
-		$pimple['confirmation-page-selector'] = $pimple->share( function() {
-			return new DonationConfirmationPageSelector( $this->config['confirmation-pages'] );
+		$pimple['confirmation-page-selector'] = $pimple->share(function() {
+			return new DonationConfirmationPageSelector($this->config['confirmation-pages']);
 		} );
 
-		$pimple['paypal-payment-notification-verifier'] = $pimple->share( function() {
+		$pimple['paypal-payment-notification-verifier'] = $pimple->share(function() {
 			return new LoggingPaymentNotificationVerifier(
 				new PayPalPaymentNotificationVerifier(
 					new Client(),
@@ -338,7 +338,7 @@ 
 			);
 		} );
 
-		$pimple['credit-card-api-service'] = $pimple->share( function() {
+		$pimple['credit-card-api-service'] = $pimple->share(function() {
 			return new McpCreditCardService(
 				new TNvpServiceDispatcher(
 					'IMcpCreditcardService_v1_5',
@@ -362,17 +362,17 @@ 
 
 	private function newDonationEventLogger(): DonationEventLogger {
 		return new BestEffortDonationEventLogger(
-			new DoctrineDonationEventLogger( $this->getEntityManager() ),
+			new DoctrineDonationEventLogger($this->getEntityManager()),
 			$this->getLogger()
 		);
 	}
 
 	public function newInstaller(): Installer {
-		return ( new StoreFactory( $this->getConnection() ) )->newInstaller();
+		return (new StoreFactory($this->getConnection()))->newInstaller();
 	}
 
 	public function newListCommentsUseCase(): ListCommentsUseCase {
-		return new ListCommentsUseCase( $this->getCommentFinder() );
+		return new ListCommentsUseCase($this->getCommentFinder());
 	}
 
 	public function newCommentListJsonPresenter(): CommentListJsonPresenter {
@@ -380,14 +380,14 @@ 
 	}
 
 	public function newCommentListRssPresenter(): CommentListRssPresenter {
-		return new CommentListRssPresenter( new TwigTemplate(
+		return new CommentListRssPresenter(new TwigTemplate(
 			$this->getTwig(),
 			'CommentList.rss.twig'
-		) );
+		));
 	}
 
 	public function newCommentListHtmlPresenter(): CommentListHtmlPresenter {
-		return new CommentListHtmlPresenter( $this->getLayoutTemplate( 'CommentList.html.twig' ) );
+		return new CommentListHtmlPresenter($this->getLayoutTemplate('CommentList.html.twig'));
 	}
 
 	private function getCommentFinder(): CommentFinder {
@@ -398,7 +398,7 @@ 
 		return $this->pimple['subscription_repository'];
 	}
 
-	public function setSubscriptionRepository( SubscriptionRepository $subscriptionRepository ) {
+	public function setSubscriptionRepository(SubscriptionRepository $subscriptionRepository) {
 		$this->pimple['subscription_repository'] = $subscriptionRepository;
 	}
 
@@ -421,26 +421,26 @@ 
 	}
 
 	public function newDisplayPagePresenter(): DisplayPagePresenter {
-		return new DisplayPagePresenter( $this->getLayoutTemplate( 'DisplayPageLayout.twig' ) );
+		return new DisplayPagePresenter($this->getLayoutTemplate('DisplayPageLayout.twig'));
 	}
 
 	public function newAddSubscriptionHTMLPresenter(): AddSubscriptionHtmlPresenter {
-		return new AddSubscriptionHtmlPresenter( $this->getLayoutTemplate( 'AddSubscription.twig' ), $this->getTranslator() );
+		return new AddSubscriptionHtmlPresenter($this->getLayoutTemplate('AddSubscription.twig'), $this->getTranslator());
 	}
 
 	public function newConfirmSubscriptionHtmlPresenter(): ConfirmSubscriptionHtmlPresenter {
 		return new ConfirmSubscriptionHtmlPresenter(
-			$this->getLayoutTemplate( 'ConfirmSubscription.html.twig' ),
+			$this->getLayoutTemplate('ConfirmSubscription.html.twig'),
 			$this->getTranslator()
 		);
 	}
 
 	public function newAddSubscriptionJSONPresenter(): AddSubscriptionJsonPresenter {
-		return new AddSubscriptionJsonPresenter( $this->getTranslator() );
+		return new AddSubscriptionJsonPresenter($this->getTranslator());
 	}
 
 	public function newGetInTouchHTMLPresenter(): GetInTouchHtmlPresenter {
-		return new GetInTouchHtmlPresenter( $this->getLayoutTemplate( 'GetInTouch.twig' ), $this->getTranslator() );
+		return new GetInTouchHtmlPresenter($this->getLayoutTemplate('GetInTouch.twig'), $this->getTranslator());
 	}
 
 	public function getTwig(): Twig_Environment {
@@ -453,7 +453,7 @@ 
 	 * @param string $templateName
 	 * @return TwigTemplate
 	 */
-	private function getLayoutTemplate( string $templateName ): TwigTemplate {
+	private function getLayoutTemplate(string $templateName): TwigTemplate {
 		 return new TwigTemplate(
 			$this->getTwig(),
 			$templateName,
@@ -467,13 +467,13 @@ 
 	 * @param string $templateName Template to include
 	 * @return TwigTemplate
 	 */
-	private function getIncludeTemplate( string $templateName ): TwigTemplate {
+	private function getIncludeTemplate(string $templateName): TwigTemplate {
 		return new TwigTemplate(
 			$this->getTwig(),
 			'IncludeInLayout.twig',
 			array_merge(
 				$this->getDefaultTwigVariables(),
-				[ 'main_template' => $templateName]
+				['main_template' => $templateName]
 			)
 		);
 	}
@@ -498,7 +498,7 @@ 
 	private function newPageRetriever(): PageRetriever {
 		return new ApiBasedPageRetriever(
 			$this->getMediaWikiApi(),
-			new ApiUser( $this->config['cms-wiki-user'], $this->config['cms-wiki-password'] ),
+			new ApiUser($this->config['cms-wiki-user'], $this->config['cms-wiki-password']),
 			$this->getLogger()
 		);
 	}
@@ -507,7 +507,7 @@ 
 		return $this->pimple['mw_api'];
 	}
 
-	public function setMediaWikiApi( MediawikiApi $api ) {
+	public function setMediaWikiApi(MediawikiApi $api) {
 		$this->pimple['mw_api'] = $api;
 	}
 
@@ -527,7 +527,7 @@ 
 		return $this->pimple['logger'];
 	}
 
-	private function newLoggerPath( string $fileName ): string {
+	private function newLoggerPath(string $fileName): string {
 		return __DIR__ . '/../../var/log/' . $fileName . '.log';
 	}
 
@@ -571,20 +571,20 @@ 
 			new TwigTemplate(
 					$this->getTwig(),
 					'Mail_Subscription_Confirmation.twig',
-					[ 'greeting_generator' => $this->getGreetingGenerator() ]
+					['greeting_generator' => $this->getGreetingGenerator()]
 			),
 			'mail_subject_membership'
 		);
 	}
 
-	private function newTemplateMailer( TwigTemplate $template, string $messageKey ): TemplateBasedMailer {
+	private function newTemplateMailer(TwigTemplate $template, string $messageKey): TemplateBasedMailer {
 		$mailer = new TemplateBasedMailer(
 			$this->getMessenger(),
 			$template,
-			$this->getTranslator()->trans( $messageKey )
+			$this->getTranslator()->trans($messageKey)
 		);
 
-		return new LoggingMailer( $mailer, $this->getLogger() );
+		return new LoggingMailer($mailer, $this->getLogger());
 	}
 
 	public function getGreetingGenerator() {
@@ -592,11 +592,11 @@ 
 	}
 
 	public function newCheckIbanUseCase(): CheckIbanUseCase {
-		return new CheckIbanUseCase( $this->newBankDataConverter() );
+		return new CheckIbanUseCase($this->newBankDataConverter());
 	}
 
 	public function newGenerateIbanUseCase(): GenerateIbanUseCase {
-		return new GenerateIbanUseCase( $this->newBankDataConverter() );
+		return new GenerateIbanUseCase($this->newBankDataConverter());
 	}
 
 	public function newIbanPresenter(): IbanPresenter {
@@ -604,14 +604,14 @@ 
 	}
 
 	public function newBankDataConverter() {
-		return new BankDataConverter( $this->config['bank-data-file'] );
+		return new BankDataConverter($this->config['bank-data-file']);
 	}
 
-	public function setSubscriptionValidator( SubscriptionValidator $subscriptionValidator ) {
+	public function setSubscriptionValidator(SubscriptionValidator $subscriptionValidator) {
 		$this->pimple['subscription_validator'] = $subscriptionValidator;
 	}
 
-	public function setPageTitlePrefix( string $prefix ) {
+	public function setPageTitlePrefix(string $prefix) {
 		$this->config['cms-wiki-title-prefix'] = $prefix;
 	}
 
@@ -625,7 +625,7 @@ 
 
 	private function newContactConfirmationMailer(): TemplateBasedMailer {
 		return $this->newTemplateMailer(
-			new TwigTemplate( $this->getTwig(), 'GetInTouchConfirmation.twig' ),
+			new TwigTemplate($this->getTwig(), 'GetInTouchConfirmation.twig'),
 			'mail_subject_getintouch'
 		);
 	}
@@ -643,12 +643,12 @@ 
 
 	private function newSubscriptionDuplicateCutoffDate(): \DateTime {
 		$cutoffDateTime = new \DateTime();
-		$cutoffDateTime->sub( new \DateInterval( $this->config['subscription-interval'] ) );
+		$cutoffDateTime->sub(new \DateInterval($this->config['subscription-interval']));
 		return $cutoffDateTime;
 	}
 
 	private function newHonorificValidator(): AllowedValuesValidator {
-		return new AllowedValuesValidator( $this->getHonorifics()->getKeys() );
+		return new AllowedValuesValidator($this->getHonorifics()->getKeys());
 	}
 
 	private function getHonorifics(): Honorifics {
@@ -656,45 +656,45 @@ 
 	}
 
 	private function newPaymentTypeValidator(): AllowedValuesValidator {
-		return new AllowedValuesValidator( PaymentType::getPaymentTypes() );
+		return new AllowedValuesValidator(PaymentType::getPaymentTypes());
 	}
 
 	private function newBankDataValidator(): BankDataValidator {
-		return new BankDataValidator( new IbanValidator( $this->newBankDataConverter() ) );
+		return new BankDataValidator(new IbanValidator($this->newBankDataConverter()));
 	}
 
 	private function getMessenger(): Messenger {
 		return $this->pimple['messenger'];
 	}
 
-	public function setMessenger( Messenger $messenger ) {
+	public function setMessenger(Messenger $messenger) {
 		$this->pimple['messenger'] = $messenger;
 	}
 
 	public function setNullMessenger() {
-		$this->setMessenger( new Messenger(
+		$this->setMessenger(new Messenger(
 			Swift_NullTransport::newInstance(),
 			$this->getOperatorAddress()
-		) );
+		));
 	}
 
 	public function getOperatorAddress() {
-		return new EmailAddress( $this->config['operator-email'] );
+		return new EmailAddress($this->config['operator-email']);
 	}
 
 	public function newInternalErrorHTMLPresenter(): InternalErrorHtmlPresenter {
-		return new InternalErrorHtmlPresenter( $this->getLayoutTemplate( 'Error.twig' ) );
+		return new InternalErrorHtmlPresenter($this->getLayoutTemplate('Error.twig'));
 	}
 
 	public function newAccessDeniedHTMLPresenter(): InternalErrorHtmlPresenter {
-		return new InternalErrorHtmlPresenter( $this->getLayoutTemplate( 'AccessDenied.twig' ) );
+		return new InternalErrorHtmlPresenter($this->getLayoutTemplate('AccessDenied.twig'));
 	}
 
 	public function getTranslator(): TranslatorInterface {
 		return $this->pimple['translator'];
 	}
 
-	public function setTranslator( TranslatorInterface $translator ) {
+	public function setTranslator(TranslatorInterface $translator) {
 		$this->pimple['translator'] = $translator;
 	}
 
@@ -702,40 +702,40 @@ 
 		return $this->pimple['twig_factory'];
 	}
 
-	private function getTextPolicyValidator( $policyName ) {
+	private function getTextPolicyValidator($policyName) {
 		$policyValidator = new TextPolicyValidator();
 
 		$contentProvider = $this->newWikiContentProvider();
 		$textPolicyConfig = $this->config['text-policies'][$policyName];
 
 		// TODO: this is not the place to retrieve resources over the network
-		$policyValidator->addBadWordsFromArray( $this->loadWordsFromWiki(
+		$policyValidator->addBadWordsFromArray($this->loadWordsFromWiki(
 			$contentProvider,
 			$textPolicyConfig['badwords'] ?? ''
-		) );
-		$policyValidator->addWhiteWordsFromArray( $this->loadWordsFromWiki(
+		));
+		$policyValidator->addWhiteWordsFromArray($this->loadWordsFromWiki(
 			$contentProvider,
 			$textPolicyConfig['whitewords'] ?? ''
-		) );
+		));
 
 		return $policyValidator;
 	}
 
-	private function loadWordsFromWiki( WikiContentProvider $contentProvider, string $pageName ): array {
-		if ( $pageName === '' ) {
-			return [ ];
+	private function loadWordsFromWiki(WikiContentProvider $contentProvider, string $pageName): array {
+		if ($pageName === '') {
+			return [];
 		}
-		$content = $contentProvider->getContent( $pageName, 'raw' );
-		$words = array_map( 'trim', explode( "\n", $content ) );
+		$content = $contentProvider->getContent($pageName, 'raw');
+		$words = array_map('trim', explode("\n", $content));
 
-		return array_filter( $words );
+		return array_filter($words);
 	}
 
-	public function newCancelDonationUseCase( string $updateToken ): CancelDonationUseCase {
+	public function newCancelDonationUseCase(string $updateToken): CancelDonationUseCase {
 		return new CancelDonationUseCase(
 			$this->getDonationRepository(),
 			$this->newCancelDonationMailer(),
-			$this->newDonationAuthorizer( $updateToken )
+			$this->newDonationAuthorizer($updateToken)
 		);
 	}
 
@@ -744,7 +744,7 @@ 
 			new TwigTemplate(
 				$this->getTwig(),
 				'Mail_Donation_Cancellation_Confirmation.twig',
-				[ 'greeting_generator' => $this->getGreetingGenerator() ]
+				['greeting_generator' => $this->getGreetingGenerator()]
 			),
 			'mail_subject_confirm_cancellation'
 		);
@@ -772,9 +772,9 @@ 
 	private function newDonationValidator(): DonationValidator {
 		return new DonationValidator(
 			$this->newAmountValidator(),
-			new AmountPolicyValidator( 1000, 200, 300 ),
+			new AmountPolicyValidator(1000, 200, 300),
 			$this->newPersonalInfoValidator(),
-			$this->getTextPolicyValidator( 'fields' ),
+			$this->getTextPolicyValidator('fields'),
 			$this->newPaymentTypeValidator(),
 			$this->newBankDataValidator()
 		);
@@ -804,19 +804,19 @@ 
 	}
 
 	public function newPayPalUrlGenerator() {
-		return new PayPalUrlGenerator( $this->getPayPalUrlConfig() );
+		return new PayPalUrlGenerator($this->getPayPalUrlConfig());
 	}
 
 	private function getPayPalUrlConfig() {
-		return PayPalUrlConfig::newFromConfig( $this->config['paypal'] );
+		return PayPalUrlConfig::newFromConfig($this->config['paypal']);
 	}
 
 	private function newCreditCardUrlGenerator() {
-		return new CreditCardUrlGenerator( $this->newCreditCardUrlConfig() );
+		return new CreditCardUrlGenerator($this->newCreditCardUrlConfig());
 	}
 
 	private function newCreditCardUrlConfig() {
-		return CreditCardUrlConfig::newFromConfig( $this->config['creditcard'] );
+		return CreditCardUrlConfig::newFromConfig($this->config['creditcard']);
 	}
 
 	public function getDonationRepository(): DonationRepository {
@@ -834,22 +834,22 @@ 
 	}
 
 	private function newAmountFormatter(): AmountFormatter {
-		return new AmountFormatter( $this->config['locale'] );
+		return new AmountFormatter($this->config['locale']);
 	}
 
 	public function newDecimalNumberFormatter(): NumberFormatter {
-		return new NumberFormatter( $this->config['locale'], NumberFormatter::DECIMAL );
+		return new NumberFormatter($this->config['locale'], NumberFormatter::DECIMAL);
 	}
 
-	public function newAddCommentUseCase( string $updateToken ): AddCommentUseCase {
+	public function newAddCommentUseCase(string $updateToken): AddCommentUseCase {
 		return new AddCommentUseCase(
 			$this->getDonationRepository(),
-			$this->newDonationAuthorizer( $updateToken ),
-			$this->getTextPolicyValidator( 'comment' )
+			$this->newDonationAuthorizer($updateToken),
+			$this->getTextPolicyValidator('comment')
 		);
 	}
 
-	private function newDonationAuthorizer( string $updateToken = null, string $accessToken = null ): DonationAuthorizer {
+	private function newDonationAuthorizer(string $updateToken = null, string $accessToken = null): DonationAuthorizer {
 		return new DoctrineDonationAuthorizer(
 			$this->getEntityManager(),
 			$updateToken,
@@ -858,10 +858,10 @@ 
 	}
 
 	public function getTokenGenerator(): TokenGenerator {
-		if ( $this->tokenGenerator === null ) {
+		if ($this->tokenGenerator === null) {
 			$this->tokenGenerator = new RandomTokenGenerator(
 				$this->config['token-length'],
-				new \DateInterval( $this->config['token-validity-timestamp'] )
+				new \DateInterval($this->config['token-validity-timestamp'])
 			);
 		}
 
@@ -870,13 +870,13 @@ 
 
 	public function newDonationConfirmationPresenter() {
 		return new DonationConfirmationHtmlPresenter(
-			$this->getLayoutTemplate( 'DonationConfirmation.twig' )
+			$this->getLayoutTemplate('DonationConfirmation.twig')
 		);
 	}
 
 	public function newCreditCardPaymentHtmlPresenter() {
 		return new CreditCardPaymentHtmlPresenter(
-			$this->getIncludeTemplate( 'CreditCardPayment.html.twig' ),
+			$this->getIncludeTemplate('CreditCardPayment.html.twig'),
 			$this->getTranslator(),
 			$this->newCreditCardUrlGenerator()
 		);
@@ -884,7 +884,7 @@ 
 
 	public function newCancelDonationHtmlPresenter() {
 		return new CancelDonationHtmlPresenter(
-			$this->getIncludeTemplate( 'Donation_Cancellation_Confirmation.twig' )
+			$this->getIncludeTemplate('Donation_Cancellation_Confirmation.twig')
 		);
 	}
 
@@ -904,7 +904,7 @@ 
 			new TwigTemplate(
 				$this->getTwig(),
 				'Mail_Membership_Application_Confirmation.twig',
-				[ 'greeting_generator' => $this->getGreetingGenerator() ]
+				['greeting_generator' => $this->getGreetingGenerator()]
 			),
 			'mail_subject_confirm_membership_application'
 		);
@@ -919,7 +919,7 @@ 
 	}
 
 	private function newMembershipApplicationTracker(): MembershipApplicationTracker {
-		return new DoctrineMembershipApplicationTracker( $this->getEntityManager() );
+		return new DoctrineMembershipApplicationTracker($this->getEntityManager());
 	}
 
 	private function newMembershipAuthUpdater(): MembershipAppAuthUpdater {
@@ -928,9 +928,9 @@ 
 		);
 	}
 
-	public function newCancelMembershipApplicationUseCase( string $updateToken ): CancelMembershipApplicationUseCase {
+	public function newCancelMembershipApplicationUseCase(string $updateToken): CancelMembershipApplicationUseCase {
 		return new CancelMembershipApplicationUseCase(
-			$this->newMembershipApplicationAuthorizer( $updateToken ),
+			$this->newMembershipApplicationAuthorizer($updateToken),
 			$this->getMembershipApplicationRepository(),
 			$this->newCancelMembershipApplicationMailer()
 		);
@@ -955,20 +955,20 @@ 
 			new TwigTemplate(
 				$this->getTwig(),
 				'Mail_Membership_Application_Cancellation_Confirmation.twig',
-				[ 'greeting_generator' => $this->getGreetingGenerator() ]
+				['greeting_generator' => $this->getGreetingGenerator()]
 			),
 			'mail_subject_confirm_membership_application_cancellation'
 		);
 	}
 
-	public function newShowDonationConfirmationUseCase( string $accessToken ): ShowDonationConfirmationUseCase {
+	public function newShowDonationConfirmationUseCase(string $accessToken): ShowDonationConfirmationUseCase {
 		return new ShowDonationConfirmationUseCase(
-			$this->newDonationAuthorizer( null, $accessToken ),
+			$this->newDonationAuthorizer(null, $accessToken),
 			$this->getDonationRepository()
 		);
 	}
 
-	public function setDonationConfirmationPageSelector( DonationConfirmationPageSelector $selector ) {
+	public function setDonationConfirmationPageSelector(DonationConfirmationPageSelector $selector) {
 		$this->pimple['confirmation-page-selector'] = $selector;
 	}
 
@@ -977,17 +977,17 @@ 
 	}
 
 	public function newDonationFormViolationPresenter() {
-		$template = $this->getLayoutTemplate( 'DisplayPageLayout.twig' );
+		$template = $this->getLayoutTemplate('DisplayPageLayout.twig');
 		// TODO make this dependent on the 'form' value from the HTTP POST request
 		// (we need different form pages for A/B testing)
 		$template->context['main_template'] = 'DonationForm.html.twig';
-		return new DonationFormViolationPresenter( $template, $this->newAmountFormatter() );
+		return new DonationFormViolationPresenter($template, $this->newAmountFormatter());
 	}
 
-	public function newHandlePayPalPaymentNotificationUseCase( string $updateToken ) {
+	public function newHandlePayPalPaymentNotificationUseCase(string $updateToken) {
 		return new HandlePayPalPaymentNotificationUseCase(
 			$this->getDonationRepository(),
-			$this->newDonationAuthorizer( $updateToken ),
+			$this->newDonationAuthorizer($updateToken),
 			$this->newDonationConfirmationMailer(),
 			$this->getLogger(),
 			$this->newDonationEventLogger()
@@ -998,14 +998,14 @@ 
 		return $this->pimple['paypal-payment-notification-verifier'];
 	}
 
-	public function setPayPalPaymentNotificationVerifier( PaymentNotificationVerifier $verifier ) {
+	public function setPayPalPaymentNotificationVerifier(PaymentNotificationVerifier $verifier) {
 		$this->pimple['paypal-payment-notification-verifier'] = $verifier;
 	}
 
-	public function newCreditCardNotificationUseCase( string $updateToken ) {
+	public function newCreditCardNotificationUseCase(string $updateToken) {
 		return new CreditCardNotificationUseCase(
 			$this->getDonationRepository(),
-			$this->newDonationAuthorizer( $updateToken ),
+			$this->newDonationAuthorizer($updateToken),
 			$this->getCreditCardService(),
 			$this->newDonationConfirmationMailer(),
 			$this->getLogger(),
@@ -1015,23 +1015,23 @@ 
 
 	public function newCancelMembershipApplicationHtmlPresenter() {
 		return new CancelMembershipApplicationHtmlPresenter(
-			$this->getIncludeTemplate( 'Membership_Application_Cancellation_Confirmation.twig' )
+			$this->getIncludeTemplate('Membership_Application_Cancellation_Confirmation.twig')
 		);
 	}
 
 	public function newMembershipApplicationConfirmationHtmlPresenter() {
 		return new MembershipApplicationConfirmationHtmlPresenter(
-			$this->getIncludeTemplate( 'MembershipApplicationConfirmation.html.twig' )
+			$this->getIncludeTemplate('MembershipApplicationConfirmation.html.twig')
 		);
 	}
 
 	public function newMembershipFormViolationPresenter() {
 		return new MembershipFormViolationPresenter(
-			$this->getIncludeTemplate( 'MembershipApplication.html.twig' )
+			$this->getIncludeTemplate('MembershipApplication.html.twig')
 		);
 	}
 
-	public function setCreditCardService( CreditCardService $ccService ) {
+	public function setCreditCardService(CreditCardService $ccService) {
 		$this->pimple['credit-card-api-service'] = $ccService;
 	}
 
@@ -1044,7 +1044,7 @@ 
 			new TwigTemplate(
 				$this->getTwig(),
 				'CreditCardPaymentNotification.twig',
-				[ 'returnUrl' => $this->config['creditcard']['return-url'] ]
+				['returnUrl' => $this->config['creditcard']['return-url']]
 			)
 		);
 	}
@@ -1057,7 +1057,7 @@ 
 		);
 	}
 
-	public function setTokenGenerator( TokenGenerator $tokenGenerator ) {
+	public function setTokenGenerator(TokenGenerator $tokenGenerator) {
 		$this->tokenGenerator = $tokenGenerator;
 	}