wmde / FundraisingFrontend

contexts/MembershipContext/src/Domain/Repositories/ApplicationPurgedException.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\MembershipContext\Domain\Repositories;
 
@@ -11,7 +11,7 @@ 
 class ApplicationPurgedException extends GetMembershipApplicationException {
 
 	public function __construct() {
-		parent::__construct( 'Tried to access a purged Application' );
+		parent::__construct('Tried to access a purged Application');
 	}
 
 }

contexts/MembershipContext/src/Domain/Repositories/GetMembershipApplicationException.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\MembershipContext\Domain\Repositories;
 
@@ -10,7 +10,7 @@ 
  */
 class GetMembershipApplicationException extends \RuntimeException {
 
-	public function __construct( string $message = null, \Exception $previous = null ) {
+	public function __construct(string $message = null, \Exception $previous = null) {
 		parent::__construct(
 			$message ?? 'Could not get membership application',
 			0,

contexts/MembershipContext/src/DataAccess/DoctrineApplicationRepository.php

Braces +4 added lines, -8 removed lines

@@ -43,8 +43,7 @@ 
 	public function storeApplication( Application $application ): void {
 		if ( $application->hasId() ) {
 			$this->updateApplication( $application );
-		}
-		else {
+		} else {
 			$this->insertApplication( $application );
 		}
 	}
@@ -56,8 +55,7 @@ 
 		try {
 			$this->entityManager->persist( $doctrineApplication );
 			$this->entityManager->flush();
-		}
-		catch ( ORMException $ex ) {
+		} catch ( ORMException $ex ) {
 			throw new StoreMembershipApplicationException( $ex );
 		}
 
@@ -76,8 +74,7 @@ 
 		try {
 			$this->entityManager->persist( $doctrineApplication );
 			$this->entityManager->flush();
-		}
-		catch ( ORMException $ex ) {
+		} catch ( ORMException $ex ) {
 			throw new StoreMembershipApplicationException( $ex );
 		}
 	}
@@ -209,8 +206,7 @@ 
 	public function getApplicationById( int $id ): ?Application {
 		try {
 			$application = $this->getDoctrineApplicationById( $id );
-		}
-		catch ( ORMException $ex ) {
+		} catch ( ORMException $ex ) {
 			throw new GetMembershipApplicationException( null, $ex );
 		}
 

Spacing +127 added lines, -127 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\MembershipContext\DataAccess;
 
@@ -36,108 +36,108 @@ 
 
 	private $entityManager;
 
-	public function __construct( EntityManager $entityManager ) {
+	public function __construct(EntityManager $entityManager) {
 		$this->entityManager = $entityManager;
 	}
 
-	public function storeApplication( Application $application ): void {
-		if ( $application->hasId() ) {
-			$this->updateApplication( $application );
+	public function storeApplication(Application $application): void {
+		if ($application->hasId()) {
+			$this->updateApplication($application);
 		}
 		else {
-			$this->insertApplication( $application );
+			$this->insertApplication($application);
 		}
 	}
 
-	private function insertApplication( Application $application ): void {
+	private function insertApplication(Application $application): void {
 		$doctrineApplication = new DoctrineApplication();
-		$this->updateDoctrineApplication( $doctrineApplication, $application );
+		$this->updateDoctrineApplication($doctrineApplication, $application);
 
 		try {
-			$this->entityManager->persist( $doctrineApplication );
+			$this->entityManager->persist($doctrineApplication);
 			$this->entityManager->flush();
 		}
-		catch ( ORMException $ex ) {
-			throw new StoreMembershipApplicationException( $ex );
+		catch (ORMException $ex) {
+			throw new StoreMembershipApplicationException($ex);
 		}
 
-		$application->assignId( $doctrineApplication->getId() );
+		$application->assignId($doctrineApplication->getId());
 	}
 
-	private function updateApplication( Application $application ): void {
-		$doctrineApplication = $this->getDoctrineApplicationById( $application->getId() );
+	private function updateApplication(Application $application): void {
+		$doctrineApplication = $this->getDoctrineApplicationById($application->getId());
 
-		if ( $doctrineApplication === null ) {
+		if ($doctrineApplication === null) {
 			throw new StoreMembershipApplicationException();
 		}
 
-		$this->updateDoctrineApplication( $doctrineApplication, $application );
+		$this->updateDoctrineApplication($doctrineApplication, $application);
 
 		try {
-			$this->entityManager->persist( $doctrineApplication );
+			$this->entityManager->persist($doctrineApplication);
 			$this->entityManager->flush();
 		}
-		catch ( ORMException $ex ) {
-			throw new StoreMembershipApplicationException( $ex );
+		catch (ORMException $ex) {
+			throw new StoreMembershipApplicationException($ex);
 		}
 	}
 
-	private function updateDoctrineApplication( DoctrineApplication $doctrineApplication, Application $application ): void {
-		$doctrineApplication->setId( $application->getId() );
-		$doctrineApplication->setMembershipType( $application->getType() );
+	private function updateDoctrineApplication(DoctrineApplication $doctrineApplication, Application $application): void {
+		$doctrineApplication->setId($application->getId());
+		$doctrineApplication->setMembershipType($application->getType());
 
-		$this->setApplicantFields( $doctrineApplication, $application->getApplicant() );
-		$this->setPaymentFields( $doctrineApplication, $application->getPayment() );
-		$doctrineApplication->setDonationReceipt( $application->getDonationReceipt() );
+		$this->setApplicantFields($doctrineApplication, $application->getApplicant());
+		$this->setPaymentFields($doctrineApplication, $application->getPayment());
+		$doctrineApplication->setDonationReceipt($application->getDonationReceipt());
 
-		$doctrineStatus = $this->getDoctrineStatus( $application );
-		$this->preserveDoctrineStatus( $doctrineApplication, $doctrineStatus );
-		$doctrineApplication->setStatus( $doctrineStatus );
+		$doctrineStatus = $this->getDoctrineStatus($application);
+		$this->preserveDoctrineStatus($doctrineApplication, $doctrineStatus);
+		$doctrineApplication->setStatus($doctrineStatus);
 	}
 
-	private function setApplicantFields( DoctrineApplication $application, Applicant $applicant ): void {
-		$application->setApplicantFirstName( $applicant->getName()->getFirstName() );
-		$application->setApplicantLastName( $applicant->getName()->getLastName() );
-		$application->setApplicantSalutation( $applicant->getName()->getSalutation() );
-		$application->setApplicantTitle( $applicant->getName()->getTitle() );
-		$application->setCompany( $applicant->getName()->getCompanyName() );
+	private function setApplicantFields(DoctrineApplication $application, Applicant $applicant): void {
+		$application->setApplicantFirstName($applicant->getName()->getFirstName());
+		$application->setApplicantLastName($applicant->getName()->getLastName());
+		$application->setApplicantSalutation($applicant->getName()->getSalutation());
+		$application->setApplicantTitle($applicant->getName()->getTitle());
+		$application->setCompany($applicant->getName()->getCompanyName());
 
-		$application->setApplicantDateOfBirth( $applicant->getDateOfBirth() );
+		$application->setApplicantDateOfBirth($applicant->getDateOfBirth());
 
-		$application->setApplicantEmailAddress( $applicant->getEmailAddress()->getFullAddress() );
-		$application->setApplicantPhoneNumber( $applicant->getPhoneNumber()->__toString() );
+		$application->setApplicantEmailAddress($applicant->getEmailAddress()->getFullAddress());
+		$application->setApplicantPhoneNumber($applicant->getPhoneNumber()->__toString());
 
 		$address = $applicant->getPhysicalAddress();
 
-		$application->setCity( $address->getCity() );
-		$application->setCountry( $address->getCountryCode() );
-		$application->setPostcode( $address->getPostalCode() );
-		$application->setAddress( $address->getStreetAddress() );
+		$application->setCity($address->getCity());
+		$application->setCountry($address->getCountryCode());
+		$application->setPostcode($address->getPostalCode());
+		$application->setAddress($address->getStreetAddress());
 	}
 
-	private function setPaymentFields( DoctrineApplication $application, Payment $payment ): void {
-		$application->setPaymentIntervalInMonths( $payment->getIntervalInMonths() );
-		$application->setPaymentAmount( (int)$payment->getAmount()->getEuroFloat() );
+	private function setPaymentFields(DoctrineApplication $application, Payment $payment): void {
+		$application->setPaymentIntervalInMonths($payment->getIntervalInMonths());
+		$application->setPaymentAmount((int)$payment->getAmount()->getEuroFloat());
 		$paymentMethod = $payment->getPaymentMethod();
 
-		$application->setPaymentType( $paymentMethod->getType() );
-		if ( $paymentMethod instanceof DirectDebitPayment ) {
-			$this->setBankDataFields( $application, $paymentMethod->getBankData() );
-		} elseif ( $paymentMethod instanceof PayPalPayment && $paymentMethod->getPayPalData() != new PayPalData() ) {
-			$this->setPayPalDataFields( $application, $paymentMethod->getPayPalData() );
+		$application->setPaymentType($paymentMethod->getType());
+		if ($paymentMethod instanceof DirectDebitPayment) {
+			$this->setBankDataFields($application, $paymentMethod->getBankData());
+		} elseif ($paymentMethod instanceof PayPalPayment && $paymentMethod->getPayPalData() != new PayPalData()) {
+			$this->setPayPalDataFields($application, $paymentMethod->getPayPalData());
 		}
 	}
 
-	private function setBankDataFields( DoctrineApplication $application, BankData $bankData ): void {
-		$application->setPaymentBankAccount( $bankData->getAccount() );
-		$application->setPaymentBankCode( $bankData->getBankCode() );
-		$application->setPaymentBankName( $bankData->getBankName() );
-		$application->setPaymentBic( $bankData->getBic() );
-		$application->setPaymentIban( $bankData->getIban()->toString() );
+	private function setBankDataFields(DoctrineApplication $application, BankData $bankData): void {
+		$application->setPaymentBankAccount($bankData->getAccount());
+		$application->setPaymentBankCode($bankData->getBankCode());
+		$application->setPaymentBankName($bankData->getBankName());
+		$application->setPaymentBic($bankData->getBic());
+		$application->setPaymentIban($bankData->getIban()->toString());
 	}
 
-	private function setPayPalDataFields( DoctrineApplication $application, PayPalData $payPalData ): void {
-		$application->encodeAndSetData( array_merge(
+	private function setPayPalDataFields(DoctrineApplication $application, PayPalData $payPalData): void {
+		$application->encodeAndSetData(array_merge(
 			$application->getDecodedData(),
 			[
 				'paypal_payer_id' => $payPalData->getPayerId(),
@@ -159,43 +159,43 @@ 
 				'ext_payment_timestamp' => $payPalData->getPaymentTimestamp(),
 				'first_payment_date' => $payPalData->getFirstPaymentDate()
 			]
-		) );
+		));
 	}
 
-	private function getDoctrineStatus( Application $application ): int {
+	private function getDoctrineStatus(Application $application): int {
 		$status = DoctrineApplication::STATUS_NEUTRAL;
 
-		if ( $application->needsModeration() ) {
+		if ($application->needsModeration()) {
 			$status += DoctrineApplication::STATUS_MODERATION;
 		}
 
-		if ( $application->isCancelled() ) {
+		if ($application->isCancelled()) {
 			$status += DoctrineApplication::STATUS_CANCELED;
 		}
 
-		if ( $application->isDeleted() ) {
+		if ($application->isDeleted()) {
 			$status += DoctrineApplication::STATUS_DELETED;
 		}
 
-		if ( $application->isConfirmed() || $this->isAutoConfirmed( $status, $application ) ) {
+		if ($application->isConfirmed() || $this->isAutoConfirmed($status, $application)) {
 			$status += DoctrineApplication::STATUS_CONFIRMED;
 		}
 
 		return $status;
 	}
 
-	private function isAutoConfirmed( int $status, Application $application ): bool {
-		return $status === DoctrineApplication::STATUS_NEUTRAL && $this->isDirectDebitPayment( $application );
+	private function isAutoConfirmed(int $status, Application $application): bool {
+		return $status === DoctrineApplication::STATUS_NEUTRAL && $this->isDirectDebitPayment($application);
 	}
 
-	private function isDirectDebitPayment( Application $application ): bool {
+	private function isDirectDebitPayment(Application $application): bool {
 		return $application->getPayment()->getPaymentMethod()->getType() === PaymentType::DIRECT_DEBIT;
 	}
 
-	private function preserveDoctrineStatus( DoctrineApplication $doctrineApplication, int $doctrineStatus ): void {
-		if ( $doctrineStatus < DoctrineApplication::STATUS_CONFIRMED ) {
-			$doctrineApplication->modifyDataObject( function ( MembershipApplicationData $data ): void {
-				$data->setPreservedStatus( DoctrineApplication::STATUS_CONFIRMED );
+	private function preserveDoctrineStatus(DoctrineApplication $doctrineApplication, int $doctrineStatus): void {
+		if ($doctrineStatus < DoctrineApplication::STATUS_CONFIRMED) {
+			$doctrineApplication->modifyDataObject(function(MembershipApplicationData $data): void {
+				$data->setPreservedStatus(DoctrineApplication::STATUS_CONFIRMED);
 			} );
 		}
 	}
@@ -206,15 +206,15 @@ 
 	 * @return Application|null
 	 * @throws GetMembershipApplicationException
 	 */
-	public function getApplicationById( int $id ): ?Application {
+	public function getApplicationById(int $id): ?Application {
 		try {
-			$application = $this->getDoctrineApplicationById( $id );
+			$application = $this->getDoctrineApplicationById($id);
 		}
-		catch ( ORMException $ex ) {
-			throw new GetMembershipApplicationException( null, $ex );
+		catch (ORMException $ex) {
+			throw new GetMembershipApplicationException(null, $ex);
 		}
 
-		if ( $application === null ) {
+		if ($application === null) {
 			return null;
 		}
 
@@ -231,25 +231,25 @@ 
 	 * @return DoctrineApplication|null
 	 * @throws ORMException
 	 */
-	public function getDoctrineApplicationById( int $id ): ?DoctrineApplication {
-		return $this->entityManager->find( DoctrineApplication::class, $id );
+	public function getDoctrineApplicationById(int $id): ?DoctrineApplication {
+		return $this->entityManager->find(DoctrineApplication::class, $id);
 	}
 
-	private function newApplicationDomainEntity( DoctrineApplication $application ): Application {
+	private function newApplicationDomainEntity(DoctrineApplication $application): Application {
 		return new Application(
 			$application->getId(),
 			$application->getMembershipType(),
 			new Applicant(
-				$this->newPersonName( $application ),
-				$this->newAddress( $application ),
-				new EmailAddress( $application->getApplicantEmailAddress() ),
-				new PhoneNumber( $application->getApplicantPhoneNumber() ),
+				$this->newPersonName($application),
+				$this->newAddress($application),
+				new EmailAddress($application->getApplicantEmailAddress()),
+				new PhoneNumber($application->getApplicantPhoneNumber()),
 				$application->getApplicantDateOfBirth()
 			),
 			new Payment(
 				$application->getPaymentIntervalInMonths(),
-				Euro::newFromFloat( $application->getPaymentAmount() ),
-				$this->newPaymentMethod( $application )
+				Euro::newFromFloat($application->getPaymentAmount()),
+				$this->newPaymentMethod($application)
 			),
 			$application->needsModeration(),
 			$application->isCancelled(),
@@ -259,77 +259,77 @@ 
 		);
 	}
 
-	private function newPersonName( DoctrineApplication $application ): ApplicantName {
-		if ( empty( $application->getCompany() ) ) {
+	private function newPersonName(DoctrineApplication $application): ApplicantName {
+		if (empty($application->getCompany())) {
 			$personName = ApplicantName::newPrivatePersonName();
-			$personName->setFirstName( $application->getApplicantFirstName() );
-			$personName->setLastName( $application->getApplicantLastName() );
-			$personName->setSalutation( $application->getApplicantSalutation() );
-			$personName->setTitle( $application->getApplicantTitle() );
+			$personName->setFirstName($application->getApplicantFirstName());
+			$personName->setLastName($application->getApplicantLastName());
+			$personName->setSalutation($application->getApplicantSalutation());
+			$personName->setTitle($application->getApplicantTitle());
 		} else {
 			$personName = ApplicantName::newCompanyName();
-			$personName->setCompanyName( $application->getCompany() );
-			$personName->setSalutation( $application->getApplicantSalutation() );
+			$personName->setCompanyName($application->getCompany());
+			$personName->setSalutation($application->getApplicantSalutation());
 		}
 
 		return $personName->freeze()->assertNoNullFields();
 	}
 
-	private function newAddress( DoctrineApplication $application ): ApplicantAddress {
+	private function newAddress(DoctrineApplication $application): ApplicantAddress {
 		$address = new ApplicantAddress();
 
-		$address->setCity( $application->getCity() );
-		$address->setCountryCode( $application->getCountry() );
-		$address->setPostalCode( $application->getPostcode() );
-		$address->setStreetAddress( $application->getAddress() );
+		$address->setCity($application->getCity());
+		$address->setCountryCode($application->getCountry());
+		$address->setPostalCode($application->getPostcode());
+		$address->setStreetAddress($application->getAddress());
 
 		return $address->freeze()->assertNoNullFields();
 	}
 
-	private function newPaymentMethod( DoctrineApplication $application ): PaymentMethod {
-		if ( $application->getPaymentType() === PaymentType::DIRECT_DEBIT ) {
-			return new DirectDebitPayment( $this->newBankData( $application ) );
+	private function newPaymentMethod(DoctrineApplication $application): PaymentMethod {
+		if ($application->getPaymentType() === PaymentType::DIRECT_DEBIT) {
+			return new DirectDebitPayment($this->newBankData($application));
 		}
 
-		if ( $application->getPaymentType() === PaymentType::PAYPAL ) {
-			return new PayPalPayment( $this->newPayPalData( $application ) );
+		if ($application->getPaymentType() === PaymentType::PAYPAL) {
+			return new PayPalPayment($this->newPayPalData($application));
 		}
 
-		throw new \RuntimeException( 'Unsupported payment type' );
+		throw new \RuntimeException('Unsupported payment type');
 	}
 
-	private function newBankData( DoctrineApplication $application ): BankData {
+	private function newBankData(DoctrineApplication $application): BankData {
 		$bankData = new BankData();
 
-		$bankData->setAccount( $application->getPaymentBankAccount() );
-		$bankData->setBankCode( $application->getPaymentBankCode() );
-		$bankData->setBankName( $application->getPaymentBankName() );
-		$bankData->setBic( $application->getPaymentBic() );
-		$bankData->setIban( new Iban( $application->getPaymentIban() ) );
+		$bankData->setAccount($application->getPaymentBankAccount());
+		$bankData->setBankCode($application->getPaymentBankCode());
+		$bankData->setBankName($application->getPaymentBankName());
+		$bankData->setBic($application->getPaymentBic());
+		$bankData->setIban(new Iban($application->getPaymentIban()));
 
 		return $bankData->freeze()->assertNoNullFields();
 	}
 
-	private function newPayPalData( DoctrineApplication $application ): ?PayPalData {
+	private function newPayPalData(DoctrineApplication $application): ?PayPalData {
 		$data = $application->getDecodedData();
 
-		return ( new PayPalData() )
-			->setPayerId( $data['paypal_payer_id'] ?? '' )
-			->setSubscriberId( $data['paypal_subscr_id'] ?? '' )
-			->setPayerStatus( $data['paypal_payer_status'] ?? '' )
-			->setAddressStatus( $data['paypal_address_status'] ?? '' )
-			->setAmount( Euro::newFromString( $data['paypal_mc_gross'] ?? '0' ) )
-			->setCurrencyCode( $data['paypal_mc_currency'] ?? '' )
-			->setFee( Euro::newFromString( $data['paypal_mc_fee'] ?? '0' ) )
-			->setSettleAmount( Euro::newFromString( $data['paypal_settle_amount'] ?? '0' ) )
-			->setFirstName( $data['paypal_first_name'] ?? '' )
-			->setLastName( $data['paypal_last_name'] ?? '' )
-			->setAddressName( $data['paypal_address_name'] ?? '' )
-			->setPaymentId( $data['ext_payment_id'] ?? '' )
-			->setPaymentType( $data['ext_payment_type'] ?? '' )
-			->setPaymentStatus( $data['ext_payment_status'] ?? '' )
-			->setPaymentTimestamp( $data['ext_payment_timestamp'] ?? '' )
-			->setFirstPaymentDate( $data['first_payment_date'] ?? '' )
+		return (new PayPalData())
+			->setPayerId($data['paypal_payer_id'] ?? '')
+			->setSubscriberId($data['paypal_subscr_id'] ?? '')
+			->setPayerStatus($data['paypal_payer_status'] ?? '')
+			->setAddressStatus($data['paypal_address_status'] ?? '')
+			->setAmount(Euro::newFromString($data['paypal_mc_gross'] ?? '0'))
+			->setCurrencyCode($data['paypal_mc_currency'] ?? '')
+			->setFee(Euro::newFromString($data['paypal_mc_fee'] ?? '0'))
+			->setSettleAmount(Euro::newFromString($data['paypal_settle_amount'] ?? '0'))
+			->setFirstName($data['paypal_first_name'] ?? '')
+			->setLastName($data['paypal_last_name'] ?? '')
+			->setAddressName($data['paypal_address_name'] ?? '')
+			->setPaymentId($data['ext_payment_id'] ?? '')
+			->setPaymentType($data['ext_payment_type'] ?? '')
+			->setPaymentStatus($data['ext_payment_status'] ?? '')
+			->setPaymentTimestamp($data['ext_payment_timestamp'] ?? '')
+			->setFirstPaymentDate($data['first_payment_date'] ?? '')
 			->freeze()->assertNoNullFields();
 	}
 

tests/Integration/Presentation/Presenters/MembershipApplicationConfirmationHtmlPresenterTest.php

Spacing +11 added lines, -11 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\Tests\Integration\Presentation\Presenters;
 
@@ -21,18 +21,18 @@ 
 	private const STATUS_UNCONFIRMED = 'status-unconfirmed';
 
 	/** @dataProvider applicationStatusProvider */
-	public function testWhenPresenterPresents_itPassesParametersToTemplate( bool $isConfirmed, string $expectedMappedStatus ): void {
-		$twig = $this->getMockBuilder( TwigTemplate::class )->disableOriginalConstructor()->getMock();
-		$twig->expects( $this->once() )
-			->method( 'render' )
-			->with( $this->getExpectedRenderParams( $expectedMappedStatus ) );
+	public function testWhenPresenterPresents_itPassesParametersToTemplate(bool $isConfirmed, string $expectedMappedStatus): void {
+		$twig = $this->getMockBuilder(TwigTemplate::class)->disableOriginalConstructor()->getMock();
+		$twig->expects($this->once())
+			->method('render')
+			->with($this->getExpectedRenderParams($expectedMappedStatus));
 
 		$membershipApplication = ValidMembershipApplication::newDomainEntityUsingPayPal();
-		if ( $isConfirmed === true ) {
+		if ($isConfirmed === true) {
 			$membershipApplication->confirm();
 		}
 
-		$presenter = new MembershipApplicationConfirmationHtmlPresenter( $twig );
+		$presenter = new MembershipApplicationConfirmationHtmlPresenter($twig);
 		$presenter->present(
 			ShowApplicationConfirmationResponse::newValidResponse(
 				$membershipApplication,
@@ -43,12 +43,12 @@ 
 
 	public function applicationStatusProvider(): array {
 		return [
-			[ true, self::STATUS_BOOKED ],
-			[ false, self::STATUS_UNCONFIRMED ]
+			[true, self::STATUS_BOOKED],
+			[false, self::STATUS_UNCONFIRMED]
 		];
 	}
 
-	private function getExpectedRenderParams( string $mappedStatus ): array {
+	private function getExpectedRenderParams(string $mappedStatus): array {
 		return [
 			'membershipApplication' => [
 				'id' => null,

src/Factories/FunFunFactory.php

Spacing +204 added lines, -204 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\Factories;
 
@@ -190,7 +190,7 @@ 
 	 */
 	private $profiler = null;
 
-	public function __construct( array $config ) {
+	public function __construct(array $config) {
 		$this->config = $config;
 		$this->pimple = $this->newPimple();
 	}
@@ -215,15 +215,15 @@ 
 		};
 
 		$pimple['dbal_connection'] = function() {
-			return DriverManager::getConnection( $this->config['db'] );
+			return DriverManager::getConnection($this->config['db']);
 		};
 
 		$pimple['entity_manager'] = function() {
-			$entityManager = ( new StoreFactory( $this->getConnection(), $this->getVarPath() . '/doctrine_proxies' ) )
+			$entityManager = (new StoreFactory($this->getConnection(), $this->getVarPath() . '/doctrine_proxies'))
 				->getEntityManager();
-			if ( $this->addDoctrineSubscribers ) {
-				$entityManager->getEventManager()->addEventSubscriber( $this->newDoctrineDonationPrePersistSubscriber() );
-				$entityManager->getEventManager()->addEventSubscriber( $this->newDoctrineMembershipApplicationPrePersistSubscriber() );
+			if ($this->addDoctrineSubscribers) {
+				$entityManager->getEventManager()->addEventSubscriber($this->newDoctrineDonationPrePersistSubscriber());
+				$entityManager->getEventManager()->addEventSubscriber($this->newDoctrineMembershipApplicationPrePersistSubscriber());
 			}
 
 			return $entityManager;
@@ -231,53 +231,53 @@ 
 
 		$pimple['subscription_repository'] = function() {
 			return new LoggingSubscriptionRepository(
-				new DoctrineSubscriptionRepository( $this->getEntityManager() ),
+				new DoctrineSubscriptionRepository($this->getEntityManager()),
 				$this->getLogger()
 			);
 		};
 
 		$pimple['donation_repository'] = function() {
 			return new LoggingDonationRepository(
-				new DoctrineDonationRepository( $this->getEntityManager() ),
+				new DoctrineDonationRepository($this->getEntityManager()),
 				$this->getLogger()
 			);
 		};
 
 		$pimple['membership_application_repository'] = function() {
 			return new LoggingApplicationRepository(
-				new DoctrineApplicationRepository( $this->getEntityManager() ),
+				new DoctrineApplicationRepository($this->getEntityManager()),
 				$this->getLogger()
 			);
 		};
 
 		$pimple['comment_repository'] = function() {
 			$finder = new LoggingCommentFinder(
-				new DoctrineCommentFinder( $this->getEntityManager() ),
+				new DoctrineCommentFinder($this->getEntityManager()),
 				$this->getLogger()
 			);
 
-			return $this->addProfilingDecorator( $finder, 'CommentFinder' );
+			return $this->addProfilingDecorator($finder, 'CommentFinder');
 		};
 
 		$pimple['mail_validator'] = function() {
-			return new EmailValidator( new InternetDomainNameValidator() );
+			return new EmailValidator(new InternetDomainNameValidator());
 		};
 
 		$pimple['subscription_validator'] = function() {
 			return new SubscriptionValidator(
 				$this->getEmailValidator(),
-				$this->newTextPolicyValidator( 'fields' ),
+				$this->newTextPolicyValidator('fields'),
 				$this->newSubscriptionDuplicateValidator(),
 				$this->newHonorificValidator()
 			);
 		};
 
 		$pimple['template_name_validator'] = function() {
-			return new TemplateNameValidator( $this->getSkinTwig() );
+			return new TemplateNameValidator($this->getSkinTwig());
 		};
 
 		$pimple['contact_validator'] = function() {
-			return new GetInTouchValidator( $this->getEmailValidator() );
+			return new GetInTouchValidator($this->getEmailValidator());
 		};
 
 		$pimple['greeting_generator'] = function() {
@@ -291,9 +291,9 @@ 
 			];
 			$locale = $this->config['locale'];
 			$messagesPath = $this->getI18nDirectory() . $this->config['translation']['message-dir'];
-			$translator = $translationFactory->create( $loaders, $locale );
+			$translator = $translationFactory->create($loaders, $locale);
 			foreach ($this->config['translation']['files'] as $domain => $file) {
-				$translator->addResource( 'json', $messagesPath . '/' . $file, $locale, $domain );
+				$translator->addResource('json', $messagesPath . '/' . $file, $locale, $domain);
 			}
 
 			return $translator;
@@ -301,27 +301,27 @@ 
 
 		// In the future, this could be locale-specific or filled from a DB table
 		$pimple['honorifics'] = function() {
-			return new Honorifics( [
+			return new Honorifics([
 				'' => 'Kein Titel',
 				'Dr.' => 'Dr.',
 				'Prof.' => 'Prof.',
 				'Prof. Dr.' => 'Prof. Dr.'
-			] );
+			]);
 		};
 
 		$pimple['twig'] = function() {
 			$config = $this->config['twig'];
 			$config['loaders']['filesystem']['template-dir'] = 'skins/' . $this->getSkinSettings()->getSkin() . '/templates';
 
-			$twigFactory = $this->newTwigFactory( $config );
+			$twigFactory = $this->newTwigFactory($config);
 			$configurator = $twigFactory->newTwigEnvironmentConfigurator();
 
-			$loaders = array_filter( [
+			$loaders = array_filter([
 				$twigFactory->newFileSystemLoader(),
 				$twigFactory->newArrayLoader(), // This is just a fallback for testing
-			] );
+			]);
 			$extensions = [
-				$twigFactory->newTranslationExtension( $this->getTranslator() ),
+				$twigFactory->newTranslationExtension($this->getTranslator()),
 				new Twig_Extensions_Extension_Intl()
 			];
 			$filters = [
@@ -332,48 +332,48 @@ 
 			$functions = [
 				new Twig_SimpleFunction(
 					'web_content',
-					function( string $name, array $context = [] ): string {
-						return $this->getContentProvider()->getWeb( $name, $context );
+					function(string $name, array $context = []): string {
+						return $this->getContentProvider()->getWeb($name, $context);
 					},
-					[ 'is_safe' => [ 'html' ] ]
+					['is_safe' => ['html']]
 				),
 			];
 
-			return $configurator->getEnvironment( $this->pimple['skin_twig_environment'], $loaders, $extensions, $filters, $functions );
+			return $configurator->getEnvironment($this->pimple['skin_twig_environment'], $loaders, $extensions, $filters, $functions);
 		};
 
 		$pimple['mailer_twig'] = function() {
-			$twigFactory = $this->newTwigFactory( $this->config['mailer-twig'] );
+			$twigFactory = $this->newTwigFactory($this->config['mailer-twig']);
 			$configurator = $twigFactory->newTwigEnvironmentConfigurator();
 
-			$loaders = array_filter( [
+			$loaders = array_filter([
 				$twigFactory->newFileSystemLoader(),
 				$twigFactory->newArrayLoader(), // This is just a fallback for testing
-			] );
+			]);
 			$extensions = [
-				$twigFactory->newTranslationExtension( $this->getTranslator() ),
+				$twigFactory->newTranslationExtension($this->getTranslator()),
 				new Twig_Extensions_Extension_Intl(),
 			];
 			$filters = [];
 			$functions = [
 				new Twig_SimpleFunction(
 					'mail_content',
-					function( string $name, array $context = [] ): string {
-						return $this->getContentProvider()->getMail( $name, $context );
+					function(string $name, array $context = []): string {
+						return $this->getContentProvider()->getMail($name, $context);
 					},
-					[ 'is_safe' => [ 'all' ] ]
+					['is_safe' => ['all']]
 				),
 				new Twig_SimpleFunction(
 					'url',
-					function( string $name, array $parameters = [] ): string {
-						return $this->getUrlGenerator()->generateUrl( $name, $parameters );
+					function(string $name, array $parameters = []): string {
+						return $this->getUrlGenerator()->generateUrl($name, $parameters);
 					}
 				)
 			];
 
 			$twigEnvironment = new Twig_Environment();
 
-			return $configurator->getEnvironment( $twigEnvironment, $loaders, $extensions, $filters, $functions );
+			return $configurator->getEnvironment($twigEnvironment, $loaders, $extensions, $filters, $functions);
 		};
 
 		$pimple['messenger_suborganization'] = function() {
@@ -393,7 +393,7 @@ 
 		};
 
 		$pimple['confirmation-page-selector'] = function() {
-			return new DonationConfirmationPageSelector( $this->config['confirmation-pages'] );
+			return new DonationConfirmationPageSelector($this->config['confirmation-pages']);
 		};
 
 		$pimple['paypal-payment-notification-verifier'] = function() {
@@ -432,14 +432,14 @@ 
 		$pimple['donation_token_generator'] = function() {
 			return new RandomTokenGenerator(
 				$this->config['token-length'],
-				new \DateInterval( $this->config['token-validity-timestamp'] )
+				new \DateInterval($this->config['token-validity-timestamp'])
 			);
 		};
 
 		$pimple['membership_token_generator'] = function() {
 			return new RandomMembershipTokenGenerator(
 				$this->config['token-length'],
-				new \DateInterval( $this->config['token-validity-timestamp'] )
+				new \DateInterval($this->config['token-validity-timestamp'])
 			);
 		};
 
@@ -451,41 +451,41 @@ 
 			return new VoidCache();
 		};
 
-		$pimple['page_view_tracker'] = function () {
-			return new PageViewTracker( $this->newServerSideTracker(), $this->config['piwik']['siteUrlBase'] );
+		$pimple['page_view_tracker'] = function() {
+			return new PageViewTracker($this->newServerSideTracker(), $this->config['piwik']['siteUrlBase']);
 		};
 
-		$pimple['cachebusting_fileprefixer'] = function () {
-			return new FilePrefixer( $this->getFilePrefix() );
+		$pimple['cachebusting_fileprefixer'] = function() {
+			return new FilePrefixer($this->getFilePrefix());
 		};
 
-		$pimple['content_page_selector'] = function () {
-			$json = (new SimpleFileFetcher())->fetchFile( $this->getI18nDirectory() . '/data/pages.json' );
-			$config = json_decode( $json, true ) ?? [];
+		$pimple['content_page_selector'] = function() {
+			$json = (new SimpleFileFetcher())->fetchFile($this->getI18nDirectory() . '/data/pages.json');
+			$config = json_decode($json, true) ?? [];
 
-			return new PageSelector( $config );
+			return new PageSelector($config);
 		};
 
-		$pimple['content_provider'] = function () {
-			return new ContentProvider( [
+		$pimple['content_provider'] = function() {
+			return new ContentProvider([
 				'content_path' => $this->getI18nDirectory(),
 				'cache' => $this->config['twig']['enable-cache'] ? $this->getCachePath() . '/content' : false,
 				'globals' => [
 					'basepath' => $this->config['web-basepath']
 				]
-			] );
+			]);
 		};
 
 		$pimple['payment-delay-calculator'] = function() {
-			return new DefaultPaymentDelayCalculator( $this->getPaymentDelayInDays() );
+			return new DefaultPaymentDelayCalculator($this->getPaymentDelayInDays());
 		};
 
-		$pimple['sofort-client'] = function () {
+		$pimple['sofort-client'] = function() {
 			$config = $this->config['sofort'];
-			return new SofortClient( $config['config-key'] );
+			return new SofortClient($config['config-key']);
 		};
 
-		$pimple['cookie-builder'] = function (): CookieBuilder {
+		$pimple['cookie-builder'] = function(): CookieBuilder {
 			return new CookieBuilder(
 				$this->config['cookie']['expiration'],
 				$this->config['cookie']['path'],
@@ -497,13 +497,13 @@ 
 			);
 		};
 
-		$pimple['skin-settings'] = function (): SkinSettings {
+		$pimple['skin-settings'] = function(): SkinSettings {
 			$config = $this->config['skin'];
-			return new SkinSettings( $config['options'], $config['default'], $config['cookie-lifetime'] );
+			return new SkinSettings($config['options'], $config['default'], $config['cookie-lifetime']);
 		};
 
-		$pimple['payment-types-settings'] = function (): PaymentTypesSettings {
-			return new PaymentTypesSettings( $this->config['payment-types'] );
+		$pimple['payment-types-settings'] = function(): PaymentTypesSettings {
+			return new PaymentTypesSettings($this->config['payment-types']);
 		};
 
 		return $pimple;
@@ -519,17 +519,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 {
@@ -537,14 +537,14 @@ 
 	}
 
 	public function newCommentListRssPresenter(): CommentListRssPresenter {
-		return new CommentListRssPresenter( new TwigTemplate(
+		return new CommentListRssPresenter(new TwigTemplate(
 			$this->getSkinTwig(),
 			'Comment_List.rss.twig'
-		) );
+		));
 	}
 
 	public function newCommentListHtmlPresenter(): CommentListHtmlPresenter {
-		return new CommentListHtmlPresenter( $this->getLayoutTemplate( 'Comment_List.html.twig', [ 'piwikGoals' => [ 1 ] ] ) );
+		return new CommentListHtmlPresenter($this->getLayoutTemplate('Comment_List.html.twig', ['piwikGoals' => [1]]));
 	}
 
 	private function getCommentFinder(): CommentFinder {
@@ -555,7 +555,7 @@ 
 		return $this->pimple['subscription_repository'];
 	}
 
-	public function setSubscriptionRepository( SubscriptionRepository $subscriptionRepository ): void {
+	public function setSubscriptionRepository(SubscriptionRepository $subscriptionRepository): void {
 		$this->pimple['subscription_repository'] = $subscriptionRepository;
 	}
 
@@ -572,25 +572,25 @@ 
 	}
 
 	public function newAddSubscriptionHtmlPresenter(): AddSubscriptionHtmlPresenter {
-		return new AddSubscriptionHtmlPresenter( $this->getLayoutTemplate( 'Subscription_Form.html.twig' ), $this->getTranslator() );
+		return new AddSubscriptionHtmlPresenter($this->getLayoutTemplate('Subscription_Form.html.twig'), $this->getTranslator());
 	}
 
 	public function newConfirmSubscriptionHtmlPresenter(): ConfirmSubscriptionHtmlPresenter {
 		return new ConfirmSubscriptionHtmlPresenter(
-			$this->getLayoutTemplate( 'Confirm_Subscription.twig' ),
+			$this->getLayoutTemplate('Confirm_Subscription.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( 'contact_form.html.twig' ), $this->getTranslator() );
+		return new GetInTouchHtmlPresenter($this->getLayoutTemplate('contact_form.html.twig'), $this->getTranslator());
 	}
 
-	public function setSkinTwigEnvironment( Twig_Environment $twig ): void {
+	public function setSkinTwigEnvironment(Twig_Environment $twig): void {
 		$this->pimple['skin_twig_environment'] = $twig;
 	}
 
@@ -609,19 +609,19 @@ 
 	 * @param array $context Additional variables for the template
 	 * @return TwigTemplate
 	 */
-	public function getLayoutTemplate( string $templateName, array $context = [] ): TwigTemplate {
+	public function getLayoutTemplate(string $templateName, array $context = []): TwigTemplate {
 		 return new TwigTemplate(
 			$this->getSkinTwig(),
 			$templateName,
-			array_merge( $this->getDefaultTwigVariables(), $context )
+			array_merge($this->getDefaultTwigVariables(), $context)
 		);
 	}
 
-	public function getMailerTemplate( string $templateName, array $context = [] ): TwigTemplate {
+	public function getMailerTemplate(string $templateName, array $context = []): TwigTemplate {
 		return new TwigTemplate(
 			$this->getMailerTwig(),
 			$templateName,
-			array_merge( $this->getDefaultTwigVariables(), $context )
+			array_merge($this->getDefaultTwigVariables(), $context)
 		);
 	}
 
@@ -702,7 +702,7 @@ 
 			new TwigTemplate(
 					$this->getMailerTwig(),
 					'Subscription_Confirmation.txt.twig',
-					[ 'greeting_generator' => $this->getGreetingGenerator() ]
+					['greeting_generator' => $this->getGreetingGenerator()]
 			),
 			'mail_subject_subscription_confirmed'
 		);
@@ -714,16 +714,16 @@ 
 	 * So much decoration going on that explicitly hinting what we return (Robustness principle) would be confusing
 	 * (you'd expect a TemplateBasedMailer, not a LoggingMailer), so we hint the interface instead.
 	 */
-	private function newTemplateMailer( Messenger $messenger, TwigTemplate $template, string $messageKey ): TemplateMailerInterface {
+	private function newTemplateMailer(Messenger $messenger, TwigTemplate $template, string $messageKey): TemplateMailerInterface {
 		$mailer = new TemplateBasedMailer(
 			$messenger,
 			$template,
-			$this->getTranslator()->trans( $messageKey )
+			$this->getTranslator()->trans($messageKey)
 		);
 
-		$mailer = new LoggingMailer( $mailer, $this->getLogger() );
+		$mailer = new LoggingMailer($mailer, $this->getLogger());
 
-		return $this->addProfilingDecorator( $mailer, 'Mailer' );
+		return $this->addProfilingDecorator($mailer, 'Mailer');
 	}
 
 	public function getGreetingGenerator(): GreetingGenerator {
@@ -731,11 +731,11 @@ 
 	}
 
 	public function newCheckIbanUseCase(): CheckIbanUseCase {
-		return new CheckIbanUseCase( $this->newBankDataConverter(), $this->newIbanValidator() );
+		return new CheckIbanUseCase($this->newBankDataConverter(), $this->newIbanValidator());
 	}
 
 	public function newGenerateIbanUseCase(): GenerateIbanUseCase {
-		return new GenerateIbanUseCase( $this->newBankDataConverter(), $this->newIbanValidator() );
+		return new GenerateIbanUseCase($this->newBankDataConverter(), $this->newIbanValidator());
 	}
 
 	public function newIbanPresenter(): IbanPresenter {
@@ -743,10 +743,10 @@ 
 	}
 
 	public function newBankDataConverter(): BankDataConverter {
-		return new BankDataConverter( $this->config['bank-data-file'] );
+		return new BankDataConverter($this->config['bank-data-file']);
 	}
 
-	public function setSubscriptionValidator( SubscriptionValidator $subscriptionValidator ): void {
+	public function setSubscriptionValidator(SubscriptionValidator $subscriptionValidator): void {
 		$this->pimple['subscription_validator'] = $subscriptionValidator;
 	}
 
@@ -761,7 +761,7 @@ 
 	private function newContactUserMailer(): TemplateMailerInterface {
 		return $this->newTemplateMailer(
 			$this->getSuborganizationMessenger(),
-			new TwigTemplate( $this->getMailerTwig(), 'Contact_Confirm_to_User.txt.twig' ),
+			new TwigTemplate($this->getMailerTwig(), 'Contact_Confirm_to_User.txt.twig'),
 			'mail_subject_getintouch'
 		);
 	}
@@ -769,8 +769,8 @@ 
 	private function newContactOperatorMailer(): OperatorMailer {
 		return new OperatorMailer(
 			$this->getSuborganizationMessenger(),
-			new TwigTemplate( $this->getMailerTwig(), 'Contact_Forward_to_Operator.txt.twig' ),
-			$this->getTranslator()->trans( 'mail_subject_getintouch_forward' )
+			new TwigTemplate($this->getMailerTwig(), 'Contact_Forward_to_Operator.txt.twig'),
+			$this->getTranslator()->trans('mail_subject_getintouch_forward')
 		);
 	}
 
@@ -787,12 +787,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 {
@@ -801,20 +801,20 @@ 
 
 	public function newAuthorizedCachePurger(): AuthorizedCachePurger {
 		return new AuthorizedCachePurger(
-			new AllOfTheCachePurger( $this->getSkinTwig(), $this->getPageCache(), $this->getRenderedPageCache() ),
+			new AllOfTheCachePurger($this->getSkinTwig(), $this->getPageCache(), $this->getRenderedPageCache()),
 			$this->config['purging-secret']
 		);
 	}
 
 	private function newBankDataValidator(): BankDataValidator {
-		return new BankDataValidator( $this->newIbanValidator() );
+		return new BankDataValidator($this->newIbanValidator());
 	}
 
 	private function getSuborganizationMessenger(): Messenger {
 		return $this->pimple['messenger_suborganization'];
 	}
 
-	public function setSuborganizationMessenger( Messenger $messenger ): void {
+	public function setSuborganizationMessenger(Messenger $messenger): void {
 		$this->pimple['messenger_suborganization'] = $messenger;
 	}
 
@@ -822,57 +822,57 @@ 
 		return $this->pimple['messenger_organization'];
 	}
 
-	public function setOrganizationMessenger( Messenger $messenger ): void {
+	public function setOrganizationMessenger(Messenger $messenger): void {
 		$this->pimple['messenger_organization'] = $messenger;
 	}
 
 	public function setNullMessenger(): void {
-		$this->setSuborganizationMessenger( new Messenger(
+		$this->setSuborganizationMessenger(new Messenger(
 			Swift_NullTransport::newInstance(),
 			$this->getSubOrganizationEmailAddress()
-		) );
-		$this->setOrganizationMessenger( new Messenger(
+		));
+		$this->setOrganizationMessenger(new Messenger(
 			Swift_NullTransport::newInstance(),
 			$this->getOrganizationEmailAddress()
-		) );
+		));
 	}
 
 	public function getSubOrganizationEmailAddress(): EmailAddress {
-		return new EmailAddress( $this->config['contact-info']['suborganization']['email'] );
+		return new EmailAddress($this->config['contact-info']['suborganization']['email']);
 	}
 
 	public function getOrganizationEmailAddress(): EmailAddress {
-		return new EmailAddress( $this->config['contact-info']['organization']['email'] );
+		return new EmailAddress($this->config['contact-info']['organization']['email']);
 	}
 
 	public function newInternalErrorHtmlPresenter(): InternalErrorHtmlPresenter {
-		return new InternalErrorHtmlPresenter( $this->getLayoutTemplate( 'Error_Page.html.twig' ) );
+		return new InternalErrorHtmlPresenter($this->getLayoutTemplate('Error_Page.html.twig'));
 	}
 
 	public function newAccessDeniedHtmlPresenter(): InternalErrorHtmlPresenter {
-		return new InternalErrorHtmlPresenter( $this->getLayoutTemplate( 'Access_Denied.twig' ) );
+		return new InternalErrorHtmlPresenter($this->getLayoutTemplate('Access_Denied.twig'));
 	}
 
 	public function getTranslator(): TranslatorInterface {
 		return $this->pimple['translator'];
 	}
 
-	public function setTranslator( TranslatorInterface $translator ): void {
+	public function setTranslator(TranslatorInterface $translator): void {
 		$this->pimple['translator'] = $translator;
 	}
 
-	private function newTwigFactory( array $twigConfig ): TwigFactory {
+	private function newTwigFactory(array $twigConfig): TwigFactory {
 		return new TwigFactory(
 			array_merge_recursive(
 				$twigConfig,
-				[ 'web-basepath' => $this->config['web-basepath'] ]
+				['web-basepath' => $this->config['web-basepath']]
 			),
 			$this->getCachePath() . '/twig',
 			$this->config['locale']
 		);
 	}
 
-	private function newTextPolicyValidator( string $policyName ): TextPolicyValidator {
+	private function newTextPolicyValidator(string $policyName): TextPolicyValidator {
 		$fetcher = new ErrorLoggingFileFetcher(
 			new SimpleFileFetcher(),
 			$this->getLogger()
@@ -881,24 +881,24 @@ 
 		return new TextPolicyValidator(
 			new WordListFileReader(
 				$fetcher,
-				$textPolicyConfig['badwords'] ? $this->getAbsolutePath( $textPolicyConfig['badwords'] ) : ''
+				$textPolicyConfig['badwords'] ? $this->getAbsolutePath($textPolicyConfig['badwords']) : ''
 			),
 			new WordListFileReader(
 				$fetcher,
-				$textPolicyConfig['whitewords'] ? $this->getAbsolutePath( $textPolicyConfig['whitewords'] ) : ''
+				$textPolicyConfig['whitewords'] ? $this->getAbsolutePath($textPolicyConfig['whitewords']) : ''
 			)
 		);
 	}
 
 	private function newCommentPolicyValidator(): TextPolicyValidator {
-		return $this->newTextPolicyValidator( 'comment' );
+		return $this->newTextPolicyValidator('comment');
 	}
 
-	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),
 			$this->newDonationEventLogger()
 		);
 	}
@@ -909,7 +909,7 @@ 
 			new TwigTemplate(
 				$this->getMailerTwig(),
 				'Donation_Cancellation_Confirmation.txt.twig',
-				[ 'greeting_generator' => $this->getGreetingGenerator() ]
+				['greeting_generator' => $this->getGreetingGenerator()]
 			),
 			'mail_subject_confirm_cancellation'
 		);
@@ -968,23 +968,23 @@ 
 	public function newPayPalUrlGeneratorForDonations(): PayPalUrlGenerator {
 		return new PayPalUrlGenerator(
 			$this->getPayPalUrlConfigForDonations(),
-			$this->getTranslator()->trans( 'item_name_donation' )
+			$this->getTranslator()->trans('item_name_donation')
 		);
 	}
 
 	public function newPayPalUrlGeneratorForMembershipApplications(): PayPalUrlGenerator {
 		return new PayPalUrlGenerator(
 			$this->getPayPalUrlConfigForMembershipApplications(),
-			$this->getTranslator()->trans( 'item_name_membership' )
+			$this->getTranslator()->trans('item_name_membership')
 		);
 	}
 
 	private function getPayPalUrlConfigForDonations(): PayPalConfig {
-		return PayPalConfig::newFromConfig( $this->config['paypal-donation'] );
+		return PayPalConfig::newFromConfig($this->config['paypal-donation']);
 	}
 
 	private function getPayPalUrlConfigForMembershipApplications(): PayPalConfig {
-		return PayPalConfig::newFromConfig( $this->config['paypal-membership'] );
+		return PayPalConfig::newFromConfig($this->config['paypal-membership']);
 	}
 
 	public function newSofortUrlGeneratorForDonations(): SofortUrlGenerator {
@@ -992,7 +992,7 @@ 
 
 		return new SofortUrlGenerator(
 			new SofortConfig(
-				$this->getTranslator()->trans( 'item_name_donation', [], 'messages' ),
+				$this->getTranslator()->trans('item_name_donation', [], 'messages'),
 				$config['return-url'],
 				$config['cancel-url'],
 				$config['notification-url']
@@ -1001,7 +1001,7 @@ 
 		);
 	}
 
-	public function setSofortClient( SofortClient $client ): void {
+	public function setSofortClient(SofortClient $client): void {
 		$this->pimple['sofort-client'] = $client;
 	}
 
@@ -1010,11 +1010,11 @@ 
 	}
 
 	private function newCreditCardUrlGenerator(): CreditCardUrlGenerator {
-		return new CreditCardUrlGenerator( $this->newCreditCardUrlConfig() );
+		return new CreditCardUrlGenerator($this->newCreditCardUrlConfig());
 	}
 
 	private function newCreditCardUrlConfig(): CreditCardConfig {
-		return CreditCardConfig::newFromConfig( $this->config['creditcard'] );
+		return CreditCardConfig::newFromConfig($this->config['creditcard']);
 	}
 
 	public function getDonationRepository(): DonationRepository {
@@ -1030,23 +1030,23 @@ 
 	}
 
 	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->newDonationAuthorizer($updateToken),
 			$this->newCommentPolicyValidator(),
 			$this->newAddCommentValidator()
 		);
 	}
 
-	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,
@@ -1062,12 +1062,12 @@ 
 		return $this->pimple['membership_token_generator'];
 	}
 
-	public function newDonationConfirmationPresenter( string $templateName = 'Donation_Confirmation.html.twig' ): DonationConfirmationHtmlPresenter {
+	public function newDonationConfirmationPresenter(string $templateName = 'Donation_Confirmation.html.twig'): DonationConfirmationHtmlPresenter {
 		return new DonationConfirmationHtmlPresenter(
 			$this->getLayoutTemplate(
 				$templateName,
 				[
-					'piwikGoals' => [ 3 ],
+					'piwikGoals' => [3],
 					'paymentTypes' => $this->getPaymentTypesSettings()->getEnabledForMembershipApplication()
 				]
 			),
@@ -1084,7 +1084,7 @@ 
 
 	public function newCancelDonationHtmlPresenter(): CancelDonationHtmlPresenter {
 		return new CancelDonationHtmlPresenter(
-			$this->getLayoutTemplate( 'Donation_Cancellation_Confirmation.html.twig' )
+			$this->getLayoutTemplate('Donation_Cancellation_Confirmation.html.twig')
 		);
 	}
 
@@ -1107,7 +1107,7 @@ 
 			new TwigTemplate(
 				$this->getMailerTwig(),
 				'Membership_Application_Confirmation.txt.twig',
-				[ 'greeting_generator' => $this->getGreetingGenerator() ]
+				['greeting_generator' => $this->getGreetingGenerator()]
 			),
 			'mail_subject_confirm_membership_application'
 		);
@@ -1122,11 +1122,11 @@ 
 	}
 
 	private function newMembershipApplicationTracker(): ApplicationTracker {
-		return new DoctrineApplicationTracker( $this->getEntityManager() );
+		return new DoctrineApplicationTracker($this->getEntityManager());
 	}
 
 	private function newMembershipApplicationPiwikTracker(): ApplicationPiwikTracker {
-		return new DoctrineApplicationPiwikTracker( $this->getEntityManager() );
+		return new DoctrineApplicationPiwikTracker($this->getEntityManager());
 	}
 
 	private function getPaymentDelayCalculator(): PaymentDelayCalculator {
@@ -1137,20 +1137,20 @@ 
 		return $this->getPayPalUrlConfigForMembershipApplications()->getDelayInDays();
 	}
 
-	public function setPaymentDelayCalculator( PaymentDelayCalculator $paymentDelayCalculator ): void {
+	public function setPaymentDelayCalculator(PaymentDelayCalculator $paymentDelayCalculator): void {
 		$this->pimple['payment-delay-calculator'] = $paymentDelayCalculator;
 	}
 
 	private function newApplyForMembershipPolicyValidator(): ApplyForMembershipPolicyValidator {
 		return new ApplyForMembershipPolicyValidator(
-			$this->newTextPolicyValidator( 'fields' ),
+			$this->newTextPolicyValidator('fields'),
 			$this->config['email-address-blacklist']
 		);
 	}
 
-	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()
 		);
@@ -1176,29 +1176,29 @@ 
 			new TwigTemplate(
 				$this->getMailerTwig(),
 				'Membership_Application_Cancellation_Confirmation.txt.twig',
-				[ 'greeting_generator' => $this->getGreetingGenerator() ]
+				['greeting_generator' => $this->getGreetingGenerator()]
 			),
 			'mail_subject_confirm_membership_application_cancellation'
 		);
 	}
 
-	public function newMembershipApplicationConfirmationUseCase( string $accessToken ): ShowApplicationConfirmationUseCase {
+	public function newMembershipApplicationConfirmationUseCase(string $accessToken): ShowApplicationConfirmationUseCase {
 		return new ShowApplicationConfirmationUseCase(
-			$this->newMembershipApplicationAuthorizer( null, $accessToken ),
+			$this->newMembershipApplicationAuthorizer(null, $accessToken),
 			$this->getMembershipApplicationRepository(),
 			$this->newMembershipApplicationTokenFetcher()
 		);
 	}
 
-	public function newShowDonationConfirmationUseCase( string $accessToken ): ShowDonationConfirmationUseCase {
+	public function newShowDonationConfirmationUseCase(string $accessToken): ShowDonationConfirmationUseCase {
 		return new ShowDonationConfirmationUseCase(
-			$this->newDonationAuthorizer( null, $accessToken ),
+			$this->newDonationAuthorizer(null, $accessToken),
 			$this->newDonationTokenFetcher(),
 			$this->getDonationRepository()
 		);
 	}
 
-	public function setDonationConfirmationPageSelector( DonationConfirmationPageSelector $selector ): void {
+	public function setDonationConfirmationPageSelector(DonationConfirmationPageSelector $selector): void {
 		$this->pimple['confirmation-page-selector'] = $selector;
 	}
 
@@ -1207,57 +1207,57 @@ 
 	}
 
 	public function newDonationFormViolationPresenter(): DonationFormViolationPresenter {
-		return new DonationFormViolationPresenter( $this->getDonationFormTemplate(), $this->newAmountFormatter() );
+		return new DonationFormViolationPresenter($this->getDonationFormTemplate(), $this->newAmountFormatter());
 	}
 
 	public function newDonationFormPresenter(): DonationFormPresenter {
-		return new DonationFormPresenter( $this->getDonationFormTemplate(), $this->newAmountFormatter() );
+		return new DonationFormPresenter($this->getDonationFormTemplate(), $this->newAmountFormatter());
 	}
 
 	private function getDonationFormTemplate(): TwigTemplate {
 		// TODO make the template name dependent on the 'form' value from the HTTP POST request
 		// (we need different form pages for A/B testing)
-		return $this->getLayoutTemplate( 'Donation_Form.html.twig', [
+		return $this->getLayoutTemplate('Donation_Form.html.twig', [
 			'paymentTypes' => $this->getPaymentTypesSettings()->getEnabledForDonation()
-		] );
+		]);
 	}
 
 	public function getMembershipApplicationFormTemplate(): TwigTemplate {
-		return $this->getLayoutTemplate( 'Membership_Application.html.twig', [
+		return $this->getLayoutTemplate('Membership_Application.html.twig', [
 			'paymentTypes' => $this->getPaymentTypesSettings()->getEnabledForMembershipApplication()
-		] );
+		]);
 	}
 
-	public function newHandleSofortPaymentNotificationUseCase( string $updateToken ): SofortPaymentNotificationUseCase {
+	public function newHandleSofortPaymentNotificationUseCase(string $updateToken): SofortPaymentNotificationUseCase {
 		return new SofortPaymentNotificationUseCase(
 			$this->getDonationRepository(),
-			$this->newDonationAuthorizer( $updateToken ),
+			$this->newDonationAuthorizer($updateToken),
 			$this->newDonationConfirmationMailer()
 		);
 	}
 
-	public function newHandlePayPalPaymentCompletionNotificationUseCase( string $updateToken ): HandlePayPalPaymentCompletionNotificationUseCase {
+	public function newHandlePayPalPaymentCompletionNotificationUseCase(string $updateToken): HandlePayPalPaymentCompletionNotificationUseCase {
 		return new HandlePayPalPaymentCompletionNotificationUseCase(
 			$this->getDonationRepository(),
-			$this->newDonationAuthorizer( $updateToken ),
+			$this->newDonationAuthorizer($updateToken),
 			$this->newDonationConfirmationMailer(),
 			$this->newDonationEventLogger()
 		);
 	}
 
-	public function newMembershipApplicationSubscriptionSignupNotificationUseCase( string $updateToken ): HandleSubscriptionSignupNotificationUseCase {
+	public function newMembershipApplicationSubscriptionSignupNotificationUseCase(string $updateToken): HandleSubscriptionSignupNotificationUseCase {
 		return new HandleSubscriptionSignupNotificationUseCase(
 			$this->getMembershipApplicationRepository(),
-			$this->newMembershipApplicationAuthorizer( $updateToken ),
+			$this->newMembershipApplicationAuthorizer($updateToken),
 			$this->newApplyForMembershipMailer(),
 			$this->getLogger()
 		);
 	}
 
-	public function newMembershipApplicationSubscriptionPaymentNotificationUseCase( string $updateToken ): HandleSubscriptionPaymentNotificationUseCase {
+	public function newMembershipApplicationSubscriptionPaymentNotificationUseCase(string $updateToken): HandleSubscriptionPaymentNotificationUseCase {
 		return new HandleSubscriptionPaymentNotificationUseCase(
 			$this->getMembershipApplicationRepository(),
-			$this->newMembershipApplicationAuthorizer( $updateToken ),
+			$this->newMembershipApplicationAuthorizer($updateToken),
 			$this->newApplyForMembershipMailer(),
 			$this->getLogger()
 		);
@@ -1267,7 +1267,7 @@ 
 		return $this->pimple['paypal-payment-notification-verifier'];
 	}
 
-	public function setPayPalPaymentNotificationVerifier( PaymentNotificationVerifier $verifier ): void {
+	public function setPayPalPaymentNotificationVerifier(PaymentNotificationVerifier $verifier): void {
 		$this->pimple['paypal-payment-notification-verifier'] = $verifier;
 	}
 
@@ -1275,14 +1275,14 @@ 
 		return $this->pimple['paypal-membership-fee-notification-verifier'];
 	}
 
-	public function setPayPalMembershipFeeNotificationVerifier( PaymentNotificationVerifier $verifier ): void {
+	public function setPayPalMembershipFeeNotificationVerifier(PaymentNotificationVerifier $verifier): void {
 		$this->pimple['paypal-membership-fee-notification-verifier'] = $verifier;
 	}
 
-	public function newCreditCardNotificationUseCase( string $updateToken ): CreditCardNotificationUseCase {
+	public function newCreditCardNotificationUseCase(string $updateToken): CreditCardNotificationUseCase {
 		return new CreditCardNotificationUseCase(
 			$this->getDonationRepository(),
-			$this->newDonationAuthorizer( $updateToken ),
+			$this->newDonationAuthorizer($updateToken),
 			$this->getCreditCardService(),
 			$this->newDonationConfirmationMailer(),
 			$this->getLogger(),
@@ -1292,13 +1292,13 @@ 
 
 	public function newCancelMembershipApplicationHtmlPresenter(): CancelMembershipApplicationHtmlPresenter {
 		return new CancelMembershipApplicationHtmlPresenter(
-			$this->getLayoutTemplate( 'Membership_Application_Cancellation_Confirmation.html.twig' )
+			$this->getLayoutTemplate('Membership_Application_Cancellation_Confirmation.html.twig')
 		);
 	}
 
 	public function newMembershipApplicationConfirmationHtmlPresenter(): MembershipApplicationConfirmationHtmlPresenter {
 		return new MembershipApplicationConfirmationHtmlPresenter(
-			$this->getLayoutTemplate( 'Membership_Application_Confirmation.html.twig' )
+			$this->getLayoutTemplate('Membership_Application_Confirmation.html.twig')
 		);
 	}
 
@@ -1308,7 +1308,7 @@ 
 		);
 	}
 
-	public function setCreditCardService( CreditCardService $ccService ): void {
+	public function setCreditCardService(CreditCardService $ccService): void {
 		$this->pimple['credit-card-api-service'] = $ccService;
 	}
 
@@ -1321,7 +1321,7 @@ 
 			new TwigTemplate(
 				$this->getSkinTwig(),
 				'Credit_Card_Payment_Notification.txt.twig',
-				[ 'returnUrl' => $this->config['creditcard']['return-url'] ]
+				['returnUrl' => $this->config['creditcard']['return-url']]
 			)
 		);
 	}
@@ -1340,11 +1340,11 @@ 
 		);
 	}
 
-	public function setDonationTokenGenerator( TokenGenerator $tokenGenerator ): void {
+	public function setDonationTokenGenerator(TokenGenerator $tokenGenerator): void {
 		$this->pimple['donation_token_generator'] = $tokenGenerator;
 	}
 
-	public function setMembershipTokenGenerator( MembershipTokenGenerator $tokenGenerator ): void {
+	public function setMembershipTokenGenerator(MembershipTokenGenerator $tokenGenerator): void {
 		$this->pimple['membership_token_generator'] = $tokenGenerator;
 	}
 
@@ -1367,23 +1367,23 @@ 
 	private function newDonationPolicyValidator(): AddDonationPolicyValidator {
 		return new AddDonationPolicyValidator(
 			$this->newDonationAmountPolicyValidator(),
-			$this->newTextPolicyValidator( 'fields' ),
+			$this->newTextPolicyValidator('fields'),
 			$this->config['email-address-blacklist']
 		);
 	}
 
 	private function newDonationAmountPolicyValidator(): AmountPolicyValidator {
 		// in the future, this might come from the configuration
-		return new AmountPolicyValidator( 1000, 1000 );
+		return new AmountPolicyValidator(1000, 1000);
 	}
 
 	public function getDonationTimeframeLimit(): string {
 		return $this->config['donation-timeframe-limit'];
 	}
 
-	public function newSystemMessageResponse( string $message ): string {
-		return $this->getLayoutTemplate( 'System_Message.html.twig' )
-			->render( [ 'message' => $message ] );
+	public function newSystemMessageResponse(string $message): string {
+		return $this->getLayoutTemplate('System_Message.html.twig')
+			->render(['message' => $message]);
 	}
 
 	public function getMembershipApplicationTimeframeLimit(): string {
@@ -1404,41 +1404,41 @@ 
 
 	public function enablePageCache(): void {
 		$this->pimple['page_cache'] = function() {
-			return new FilesystemCache( $this->getCachePath() . '/pages/raw' );
+			return new FilesystemCache($this->getCachePath() . '/pages/raw');
 		};
 
 		$this->pimple['rendered_page_cache'] = function() {
-			return new FilesystemCache( $this->getCachePath() . '/pages/rendered' );
+			return new FilesystemCache($this->getCachePath() . '/pages/rendered');
 		};
 	}
 
-	private function addProfilingDecorator( $objectToDecorate, string $profilingLabel ) {	// @codingStandardsIgnoreLine
-		if ( $this->profiler === null ) {
+	private function addProfilingDecorator($objectToDecorate, string $profilingLabel) {	// @codingStandardsIgnoreLine
+		if ($this->profiler === null) {
 			return $objectToDecorate;
 		}
 
-		$builder = new ProfilingDecoratorBuilder( $this->profiler, $this->getProfilerDataCollector() );
+		$builder = new ProfilingDecoratorBuilder($this->profiler, $this->getProfilerDataCollector());
 
-		return $builder->decorate( $objectToDecorate, $profilingLabel );
+		return $builder->decorate($objectToDecorate, $profilingLabel);
 	}
 
-	public function setProfiler( Stopwatch $profiler ): void {
+	public function setProfiler(Stopwatch $profiler): void {
 		$this->profiler = $profiler;
 	}
 
-	public function setEmailValidator( EmailValidator $validator ): void {
+	public function setEmailValidator(EmailValidator $validator): void {
 		$this->pimple['mail_validator'] = $validator;
 	}
 
-	public function setLogger( LoggerInterface $logger ): void {
+	public function setLogger(LoggerInterface $logger): void {
 		$this->pimple['logger'] = $logger;
 	}
 
-	public function setPaypalLogger( LoggerInterface $logger ): void {
+	public function setPaypalLogger(LoggerInterface $logger): void {
 		$this->pimple['paypal_logger'] = $logger;
 	}
 
-	public function setSofortLogger( LoggerInterface $logger ): void {
+	public function setSofortLogger(LoggerInterface $logger): void {
 		$this->pimple['sofort_logger'] = $logger;
 	}
 
@@ -1447,10 +1447,10 @@ 
 	}
 
 	private function newIbanValidator(): KontoCheckIbanValidator {
-		return new KontoCheckIbanValidator( $this->newBankDataConverter(), $this->config['banned-ibans'] );
+		return new KontoCheckIbanValidator($this->newBankDataConverter(), $this->config['banned-ibans']);
 	}
 
-	public function setFilePrefixer( FilePrefixer $prefixer ): void {
+	public function setFilePrefixer(FilePrefixer $prefixer): void {
 		$this->pimple['cachebusting_fileprefixer'] = $prefixer;
 	}
 
@@ -1460,26 +1460,26 @@ 
 
 	private function getFilePrefix(): string {
 		$prefixContentFile = $this->getVarPath() . '/file_prefix.txt';
-		if ( !file_exists( $prefixContentFile ) ) {
+		if (!file_exists($prefixContentFile)) {
 			return '';
 		}
-		return $prefix = preg_replace( '/[^0-9a-f]/', '', file_get_contents( $prefixContentFile ) );
+		return $prefix = preg_replace('/[^0-9a-f]/', '', file_get_contents($prefixContentFile));
 	}
 
-	public function newDonationAcceptedEventHandler( string $updateToken ): DonationAcceptedEventHandler {
+	public function newDonationAcceptedEventHandler(string $updateToken): DonationAcceptedEventHandler {
 		return new DonationAcceptedEventHandler(
-			$this->newDonationAuthorizer( $updateToken ),
+			$this->newDonationAuthorizer($updateToken),
 			$this->getDonationRepository(),
 			$this->newDonationConfirmationMailer()
 		);
 	}
 
 	public function newPageNotFoundHtmlPresenter(): PageNotFoundPresenter {
-		return new PageNotFoundPresenter( $this->getLayoutTemplate( 'Page_not_found.html.twig' ) );
+		return new PageNotFoundPresenter($this->getLayoutTemplate('Page_not_found.html.twig'));
 	}
 
-	public function setPageViewTracker( PageViewTracker $tracker ): void {
-		$this->pimple['page_view_tracker'] = function () use ( $tracker )  {
+	public function setPageViewTracker(PageViewTracker $tracker): void {
+		$this->pimple['page_view_tracker'] = function() use ($tracker)  {
 			return $tracker;
 		};
 	}
@@ -1492,25 +1492,25 @@ 
 		// the "https:" prefix does NOT get any slashes because baseURL is stored in a protocol-agnostic way
 		// (e.g. "//tracking.wikimedia.de" )
 		return new PiwikServerSideTracker(
-			new \PiwikTracker( $this->config['piwik']['siteId'], 'https:' . $this->config['piwik']['baseUrl'] )
+			new \PiwikTracker($this->config['piwik']['siteId'], 'https:' . $this->config['piwik']['baseUrl'])
 		);
 	}
 
 	public function getI18nDirectory(): string {
-		return $this->getAbsolutePath( $this->config['i18n-base-path'] ) . '/' . $this->config['locale'];
+		return $this->getAbsolutePath($this->config['i18n-base-path']) . '/' . $this->config['locale'];
 	}
 
 	/**
 	 * If the pathname does not start with a slash, make the path absolute to root dir of application
 	 */
-	private function getAbsolutePath( string $path ): string {
-		if ( $path[0] === '/' ) {
+	private function getAbsolutePath(string $path): string {
+		if ($path[0] === '/') {
 			return $path;
 		}
 		return __DIR__ . '/../../' . $path;
 	}
 
-	public function setContentPagePageSelector( PageSelector $pageSelector ): void {
+	public function setContentPagePageSelector(PageSelector $pageSelector): void {
 		$this->pimple['content_page_selector'] = $pageSelector;
 	}
 
@@ -1518,7 +1518,7 @@ 
 		return $this->pimple['content_page_selector'];
 	}
 
-	public function setContentProvider( ContentProvider $contentProvider ): void {
+	public function setContentProvider(ContentProvider $contentProvider): void {
 		$this->pimple['content_provider'] = $contentProvider;
 	}
 
@@ -1536,7 +1536,7 @@ 
 		return $this->pimple['url_generator'];
 	}
 
-	public function setUrlGenerator( UrlGenerator $urlGenerator ): void {
+	public function setUrlGenerator(UrlGenerator $urlGenerator): void {
 		$this->pimple['url_generator'] = $urlGenerator;
 	}
 
@@ -1553,12 +1553,12 @@ 
 	}
 
 	public function newDonationAmountConstraint(): ValidatorConstraint {
-		return new RequiredConstraint( [
-			new TypeConstraint( [ 'type' => 'digit' ] ),
-			new RangeConstraint( [
-				'min' => Euro::newFromInt( $this->config['donation-minimum-amount'] )->getEuroCents(),
-				'max' => Euro::newFromInt( $this->config['donation-maximum-amount'] )->getEuroCents()
-			] )
-		] );
+		return new RequiredConstraint([
+			new TypeConstraint(['type' => 'digit']),
+			new RangeConstraint([
+				'min' => Euro::newFromInt($this->config['donation-minimum-amount'])->getEuroCents(),
+				'max' => Euro::newFromInt($this->config['donation-maximum-amount'])->getEuroCents()
+			])
+		]);
 	}
 }

src/Presentation/Presenters/MembershipApplicationConfirmationHtmlPresenter.php

Spacing +16 added lines, -16 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\Presentation\Presenters;
 
@@ -23,11 +23,11 @@ 
 
 	private $template;
 
-	public function __construct( TwigTemplate $template ) {
+	public function __construct(TwigTemplate $template) {
 		$this->template = $template;
 	}
 
-	public function present( ShowApplicationConfirmationResponse $response ): string {
+	public function present(ShowApplicationConfirmationResponse $response): string {
 		return $this->template->render(
 			$this->getConfirmationPageArguments(
 				$response->getApplication(),
@@ -36,30 +36,30 @@ 
 		);
 	}
 
-	private function getConfirmationPageArguments( Application $membershipApplication, string $updateToken ): array {
+	private function getConfirmationPageArguments(Application $membershipApplication, string $updateToken): array {
 		return [
-			'membershipApplication' => $this->getApplicationArguments( $membershipApplication, $updateToken ),
-			'person' => $this->getPersonArguments( $membershipApplication->getApplicant() ),
-			'bankData' => $this->getBankDataArguments( $membershipApplication->getPayment()->getPaymentMethod() ),
+			'membershipApplication' => $this->getApplicationArguments($membershipApplication, $updateToken),
+			'person' => $this->getPersonArguments($membershipApplication->getApplicant()),
+			'bankData' => $this->getBankDataArguments($membershipApplication->getPayment()->getPaymentMethod()),
 			'payPalData' => $this->getPayPalDataArguments(
 				$membershipApplication->getPayment()->getPaymentMethod()
 			)
 		];
 	}
 
-	private function getApplicationArguments( Application $membershipApplication, string $updateToken ): array {
+	private function getApplicationArguments(Application $membershipApplication, string $updateToken): array {
 		return [
 			'id' => $membershipApplication->getId(),
 			'membershipType' => $membershipApplication->getType(),
 			'paymentType' => $membershipApplication->getPayment()->getPaymentMethod()->getType(),
-			'status' => $this->mapStatus( $membershipApplication->isConfirmed() ),
+			'status' => $this->mapStatus($membershipApplication->isConfirmed()),
 			'membershipFee' => $membershipApplication->getPayment()->getAmount()->getEuroString(),
 			'paymentIntervalInMonths' => $membershipApplication->getPayment()->getIntervalInMonths(),
 			'updateToken' => $updateToken
 		];
 	}
 
-	private function getPersonArguments( Applicant $applicant ): array {
+	private function getPersonArguments(Applicant $applicant): array {
 		return [
 			'salutation' => $applicant->getName()->getSalutation(),
 			'title' => $applicant->getName()->getTitle(),
@@ -71,8 +71,8 @@ 
 		];
 	}
 
-	private function getBankDataArguments( PaymentMethod $payment ): array {
-		if ( $payment instanceof DirectDebitPayment ) {
+	private function getBankDataArguments(PaymentMethod $payment): array {
+		if ($payment instanceof DirectDebitPayment) {
 			return [
 				'iban' => $payment->getBankData()->getIban()->toString(),
 				'bic' => $payment->getBankData()->getBic(),
@@ -83,10 +83,10 @@ 
 		return [];
 	}
 
-	private function getPayPalDataArguments( PaymentMethod $payment ): array {
-		if ( $payment instanceof PayPalPayment ) {
+	private function getPayPalDataArguments(PaymentMethod $payment): array {
+		if ($payment instanceof PayPalPayment) {
 			return [
-				'firstPaymentDate' => ( new DateTime( $payment->getPayPalData()->getFirstPaymentDate() ) )->format( 'd.m.Y' )
+				'firstPaymentDate' => (new DateTime($payment->getPayPalData()->getFirstPaymentDate()))->format('d.m.Y')
 			];
 		}
 
@@ -99,7 +99,7 @@ 
 	 * @param bool $isConfirmed
 	 * @return string
 	 */
-	private function mapStatus( bool $isConfirmed ): string {
+	private function mapStatus(bool $isConfirmed): string {
 		return $isConfirmed ? 'status-booked' : 'status-unconfirmed';
 	}
 

contexts/MembershipContext/src/UseCases/ShowMembershipApplicationConfirmation/ShowApplicationConfirmationUseCase.php

Spacing +9 added lines, -9 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\MembershipContext\UseCases\ShowMembershipApplicationConfirmation;
 
@@ -20,32 +20,32 @@ 
 	private $repository;
 	private $tokenFetcher;
 
-	public function __construct( ApplicationAuthorizer $authorizer, ApplicationRepository $repository,
-								 ApplicationTokenFetcher $tokenFetcher ) {
+	public function __construct(ApplicationAuthorizer $authorizer, ApplicationRepository $repository,
+								 ApplicationTokenFetcher $tokenFetcher) {
 		$this->authorizer = $authorizer;
 		$this->repository = $repository;
 		$this->tokenFetcher = $tokenFetcher;
 	}
 
-	public function showConfirmation( ShowAppConfirmationRequest $request ): ShowApplicationConfirmationResponse {
-		if ( !$this->authorizer->canAccessApplication( $request->getApplicationId() ) ) {
+	public function showConfirmation(ShowAppConfirmationRequest $request): ShowApplicationConfirmationResponse {
+		if (!$this->authorizer->canAccessApplication($request->getApplicationId())) {
 			return ShowApplicationConfirmationResponse::newNotAllowedResponse();
 		}
 
 		try {
-			$application = $this->repository->getApplicationById( $request->getApplicationId() );
+			$application = $this->repository->getApplicationById($request->getApplicationId());
 		}
-		catch ( ApplicationPurgedException $ex ) {
+		catch (ApplicationPurgedException $ex) {
 			// TODO: success response without the Application
 			return ShowApplicationConfirmationResponse::newNotAllowedResponse();
 		}
-		catch ( GetMembershipApplicationException $ex ) {
+		catch (GetMembershipApplicationException $ex) {
 			return ShowApplicationConfirmationResponse::newNotAllowedResponse();
 		}
 
 		return ShowApplicationConfirmationResponse::newValidResponse(
 			$application, // TODO: use DTO instead of Entity (currently violates the architecture)
-			$this->tokenFetcher->getTokens( $request->getApplicationId() )->getUpdateToken()
+			$this->tokenFetcher->getTokens($request->getApplicationId())->getUpdateToken()
 		);
 	}
 

Braces +2 added lines, -4 removed lines

@@ -34,12 +34,10 @@
 
 		try {
 			$application = $this->repository->getApplicationById( $request->getApplicationId() );
-		}
-		catch ( ApplicationPurgedException $ex ) {
+		} catch ( ApplicationPurgedException $ex ) {
 			// TODO: success response without the Application
 			return ShowApplicationConfirmationResponse::newNotAllowedResponse();
-		}
-		catch ( GetMembershipApplicationException $ex ) {
+		} catch ( GetMembershipApplicationException $ex ) {
 			return ShowApplicationConfirmationResponse::newNotAllowedResponse();
 		}
 

app/routes.php

Spacing +205 added lines, -205 removed lines

@@ -7,7 +7,7 @@ 
  * @var \WMDE\Fundraising\Frontend\Factories\FunFunFactory $ffFactory
  */
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 use Silex\Application;
 use Symfony\Component\HttpFoundation\Request;
@@ -49,96 +49,96 @@ 
 
 $app->post(
 	'validate-email',
-	function( Request $request ) use ( $app, $ffFactory ) {
-		$validationResult = $ffFactory->getEmailValidator()->validate( $request->request->get( 'email', '' ) );
-		return $app->json( [ 'status' => $validationResult->isSuccessful() ? 'OK' : 'ERR' ] );
+	function(Request $request) use ($app, $ffFactory) {
+		$validationResult = $ffFactory->getEmailValidator()->validate($request->request->get('email', ''));
+		return $app->json(['status' => $validationResult->isSuccessful() ? 'OK' : 'ERR']);
 	}
 );
 
 $app->post(
 	'validate-payment-data',
-	function( Request $request ) use ( $app, $ffFactory ) {
+	function(Request $request) use ($app, $ffFactory) {
 
-		$amount = (float) $ffFactory->newDecimalNumberFormatter()->parse( $request->get( 'amount', '0' ) );
+		$amount = (float)$ffFactory->newDecimalNumberFormatter()->parse($request->get('amount', '0'));
 		$validator = $ffFactory->newPaymentDataValidator();
-		$validationResult = $validator->validate( $amount, (string) $request->get( 'paymentType', '' ) );
+		$validationResult = $validator->validate($amount, (string)$request->get('paymentType', ''));
 
-		if ( $validationResult->isSuccessful() ) {
-			return $app->json( [ 'status' => 'OK' ] );
+		if ($validationResult->isSuccessful()) {
+			return $app->json(['status' => 'OK']);
 		} else {
 			$errors = [];
-			foreach( $validationResult->getViolations() as $violation ) {
-				$errors[ $violation->getSource() ] = $ffFactory->getTranslator()->trans( $violation->getMessageIdentifier() );
+			foreach ($validationResult->getViolations() as $violation) {
+				$errors[$violation->getSource()] = $ffFactory->getTranslator()->trans($violation->getMessageIdentifier());
 			}
-			return $app->json( [ 'status' => 'ERR', 'messages' => $errors ] );
+			return $app->json(['status' => 'ERR', 'messages' => $errors]);
 		}
 	}
 );
 
 $app->post(
 	'validate-address', // Validates donor information. This route is named badly.
-	function( Request $request ) use ( $app, $ffFactory ) {
-		return ( new ValidateDonorHandler( $ffFactory, $app ) )->handle( $request );
+	function(Request $request) use ($app, $ffFactory) {
+		return (new ValidateDonorHandler($ffFactory, $app))->handle($request);
 	}
 );
 
 $app->post(
 	'validate-donation-amount',
-	function( Request $httpRequest ) use ( $app, $ffFactory ) {
+	function(Request $httpRequest) use ($app, $ffFactory) {
 
-		$constraint = new Collection( [
+		$constraint = new Collection([
 			'allowExtraFields' => false,
 			'fields' => [
 				'amount' => $ffFactory->newDonationAmountConstraint()
 			]
-		] );
+		]);
 
-		$violations = Validation::createValidator()->validate( $httpRequest->request->all(), $constraint );
+		$violations = Validation::createValidator()->validate($httpRequest->request->all(), $constraint);
 
-		if ( $violations->count() > 0 ) {
+		if ($violations->count() > 0) {
 			$mapper = new ConstraintViolationListMapper();
-			return $app->json( [ 'status' => 'ERR', 'messages' => $mapper->map( $violations ) ] );
+			return $app->json(['status' => 'ERR', 'messages' => $mapper->map($violations)]);
 		}
 
-		return $app->json( [ 'status' => 'OK' ] );
+		return $app->json(['status' => 'OK']);
 	}
 );
 
 $app->post(
 	'validate-fee',
-	function( Request $httpRequest ) use ( $app, $ffFactory ) {
+	function(Request $httpRequest) use ($app, $ffFactory) {
 		$validator = new MembershipFeeValidator();
 		$result = $validator->validate(
-			str_replace( ',', '.', $httpRequest->request->get( 'amount', '' ) ),
-			(int) $httpRequest->request->get( 'paymentIntervalInMonths', '0' ),
-			$httpRequest->request->get( 'addressType', '' )
+			str_replace(',', '.', $httpRequest->request->get('amount', '')),
+			(int)$httpRequest->request->get('paymentIntervalInMonths', '0'),
+			$httpRequest->request->get('addressType', '')
 		);
 
-		if ( $result->isSuccessful() ) {
-			return $app->json( [ 'status' => 'OK' ] );
+		if ($result->isSuccessful()) {
+			return $app->json(['status' => 'OK']);
 		} else {
 			$errors = $result->getViolations();
-			return $app->json( [ 'status' => 'ERR', 'messages' => $errors ] );
+			return $app->json(['status' => 'ERR', 'messages' => $errors]);
 		}
 	}
 );
 
 $app->get(
 	'list-comments.json',
-	function( Request $request ) use ( $app, $ffFactory ) {
+	function(Request $request) use ($app, $ffFactory) {
 		$response = $app->json(
 			$ffFactory->newCommentListJsonPresenter()->present(
 				$ffFactory->newListCommentsUseCase()->listComments(
 					new CommentListingRequest(
-						(int)$request->query->get( 'n', '10' ),
-						(int)$request->query->get( 'page', '1' )
+						(int)$request->query->get('n', '10'),
+						(int)$request->query->get('page', '1')
 					)
 				)
 			)
 		);
 
-		if ( $request->query->get( 'f' ) ) {
-			$response->setCallback( $request->query->get( 'f' ) );
+		if ($request->query->get('f')) {
+			$response->setCallback($request->query->get('f'));
 		}
 
 		return $response;
@@ -147,10 +147,10 @@ 
 
 $app->get(
 	'list-comments.rss',
-	function() use ( $app, $ffFactory ) {
+	function() use ($app, $ffFactory) {
 		$rss = $ffFactory->newCommentListRssPresenter()->present(
 			$ffFactory->newListCommentsUseCase()->listComments(
-				new CommentListingRequest( 100, CommentListingRequest::FIRST_PAGE )
+				new CommentListingRequest(100, CommentListingRequest::FIRST_PAGE)
 			)
 		);
 
@@ -163,101 +163,101 @@ 
 			]
 		);
 	}
-)->bind( 'list-comments.rss' );
+)->bind('list-comments.rss');
 
 $app->get(
 	'list-comments.html',
-	function( Request $request ) use ( $app, $ffFactory ) {
+	function(Request $request) use ($app, $ffFactory) {
 		return new Response(
 			$ffFactory->newCommentListHtmlPresenter()->present(
 				$ffFactory->newListCommentsUseCase()->listComments(
 					new CommentListingRequest(
 						10,
-						(int)$request->query->get( 'page', '1' )
+						(int)$request->query->get('page', '1')
 					)
 				),
-				(int)$request->query->get( 'page', '1' )
+				(int)$request->query->get('page', '1')
 			)
 		);
 	}
-)->bind( 'list-comments.html' );
+)->bind('list-comments.html');
 
 $app->get(
 	'page/{pageName}',
-	function( $pageName ) use ( $ffFactory ) {
+	function($pageName) use ($ffFactory) {
 		$pageSelector = $ffFactory->getContentPagePageSelector();
 
 		try {
-			$pageId = $pageSelector->getPageId( $pageName );
-		} catch ( PageNotFoundException $exception ) {
-			throw new NotFoundHttpException( "Page page name '$pageName' not found." );
+			$pageId = $pageSelector->getPageId($pageName);
+		} catch (PageNotFoundException $exception) {
+			throw new NotFoundHttpException("Page page name '$pageName' not found.");
 		}
 
 		try {
-			return $ffFactory->getLayoutTemplate( 'Display_Page_Layout.twig' )->render( [
+			return $ffFactory->getLayoutTemplate('Display_Page_Layout.twig')->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." );
+				throw new NotFoundHttpException("Content for page id '$pageId' not found.");
 			}
 
 			throw $exception;
 		}
 	}
 )
-->bind( 'page' );
+->bind('page');
 
 // Form for this is provided by route page/Subscription_Form
 $app->match(
 	'contact/subscribe',
-	function( Application $app, Request $request ) use ( $ffFactory ) {
-		return ( new AddSubscriptionHandler( $ffFactory, $app ) )
-			->handle( $request );
+	function(Application $app, Request $request) use ($ffFactory) {
+		return (new AddSubscriptionHandler($ffFactory, $app))
+			->handle($request);
 	}
 )
-->method( 'GET|POST' )
-->bind( 'subscribe' );
+->method('GET|POST')
+->bind('subscribe');
 
-$app->get( 'contact/confirm-subscription/{confirmationCode}', function ( $confirmationCode ) use ( $ffFactory ) {
+$app->get('contact/confirm-subscription/{confirmationCode}', function($confirmationCode) use ($ffFactory) {
 	$useCase = $ffFactory->newConfirmSubscriptionUseCase();
-	$response = $useCase->confirmSubscription( $confirmationCode );
-	return $ffFactory->newConfirmSubscriptionHtmlPresenter()->present( $response );
+	$response = $useCase->confirmSubscription($confirmationCode);
+	return $ffFactory->newConfirmSubscriptionHtmlPresenter()->present($response);
 } )
-->assert( 'confirmationCode', '^[0-9a-f]+$' )
-->bind( 'confirm-subscription' );
+->assert('confirmationCode', '^[0-9a-f]+$')
+->bind('confirm-subscription');
 
 $app->get(
 	'check-iban',
-	function( Request $request ) use ( $app, $ffFactory ) {
+	function(Request $request) use ($app, $ffFactory) {
 		$useCase = $ffFactory->newCheckIbanUseCase();
-		$checkIbanResponse = $useCase->checkIban( new Iban( $request->query->get( 'iban', '' ) ) );
-		return $app->json( $ffFactory->newIbanPresenter()->present( $checkIbanResponse ) );
+		$checkIbanResponse = $useCase->checkIban(new Iban($request->query->get('iban', '')));
+		return $app->json($ffFactory->newIbanPresenter()->present($checkIbanResponse));
 	}
 );
 
 $app->get(
 	'generate-iban',
-	function( Request $request ) use ( $app, $ffFactory ) {
+	function(Request $request) use ($app, $ffFactory) {
 		$generateIbanRequest = new GenerateIbanRequest(
-			$request->query->get( 'accountNumber', '' ),
-			$request->query->get( 'bankCode', '' )
+			$request->query->get('accountNumber', ''),
+			$request->query->get('bankCode', '')
 		);
 
-		$generateIbanResponse = $ffFactory->newGenerateIbanUseCase()->generateIban( $generateIbanRequest );
-		return $app->json( $ffFactory->newIbanPresenter()->present( $generateIbanResponse ) );
+		$generateIbanResponse = $ffFactory->newGenerateIbanUseCase()->generateIban($generateIbanRequest);
+		return $app->json($ffFactory->newIbanPresenter()->present($generateIbanResponse));
 	}
 );
 
 $app->post(
 	'add-comment',
-	function( Request $request ) use ( $app, $ffFactory ) {
+	function(Request $request) use ($app, $ffFactory) {
 		$addCommentRequest = new AddCommentRequest();
-		$addCommentRequest->setCommentText( trim( $request->request->get( 'comment', '' ) ) );
-		$addCommentRequest->setIsPublic( $request->request->get( 'public', '0' ) === '1' );
-		$addCommentRequest->setDonationId( (int)$request->request->get( 'donationId', '' ) );
+		$addCommentRequest->setCommentText(trim($request->request->get('comment', '')));
+		$addCommentRequest->setIsPublic($request->request->get('public', '0') === '1');
+		$addCommentRequest->setDonationId((int)$request->request->get('donationId', ''));
 
-		if ( $request->request->get( 'isAnonymous', '0' ) === '1' ) {
+		if ($request->request->get('isAnonymous', '0') === '1') {
 			$addCommentRequest->setIsAnonymous();
 		}
 		else {
@@ -266,34 +266,34 @@ 
 
 		$addCommentRequest->freeze()->assertNoNullFields();
 
-		$updateToken = $request->request->get( 'updateToken', '' );
+		$updateToken = $request->request->get('updateToken', '');
 
-		if ( $updateToken === '' ) {
-			return $app->json( [
+		if ($updateToken === '') {
+			return $app->json([
 				'status' => 'ERR',
-				'message' => $ffFactory->getTranslator()->trans( 'comment_failure_access_denied' ),
-			] );
+				'message' => $ffFactory->getTranslator()->trans('comment_failure_access_denied'),
+			]);
 		}
 
-		$response = $ffFactory->newAddCommentUseCase( $updateToken )->addComment( $addCommentRequest );
+		$response = $ffFactory->newAddCommentUseCase($updateToken)->addComment($addCommentRequest);
 
-		if ( $response->isSuccessful() ) {
-			return $app->json( [
+		if ($response->isSuccessful()) {
+			return $app->json([
 				'status' => 'OK',
-				'message' => $ffFactory->getTranslator()->trans( $response->getSuccessMessage() ),
-			] );
+				'message' => $ffFactory->getTranslator()->trans($response->getSuccessMessage()),
+			]);
 		}
 
-		return $app->json( [
+		return $app->json([
 			'status' => 'ERR',
-			'message' => $ffFactory->getTranslator()->trans( $response->getErrorMessage() ),
-		] );
+			'message' => $ffFactory->getTranslator()->trans($response->getErrorMessage()),
+		]);
 	}
-)->bind( 'PostComment' );
+)->bind('PostComment');
 
 $app->get(
 	'add-comment',
-	function( Request $request ) use ( $app, $ffFactory ) {
+	function(Request $request) use ($app, $ffFactory) {
 		$template = $ffFactory->getLayoutTemplate(
 			'Donation_Comment.html.twig'
 		);
@@ -301,61 +301,61 @@ 
 		return new Response(
 			$template->render(
 				[
-					'donationId' => (int)$request->query->get( 'donationId', '' ),
-					'updateToken' => $request->query->get( 'updateToken', '' ),
+					'donationId' => (int)$request->query->get('donationId', ''),
+					'updateToken' => $request->query->get('updateToken', ''),
 					'cancelUrl' => $app['url_generator']->generate(
 						'show-donation-confirmation',
 						[
-							'id' => (int)$request->query->get( 'donationId', '' ),
-							'accessToken' => $request->query->get( 'accessToken', '' )
+							'id' => (int)$request->query->get('donationId', ''),
+							'accessToken' => $request->query->get('accessToken', '')
 						]
 					)
 				]
 			)
 		);
 	}
-)->bind( 'AddCommentPage' );
+)->bind('AddCommentPage');
 
 $app->post(
 	'contact/get-in-touch',
-	function( Request $request ) use ( $app, $ffFactory ) {
+	function(Request $request) use ($app, $ffFactory) {
 		$contactFormRequest = new GetInTouchRequest(
-			$request->get( 'firstname', '' ),
-			$request->get( 'lastname', '' ),
-			$request->get( 'email', '' ),
-			$request->get( 'subject', '' ),
-			$request->get( 'messageBody', '' )
+			$request->get('firstname', ''),
+			$request->get('lastname', ''),
+			$request->get('email', ''),
+			$request->get('subject', ''),
+			$request->get('messageBody', '')
 		);
 
-		$contactFormResponse = $ffFactory->newGetInTouchUseCase()->processContactRequest( $contactFormRequest );
-		if ( $contactFormResponse->isSuccessful() ) {
-			return $app->redirect( $app['url_generator']->generate( 'page', [ 'pageName' => 'Kontakt_Bestaetigung' ] ) );
+		$contactFormResponse = $ffFactory->newGetInTouchUseCase()->processContactRequest($contactFormRequest);
+		if ($contactFormResponse->isSuccessful()) {
+			return $app->redirect($app['url_generator']->generate('page', ['pageName' => 'Kontakt_Bestaetigung']));
 		}
 
-		return $ffFactory->newGetInTouchHtmlPresenter()->present( $contactFormResponse, $request->request->all() );
+		return $ffFactory->newGetInTouchHtmlPresenter()->present($contactFormResponse, $request->request->all());
 	}
 );
 
 $app->get(
 	'contact/get-in-touch',
-	function() use ( $app, $ffFactory ) {
-		return $ffFactory->getLayoutTemplate( 'contact_form.html.twig' )->render( [ ] );
+	function() use ($app, $ffFactory) {
+		return $ffFactory->getLayoutTemplate('contact_form.html.twig')->render([]);
 	}
 )->bind('contact');
 
 $app->post(
 	'donation/cancel',
-	function( Request $request ) use ( $app, $ffFactory ) {
+	function(Request $request) use ($app, $ffFactory) {
 		$cancellationRequest = new CancelDonationRequest(
-			(int)$request->request->get( 'sid', '' )
+			(int)$request->request->get('sid', '')
 		);
 
-		$responseModel = $ffFactory->newCancelDonationUseCase( $request->request->get( 'utoken', '' ) )
-			->cancelDonation( $cancellationRequest );
+		$responseModel = $ffFactory->newCancelDonationUseCase($request->request->get('utoken', ''))
+			->cancelDonation($cancellationRequest);
 
-		$httpResponse = new Response( $ffFactory->newCancelDonationHtmlPresenter()->present( $responseModel ) );
-		if ( $responseModel->cancellationSucceeded() ) {
-			$httpResponse->headers->clearCookie( 'donation_timestamp' );
+		$httpResponse = new Response($ffFactory->newCancelDonationHtmlPresenter()->present($responseModel));
+		if ($responseModel->cancellationSucceeded()) {
+			$httpResponse->headers->clearCookie('donation_timestamp');
 		}
 
 		return $httpResponse;
@@ -364,184 +364,184 @@ 
 
 $app->post(
 	'donation/add',
-	function( Application $app, Request $request ) use ( $ffFactory ) {
-		return ( new AddDonationHandler( $ffFactory, $app ) )
-			->handle( $request );
+	function(Application $app, Request $request) use ($ffFactory) {
+		return (new AddDonationHandler($ffFactory, $app))
+			->handle($request);
 	}
 );
 
 // Show a donation form with pre-filled payment values, e.g. when coming from a banner
-$app->get( 'donation/new', function ( Request $request ) use ( $ffFactory ) {
+$app->get('donation/new', function(Request $request) use ($ffFactory) {
 	try {
-		$amount = Euro::newFromFloat( ( new AmountParser( 'en_EN' ) )->parseAsFloat(
-			$request->get( 'betrag_auswahl', $request->get( 'amountGiven', '' ) ) )
+		$amount = Euro::newFromFloat((new AmountParser('en_EN'))->parseAsFloat(
+			$request->get('betrag_auswahl', $request->get('amountGiven', '')) )
 		);
-	} catch ( \InvalidArgumentException $ex ) {
-		$amount = Euro::newFromCents( 0 );
+	} catch (\InvalidArgumentException $ex) {
+		$amount = Euro::newFromCents(0);
 	}
-	$validationResult = $ffFactory->newPaymentDataValidator()->validate( $amount, (string) $request->get( 'zahlweise', '' ) );
+	$validationResult = $ffFactory->newPaymentDataValidator()->validate($amount, (string)$request->get('zahlweise', ''));
 
 	$trackingInfo = new DonationTrackingInfo();
-	$trackingInfo->setTotalImpressionCount( intval( $request->get( 'impCount' ) ) );
-	$trackingInfo->setSingleBannerImpressionCount( intval( $request->get( 'bImpCount' ) ) );
+	$trackingInfo->setTotalImpressionCount(intval($request->get('impCount')));
+	$trackingInfo->setSingleBannerImpressionCount(intval($request->get('bImpCount')));
 
 	// TODO: don't we want to use newDonationFormViolationPresenter when !$validationResult->isSuccessful()?
 
 	return new Response(
 		$ffFactory->newDonationFormPresenter()->present(
 			$amount,
-			$request->get( 'zahlweise', '' ),
-			intval( $request->get( 'periode', 0 ) ),
+			$request->get('zahlweise', ''),
+			intval($request->get('periode', 0)),
 			$validationResult->isSuccessful(),
 			$trackingInfo,
-			$request->get( 'addressType', 'person' )
+			$request->get('addressType', 'person')
 		)
 	);
-} )->method( 'POST|GET' );
+} )->method('POST|GET');
 
 $app->post(
 	'apply-for-membership',
-	function( Application $app, Request $httpRequest ) use ( $ffFactory ) {
-		return ( new ApplyForMembershipHandler( $ffFactory, $app ) )->handle( $httpRequest );
+	function(Application $app, Request $httpRequest) use ($ffFactory) {
+		return (new ApplyForMembershipHandler($ffFactory, $app))->handle($httpRequest);
 	}
 );
 
 $app->get(
 	'apply-for-membership',
-	function( Request $request ) use ( $ffFactory ) {
+	function(Request $request) use ($ffFactory) {
 		$params = [];
 
-		if ( $request->query->get('type' ) === 'sustaining' ) {
-			$params['showMembershipTypeOption'] = false ;
+		if ($request->query->get('type') === 'sustaining') {
+			$params['showMembershipTypeOption'] = false;
 		}
 
 		try {
-			$useCase = $ffFactory->newShowDonationConfirmationUseCase( $request->query->get( 'donationAccessToken', '' ) );
-			$responseModel = $useCase->showConfirmation( new ShowDonationConfirmationRequest(
-				$request->query->getInt( 'donationId' )
-			) );
+			$useCase = $ffFactory->newShowDonationConfirmationUseCase($request->query->get('donationAccessToken', ''));
+			$responseModel = $useCase->showConfirmation(new ShowDonationConfirmationRequest(
+				$request->query->getInt('donationId')
+			));
 
-			if ( $responseModel->accessIsPermitted() ) {
+			if ($responseModel->accessIsPermitted()) {
 				$adapter = new DonationMembershipApplicationAdapter();
-				$params['initialFormValues'] = $adapter->getInitialMembershipFormValues( $responseModel->getDonation() );
+				$params['initialFormValues'] = $adapter->getInitialMembershipFormValues($responseModel->getDonation());
 			}
-		} catch ( Exception $e ) {
+		} catch (Exception $e) {
 		}
 
-		return $ffFactory->getMembershipApplicationFormTemplate()->render( $params );
+		return $ffFactory->getMembershipApplicationFormTemplate()->render($params);
 	}
 );
 
 $app->get(
 	'show-membership-confirmation',
-	function( Request $request ) use ( $ffFactory ) {
-		$confirmationRequest = new ShowAppConfirmationRequest( (int)$request->query->get( 'id', 0 ) );
+	function(Request $request) use ($ffFactory) {
+		$confirmationRequest = new ShowAppConfirmationRequest((int)$request->query->get('id', 0));
 
 		return $ffFactory->newMembershipApplicationConfirmationHtmlPresenter()->present(
-			$ffFactory->newMembershipApplicationConfirmationUseCase( $request->query->get( 'accessToken', '' ) )
-				->showConfirmation( $confirmationRequest )
+			$ffFactory->newMembershipApplicationConfirmationUseCase($request->query->get('accessToken', ''))
+				->showConfirmation($confirmationRequest)
 		);
 	}
-)->bind( 'show-membership-confirmation' );
+)->bind('show-membership-confirmation');
 
 $app->get(
 	'cancel-membership-application',
-	function( Request $request ) use ( $ffFactory ) {
+	function(Request $request) use ($ffFactory) {
 		$cancellationRequest = new CancellationRequest(
-			(int)$request->query->get( 'id', '' )
+			(int)$request->query->get('id', '')
 		);
 
 		return $ffFactory->newCancelMembershipApplicationHtmlPresenter()->present(
-			$ffFactory->newCancelMembershipApplicationUseCase( $request->query->get( 'updateToken', '' ) )
-				->cancelApplication( $cancellationRequest )
+			$ffFactory->newCancelMembershipApplicationUseCase($request->query->get('updateToken', ''))
+				->cancelApplication($cancellationRequest)
 		);
 	}
 );
 
 $app->match(
 	'show-donation-confirmation',
-	function( Application $app, Request $request ) use ( $ffFactory ) {
-		return ( new ShowDonationConfirmationHandler( $ffFactory ) )->handle(
+	function(Application $app, Request $request) use ($ffFactory) {
+		return (new ShowDonationConfirmationHandler($ffFactory))->handle(
 			$request,
-			$app['session']->get( 'piwikTracking', [] )
+			$app['session']->get('piwikTracking', [])
 		);
 	}
-)->bind( 'show-donation-confirmation' )
-->method( 'GET|POST' );
+)->bind('show-donation-confirmation')
+->method('GET|POST');
 
 $app->post(
 	'handle-paypal-payment-notification',
-	function ( Request $request ) use ( $ffFactory ) {
-		return ( new PayPalNotificationHandler( $ffFactory ) )->handle( $request );
+	function(Request $request) use ($ffFactory) {
+		return (new PayPalNotificationHandler($ffFactory))->handle($request);
 	}
 );
 
 $app->post(
 	'sofort-payment-notification',
-	function ( Request $request ) use ( $ffFactory ) {
-		return ( new SofortNotificationHandler( $ffFactory ) )->handle( $request );
+	function(Request $request) use ($ffFactory) {
+		return (new SofortNotificationHandler($ffFactory))->handle($request);
 	}
 );
 
 $app->get(
 	'handle-creditcard-payment-notification',
-	function ( Request $request ) use ( $ffFactory ) {
+	function(Request $request) use ($ffFactory) {
 		try {
-			$ffFactory->newCreditCardNotificationUseCase( $request->query->get( 'utoken', '' ) )
+			$ffFactory->newCreditCardNotificationUseCase($request->query->get('utoken', ''))
 				->handleNotification(
-					( new CreditCardPaymentNotificationRequest() )
-						->setTransactionId( $request->query->get( 'transactionId', '' ) )
-						->setDonationId( (int)$request->query->get( 'donation_id', '' ) )
-						->setAmount( Euro::newFromCents( (int)$request->query->get( 'amount' ) ) )
-						->setCustomerId( $request->query->get( 'customerId', '' ) )
-						->setSessionId( $request->query->get( 'sessionId', '' ) )
-						->setAuthId( $request->query->get(  'auth', '' ) )
-						->setTitle( $request->query->get( 'title', '' ) )
-						->setCountry( $request->query->get( 'country', '' ) )
-						->setCurrency( $request->query->get( 'currency', '' ) )
+					(new CreditCardPaymentNotificationRequest())
+						->setTransactionId($request->query->get('transactionId', ''))
+						->setDonationId((int)$request->query->get('donation_id', ''))
+						->setAmount(Euro::newFromCents((int)$request->query->get('amount')))
+						->setCustomerId($request->query->get('customerId', ''))
+						->setSessionId($request->query->get('sessionId', ''))
+						->setAuthId($request->query->get('auth', ''))
+						->setTitle($request->query->get('title', ''))
+						->setCountry($request->query->get('country', ''))
+						->setCurrency($request->query->get('currency', ''))
 				);
 
 			$response = CreditCardNotificationResponse::newSuccessResponse(
-				(int)$request->query->get( 'donation_id', '' ),
-				$request->query->get( 'token', '' )
+				(int)$request->query->get('donation_id', ''),
+				$request->query->get('token', '')
  			);
-		} catch ( CreditCardPaymentHandlerException $e ) {
-			$response = CreditCardNotificationResponse::newFailureResponse( $e->getMessage() );
+		} catch (CreditCardPaymentHandlerException $e) {
+			$response = CreditCardNotificationResponse::newFailureResponse($e->getMessage());
 		}
 
-		return new Response( $ffFactory->newCreditCardNotificationPresenter()->present( $response ) );
+		return new Response($ffFactory->newCreditCardNotificationPresenter()->present($response));
 	}
 );
 
 $app->get(
 	'donation-accepted',
-	function( Request $request ) use ( $app, $ffFactory ) {
+	function(Request $request) use ($app, $ffFactory) {
 
-		$eventHandler = $ffFactory->newDonationAcceptedEventHandler( $request->query->get( 'update_token', '' ) );
-		$result = $eventHandler->onDonationAccepted( (int)$request->query->get( 'donation_id', '' ) );
+		$eventHandler = $ffFactory->newDonationAcceptedEventHandler($request->query->get('update_token', ''));
+		$result = $eventHandler->onDonationAccepted((int)$request->query->get('donation_id', ''));
 
 		return $app->json(
-			$result === null ? [ 'status' => 'OK' ] : [ 'status' => 'ERR', 'message' => $result ]
+			$result === null ? ['status' => 'OK'] : ['status' => 'ERR', 'message' => $result]
 		);
 	}
 );
 
 $app->post(
 	'handle-paypal-membership-fee-payments',
-	function ( Request $request ) use ( $ffFactory ) {
-		return ( new PayPalNotificationHandlerForMembershipFee( $ffFactory ) )->handle( $request->request );
+	function(Request $request) use ($ffFactory) {
+		return (new PayPalNotificationHandlerForMembershipFee($ffFactory))->handle($request->request);
 	}
 );
 
-$app->get( '/', function ( Application $app, Request $request ) {
-	$app['session']->set( 'piwikTracking', array_filter(
+$app->get('/', function(Application $app, Request $request) {
+	$app['session']->set('piwikTracking', array_filter(
 			[
-				'paymentType' => $request->get( 'zahlweise', '' ),
-				'paymentAmount' => $request->get( 'betrag', '' ),
-				'paymentInterval' => $request->get( 'periode', '' )
+				'paymentType' => $request->get('zahlweise', ''),
+				'paymentAmount' => $request->get('betrag', ''),
+				'paymentInterval' => $request->get('periode', '')
 			],
-			function ( string $value ) {
-				return $value !== '' && strlen( $value ) < 20;
+			function(string $value) {
+				return $value !== '' && strlen($value) < 20;
 			} )
 	);
 
@@ -556,43 +556,43 @@ 
 		),
 		HttpKernelInterface::SUB_REQUEST
 	);
-} )->bind( '/' );
+} )->bind('/');
 
 // TODO Figure out how to rewrite with Nginx
 // See https://serverfault.com/questions/805881/nginx-populate-request-uri-with-rewritten-url
 $app->post(
 	'/spenden/paypal_handler.php',
-	function ( Request $request ) use ( $ffFactory ) {
-		return ( new PayPalNotificationHandler( $ffFactory ) )->handle( $request );
+	function(Request $request) use ($ffFactory) {
+		return (new PayPalNotificationHandler($ffFactory))->handle($request);
 	}
 );
 
 // redirect display page requests from old URLs
-$app->get( '/spenden/{page}', function( Application $app, Request $request, string $page ) {
+$app->get('/spenden/{page}', function(Application $app, Request $request, string $page) {
 	// Poor man's rewrite until someone has figured out how to do this with Nginx without breaking REQUEST_URI
 	// See https://serverfault.com/questions/805881/nginx-populate-request-uri-with-rewritten-url
-	switch ( $page ) {
+	switch ($page) {
 		case 'Mitgliedschaft':
-			return ( new RouteRedirectionHandler( $app, $request->getQueryString() ) )->handle( '/page/Membership_Application' );
+			return (new RouteRedirectionHandler($app, $request->getQueryString()))->handle('/page/Membership_Application');
 		default:
-			return ( new RouteRedirectionHandler( $app, $request->getQueryString() ) )->handle( '/page/' . $page );
+			return (new RouteRedirectionHandler($app, $request->getQueryString()))->handle('/page/' . $page);
 	}
-} )->assert( 'page', '[a-zA-Z_\-\s\x7f-\xff]+' );
+} )->assert('page', '[a-zA-Z_\-\s\x7f-\xff]+');
 
 // redirect different formats of comment lists
-$app->get( '/spenden/{outputFormat}.php', function( Application $app, Request $request, string $outputFormat ) {
-	return ( new RouteRedirectionHandler( $app, $request->getQueryString() ) )->handle(
-		'/list-comments.' . ( $outputFormat === 'list' ? 'html' : $outputFormat )
+$app->get('/spenden/{outputFormat}.php', function(Application $app, Request $request, string $outputFormat) {
+	return (new RouteRedirectionHandler($app, $request->getQueryString()))->handle(
+		'/list-comments.' . ($outputFormat === 'list' ? 'html' : $outputFormat)
 	);
-} )->assert( 'outputFormat', 'list|rss|json' );
+} )->assert('outputFormat', 'list|rss|json');
 
 // redirect all other calls to default route
-$app->get( '/spenden{page}', function( Application $app, Request $request ) {
-	return ( new RouteRedirectionHandler( $app, $request->getQueryString() ) )->handle( '/' );
-} )->assert( 'page', '/?([a-z]+\.php)?' );
+$app->get('/spenden{page}', function(Application $app, Request $request) {
+	return (new RouteRedirectionHandler($app, $request->getQueryString()))->handle('/');
+} )->assert('page', '/?([a-z]+\.php)?');
 
-$app->get( '/purge-cache', function( Request $request ) use ( $ffFactory ) {
-	$response = $ffFactory->newAuthorizedCachePurger()->purgeCache( $request->query->get( 'secret', '' ) );
+$app->get('/purge-cache', function(Request $request) use ($ffFactory) {
+	$response = $ffFactory->newAuthorizedCachePurger()->purgeCache($request->query->get('secret', ''));
 
 	return new Response(
 		[
@@ -603,7 +603,7 @@ 
 	);
 } );
 
-$app->get( 'status', function() {
+$app->get('status', function() {
 	return 'Status: OK (Online)';
 } );
 

app/RouteHandlers/ShowMembershipConfirmationHandler.php

Spacing +12 added lines, -12 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\App\RouteHandlers;
 
@@ -22,33 +22,33 @@ 
 
 	private $ffFactory;
 
-	public function __construct( FunFunFactory $ffFactory ) {
+	public function __construct(FunFunFactory $ffFactory) {
 		$this->ffFactory = $ffFactory;
 	}
 
-	public function handle( Request $request ): Response {
-		$useCase = $this->ffFactory->newMembershipApplicationConfirmationUseCase( $request->get( 'accessToken', '' ) );
+	public function handle(Request $request): Response {
+		$useCase = $this->ffFactory->newMembershipApplicationConfirmationUseCase($request->get('accessToken', ''));
 
-		$responseModel = $useCase->showConfirmation( new ShowAppConfirmationRequest(
-			(int)$request->get( 'id', '' )
-		) );
+		$responseModel = $useCase->showConfirmation(new ShowAppConfirmationRequest(
+			(int)$request->get('id', '')
+		));
 
-		if ( $responseModel->accessIsPermitted() ) {
+		if ($responseModel->accessIsPermitted()) {
 			$httpResponse = new Response(
-				$this->ffFactory->newMembershipApplicationConfirmationHtmlPresenter()->present( $responseModel )
+				$this->ffFactory->newMembershipApplicationConfirmationHtmlPresenter()->present($responseModel)
 			);
 
-			if ( $request->cookies->get( self::SUBMISSION_COOKIE_NAME ) ) {
+			if ($request->cookies->get(self::SUBMISSION_COOKIE_NAME)) {
 				$cookie = $this->ffFactory->getCookieBuilder();
 				$httpResponse->headers->setCookie(
-					$cookie->newCookie( self::SUBMISSION_COOKIE_NAME, date( self::TIMESTAMP_FORMAT ) )
+					$cookie->newCookie(self::SUBMISSION_COOKIE_NAME, date(self::TIMESTAMP_FORMAT))
 				);
 			}
 
 			return $httpResponse;
 		}
 
-		throw new AccessDeniedException( 'access_denied_donation_confirmation' );
+		throw new AccessDeniedException('access_denied_donation_confirmation');
 	}
 
 }
\ No newline at end of file