actionm / laravel-webmoney-merchant

src/WebMoneyMerchant.php

Doc Comments +6 added lines, -6 removed lines

@@ -15,7 +15,7 @@ 
 
     /**
      * Allow access, if the ip address is in the whitelist.
-     * @param $ip
+     * @param string $ip
      * @return bool
      */
     public function allowIP($ip)
@@ -30,7 +30,7 @@ 
 
     /**
      * Return 403 error code.
-     * @param $message
+     * @param string $message
      * @return mixed
      */
     public function responseError($message)
@@ -40,7 +40,7 @@ 
 
     /**
      * Return YES success message.
-     * @return mixed
+     * @return string
      */
     public function responseOK()
     {
@@ -49,8 +49,8 @@ 
 
     /**
      * Fill event details to pass the title and request params as array.
-     * @param $event_type
-     * @param $event_title
+     * @param string $event_type
+     * @param string $event_title
      * @param Request $request
      */
     public function eventFillAndSend($event_type, $event_title, Request $request)
@@ -317,7 +317,7 @@ 
     /**
      * Call PaidOrderFilter if order not paid.
      * @param Request $request
-     * @param $order
+     * @param boolean $order
      * @return mixed
      * @throws InvalidConfiguration
      */

Spacing +12 added lines, -12 removed lines

@@ -135,13 +135,13 @@ 
         ];
 
         foreach ($required_fields as $key => $value) {
-            if (! array_key_exists($value, $order) || empty($order[$value])) {
+            if (!array_key_exists($value, $order) || empty($order[$value])) {
                 throw InvalidConfiguration::generatePaymentFormOrderParamsNotSet($value);
             }
         }
 
         // check if PAYMENT_NO is numeric
-        if (! is_numeric($order['PAYMENT_NO'])) {
+        if (!is_numeric($order['PAYMENT_NO'])) {
             throw InvalidConfiguration::generatePaymentFormOrderInvalidPaymentNo('PAYMENT_NO');
         }
 
@@ -238,7 +238,7 @@ 
      */
     public function validateOrderRequestFromGate(Request $request)
     {
-        if (! $this->AllowIP($request->ip()) || ! $this->validate($request) || ! $this->validatePayeePurse($request) || ! $this->validateSignature($request)) {
+        if (!$this->AllowIP($request->ip()) || !$this->validate($request) || !$this->validatePayeePurse($request) || !$this->validateSignature($request)) {
             $this->eventFillAndSend('webmoneymerchant.error', 'validateOrderRequestFromGate', $request);
 
             return false;
@@ -255,7 +255,7 @@ 
      */
     public function validateSearchOrderRequiredAttributes(Request $request, $order)
     {
-        if (! $order) {
+        if (!$order) {
             $this->eventFillAndSend('webmoneymerchant.error', 'orderNotFound', $request);
 
             return false;
@@ -265,7 +265,7 @@ 
         $attr = ['WEBMONEY_orderStatus', 'WEBMONEY_orderSum'];
 
         foreach ($attr as $k => $value) {
-            if (! $order->getAttribute($value)) {
+            if (!$order->getAttribute($value)) {
                 $this->eventFillAndSend('webmoneymerchant.error', $value.'Invalid', $request);
 
                 return false;
@@ -292,7 +292,7 @@ 
     {
         $callable = config('webmoney-merchant.searchOrderFilter');
 
-        if (! is_callable($callable)) {
+        if (!is_callable($callable)) {
             throw InvalidConfiguration::searchOrderFilterInvalid();
         }
 
@@ -307,7 +307,7 @@ 
 
         $order = $callable($request, $request->input('LMI_PAYMENT_NO'));
 
-        if (! $this->validateSearchOrderRequiredAttributes($request, $order)) {
+        if (!$this->validateSearchOrderRequiredAttributes($request, $order)) {
             return false;
         }
 
@@ -325,7 +325,7 @@ 
     {
         $callable = config('webmoney-merchant.paidOrderFilter');
 
-        if (! is_callable($callable)) {
+        if (!is_callable($callable)) {
             throw InvalidConfiguration::orderPaidFilterInvalid();
         }
 
@@ -343,7 +343,7 @@ 
      */
     public function payOrderFromGate(Request $request)
     {
-        if (! $request->has('LMI_HASH')) {
+        if (!$request->has('LMI_HASH')) {
             return $this->responseError('LMI_HASH not set');
         }
 
@@ -352,14 +352,14 @@ 
         }
 
         // Validate request params from WebMoney Merchant server.
-        if (! $this->validateOrderRequestFromGate($request)) {
+        if (!$this->validateOrderRequestFromGate($request)) {
             return $this->responseError('validateOrderRequestFromGate');
         }
 
         // Search and return order
         $order = $this->callFilterSearchOrder($request);
 
-        if (! $order) {
+        if (!$order) {
             return $this->responseError('searchOrderFilter');
         }
 
@@ -377,7 +377,7 @@ 
 
         // PaidOrderFilter - update order into DB as paid & other actions
         // if return false then error
-        if (! $this->callFilterPaidOrder($request, $order)) {
+        if (!$this->callFilterPaidOrder($request, $order)) {
             $this->eventFillAndSend('webmoneymerchant.error', 'callFilterPaidOrder', $request);
 
             return $this->responseError('callFilterPaidOrder');

src/WebMoneyMerchantNotifier.php

Spacing +4 added lines, -4 removed lines

@@ -14,10 +14,10 @@ 
     public function subscribe(Dispatcher $events)
     {
         // Listen events and send notification
-        $events->listen(WebMoneyMerchantEvent::class, function ($event) {
+        $events->listen(WebMoneyMerchantEvent::class, function($event) {
             $event->type = str_replace('webmoneymerchant.', '', $event->type);
 
-            if (! in_array($event->type, ['info', 'success', 'error'])) {
+            if (!in_array($event->type, ['info', 'success', 'error'])) {
                 $event->type = 'error';
             }
 
@@ -26,7 +26,7 @@ 
             $notification = app(config('webmoney-merchant.notification'));
             $notification->setEvent($event);
 
-            if (! $this->isValidNotificationClass($notification)) {
+            if (!$this->isValidNotificationClass($notification)) {
                 throw InvalidConfiguration::notificationClassInvalid(get_class($notification));
             }
 
@@ -53,7 +53,7 @@ 
     {
         $callable = config('webmoney-merchant.notificationFilter');
 
-        if (! is_callable($callable)) {
+        if (!is_callable($callable)) {
             return true;
         }
 

src/WebMoneyMerchantNotification.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
 
         return $slack_message
             ->content('WebMoneyMerchant payment message from '.config('app.url'))
-            ->attachment(function (SlackAttachment $attachment) {
+            ->attachment(function(SlackAttachment $attachment) {
                 $attachment->fields([
                     'Title' => $this->event->title,
                     'IP' => $this->event->ip,

src/WebMoneyMerchantServiceProvider.php

Spacing +4 added lines, -4 removed lines

@@ -35,7 +35,7 @@ 
 
         $this->app['events']->subscribe(WebMoneyMerchantNotifier::class);
 
-        $this->app->singleton('webmoneymerchant', function () {
+        $this->app->singleton('webmoneymerchant', function() {
             return $this->app->make('ActionM\WebMoneyMerchant\WebMoneyMerchant');
         });
 
@@ -50,16 +50,16 @@ 
      */
     public function testingEnv()
     {
-        if (! App::environment('testing')) {
+        if (!App::environment('testing')) {
             $callable = config('webmoney-merchant.searchOrderFilter');
 
-            if (! is_callable($callable)) {
+            if (!is_callable($callable)) {
                 throw InvalidConfiguration::searchOrderFilterInvalid();
             }
 
             $callable = config('webmoney-merchant.paidOrderFilter');
 
-            if (! is_callable($callable)) {
+            if (!is_callable($callable)) {
                 throw InvalidConfiguration::orderPaidFilterInvalid();
             }
         }

config/webmoney-merchant.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
     /*
      * locale for payment form
      */
-    'locale' => 'ru-RU',  // ru-RU || en-US
+    'locale' => 'ru-RU', // ru-RU || en-US
 
     /*
      *  SearchOrderFilter
@@ -82,7 +82,7 @@ 
     'channels' => ['mail', 'slack'],
 
     'mail' => [
-        'to' => '',  // your email
+        'to' => '', // your email
     ],
 
     'slack' => [