gegmar / cluber

app/PaymentProvider/Klarna.php

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
     {
         $this->sofortApi->setAmount($purchase->total());
         $this->sofortApi->setCurrencyCode('EUR');
-        $this->sofortApi->setReason('#'.$purchase->id . ' Purchase');
+        $this->sofortApi->setReason('#' . $purchase->id . ' Purchase');
         $this->sofortApi->setSuccessUrl(route('ts.payment.successful', [
             'purchase' => $purchase->random_id,
             'secret' => $purchase->payment_secret
@@ -52,7 +52,7 @@ 
 
         if ($this->sofortApi->isError()) {
             // SOFORT-API didn't accept the data
-            foreach($this->sofortApi->getErrors() as $error) {
+            foreach ($this->sofortApi->getErrors() as $error) {
                 Log::error($error);
             }
             throw new PaymentProviderException("SOFORT got errors...");

app/Http/Controllers/Admin/SeatMapController.php

Spacing +2 added lines, -2 removed lines

@@ -50,10 +50,10 @@
         $seatMap->description = $validated['description'];
         $seatMap->seats = $validated['seats'];
 
-        if( !empty(json_decode($validated['layout'])) )
+        if (!empty(json_decode($validated['layout'])))
         {
             $numberOfSeatsInLayout = preg_match_all('/a/', $validated['layout']);
-            if($numberOfSeatsInLayout != $validated['seats']) {
+            if ($numberOfSeatsInLayout != $validated['seats']) {
                 return redirect()
                     ->route('admin.dependencies.seatmap.get', $seatMap)
                     ->with('status', 'Counted seats of layout (' . $numberOfSeatsInLayout . ') does not match the given seats (' . $validated['seats'] . ')!');

app/Rules/SeatMapLayout.php

Spacing +4 added lines, -4 removed lines

@@ -35,7 +35,7 @@ 
      */
     public function passes($attribute, $value)
     {
-        if($value === NULL || $value === "") {
+        if ($value === NULL || $value === "") {
             // it is allowed for the value to be empty
             return true;
         }
@@ -45,20 +45,20 @@ 
 
         // if json_decode result is falsy, then json object could
         // not be decoded and therefore the input is invalid
-        if(!$rows) {
+        if (!$rows) {
             return false;
         }
 
         // iterate over all rows to check that only valid
         // characters ('a' and '_') have been submitted
-        foreach( $rows as $row ) {
+        foreach ($rows as $row) {
             // explanation for regex ^[a_]*$
             //  ^     = Start of string
             //  [a_]  = Only characters "a" and "_" allowed
             //  *     = zero or more characters
             //  $     = End of string
             // Complete: Only match strings that contain from start to end only zero or more characters on the whitelist of a and _
-            if( !preg_match('/^[a_]*$/', $row) ) {
+            if (!preg_match('/^[a_]*$/', $row)) {
                 return false;
             }
         }

app/Ticket.php

Spacing +6 added lines, -6 removed lines

@@ -41,7 +41,7 @@ 
 
     public function getRowAndSeat()
     {
-        if( !$this->event->seatMap->layout ) {
+        if (!$this->event->seatMap->layout) {
             return null;
         }
 
@@ -49,15 +49,15 @@ 
         $columnCounter = 0;
         $result = [];
         $rows = json_decode($this->event->seatMap->layout);
-        foreach( $rows as $rowId => $row ) {
+        foreach ($rows as $rowId => $row) {
             $columnCounter = 0;
-            foreach( str_split($row) as $charId => $char) {
-                if($char === 'a') {
+            foreach (str_split($row) as $charId => $char) {
+                if ($char === 'a') {
                     $counter++;
                     $columnCounter++;
                 }
-                if($counter === $this->seat_number) {
-                    $result['row'] = $rowId+1;
+                if ($counter === $this->seat_number) {
+                    $result['row'] = $rowId + 1;
                     $result['seat'] = $columnCounter;
                 break;
                 }

app/SeatMap.php

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
 
     public function getRowsAndColumns()
     {
-        if(!$this->layout) {
+        if (!$this->layout) {
             return null;
         }
 
@@ -25,7 +25,7 @@ 
         $result['rows'] = count($rows);
         $result['columns'] = array_reduce($rows, function($intermediate, $row) {
             $columns = strlen($row);
-            if($intermediate < $columns) {
+            if ($intermediate < $columns) {
                 return $columns;
             } else {
                 return $intermediate;

app/Console/Commands/CheckDBConnection.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         try {
             DB::statement('show tables');
             $this->info('Database connection available!');
-        } catch( \Illuminate\Database\QueryException $e) {
+        } catch (\Illuminate\Database\QueryException $e) {
             $this->error('Database connection not available!');
             exit(1); // return a non-0-code to be used in shell scripts as a falsy exit state
         }

app/Providers/AppServiceProvider.php

Spacing +5 added lines, -5 removed lines

@@ -17,10 +17,10 @@ 
      */
     public function boot()
     {
-        Blade::directive('time', function ($expression) {
+        Blade::directive('time', function($expression) {
             return "<?php echo strftime('%H:%M', date_create($expression)->getTimestamp()); ?>";
         });
-        Blade::directive('datetime', function ($expression) {
+        Blade::directive('datetime', function($expression) {
             return "<?php echo strftime('%A, %d.%m.%Y', date_create($expression)->getTimestamp()); ?>";
         });
     }
@@ -33,18 +33,18 @@ 
     public function register()
     {
         $this->app->bind('App\PaymentProvider\Klarna', function() {
-            return new Klarna( config('paymentprovider.sofortConfigKey') );
+            return new Klarna(config('paymentprovider.sofortConfigKey'));
         });
 
         $this->app->bind('App\PaymentProvider\PayPal', function() {
             return new PayPal(
-                config('paymentprovider.payPalClientId'),     // ClientID
+                config('paymentprovider.payPalClientId'), // ClientID
                 config('paymentprovider.payPalClientSecret')  // ClientSecret
             );
         });
 
         $this->app->bind('App\PaymentProvider\Mollie', function() {
-            return new Mollie( config('paymentprovider.mollieApiKey') );
+            return new Mollie(config('paymentprovider.mollieApiKey'));
         });
     }
 }

app/PaymentProvider/Mollie.php

Spacing +13 added lines, -13 removed lines

@@ -68,8 +68,8 @@ 
             $purchase->payment_id = $payment->id;
             $purchase->save();
 
-        } catch( \Mollie\Api\Exceptions\ApiException $e ) {
-            throw new PaymentProviderException( $e );
+        } catch (\Mollie\Api\Exceptions\ApiException $e) {
+            throw new PaymentProviderException($e);
         }
 
         // Set a reference to the customer, depending if a user object exists
@@ -97,48 +97,48 @@ 
             $purchaseId = $payment->metadata->purchase_id;
             $purchase = Purchase::find($purchaseId);
 
-            if( !$purchase ) {
+            if (!$purchase) {
                 throw new PaymentProviderException('Payment-Id "' . $paymentId . '" has no matching purchase!');
             }
 
-            if( $payment->isPaid() && !$payment->hasRefunds() && !$payment->hasChargebacks() ) {
+            if ($payment->isPaid() && !$payment->hasRefunds() && !$payment->hasChargebacks()) {
                 /*
                 * The payment is paid and isn't refunded or charged back.
                 */
                 // Only send an email with the tickets on the change of state to "paid"
-                if($purchase->state != 'paid') {
+                if ($purchase->state != 'paid') {
                     Log::info('[Purchase#' . $purchase->id . '] Sending Ticket-Mail.');
                     Mail::to($purchase->customer)->send(new TicketsPaid($purchase));
                 }
-            } elseif( $payment->isOpen() ) {
+            } elseif ($payment->isOpen()) {
                 /*
                 * The payment is open.
                 */
-            } elseif( $payment->isPending() ) {
+            } elseif ($payment->isPending()) {
                 /*
                 * The payment is pending.
                 */
-            } elseif( $payment->isFailed() ) {
+            } elseif ($payment->isFailed()) {
                 /*
                 * The payment has failed.
                 */
                 $purchase->deleteWithAllData();
-            } elseif( $payment->isExpired() ) {
+            } elseif ($payment->isExpired()) {
                 /*
                 * The payment is expired.
                 */
                 $purchase->deleteWithAllData();
-            } elseif( $payment->isCanceled() ) {
+            } elseif ($payment->isCanceled()) {
                 /*
                 * The payment has been canceled.
                 */
                 $purchase->deleteWithAllData();
-            } elseif( $payment->hasRefunds() ) {
+            } elseif ($payment->hasRefunds()) {
                 /*
                 * The payment has been (partially) refunded.
                 * The status of the payment is still "paid"
                 */
-            } elseif( $payment->hasChargebacks() ) {
+            } elseif ($payment->hasChargebacks()) {
                 /*
                 * The payment has been (partially) charged back.
                 * The status of the payment is still "paid"
@@ -151,7 +151,7 @@ 
             $purchase->save();
 
         } catch (\Mollie\Api\Exceptions\ApiException $e) {
-            throw new PaymentProviderException( $e );
+            throw new PaymentProviderException($e);
         }
     }
 

app/Http/Controllers/TicketShop/MollieController.php

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
         // Someone could try to cause high load on webservice by
         // requesting the webhook endpoint with fake ids
         $purchase = Purchase::firstWhere('payment_id', $id);
-        if( !$purchase ) {
+        if (!$purchase) {
             return response('PaymentId invalid', 403);
         }
 
@@ -73,7 +73,7 @@ 
         // First check, if the purchase is linked to mollie.
         // Else we would trigger unneccessary api calls
         $mollie = User::firstWhere('email', 'mollie@system.local');
-        if( $purchase->vendor->id !== $mollie->id ) {
+        if ($purchase->vendor->id !== $mollie->id) {
             return response('Not a mollie purchase!', 403); // 403 = Forbidden
         }