wmde / FundraisingFrontend

app/RouteHandlers/AddDonationHandler.php

Spacing +75 added lines, -75 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\App\RouteHandlers;
 
@@ -27,47 +27,47 @@ 
 	private $ffFactory;
 	private $app;
 
-	public function __construct( FunFunFactory $ffFactory, Application $app ) {
+	public function __construct(FunFunFactory $ffFactory, Application $app) {
 		$this->ffFactory = $ffFactory;
 		$this->app = $app;
 	}
 
-	public function handle( Request $request ): Response {
-		if ( !$this->isSubmissionAllowed( $request ) ) {
-			return new Response( $this->ffFactory->newSystemMessageResponse( 'donation_rejected_limit' ) );
+	public function handle(Request $request): Response {
+		if (!$this->isSubmissionAllowed($request)) {
+			return new Response($this->ffFactory->newSystemMessageResponse('donation_rejected_limit'));
 		}
 
-		$addDonationRequest = $this->createDonationRequest( $request );
-		$responseModel = $this->ffFactory->newAddDonationUseCase()->addDonation( $addDonationRequest );
+		$addDonationRequest = $this->createDonationRequest($request);
+		$responseModel = $this->ffFactory->newAddDonationUseCase()->addDonation($addDonationRequest);
 
-		if ( !$responseModel->isSuccessful() ) {
+		if (!$responseModel->isSuccessful()) {
 			return new Response(
 				$this->ffFactory->newDonationFormViolationPresenter()->present(
 					$responseModel->getValidationErrors(), $addDonationRequest,
-					$this->newTrackingInfoFromRequest( $request )
+					$this->newTrackingInfoFromRequest($request)
 				)
 			);
 		}
 
-		$this->sendTrackingDataIfNeeded( $request, $responseModel );
+		$this->sendTrackingDataIfNeeded($request, $responseModel);
 
-		return $this->newHttpResponse( $responseModel );
+		return $this->newHttpResponse($responseModel);
 	}
 
-	private function sendTrackingDataIfNeeded( Request $request, AddDonationResponse $responseModel ) {
-		if ( $request->get( 'mbt', '' ) !== '1' || $responseModel->getDonation()->getPaymentType() !== PaymentType::PAYPAL ) {
+	private function sendTrackingDataIfNeeded(Request $request, AddDonationResponse $responseModel) {
+		if ($request->get('mbt', '') !== '1' || $responseModel->getDonation()->getPaymentType() !== PaymentType::PAYPAL) {
 			return;
 		}
 
-		$trackingCode = explode( '/', $request->attributes->get( 'trackingCode' ) );
+		$trackingCode = explode('/', $request->attributes->get('trackingCode'));
 		$campaign = $trackingCode[0];
 		$keyword = $trackingCode[1] ?? '';
 
-		$this->ffFactory->getPageViewTracker()->trackPaypalRedirection( $campaign, $keyword, $request->getClientIp() );
+		$this->ffFactory->getPageViewTracker()->trackPaypalRedirection($campaign, $keyword, $request->getClientIp());
 	}
 
-	private function newHttpResponse( AddDonationResponse $responseModel ): Response {
-		switch( $responseModel->getDonation()->getPaymentType() ) {
+	private function newHttpResponse(AddDonationResponse $responseModel): Response {
+		switch ($responseModel->getDonation()->getPaymentType()) {
 			case PaymentType::DIRECT_DEBIT:
 			case PaymentType::BANK_TRANSFER:
 				return $this->app->redirect(
@@ -105,107 +105,107 @@ 
 				break;
 			case PaymentType::CREDIT_CARD:
 				return $this->app->redirect(
-					$this->ffFactory->newCreditCardPaymentUrlGenerator()->buildUrl( $responseModel )
+					$this->ffFactory->newCreditCardPaymentUrlGenerator()->buildUrl($responseModel)
 				);
 				break;
 			default:
-				throw new \LogicException( 'This code should not be reached' );
+				throw new \LogicException('This code should not be reached');
 		}
 	}
 
-	private function createDonationRequest( Request $request ): AddDonationRequest {
+	private function createDonationRequest(Request $request): AddDonationRequest {
 		$donationRequest = new AddDonationRequest();
 
-		$donationRequest->setAmount( $this->getEuroAmountFromString( $request->get( 'betrag', '' ) ) );
-
-		$donationRequest->setPaymentType( $request->get( 'zahlweise', '' ) );
-		$donationRequest->setInterval( intval( $request->get( 'periode', 0 ) ) );
-
-		$donationRequest->setDonorType( $request->get( 'addressType', '' ) );
-		$donationRequest->setDonorSalutation( $request->get( 'salutation', '' ) );
-		$donationRequest->setDonorTitle( $request->get( 'title', '' ) );
-		$donationRequest->setDonorCompany( $request->get( 'companyName', '' ) );
-		$donationRequest->setDonorFirstName( $request->get( 'firstName', '' ) );
-		$donationRequest->setDonorLastName( $request->get( 'lastName', '' ) );
-		$donationRequest->setDonorStreetAddress( $this->filterAutofillCommas( $request->get( 'street', '' ) ) );
-		$donationRequest->setDonorPostalCode( $request->get( 'postcode', '' ) );
-		$donationRequest->setDonorCity( $request->get( 'city', '' ) );
-		$donationRequest->setDonorCountryCode( $request->get( 'country', '' ) );
-		$donationRequest->setDonorEmailAddress( $request->get( 'email', '' ) );
-
-		if ( $request->get( 'zahlweise', '' ) === PaymentType::DIRECT_DEBIT ) {
-			$donationRequest->setBankData( $this->getBankDataFromRequest( $request ) );
+		$donationRequest->setAmount($this->getEuroAmountFromString($request->get('betrag', '')));
+
+		$donationRequest->setPaymentType($request->get('zahlweise', ''));
+		$donationRequest->setInterval(intval($request->get('periode', 0)));
+
+		$donationRequest->setDonorType($request->get('addressType', ''));
+		$donationRequest->setDonorSalutation($request->get('salutation', ''));
+		$donationRequest->setDonorTitle($request->get('title', ''));
+		$donationRequest->setDonorCompany($request->get('companyName', ''));
+		$donationRequest->setDonorFirstName($request->get('firstName', ''));
+		$donationRequest->setDonorLastName($request->get('lastName', ''));
+		$donationRequest->setDonorStreetAddress($this->filterAutofillCommas($request->get('street', '')));
+		$donationRequest->setDonorPostalCode($request->get('postcode', ''));
+		$donationRequest->setDonorCity($request->get('city', ''));
+		$donationRequest->setDonorCountryCode($request->get('country', ''));
+		$donationRequest->setDonorEmailAddress($request->get('email', ''));
+
+		if ($request->get('zahlweise', '') === PaymentType::DIRECT_DEBIT) {
+			$donationRequest->setBankData($this->getBankDataFromRequest($request));
 		}
 
-		$donationRequest->setTracking( $request->attributes->get( 'trackingCode' ) );
-		$donationRequest->setOptIn( $request->get( 'info', '' ) );
-		$donationRequest->setSource( $request->attributes->get( 'trackingSource' ) );
-		$donationRequest->setTotalImpressionCount( intval( $request->get( 'impCount', 0 ) ) );
-		$donationRequest->setSingleBannerImpressionCount( intval( $request->get( 'bImpCount', 0 ) ) );
-		$donationRequest->setOptsIntoDonationReceipt( $request->request->getBoolean( 'donationReceipt', true ) );
+		$donationRequest->setTracking($request->attributes->get('trackingCode'));
+		$donationRequest->setOptIn($request->get('info', ''));
+		$donationRequest->setSource($request->attributes->get('trackingSource'));
+		$donationRequest->setTotalImpressionCount(intval($request->get('impCount', 0)));
+		$donationRequest->setSingleBannerImpressionCount(intval($request->get('bImpCount', 0)));
+		$donationRequest->setOptsIntoDonationReceipt($request->request->getBoolean('donationReceipt', true));
 
 		return $donationRequest;
 	}
 
-	private function getBankDataFromRequest( Request $request ): BankData {
+	private function getBankDataFromRequest(Request $request): BankData {
 		$bankData = new BankData();
-		$bankData->setIban( new Iban( $request->get( 'iban', '' ) ) )
-			->setBic( $request->get( 'bic', '' ) )
-			->setAccount( $request->get( 'konto', '' ) )
-			->setBankCode( $request->get( 'blz', '' ) )
-			->setBankName( $request->get( 'bankname', '' ) );
+		$bankData->setIban(new Iban($request->get('iban', '')))
+			->setBic($request->get('bic', ''))
+			->setAccount($request->get('konto', ''))
+			->setBankCode($request->get('blz', ''))
+			->setBankName($request->get('bankname', ''));
 
-		if ( $bankData->isComplete() ) {
+		if ($bankData->isComplete()) {
 			return $bankData->freeze()->assertNoNullFields();
 		}
 
-		if ( $bankData->hasIban() ) {
-			$bankData = $this->newBankDataFromIban( $bankData->getIban() );
-		} elseif ( $bankData->hasCompleteLegacyBankData() ) {
-			$bankData = $this->newBankDataFromAccountAndBankCode( $bankData->getAccount(), $bankData->getBankCode() );
+		if ($bankData->hasIban()) {
+			$bankData = $this->newBankDataFromIban($bankData->getIban());
+		} elseif ($bankData->hasCompleteLegacyBankData()) {
+			$bankData = $this->newBankDataFromAccountAndBankCode($bankData->getAccount(), $bankData->getBankCode());
 		}
 		return $bankData->freeze()->assertNoNullFields();
 	}
 
-	private function newBankDataFromIban( Iban $iban ): BankData {
-		return $this->ffFactory->newBankDataConverter()->getBankDataFromIban( $iban );
+	private function newBankDataFromIban(Iban $iban): BankData {
+		return $this->ffFactory->newBankDataConverter()->getBankDataFromIban($iban);
 	}
 
-	private function newBankDataFromAccountAndBankCode( string $account, string $bankCode ): BankData {
-		return $this->ffFactory->newBankDataConverter()->getBankDataFromAccountData( $account, $bankCode );
+	private function newBankDataFromAccountAndBankCode(string $account, string $bankCode): BankData {
+		return $this->ffFactory->newBankDataConverter()->getBankDataFromAccountData($account, $bankCode);
 	}
 
-	private function getEuroAmountFromString( string $amount ): Euro {
+	private function getEuroAmountFromString(string $amount): Euro {
 		$locale = 'de_DE'; // TODO: make this configurable for multilanguage support
 		try {
-			return Euro::newFromFloat( ( new AmountParser( $locale ) )->parseAsFloat( $amount ) );
-		} catch ( \InvalidArgumentException $ex ) {
-			return Euro::newFromCents( 0 );
+			return Euro::newFromFloat((new AmountParser($locale))->parseAsFloat($amount));
+		} catch (\InvalidArgumentException $ex) {
+			return Euro::newFromCents(0);
 		}
 
 	}
 
-	private function isSubmissionAllowed( Request $request ) {
-		$lastSubmission = $request->cookies->get( ShowDonationConfirmationHandler::SUBMISSION_COOKIE_NAME, '' );
-		if ( $lastSubmission === '' ) {
+	private function isSubmissionAllowed(Request $request) {
+		$lastSubmission = $request->cookies->get(ShowDonationConfirmationHandler::SUBMISSION_COOKIE_NAME, '');
+		if ($lastSubmission === '') {
 			return true;
 		}
 
 		$minNextTimestamp =
-			\DateTime::createFromFormat( ShowDonationConfirmationHandler::TIMESTAMP_FORMAT, $lastSubmission )
-			->add( new \DateInterval( $this->ffFactory->getDonationTimeframeLimit() ) );
+			\DateTime::createFromFormat(ShowDonationConfirmationHandler::TIMESTAMP_FORMAT, $lastSubmission)
+			->add(new \DateInterval($this->ffFactory->getDonationTimeframeLimit()));
 
-		if ( $minNextTimestamp > new \DateTime() ) {
+		if ($minNextTimestamp > new \DateTime()) {
 			return false;
 		}
 
 		return true;
 	}
 
-	private function newTrackingInfoFromRequest( Request $request ): DonationTrackingInfo {
+	private function newTrackingInfoFromRequest(Request $request): DonationTrackingInfo {
 		$tracking = new DonationTrackingInfo();
-		$tracking->setSingleBannerImpressionCount( intval( $request->get( 'bImpCount', 0 ) ) );
-		$tracking->setTotalImpressionCount( intval( $request->get( 'impCount', 0 ) ) );
+		$tracking->setSingleBannerImpressionCount(intval($request->get('bImpCount', 0)));
+		$tracking->setTotalImpressionCount(intval($request->get('impCount', 0)));
 
 		return $tracking;
 	}
@@ -217,7 +217,7 @@ 
 	 * @param string $value
 	 * @return string
 	 */
-	private function filterAutofillCommas( string $value ): string {
-		return trim( preg_replace( ['/,/', '/\s{2,}/'], [' ', ' '], $value ) );
+	private function filterAutofillCommas(string $value): string {
+		return trim(preg_replace(['/,/', '/\s{2,}/'], [' ', ' '], $value));
 	}
 }
\ No newline at end of file

tests/EdgeToEdge/Routes/AddDonationRouteTest.php

Spacing +223 added lines, -223 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\Tests\EdgeToEdge\Routes;
 
@@ -32,10 +32,10 @@ 
 	private const ADD_DONATION_PATH = '/donation/add';
 
 	public function testGivenValidRequest_donationGetsPersisted(): void {
-		$this->createEnvironment( [], function ( Client $client, FunFunFactory $factory ): void {
+		$this->createEnvironment([], function(Client $client, FunFunFactory $factory): void {
 
-			$client->setServerParameter( 'HTTP_REFERER', 'https://en.wikipedia.org/wiki/Karla_Kennichnich' );
-			$client->followRedirects( false );
+			$client->setServerParameter('HTTP_REFERER', 'https://en.wikipedia.org/wiki/Karla_Kennichnich');
+			$client->followRedirects(false);
 
 			$client->request(
 				'POST',
@@ -43,28 +43,28 @@ 
 				$this->newValidFormInput()
 			);
 
-			$this->assertIsExpectedDonation( $this->getDonationFromDatabase( $factory ) );
+			$this->assertIsExpectedDonation($this->getDonationFromDatabase($factory));
 		} );
 	}
 
 	public function testWhenDonationGetsPersisted_timestampIsStoredInCookie(): void {
 		$client = $this->createClient();
-		$client->followRedirects( true );
+		$client->followRedirects(true);
 		$client->request(
 			'POST',
 			'/donation/add',
 			$this->newValidFormInput()
 		);
 
-		$cookie = $client->getCookieJar()->get( 'donation_timestamp' );
-		$this->assertNotNull( $cookie );
-		$donationTimestamp = new \DateTime( $cookie->getValue() );
-		$this->assertEquals( time(), $donationTimestamp->getTimestamp(), 'Timestamp should be not more than 5 seconds old', 5.0 );
+		$cookie = $client->getCookieJar()->get('donation_timestamp');
+		$this->assertNotNull($cookie);
+		$donationTimestamp = new \DateTime($cookie->getValue());
+		$this->assertEquals(time(), $donationTimestamp->getTimestamp(), 'Timestamp should be not more than 5 seconds old', 5.0);
 	}
 
 	public function testWhenMultipleDonationFormSubmissions_requestGetsRejected(): void {
 		$client = $this->createClient();
-		$client->getCookieJar()->set( new Cookie( 'donation_timestamp', $this->getPastTimestamp() ) );
+		$client->getCookieJar()->set(new Cookie('donation_timestamp', $this->getPastTimestamp()));
 
 		$client->request(
 			'POST',
@@ -72,7 +72,7 @@ 
 			$this->newValidFormInput()
 		);
 
-		$this->assertContains( 'donation_rejected_limit', $client->getResponse()->getContent() );
+		$this->assertContains('donation_rejected_limit', $client->getResponse()->getContent());
 	}
 
 	public function testWhenMultipleDonationsInAccordanceToTimeLimit_requestIsNotRejected(): void {
@@ -80,7 +80,7 @@ 
 		$client->getCookieJar()->set(
 			new Cookie(
 				'donation_timestamp',
-				$this->getPastTimestamp( 'PT35M' )
+				$this->getPastTimestamp('PT35M')
 			)
 		);
 
@@ -90,11 +90,11 @@ 
 			$this->newValidFormInput()
 		);
 
-		$this->assertNotContains( 'donation_rejected_limit', $client->getResponse()->getContent() );
+		$this->assertNotContains('donation_rejected_limit', $client->getResponse()->getContent());
 	}
 
-	private function getPastTimestamp( string $interval = 'PT10S' ): string {
-		return ( new \DateTime() )->sub( new \DateInterval( $interval ) )->format( 'Y-m-d H:i:s' );
+	private function getPastTimestamp(string $interval = 'PT10S'): string {
+		return (new \DateTime())->sub(new \DateInterval($interval))->format('Y-m-d H:i:s');
 	}
 
 	private function newValidFormInput(): array {
@@ -129,40 +129,40 @@ 
 		];
 	}
 
-	private function assertIsExpectedDonation( Donation $donation ): void {
+	private function assertIsExpectedDonation(Donation $donation): void {
 		$data = $donation->getDecodedData();
-		$this->assertSame( '5.51', $donation->getAmount() );
-		$this->assertSame( 'BEZ', $donation->getPaymentType() );
-		$this->assertSame( 0, $donation->getPaymentIntervalInMonths() );
-		$this->assertSame( 'DE12500105170648489890', $data['iban'] );
-		$this->assertSame( 'INGDDEFFXXX', $data['bic'] );
-		$this->assertSame( '0648489890', $data['konto'] );
-		$this->assertSame( '50010517', $data['blz'] );
-		$this->assertSame( 'ING-DiBa', $data['bankname'] );
-		$this->assertSame( 'person', $data['adresstyp'] );
-		$this->assertSame( 'Frau', $data['anrede'] );
-		$this->assertSame( 'Prof. Dr.', $data['titel'] );
-		$this->assertSame( '', $data['firma'] );
-		$this->assertSame( 'Karla', $data['vorname'] );
-		$this->assertSame( 'Kennichnich', $data['nachname'] );
-		$this->assertSame( 'Prof. Dr. Karla Kennichnich', $donation->getDonorFullName() );
-		$this->assertSame( 'Lehmgasse 12', $data['strasse'] );
-		$this->assertSame( '12345', $data['plz'] );
-		$this->assertSame( 'Einort', $data['ort'] );
-		$this->assertSame( 'Einort', $donation->getDonorCity() );
-		$this->assertSame( 'DE', $data['country'] );
-		$this->assertSame( 'karla@kennichnich.de', $data['email'] );
-		$this->assertSame( 'karla@kennichnich.de', $donation->getDonorEmail() );
-		$this->assertSame( 'test/gelb', $data['tracking'] );
-		$this->assertSame( 3, $data['impCount'] );
-		$this->assertSame( 1, $data['bImpCount'] );
-		$this->assertSame( '', $data['layout'] );
-		$this->assertSame( '', $data['color'] );
-		$this->assertSame( '', $data['skin'] );
-		$this->assertSame( 'en.wikipedia.org', $data['source'] );
-		$this->assertSame( 'N', $donation->getStatus() );
-		$this->assertTrue( $donation->getDonorOptsIntoNewsletter() );
-		$this->assertTrue( $donation->getDonationReceipt() );
+		$this->assertSame('5.51', $donation->getAmount());
+		$this->assertSame('BEZ', $donation->getPaymentType());
+		$this->assertSame(0, $donation->getPaymentIntervalInMonths());
+		$this->assertSame('DE12500105170648489890', $data['iban']);
+		$this->assertSame('INGDDEFFXXX', $data['bic']);
+		$this->assertSame('0648489890', $data['konto']);
+		$this->assertSame('50010517', $data['blz']);
+		$this->assertSame('ING-DiBa', $data['bankname']);
+		$this->assertSame('person', $data['adresstyp']);
+		$this->assertSame('Frau', $data['anrede']);
+		$this->assertSame('Prof. Dr.', $data['titel']);
+		$this->assertSame('', $data['firma']);
+		$this->assertSame('Karla', $data['vorname']);
+		$this->assertSame('Kennichnich', $data['nachname']);
+		$this->assertSame('Prof. Dr. Karla Kennichnich', $donation->getDonorFullName());
+		$this->assertSame('Lehmgasse 12', $data['strasse']);
+		$this->assertSame('12345', $data['plz']);
+		$this->assertSame('Einort', $data['ort']);
+		$this->assertSame('Einort', $donation->getDonorCity());
+		$this->assertSame('DE', $data['country']);
+		$this->assertSame('karla@kennichnich.de', $data['email']);
+		$this->assertSame('karla@kennichnich.de', $donation->getDonorEmail());
+		$this->assertSame('test/gelb', $data['tracking']);
+		$this->assertSame(3, $data['impCount']);
+		$this->assertSame(1, $data['bImpCount']);
+		$this->assertSame('', $data['layout']);
+		$this->assertSame('', $data['color']);
+		$this->assertSame('', $data['skin']);
+		$this->assertSame('en.wikipedia.org', $data['source']);
+		$this->assertSame('N', $donation->getStatus());
+		$this->assertTrue($donation->getDonorOptsIntoNewsletter());
+		$this->assertTrue($donation->getDonationReceipt());
 	}
 
 	public function testGivenValidRequest_confirmationPageContainsEnteredData(): void {
@@ -176,16 +176,16 @@ 
 
 		$response = $client->getResponse()->getContent();
 
-		$this->assertContains( '5,51 €', $response );
-		$this->assertContains( 'donation.interval: 0', $response );
-		$this->assertContains( 'DE12500105170648489890', $response );
-		$this->assertContains( 'INGDDEFFXXX', $response );
-		$this->assertContains( 'ING-DiBa', $response );
-		$this->assertContains( 'Prof. Dr. Karla Kennichnich', $response );
-		$this->assertContains( 'Lehmgasse 12', $response );
-		$this->assertContains( '<span id="confirm-postcode">12345</span> <span id="confirm-city">Einort</span>', $response );
-		$this->assertContains( 'karla@kennichnich.de', $response );
-		$this->assertContains( 'send-info', $response );
+		$this->assertContains('5,51 €', $response);
+		$this->assertContains('donation.interval: 0', $response);
+		$this->assertContains('DE12500105170648489890', $response);
+		$this->assertContains('INGDDEFFXXX', $response);
+		$this->assertContains('ING-DiBa', $response);
+		$this->assertContains('Prof. Dr. Karla Kennichnich', $response);
+		$this->assertContains('Lehmgasse 12', $response);
+		$this->assertContains('<span id="confirm-postcode">12345</span> <span id="confirm-city">Einort</span>', $response);
+		$this->assertContains('karla@kennichnich.de', $response);
+		$this->assertContains('send-info', $response);
 	}
 
 	public function testGivenValidBankTransferRequest_donationGetsPersisted(): void {
@@ -194,10 +194,10 @@ 
 		 */
 		$factory = null;
 
-		$this->createEnvironment( [], function ( Client $client, FunFunFactory $factory ): void {
+		$this->createEnvironment([], function(Client $client, FunFunFactory $factory): void {
 
-			$client->setServerParameter( 'HTTP_REFERER', 'https://en.wikipedia.org/wiki/Karla_Kennichnich' );
-			$client->followRedirects( false );
+			$client->setServerParameter('HTTP_REFERER', 'https://en.wikipedia.org/wiki/Karla_Kennichnich');
+			$client->followRedirects(false);
 
 			$client->request(
 				'POST',
@@ -205,37 +205,37 @@ 
 				$this->newValidBankTransferInput()
 			);
 
-			$donation = $this->getDonationFromDatabase( $factory );
+			$donation = $this->getDonationFromDatabase($factory);
 
 			$data = $donation->getDecodedData();
-			$this->assertSame( '12.34', $donation->getAmount() );
-			$this->assertSame( 'UEB', $donation->getPaymentType() );
-			$this->assertSame( 0, $donation->getPaymentIntervalInMonths() );
-			$this->assertSame( 'person', $data['adresstyp'] );
-			$this->assertSame( 'Frau', $data['anrede'] );
-			$this->assertSame( 'Prof. Dr.', $data['titel'] );
-			$this->assertSame( '', $data['firma'] );
-			$this->assertSame( 'Karla', $data['vorname'] );
-			$this->assertSame( 'Kennichnich', $data['nachname'] );
-			$this->assertSame( 'Prof. Dr. Karla Kennichnich', $donation->getDonorFullName() );
-			$this->assertSame( 'Lehmgasse 12', $data['strasse'] );
-			$this->assertSame( '12345', $data['plz'] );
-			$this->assertSame( 'Einort', $data['ort'] );
-			$this->assertSame( 'Einort', $donation->getDonorCity() );
-			$this->assertSame( 'DE', $data['country'] );
-			$this->assertSame( 'karla@kennichnich.de', $data['email'] );
-			$this->assertSame( 'karla@kennichnich.de', $donation->getDonorEmail() );
-			$this->assertSame( 'test/gelb', $data['tracking'] );
-			$this->assertSame( 3, $data['impCount'] );
-			$this->assertSame( 1, $data['bImpCount'] );
-			$this->assertSame( '', $data['layout'] );
-			$this->assertSame( '', $data['color'] );
-			$this->assertSame( '', $data['skin'] );
-			$this->assertSame( 'en.wikipedia.org', $data['source'] );
-			$this->assertSame( true, $donation->getDonorOptsIntoNewsletter() );
-
-			$this->assertSame( 'Z', $donation->getStatus() );
-			$this->assertRegExp( '/W-Q-[A-Z]{6}-[A-Z]/', $donation->getBankTransferCode() );
+			$this->assertSame('12.34', $donation->getAmount());
+			$this->assertSame('UEB', $donation->getPaymentType());
+			$this->assertSame(0, $donation->getPaymentIntervalInMonths());
+			$this->assertSame('person', $data['adresstyp']);
+			$this->assertSame('Frau', $data['anrede']);
+			$this->assertSame('Prof. Dr.', $data['titel']);
+			$this->assertSame('', $data['firma']);
+			$this->assertSame('Karla', $data['vorname']);
+			$this->assertSame('Kennichnich', $data['nachname']);
+			$this->assertSame('Prof. Dr. Karla Kennichnich', $donation->getDonorFullName());
+			$this->assertSame('Lehmgasse 12', $data['strasse']);
+			$this->assertSame('12345', $data['plz']);
+			$this->assertSame('Einort', $data['ort']);
+			$this->assertSame('Einort', $donation->getDonorCity());
+			$this->assertSame('DE', $data['country']);
+			$this->assertSame('karla@kennichnich.de', $data['email']);
+			$this->assertSame('karla@kennichnich.de', $donation->getDonorEmail());
+			$this->assertSame('test/gelb', $data['tracking']);
+			$this->assertSame(3, $data['impCount']);
+			$this->assertSame(1, $data['bImpCount']);
+			$this->assertSame('', $data['layout']);
+			$this->assertSame('', $data['color']);
+			$this->assertSame('', $data['skin']);
+			$this->assertSame('en.wikipedia.org', $data['source']);
+			$this->assertSame(true, $donation->getDonorOptsIntoNewsletter());
+
+			$this->assertSame('Z', $donation->getStatus());
+			$this->assertRegExp('/W-Q-[A-Z]{6}-[A-Z]/', $donation->getBankTransferCode());
 		} );
 	}
 
@@ -267,10 +267,10 @@ 
 	}
 
 	public function testGivenComplementableBankData_donationStillGetsPersisted(): void {
-		$this->createEnvironment( [], function ( Client $client, FunFunFactory $factory ): void {
+		$this->createEnvironment([], function(Client $client, FunFunFactory $factory): void {
 
-			$client->setServerParameter( 'HTTP_REFERER', 'https://en.wikipedia.org/wiki/Karla_Kennichnich' );
-			$client->followRedirects( false );
+			$client->setServerParameter('HTTP_REFERER', 'https://en.wikipedia.org/wiki/Karla_Kennichnich');
+			$client->followRedirects(false);
 
 			$client->request(
 				'POST',
@@ -278,14 +278,14 @@ 
 				$this->newComplementableFormInput()
 			);
 
-			$donation = $this->getDonationFromDatabase( $factory );
+			$donation = $this->getDonationFromDatabase($factory);
 
 			$data = $donation->getDecodedData();
-			$this->assertSame( 'DE12500105170648489890', $data['iban'] );
-			$this->assertSame( 'INGDDEFFXXX', $data['bic'] );
-			$this->assertSame( '0648489890', $data['konto'] );
-			$this->assertSame( '50010517', $data['blz'] );
-			$this->assertSame( 'ING-DiBa', $data['bankname'] );
+			$this->assertSame('DE12500105170648489890', $data['iban']);
+			$this->assertSame('INGDDEFFXXX', $data['bic']);
+			$this->assertSame('0648489890', $data['konto']);
+			$this->assertSame('50010517', $data['blz']);
+			$this->assertSame('ING-DiBa', $data['bankname']);
 		} );
 	}
 
@@ -308,16 +308,16 @@ 
 		];
 	}
 
-	private function getDonationFromDatabase( FunFunFactory $factory ): Donation {
-		$donationRepo = $factory->getEntityManager()->getRepository( Donation::class );
-		$donation = $donationRepo->find( 1 );
-		$this->assertInstanceOf( Donation::class, $donation );
+	private function getDonationFromDatabase(FunFunFactory $factory): Donation {
+		$donationRepo = $factory->getEntityManager()->getRepository(Donation::class);
+		$donation = $donationRepo->find(1);
+		$this->assertInstanceOf(Donation::class, $donation);
 		return $donation;
 	}
 
 	public function testGivenValidPayPalData_redirectsToPayPal(): void {
 		$client = $this->createClient();
-		$client->followRedirects( false );
+		$client->followRedirects(false);
 
 		$client->request(
 			'POST',
@@ -326,13 +326,13 @@ 
 		);
 
 		$response = $client->getResponse();
-		$this->assertSame( Response::HTTP_FOUND, $response->getStatusCode() );
-		$this->assertContains( 'sandbox.paypal.com', $response->getContent() );
+		$this->assertSame(Response::HTTP_FOUND, $response->getStatusCode());
+		$this->assertContains('sandbox.paypal.com', $response->getContent());
 	}
 
 	public function testWhenRedirectingToPayPal_translatedItemNameIsPassed(): void {
 		$client = $this->createClient();
-		$client->followRedirects( false );
+		$client->followRedirects(false);
 
 		$client->request(
 			'POST',
@@ -341,8 +341,8 @@ 
 		);
 
 		$response = $client->getResponse();
-		$this->assertSame( Response::HTTP_FOUND, $response->getStatusCode() );
-		$this->assertContains( 'item_name=item_name_donation', $response->getContent() );
+		$this->assertSame(Response::HTTP_FOUND, $response->getStatusCode());
+		$this->assertContains('item_name=item_name_donation', $response->getContent());
 	}
 
 	private function newValidPayPalInput(): array {
@@ -363,37 +363,37 @@ 
 		);
 
 		$response = $client->getResponse();
-		$this->assertSame( Response::HTTP_FOUND, $response->getStatusCode() );
-		$this->assertTrue( $response->isRedirect() );
-		$this->assertContains( 'amount=1234', $response->headers->get( 'Location' ) );
-		$this->assertContains( 'thatother.paymentprovider.com', $response->headers->get( 'Location' ) );
+		$this->assertSame(Response::HTTP_FOUND, $response->getStatusCode());
+		$this->assertTrue($response->isRedirect());
+		$this->assertContains('amount=1234', $response->headers->get('Location'));
+		$this->assertContains('thatother.paymentprovider.com', $response->headers->get('Location'));
 	}
 
 	public function testValidSofortInput_savesDonationAndRedirectsTo3rdPartyPage(): void {
-		$this->createEnvironment( [], function ( Client $client, FunFunFactory $factory ): void {
+		$this->createEnvironment([], function(Client $client, FunFunFactory $factory): void {
 			$response = new SofortResponse();
-			$response->setPaymentUrl( 'https://bankingpin.please' );
+			$response->setPaymentUrl('https://bankingpin.please');
 
-			$sofortClient = $this->createMock( SofortClient::class );
+			$sofortClient = $this->createMock(SofortClient::class);
 			$sofortClient
-				->method( 'get' )
-				->willReturn( $response );
-			$factory->setSofortClient( $sofortClient );
+				->method('get')
+				->willReturn($response);
+			$factory->setSofortClient($sofortClient);
 
 
-			$client->followRedirects( false );
+			$client->followRedirects(false);
 			$client->request(
 				'POST',
 				'/donation/add',
 				$this->newValidSofortInput()
 			);
 
-			$donation = $this->getDonationFromDatabase( $factory );
+			$donation = $this->getDonationFromDatabase($factory);
 
-			$this->assertSame( 'Z', $donation->getStatus() );
-			$this->assertRegExp( '/W-Q-[A-Z]{6}-[A-Z]/', $donation->getBankTransferCode() );
+			$this->assertSame('Z', $donation->getStatus());
+			$this->assertRegExp('/W-Q-[A-Z]{6}-[A-Z]/', $donation->getBankTransferCode());
 
-			$this->assertTrue( $client->getResponse()->isRedirect( 'https://bankingpin.please' ) );
+			$this->assertTrue($client->getResponse()->isRedirect('https://bankingpin.please'));
 		} );
 	}
 
@@ -425,23 +425,23 @@ 
 
 		$response = $client->getResponse()->getContent();
 
-		$this->assertContains( 'Amount: 0,00', $response );
-		$this->assertContains( 'Payment type: BEZ', $response );
-		$this->assertContains( 'Interval: 3', $response );
-		$this->assertContains( 'IBAN: DE12500105170648489890', $response );
-		$this->assertContains( 'BIC: INGDDEFFXXX', $response );
-		$this->assertContains( 'Bank name: ING-DiBa', $response );
-		$this->assertContains( 'Address type: person', $response );
-		$this->assertContains( 'Salutation: Frau', $response );
-		$this->assertContains( 'Title: Prof. Dr.', $response );
-		$this->assertContains( 'Company: ', $response );
-		$this->assertContains( 'First name: Karla', $response );
-		$this->assertContains( 'Last name: Kennichnich', $response );
-		$this->assertContains( 'Street: Lehmgasse 12', $response );
-		$this->assertContains( 'Postal code: 12345', $response );
-		$this->assertContains( 'City: Einort', $response );
-		$this->assertContains( 'Country code: DE', $response );
-		$this->assertContains( 'Email address: karla@kennichnich.de', $response );
+		$this->assertContains('Amount: 0,00', $response);
+		$this->assertContains('Payment type: BEZ', $response);
+		$this->assertContains('Interval: 3', $response);
+		$this->assertContains('IBAN: DE12500105170648489890', $response);
+		$this->assertContains('BIC: INGDDEFFXXX', $response);
+		$this->assertContains('Bank name: ING-DiBa', $response);
+		$this->assertContains('Address type: person', $response);
+		$this->assertContains('Salutation: Frau', $response);
+		$this->assertContains('Title: Prof. Dr.', $response);
+		$this->assertContains('Company: ', $response);
+		$this->assertContains('First name: Karla', $response);
+		$this->assertContains('Last name: Kennichnich', $response);
+		$this->assertContains('Street: Lehmgasse 12', $response);
+		$this->assertContains('Postal code: 12345', $response);
+		$this->assertContains('City: Einort', $response);
+		$this->assertContains('Country code: DE', $response);
+		$this->assertContains('Email address: karla@kennichnich.de', $response);
 	}
 
 	public function testGivenInvalidRequest_formStillContainsBannerTrackingData(): void {
@@ -457,8 +457,8 @@ 
 
 		$response = $client->getResponse()->getContent();
 
-		$this->assertContains( 'Impression Count: 12', $response );
-		$this->assertContains( 'Banner Impression Count: 3', $response );
+		$this->assertContains('Impression Count: 12', $response);
+		$this->assertContains('Banner Impression Count: 3', $response);
 	}
 
 	public function testGivenNegativeDonationAmount_formIsReloadedAndPrefilledWithZero(): void {
@@ -475,7 +475,7 @@ 
 
 		$response = $client->getResponse()->getContent();
 
-		$this->assertContains( 'Amount: 0,00', $response );
+		$this->assertContains('Amount: 0,00', $response);
 	}
 
 	private function newInvalidFormInput(): array {
@@ -520,10 +520,10 @@ 
 
 		$response = $client->getResponse()->getContent();
 
-		$this->assertContains( 'Amount: 0', $response );
-		$this->assertContains( 'Payment type: UEB', $response );
-		$this->assertContains( 'Interval: 1', $response );
-		$this->assertContains( 'Value of field "amount" violates rule: Amount too low', $response );
+		$this->assertContains('Amount: 0', $response);
+		$this->assertContains('Payment type: UEB', $response);
+		$this->assertContains('Interval: 1', $response);
+		$this->assertContains('Value of field "amount" violates rule: Amount too low', $response);
 	}
 
 	private function newAnonymousFormInput(): array {
@@ -536,11 +536,11 @@ 
 	}
 
 	public function testGivenValidRequest_tokensAreReturned(): void {
-		$this->createEnvironment( [], function ( Client $client, FunFunFactory $factory ): void {
-			$factory->setTokenGenerator( new FixedTokenGenerator( self::SOME_TOKEN ) );
+		$this->createEnvironment([], function(Client $client, FunFunFactory $factory): void {
+			$factory->setTokenGenerator(new FixedTokenGenerator(self::SOME_TOKEN));
 
-			$client->setServerParameter( 'HTTP_REFERER', 'https://en.wikipedia.org/wiki/Karla_Kennichnich' );
-			$client->followRedirects( false );
+			$client->setServerParameter('HTTP_REFERER', 'https://en.wikipedia.org/wiki/Karla_Kennichnich');
+			$client->followRedirects(false);
 
 			$client->request(
 				'POST',
@@ -550,14 +550,14 @@ 
 
 			$response = $client->getResponse()->getContent();
 
-			$this->assertContains( self::SOME_TOKEN, $response );
+			$this->assertContains(self::SOME_TOKEN, $response);
 		} );
 	}
 
 	public function testGivenValidRequest_clientIsRedirected(): void {
-		$this->createEnvironment( [], function ( Client $client, FunFunFactory $factory ): void {
-			$factory->setTokenGenerator( new FixedTokenGenerator( self::SOME_TOKEN ) );
-			$client->followRedirects( false );
+		$this->createEnvironment([], function(Client $client, FunFunFactory $factory): void {
+			$factory->setTokenGenerator(new FixedTokenGenerator(self::SOME_TOKEN));
+			$client->followRedirects(false);
 
 			$client->request(
 				'POST',
@@ -565,13 +565,13 @@ 
 				$this->newValidFormInput()
 			);
 
-			$this->assertTrue( $client->getResponse()->isRedirect() );
+			$this->assertTrue($client->getResponse()->isRedirect());
 		} );
 	}
 
 	public function testWhenTrackingCookieExists_valueIsPersisted(): void {
-		$this->createEnvironment( [], function ( Client $client, FunFunFactory $factory ): void {
-			$client->getCookieJar()->set( new Cookie( 'spenden_tracking', 'test/blue' ) );
+		$this->createEnvironment([], function(Client $client, FunFunFactory $factory): void {
+			$client->getCookieJar()->set(new Cookie('spenden_tracking', 'test/blue'));
 
 			$client->request(
 				'POST',
@@ -579,15 +579,15 @@ 
 				$this->newComplementableFormInput()
 			);
 
-			$donation = $this->getDonationFromDatabase( $factory );
+			$donation = $this->getDonationFromDatabase($factory);
 			$data = $donation->getDecodedData();
 
-			$this->assertSame( 'test/blue', $data['tracking'] );
+			$this->assertSame('test/blue', $data['tracking']);
 		} );
 	}
 
 	public function testWhenTrackableInputDataIsSubmitted_theyAreStoredInSession(): void {
-		$this->createAppEnvironment( [], function ( Client $client, FunFunFactory $factory, Application $app ): void {
+		$this->createAppEnvironment([], function(Client $client, FunFunFactory $factory, Application $app): void {
 
 			$client->request(
 				'GET',
@@ -599,15 +599,15 @@ 
 				]
 			);
 
-			$piwikTracking = $app['session']->get( 'piwikTracking' );
-			$this->assertSame( 'BEZ', $piwikTracking['paymentType'] );
-			$this->assertSame( 3, $piwikTracking['paymentInterval'] );
-			$this->assertSame( '5,00', $piwikTracking['paymentAmount'] );
+			$piwikTracking = $app['session']->get('piwikTracking');
+			$this->assertSame('BEZ', $piwikTracking['paymentType']);
+			$this->assertSame(3, $piwikTracking['paymentInterval']);
+			$this->assertSame('5,00', $piwikTracking['paymentAmount']);
 		} );
 	}
 
 	public function testWhenTolstojNovelIsPassed_isIsNotStoredInSession(): void {
-		$this->createAppEnvironment( [], function ( Client $client, FunFunFactory $factory, Application $app ): void {
+		$this->createAppEnvironment([], function(Client $client, FunFunFactory $factory, Application $app): void {
 
 			$client->request(
 				'GET',
@@ -625,15 +625,15 @@ 
 				]
 			);
 
-			$piwikTracking = $app['session']->get( 'piwikTracking' );
-			$this->assertArrayNotHasKey( 'paymentType', $piwikTracking );
-			$this->assertSame( 3, $piwikTracking['paymentInterval'] );
-			$this->assertSame( '5,00', $piwikTracking['paymentAmount'] );
+			$piwikTracking = $app['session']->get('piwikTracking');
+			$this->assertArrayNotHasKey('paymentType', $piwikTracking);
+			$this->assertSame(3, $piwikTracking['paymentInterval']);
+			$this->assertSame('5,00', $piwikTracking['paymentAmount']);
 		} );
 	}
 
 	public function testWhenParameterIsOmitted_itIsNotStoredInSession(): void {
-		$this->createAppEnvironment( [], function ( Client $client, FunFunFactory $factory, Application $app ): void {
+		$this->createAppEnvironment([], function(Client $client, FunFunFactory $factory, Application $app): void {
 
 			$client->request(
 				'GET',
@@ -644,15 +644,15 @@ 
 				]
 			);
 
-			$piwikTracking = $app['session']->get( 'piwikTracking' );
-			$this->assertSame( 'BEZ', $piwikTracking['paymentType'] );
-			$this->assertSame( '5,00', $piwikTracking['paymentAmount'] );
-			$this->assertArrayNotHasKey( 'paymentInterval', $piwikTracking );
+			$piwikTracking = $app['session']->get('piwikTracking');
+			$this->assertSame('BEZ', $piwikTracking['paymentType']);
+			$this->assertSame('5,00', $piwikTracking['paymentAmount']);
+			$this->assertArrayNotHasKey('paymentInterval', $piwikTracking);
 		} );
 	}
 
 	public function testWhenInitiallyIntendedPaymentOptionsDifferFromActual_itIsReflectedInPiwikTrackingEvents(): void {
-		$client = $this->createClient( [] );
+		$client = $this->createClient([]);
 		$client->request(
 			'GET',
 			'/',
@@ -676,21 +676,21 @@ 
 		$client->followRedirect();
 
 		$responseContent = $client->getResponse()->getContent();
-		$this->assertContains( 'BEZ/UEB', $responseContent );
-		$this->assertContains( '5.00/12.34', $responseContent );
-		$this->assertContains( '12/0', $responseContent );
+		$this->assertContains('BEZ/UEB', $responseContent);
+		$this->assertContains('5.00/12.34', $responseContent);
+		$this->assertContains('12/0', $responseContent);
 	}
 
 	public function testWhenMobileTrackingIsRequested_piwikTrackerIsCalledForPaypalPayment(): void {
-		$this->createEnvironment( [], function ( Client $client, FunFunFactory $factory ): void {
+		$this->createEnvironment([], function(Client $client, FunFunFactory $factory): void {
 			$factory->setNullMessenger();
-			$client->followRedirects( false );
+			$client->followRedirects(false);
 
-			$tracker = $this->getMockBuilder( PageViewTracker::class )->disableOriginalConstructor()->getMock();
-			$tracker->expects( $this->once() )
-				->method( 'trackPaypalRedirection' )
-				->with( 'test', 'gelb', '10.1.2.3' );
-			$factory->setPageViewTracker( $tracker );
+			$tracker = $this->getMockBuilder(PageViewTracker::class)->disableOriginalConstructor()->getMock();
+			$tracker->expects($this->once())
+				->method('trackPaypalRedirection')
+				->with('test', 'gelb', '10.1.2.3');
+			$factory->setPageViewTracker($tracker);
 
 			$client->request(
 				'POST',
@@ -717,14 +717,14 @@ 
 	}
 
 	public function testWhenMobileTrackingIsRequested_piwikTrackerIsNotCalledForNonPaypalPayment(): void {
-		$this->createEnvironment( [], function ( Client $client, FunFunFactory $factory ): void {
+		$this->createEnvironment([], function(Client $client, FunFunFactory $factory): void {
 			$factory->setNullMessenger();
-			$client->followRedirects( false );
+			$client->followRedirects(false);
 
-			$tracker = $this->getMockBuilder( PageViewTracker::class )->disableOriginalConstructor()->getMock();
-			$tracker->expects( $this->never() )
-				->method( 'trackPaypalRedirection' );
-			$factory->setPageViewTracker( $tracker );
+			$tracker = $this->getMockBuilder(PageViewTracker::class)->disableOriginalConstructor()->getMock();
+			$tracker->expects($this->never())
+				->method('trackPaypalRedirection');
+			$factory->setPageViewTracker($tracker);
 
 			$client->request(
 				'POST',
@@ -740,10 +740,10 @@ 
 	}
 
 	public function testGivenCommasInStreetInput_donationGetsPersisted(): void {
-		$this->createEnvironment( [], function ( Client $client, FunFunFactory $factory ): void {
+		$this->createEnvironment([], function(Client $client, FunFunFactory $factory): void {
 
-			$client->setServerParameter( 'HTTP_REFERER', 'https://en.wikipedia.org/wiki/Karla_Kennichnich' );
-			$client->followRedirects( false );
+			$client->setServerParameter('HTTP_REFERER', 'https://en.wikipedia.org/wiki/Karla_Kennichnich');
+			$client->followRedirects(false);
 
 			$formInput = $this->newValidFormInput();
 			$formInput['street'] = ',Lehmgasse, 12,';
@@ -754,12 +754,12 @@ 
 				$formInput
 			);
 
-			$this->assertIsExpectedDonation( $this->getDonationFromDatabase( $factory ) );
+			$this->assertIsExpectedDonation($this->getDonationFromDatabase($factory));
 		} );
 	}
 
 	public function testGivenSufficientForeignBankData_donationGetsPersisted(): void {
-		$this->createEnvironment( [], function ( Client $client, FunFunFactory $factory ): void {
+		$this->createEnvironment([], function(Client $client, FunFunFactory $factory): void {
 			$formInput = $this->newValidFormInput();
 			$formInput['iban'] = 'AT022050302101023600';
 			$formInput['bic'] = 'SPIHAT22XXX';
@@ -772,23 +772,23 @@ 
 				$formInput
 			);
 
-			$donation = $this->getDonationFromDatabase( $factory );
+			$donation = $this->getDonationFromDatabase($factory);
 			$data = $donation->getDecodedData();
 
-			$this->assertSame( 'AT022050302101023600', $data['iban'] );
-			$this->assertSame( 'SPIHAT22XXX', $data['bic'] );
-			$this->assertSame( '', $data['konto'] );
-			$this->assertSame( '', $data['blz'] );
-			$this->assertSame( '', $data['bankname'] );
+			$this->assertSame('AT022050302101023600', $data['iban']);
+			$this->assertSame('SPIHAT22XXX', $data['bic']);
+			$this->assertSame('', $data['konto']);
+			$this->assertSame('', $data['blz']);
+			$this->assertSame('', $data['bankname']);
 		} );
 	}
 
 	public function testCookieFlagsSecureAndHttpOnlyAreSet(): void {
 		$client = new Client(
 			$this->createSilexApplication(),
-			[ 'HTTPS' => true ]
+			['HTTPS' => true]
 		);
-		$client->followRedirects( true );
+		$client->followRedirects(true);
 
 		$client->request(
 			'POST',
@@ -797,47 +797,47 @@ 
 		);
 
 		$cookieJar = $client->getCookieJar();
-		$cookieJar->updateFromResponse( $client->getInternalResponse() );
-		$cookie = $cookieJar->get( ShowDonationConfirmationHandler::SUBMISSION_COOKIE_NAME );
-		$this->assertTrue( $cookie->isSecure() );
-		$this->assertTrue( $cookie->isHttpOnly() );
+		$cookieJar->updateFromResponse($client->getInternalResponse());
+		$cookie = $cookieJar->get(ShowDonationConfirmationHandler::SUBMISSION_COOKIE_NAME);
+		$this->assertTrue($cookie->isSecure());
+		$this->assertTrue($cookie->isHttpOnly());
 	}
 
 	public function testAllPaymentTypesAreOffered(): void {
-		$client = $this->createClient( [ 'skin' => [ 'default' => '10h16' ] ] );
+		$client = $this->createClient(['skin' => ['default' => '10h16']]);
 		$client->request(
 			'GET',
 			'/donation/new'
 		);
 		$crawler = $client->getCrawler();
 
-		$this->assertSame( 1, $crawler->filter( '#donation-payment input[name="zahlweise"][value="BEZ"]' )->count() );
-		$this->assertSame( 1, $crawler->filter( '#donation-payment input[name="zahlweise"][value="UEB"]' )->count() );
-		$this->assertSame( 1, $crawler->filter( '#donation-payment input[name="zahlweise"][value="MCP"]' )->count() );
-		$this->assertSame( 1, $crawler->filter( '#donation-payment input[name="zahlweise"][value="PPL"]' )->count() );
-		$this->assertSame( 1, $crawler->filter( '#donation-payment input[name="zahlweise"][value="SUB"]' )->count() );
+		$this->assertSame(1, $crawler->filter('#donation-payment input[name="zahlweise"][value="BEZ"]')->count());
+		$this->assertSame(1, $crawler->filter('#donation-payment input[name="zahlweise"][value="UEB"]')->count());
+		$this->assertSame(1, $crawler->filter('#donation-payment input[name="zahlweise"][value="MCP"]')->count());
+		$this->assertSame(1, $crawler->filter('#donation-payment input[name="zahlweise"][value="PPL"]')->count());
+		$this->assertSame(1, $crawler->filter('#donation-payment input[name="zahlweise"][value="SUB"]')->count());
 	}
 
 	public function testSofortPaymentTypeCanByDisabledViaQuery(): void {
-		$client = $this->createClient( [ 'skin' => [ 'default' => '10h16' ] ] );
+		$client = $this->createClient(['skin' => ['default' => '10h16']]);
 		$client->request(
 			'GET',
 			'/donation/new',
-			[ 'pmt' => '0' ]
+			['pmt' => '0']
 		);
 		$crawler = $client->getCrawler();
 
-		$this->assertSame( 0, $crawler->filter( '#donation-payment input[name="zahlweise"][value="SUB"]' )->count() );
+		$this->assertSame(0, $crawler->filter('#donation-payment input[name="zahlweise"][value="SUB"]')->count());
 	}
 
 	public function testDonationReceiptOptOut_persistedInDonation(): void {
-		$this->createEnvironment( [], function ( Client $client, FunFunFactory $factory ): void {
+		$this->createEnvironment([], function(Client $client, FunFunFactory $factory): void {
 			$parameters = $this->newValidFormInput();
 			$parameters['donationReceipt'] = '0';
 
-			$client->request( Request::METHOD_POST, self::ADD_DONATION_PATH, $parameters );
+			$client->request(Request::METHOD_POST, self::ADD_DONATION_PATH, $parameters);
 
-			$this->assertFalse( $this->getDonationFromDatabase( $factory )->getDonationReceipt() );
+			$this->assertFalse($this->getDonationFromDatabase($factory)->getDonationReceipt());
 		} );
 	}
 }