wmde / FundraisingFrontend

contexts/DonationContext/src/Domain/Model/Donation.php

Spacing +35 added lines, -35 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\DonationContext\Domain\Model;
 
@@ -55,11 +55,11 @@ 
 	 *
 	 * @throws \InvalidArgumentException
 	 */
-	public function __construct( ?int $id, string $status, ?Donor $donor, DonationPayment $payment,
-		bool $optsIntoNewsletter, DonationTrackingInfo $trackingInfo, DonationComment $comment = null ) {
+	public function __construct(? int $id, string $status, ? Donor $donor, DonationPayment $payment,
+		bool $optsIntoNewsletter, DonationTrackingInfo $trackingInfo, DonationComment $comment = null) {
 
 		$this->id = $id;
-		$this->setStatus( $status );
+		$this->setStatus($status);
 		$this->donor = $donor;
 		$this->payment = $payment;
 		$this->optsIntoNewsletter = $optsIntoNewsletter;
@@ -67,15 +67,15 @@ 
 		$this->comment = $comment;
 	}
 
-	private function setStatus( string $status ) {
-		if ( !$this->isValidStatus( $status ) ) {
-			throw new \InvalidArgumentException( 'Invalid donation status' );
+	private function setStatus(string $status) {
+		if (!$this->isValidStatus($status)) {
+			throw new \InvalidArgumentException('Invalid donation status');
 		}
 
 		$this->status = $status;
 	}
 
-	private function isValidStatus( string $status ): bool {
+	private function isValidStatus(string $status): bool {
 		return in_array(
 			$status,
 			[
@@ -89,7 +89,7 @@ 
 		);
 	}
 
-	public function getId(): ?int {
+	public function getId(): ? int {
 		return $this->id;
 	}
 
@@ -97,9 +97,9 @@ 
 	 * @param int $id
 	 * @throws \RuntimeException
 	 */
-	public function assignId( int $id ) {
-		if ( $this->id !== null && $this->id !== $id ) {
-			throw new \RuntimeException( 'Id cannot be changed after initial assignment' );
+	public function assignId(int $id) {
+		if ($this->id !== null && $this->id !== $id) {
+			throw new \RuntimeException('Id cannot be changed after initial assignment');
 		}
 
 		$this->id = $id;
@@ -129,20 +129,20 @@ 
 	/**
 	 * Returns the Donor or null for anonymous donations.
 	 */
-	public function getDonor(): ?Donor {
+	public function getDonor(): ? Donor {
 		return $this->donor;
 	}
 
 	/**
 	 * Returns the DonationComment or null for when there is none.
 	 */
-	public function getComment(): ?DonationComment {
+	public function getComment(): ? DonationComment {
 		return $this->comment;
 	}
 
-	public function addComment( DonationComment $comment ) {
-		if ( $this->hasComment() ) {
-			throw new RuntimeException( 'Can only add a single comment to a donation' );
+	public function addComment(DonationComment $comment) {
+		if ($this->hasComment()) {
+			throw new RuntimeException('Can only add a single comment to a donation');
 		}
 
 		$this->comment = $comment;
@@ -164,12 +164,12 @@ 
 	 * @throws RuntimeException
 	 */
 	public function cancel() {
-		if ( $this->getPaymentType() !== PaymentType::DIRECT_DEBIT ) {
-			throw new RuntimeException( 'Can only cancel direct debit' );
+		if ($this->getPaymentType() !== PaymentType::DIRECT_DEBIT) {
+			throw new RuntimeException('Can only cancel direct debit');
 		}
 
-		if ( !$this->statusIsCancellable() ) {
-			throw new RuntimeException( 'Can only cancel new donations' );
+		if (!$this->statusIsCancellable()) {
+			throw new RuntimeException('Can only cancel new donations');
 		}
 
 		$this->status = self::STATUS_CANCELLED;
@@ -179,15 +179,15 @@ 
 	 * @throws RuntimeException
 	 */
 	public function confirmBooked() {
-		if ( !$this->hasExternalPayment() ) {
-			throw new RuntimeException( 'Only external payments can be confirmed as booked' );
+		if (!$this->hasExternalPayment()) {
+			throw new RuntimeException('Only external payments can be confirmed as booked');
 		}
 
-		if ( !$this->statusAllowsForBooking() ) {
-			throw new RuntimeException( 'Only incomplete donations can be confirmed as booked' );
+		if (!$this->statusAllowsForBooking()) {
+			throw new RuntimeException('Only incomplete donations can be confirmed as booked');
 		}
 
-		if ( $this->hasComment() && ( $this->needsModeration() || $this->isCancelled() ) ) {
+		if ($this->hasComment() && ($this->needsModeration() || $this->isCancelled())) {
 			$this->makeCommentPrivate();
 		}
 
@@ -222,14 +222,14 @@ 
 	 * @param PayPalData $payPalData
 	 * @throws RuntimeException
 	 */
-	public function addPayPalData( PayPalData $payPalData ) {
+	public function addPayPalData(PayPalData $payPalData) {
 		$paymentMethod = $this->payment->getPaymentMethod();
 
-		if ( !( $paymentMethod instanceof PayPalPayment ) ) {
-			throw new RuntimeException( 'Cannot set PayPal data on a non PayPal payment' );
+		if (!($paymentMethod instanceof PayPalPayment)) {
+			throw new RuntimeException('Cannot set PayPal data on a non PayPal payment');
 		}
 
-		$paymentMethod->addPayPalData( $payPalData );
+		$paymentMethod->addPayPalData($payPalData);
 	}
 
 	/**
@@ -237,14 +237,14 @@ 
 	 *
 	 * @throws RuntimeException
 	 */
-	public function addCreditCardData( CreditCardTransactionData $creditCardData ) {
+	public function addCreditCardData(CreditCardTransactionData $creditCardData) {
 		$paymentMethod = $this->payment->getPaymentMethod();
 
-		if ( !( $paymentMethod instanceof CreditCardPayment ) ) {
-			throw new RuntimeException( 'Cannot set credit card transaction data on a non credit card payment' );
+		if (!($paymentMethod instanceof CreditCardPayment)) {
+			throw new RuntimeException('Cannot set credit card transaction data on a non credit card payment');
 		}
 
-		$paymentMethod->addCreditCardTransactionData( $creditCardData );
+		$paymentMethod->addCreditCardTransactionData($creditCardData);
 	}
 
 	private function statusIsCancellable(): bool {
@@ -252,7 +252,7 @@ 
 	}
 
 	public function hasExternalPayment(): bool {
-		return in_array( $this->getPaymentType(), [ PaymentType::PAYPAL, PaymentType::CREDIT_CARD ] );
+		return in_array($this->getPaymentType(), [PaymentType::PAYPAL, PaymentType::CREDIT_CARD]);
 	}
 
 	private function isIncomplete(): bool {