WebMoneyMerchant - Code Metrics - Inspection of "Merge remote-tracking branch 'origin/master'" - actionm/laravel-webmoney-merchant - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 6624a9...525711 )

by Action

created 2017-02-23 22:22 UTC

WebMoneyMerchant B

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	391
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	3

Importance

Changes	2
Bugs	2	Features	0

Metric	Value
wmc	47
c	2
b	2
f	0
lcom	2
cbo	3
dl	0
loc	391
rs	8.439

19 Methods

Rating	Name	Size	Complexity
A	__construct()	3	1
A	allowIP()	9	3
A	responseError()	4	1
A	responseOK()	4	1
A	eventFillAndSend()	12	1
A	getFormSignature()	6	1
A	getSignature()	16	1
A	generateWebMoneyMerchantOrderWithRequiredFields()	12	1
C	requiredOrderParamsCheck()	24	7
A	generatePaymentForm()	17	1
A	validate()	17	2
A	validatePayeePurse()	8	2
A	validateSignature()	10	2
B	validateOrderRequestFromGate()	10	5
B	validateSearchOrderRequiredAttributes()	28	5
B	callFilterSearchOrder()	25	3
A	callFilterPaidOrder()	14	2
A	payOrderFromGateOK()	4	1
C	payOrderFromGate()	49	7

How to fix Complexity

<?php

namespace ActionM\WebMoneyMerchant;

use Illuminate\Http\Request;
use Illuminate\Support\Facades\Validator;
use ActionM\WebMoneyMerchant\Events\WebMoneyMerchantEvent;
use ActionM\WebMoneyMerchant\Exceptions\InvalidConfiguration;

class WebMoneyMerchant
{
    public function __construct()
    {
    }

    /**
     * Allow the access, if the ip address is in the whitelist.
     * @param $ip
     * @return bool
     */
    public function allowIP($ip)
    {
        // Allow the local ip or any other ip address
        if ($ip == '127.0.0.1' || in_array('*', config('webmoney-merchant.allowed_ips'))) {
            return true;
        }

        return in_array($ip, config('webmoney-merchant.allowed_ips'));
    }

    /**
     * Generates the '403' error code.
     * @param $message
     * @return mixed
     */
    public function responseError($message)
    {
        return abort(403, $message);
    }

    /**
     * Returns the 'YES' success message.
     * @return string
     */
    public function responseOK()
    {
        return 'YES';
    }

    /**
     * Fills in the event details to pass the title and request params as array.
     * @param $event_type
     * @param $event_title
     * @param Request $request
     */
    public function eventFillAndSend($event_type, $event_title, Request $request)
    {
        $event_details = [
            'title' => 'WebMoneyMerchant: '.$event_title,
            'ip' => $request->ip(),
            'request' => $request->all(),
        ];

        event(
            new WebMoneyMerchantEvent($event_type, $event_details)
        );
    }

    /**
     * Calculates the signature for the order form.
     * @param $LMI_PAYMENT_AMOUNT
     * @param $LMI_PAYMENT_NO
     * @return string
     */
    public function getFormSignature($LMI_PAYMENT_AMOUNT, $LMI_PAYMENT_NO)
    {
        $hashStr = config('webmoney-merchant.WM_LMI_PAYEE_PURSE').';'.$LMI_PAYMENT_AMOUNT.';'.$LMI_PAYMENT_NO.';'.config('webmoney-merchant.WM_LMI_SECRET_X20').';';

        return hash('sha256', $hashStr);
    }

    /**
     * Returns the hash for the params from WebMoneyMerchant.
     * @param Request $request
     * @return string
     */
    public function getSignature(Request $request)
    {
        $hashStr =
            $request->get('LMI_PAYEE_PURSE').
            $request->get('LMI_PAYMENT_AMOUNT').
            $request->get('LMI_PAYMENT_NO').
            $request->get('LMI_MODE').
            $request->get('LMI_SYS_INVS_NO').
            $request->get('LMI_SYS_TRANS_NO').
            $request->get('LMI_SYS_TRANS_DATE').
            config('webmoney-merchant.WM_LMI_SECRET_KEY').
            $request->get('LMI_PAYER_PURSE').
            $request->get('LMI_PAYER_WM');

        return hash('sha256', $hashStr);
    }

    /**
     * Generates the order array with required fields for the order form.
     * @param $payment_amount
     * @param $payment_no
     * @param $item_name
     * @return array
     */
    public function generateWebMoneyMerchantOrderWithRequiredFields($payment_amount, $payment_no, $item_name)
    {
        $order = [
            'PAYMENT_AMOUNT' => $payment_amount,
            'PAYMENT_NO' => $payment_no,
            'ITEM_NAME' => base64_encode($item_name),
        ];

        $this->requiredOrderParamsCheck($order);

        return $order;
    }

    /**
     * Checks required order params for the order form and raise an exception if it fails.
     * @param $order
     * @throws InvalidConfiguration
     */
    public function requiredOrderParamsCheck($order)
    {
        $required_fields = [
            'PAYMENT_AMOUNT',
            'PAYMENT_NO',
            'ITEM_NAME',
        ];

        foreach ($required_fields as $key => $value) {
            if (! array_key_exists($value, $order) || empty($order[$value])) {
                throw InvalidConfiguration::generatePaymentFormOrderParamsNotSet($value);
            }
        }

        // Checks if PAYMENT_NO is numeric.
        if (! is_numeric($order['PAYMENT_NO'])) {
            throw InvalidConfiguration::generatePaymentFormOrderInvalidPaymentNo('PAYMENT_NO');
        }

        // Checks if PAYMENT_NO > 0 and < 2147483647
        if (intval($order['PAYMENT_NO']) < 1 || intval($order['PAYMENT_NO']) > 2147483647) {
            throw InvalidConfiguration::generatePaymentFormOrderInvalidPaymentNo($order['PAYMENT_NO']);
        }
    }

    /**
     * Generates html forms from view with payment buttons
     * Note: you can customise the view via artisan:publish.
     * @param $payment_amount
     * @param $payment_no
     * @param $item_name
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function generatePaymentForm($payment_amount, $payment_no, $item_name)
    {
        $order = $this->generateWebMoneyMerchantOrderWithRequiredFields($payment_amount, $payment_no, $item_name);

        $this->requiredOrderParamsCheck($order);

        /* WM Merchant accepts windows-1251; use only latin characters for the product name*/
        $payment_fields = [];
        $payment_fields['LMI_PAYMENT_AMOUNT'] = $order['PAYMENT_AMOUNT'];
        $payment_fields['LMI_PAYMENT_NO'] = $order['PAYMENT_NO'];
        $payment_fields['LMI_PAYMENT_DESC_BASE64'] = $order['ITEM_NAME'];
        $payment_fields['LOCALE'] = config('webmoney-merchant.locale');
        $payment_fields['LMI_PAYEE_PURSE'] = config('webmoney-merchant.WM_LMI_PAYEE_PURSE');
        $payment_fields['LMI_PAYMENTFORM_SIGN'] = $this->getFormSignature($payment_fields['LMI_PAYMENT_AMOUNT'], $payment_fields['LMI_PAYMENT_NO']);

        return view('webmoney-merchant::payment_form', compact('payment_fields'));
    }

    /**
     * Validates the request params from WebMoneyMerchant.
     * @param Request $request
     * @return bool
     */
    public function validate(Request $request)
    {
        $validator = Validator::make($request->all(), [
            'LMI_PAYEE_PURSE' => 'required',
            'LMI_PAYMENT_AMOUNT' => 'required',
            'LMI_PAYMENT_NO' => 'required',
            'LMI_PAYER_IP' => 'required',
            'LMI_HASH' => 'required',
            'LMI_HASH2' => 'required',
        ]);

        if ($validator->fails()) {
            return false;
        }

        return true;
    }

    /**
     * Validates the payee purse from WebMoneyMerchant.
     * @param Request $request
     * @return bool
     */
    public function validatePayeePurse(Request $request)
    {
        if ($request->get('LMI_PAYEE_PURSE') != config('webmoney-merchant.WM_LMI_PAYEE_PURSE')) {
            return false;
        }

        return true;
    }

    /**
     * Validates the request signature from WebMoneyMerchant.
     * @param Request $request
     * @return bool
     */
    public function validateSignature(Request $request)
    {
        $sign = $this->getSignature($request);

        if (mb_strtoupper($request->get('LMI_HASH')) != mb_strtoupper($sign)) {
            return false;
        }

        return true;
    }

    /**
     * Validates the allowed ip, request params and signature from WebMoneyMerchant.
     * @param Request $request
     * @return bool
     */
    public function validateOrderRequestFromGate(Request $request)
    {
        if (! $this->AllowIP($request->ip()) || ! $this->validate($request) || ! $this->validatePayeePurse($request) || ! $this->validateSignature($request)) {
            $this->eventFillAndSend('webmoneymerchant.error', 'validateOrderRequestFromGate', $request);

            return false;
        }

        return true;
    }

    /**
     * Validates the required attributes of the found order.
     * @param Request $request
     * @param $order
     * @return bool
     */
    public function validateSearchOrderRequiredAttributes(Request $request, $order)
    {
        if (! $order) {
            $this->eventFillAndSend('webmoneymerchant.error', 'orderNotFound', $request);

            return false;
        }

        // Checks required found order attributes.
        $attr = ['WEBMONEY_orderStatus', 'WEBMONEY_orderSum'];

        foreach ($attr as $k => $value) {
            if (! $order->getAttribute($value)) {
                $this->eventFillAndSend('webmoneymerchant.error', $value.'Invalid', $request);

                return false;
            }
        }

        // Compares order attributes with request params.
        if ($order->getAttribute('WEBMONEY_orderSum') != $request->input('LMI_PAYMENT_AMOUNT')) {
            $this->eventFillAndSend('webmoneymerchant.error', $value.'Invalid', $request);
foreach ($a as $b) {
}

// $b is defined here only if $a has elements, for example if $a is array()
// then $b would not be defined here. To avoid that, we recommend to set a
// default value for $b.


// Better
$b = 0; // or whatever default makes sense in your context
foreach ($a as $b) {
}

// $b is now guaranteed to be defined here.

            return false;
        }

        return true;
    }

    /**
     * Calls SearchOrderFilter and check return order params.
     * @param Request $request
     * @return bool
     * @throws InvalidConfiguration
     */
    public function callFilterSearchOrder(Request $request)
    {
        $callable = config('webmoney-merchant.searchOrderFilter');

        if (! is_callable($callable)) {
            throw InvalidConfiguration::searchOrderFilterInvalid();
        }

        /*
         *  SearchOrderFilter
         *  Searches the order in the database and return the order details.
         *  Must return the array with:
         *
         *  orderStatus
         *  orderSum
         */

        $order = $callable($request, $request->input('LMI_PAYMENT_NO'));

        if (! $this->validateSearchOrderRequiredAttributes($request, $order)) {
            return false;
        }

        return $order;
    }

    /**
     * Calls PaidOrderFilter if the order is not paid.
     * @param Request $request
     * @param $order
     * @return mixed
     * @throws InvalidConfiguration
     */
    public function callFilterPaidOrder(Request $request, $order)
    {
        $callable = config('webmoney-merchant.paidOrderFilter');

        if (! is_callable($callable)) {
            throw InvalidConfiguration::orderPaidFilterInvalid();
        }

        // Unset the custom order attributes; for Eloquent support.
        unset($order['WEBMONEY_orderSum'], $order['WEBMONEY_orderStatus']);

        // Runs the `PaidOrderFilter` callback.
        return $callable($request, $order);
    }

    /** Returns "YES"
     * @return string
     */
    public function payOrderFromGateOK()
    {
        return "YES";
    }

    /**
     * Runs WebMoneyMerchant::payOrderFromGate($request) when the request from WebMoney Merchant has been received.
     * @param Request $request
     * @return mixed
     */
    public function payOrderFromGate(Request $request)
    {
        if (! $request->has('LMI_HASH')) {
            return "OK";
        }

        if ($request->has('LMI_PREREQUEST')) {
            return 'YES';
        }

        // Validates the request params from the WebMoney Merchant server.
        if (! $this->validateOrderRequestFromGate($request)) {
            $this->eventFillAndSend('webmoneymerchant.error', 'validateOrderRequestFromGate', $request);

            return $this->responseError('validateOrderRequestFromGate');
        }

        // Searches and returns the order
        $order = $this->callFilterSearchOrder($request);

        if (! $order) {
            $this->eventFillAndSend('webmoneymerchant.error', 'searchOrderFilter', $request);

            return $this->responseError('searchOrderFilter');
        }

        // If the current order status is `paid`.
        // Sends the notification and returns the success response.
        if (mb_strtolower($order->WEBMONEY_orderStatus) === 'paid') {
            $this->eventFillAndSend('webmoneymerchant.info', 'The order is already paid', $request);

            return $this->responseError('The order is already paid');
        }

        // The current order is paid on WebMoney Merchant and not paid in the database.

        $this->eventFillAndSend('webmoneymerchant.success', 'paid order', $request);

        // PaidOrderFilter - update the order into the DB as paid & other actions.
        // If it returns false, then some error has occurred.
        if (! $this->callFilterPaidOrder($request, $order)) {
            $this->eventFillAndSend('webmoneymerchant.error', 'callFilterPaidOrder', $request);

            return $this->responseError('callFilterPaidOrder');
        }

        // The order is paid on WebMoney Merchant and updated in the database.
        return $this->responseOK();
    }
}


1			<?php
2
3			namespace ActionM\WebMoneyMerchant;
4
5			use Illuminate\Http\Request;
6			use Illuminate\Support\Facades\Validator;
7			use ActionM\WebMoneyMerchant\Events\WebMoneyMerchantEvent;
8			use ActionM\WebMoneyMerchant\Exceptions\InvalidConfiguration;
9
10			class WebMoneyMerchant
11			{
12			public function __construct()
13			{
14			}
15
16			/**
17			* Allow the access, if the ip address is in the whitelist.
18			* @param $ip
19			* @return bool
20			*/
21			public function allowIP($ip)
22			{
23			// Allow the local ip or any other ip address
24			if ($ip == '127.0.0.1' \|\| in_array('*', config('webmoney-merchant.allowed_ips'))) {
25			return true;
26			}
27
28			return in_array($ip, config('webmoney-merchant.allowed_ips'));
29			}
30
31			/**
32			* Generates the '403' error code.
33			* @param $message
34			* @return mixed
35			*/
36			public function responseError($message)
37			{
38			return abort(403, $message);
39			}
40
41			/**
42			* Returns the 'YES' success message.
43			* @return string
44			*/
45			public function responseOK()
46			{
47			return 'YES';
48			}
49
50			/**
51			* Fills in the event details to pass the title and request params as array.
52			* @param $event_type
53			* @param $event_title
54			* @param Request $request
55			*/
56			public function eventFillAndSend($event_type, $event_title, Request $request)
57			{
58			$event_details = [
59			'title' => 'WebMoneyMerchant: '.$event_title,
60			'ip' => $request->ip(),
61			'request' => $request->all(),
62			];
63
64			event(
65			new WebMoneyMerchantEvent($event_type, $event_details)
66			);
67			}
68
69			/**
70			* Calculates the signature for the order form.
71			* @param $LMI_PAYMENT_AMOUNT
72			* @param $LMI_PAYMENT_NO
73			* @return string
74			*/
75			public function getFormSignature($LMI_PAYMENT_AMOUNT, $LMI_PAYMENT_NO)
76			{
77			$hashStr = config('webmoney-merchant.WM_LMI_PAYEE_PURSE').';'.$LMI_PAYMENT_AMOUNT.';'.$LMI_PAYMENT_NO.';'.config('webmoney-merchant.WM_LMI_SECRET_X20').';';
78
79			return hash('sha256', $hashStr);
80			}
81
82			/**
83			* Returns the hash for the params from WebMoneyMerchant.
84			* @param Request $request
85			* @return string
86			*/
87			public function getSignature(Request $request)
88			{
89			$hashStr =
90			$request->get('LMI_PAYEE_PURSE').
91			$request->get('LMI_PAYMENT_AMOUNT').
92			$request->get('LMI_PAYMENT_NO').
93			$request->get('LMI_MODE').
94			$request->get('LMI_SYS_INVS_NO').
95			$request->get('LMI_SYS_TRANS_NO').
96			$request->get('LMI_SYS_TRANS_DATE').
97			config('webmoney-merchant.WM_LMI_SECRET_KEY').
98			$request->get('LMI_PAYER_PURSE').
99			$request->get('LMI_PAYER_WM');
100
101			return hash('sha256', $hashStr);
102			}
103
104			/**
105			* Generates the order array with required fields for the order form.
106			* @param $payment_amount
107			* @param $payment_no
108			* @param $item_name
109			* @return array
110			*/
111			public function generateWebMoneyMerchantOrderWithRequiredFields($payment_amount, $payment_no, $item_name)
112			{
113			$order = [
114			'PAYMENT_AMOUNT' => $payment_amount,
115			'PAYMENT_NO' => $payment_no,
116			'ITEM_NAME' => base64_encode($item_name),
117			];
118
119			$this->requiredOrderParamsCheck($order);
120
121			return $order;
122			}
123
124			/**
125			* Checks required order params for the order form and raise an exception if it fails.
126			* @param $order
127			* @throws InvalidConfiguration
128			*/
129			public function requiredOrderParamsCheck($order)
130			{
131			$required_fields = [
132			'PAYMENT_AMOUNT',
133			'PAYMENT_NO',
134			'ITEM_NAME',
135			];
136
137			foreach ($required_fields as $key => $value) {
138			if (! array_key_exists($value, $order) \|\| empty($order[$value])) {
139			throw InvalidConfiguration::generatePaymentFormOrderParamsNotSet($value);
140			}
141			}
142
143			// Checks if PAYMENT_NO is numeric.
144			if (! is_numeric($order['PAYMENT_NO'])) {
145			throw InvalidConfiguration::generatePaymentFormOrderInvalidPaymentNo('PAYMENT_NO');
146			}
147
148			// Checks if PAYMENT_NO > 0 and < 2147483647
149			if (intval($order['PAYMENT_NO']) < 1 \|\| intval($order['PAYMENT_NO']) > 2147483647) {
150			throw InvalidConfiguration::generatePaymentFormOrderInvalidPaymentNo($order['PAYMENT_NO']);
151			}
152			}
153
154			/**
155			* Generates html forms from view with payment buttons
156			* Note: you can customise the view via artisan:publish.
157			* @param $payment_amount
158			* @param $payment_no
159			* @param $item_name
160			* @return \Illuminate\Contracts\View\Factory\|\Illuminate\View\View
161			*/
162			public function generatePaymentForm($payment_amount, $payment_no, $item_name)
163			{
164			$order = $this->generateWebMoneyMerchantOrderWithRequiredFields($payment_amount, $payment_no, $item_name);
165
166			$this->requiredOrderParamsCheck($order);
167
168			/* WM Merchant accepts windows-1251; use only latin characters for the product name*/
169			$payment_fields = [];
170			$payment_fields['LMI_PAYMENT_AMOUNT'] = $order['PAYMENT_AMOUNT'];
171			$payment_fields['LMI_PAYMENT_NO'] = $order['PAYMENT_NO'];
172			$payment_fields['LMI_PAYMENT_DESC_BASE64'] = $order['ITEM_NAME'];
173			$payment_fields['LOCALE'] = config('webmoney-merchant.locale');
174			$payment_fields['LMI_PAYEE_PURSE'] = config('webmoney-merchant.WM_LMI_PAYEE_PURSE');
175			$payment_fields['LMI_PAYMENTFORM_SIGN'] = $this->getFormSignature($payment_fields['LMI_PAYMENT_AMOUNT'], $payment_fields['LMI_PAYMENT_NO']);
176
177			return view('webmoney-merchant::payment_form', compact('payment_fields'));
178			}
179
180			/**
181			* Validates the request params from WebMoneyMerchant.
182			* @param Request $request
183			* @return bool
184			*/
185			public function validate(Request $request)
186			{
187			$validator = Validator::make($request->all(), [
188			'LMI_PAYEE_PURSE' => 'required',
189			'LMI_PAYMENT_AMOUNT' => 'required',
190			'LMI_PAYMENT_NO' => 'required',
191			'LMI_PAYER_IP' => 'required',
192			'LMI_HASH' => 'required',
193			'LMI_HASH2' => 'required',
194			]);
195
196			if ($validator->fails()) {
197			return false;
198			}
199
200			return true;
201			}
202
203			/**
204			* Validates the payee purse from WebMoneyMerchant.
205			* @param Request $request
206			* @return bool
207			*/
208			public function validatePayeePurse(Request $request)
209			{
210			if ($request->get('LMI_PAYEE_PURSE') != config('webmoney-merchant.WM_LMI_PAYEE_PURSE')) {
211			return false;
212			}
213
214			return true;
215			}
216
217			/**
218			* Validates the request signature from WebMoneyMerchant.
219			* @param Request $request
220			* @return bool
221			*/
222			public function validateSignature(Request $request)
223			{
224			$sign = $this->getSignature($request);
225
226			if (mb_strtoupper($request->get('LMI_HASH')) != mb_strtoupper($sign)) {
227			return false;
228			}
229
230			return true;
231			}
232
233			/**
234			* Validates the allowed ip, request params and signature from WebMoneyMerchant.
235			* @param Request $request
236			* @return bool
237			*/
238			public function validateOrderRequestFromGate(Request $request)
239			{
240			if (! $this->AllowIP($request->ip()) \|\| ! $this->validate($request) \|\| ! $this->validatePayeePurse($request) \|\| ! $this->validateSignature($request)) {
241			$this->eventFillAndSend('webmoneymerchant.error', 'validateOrderRequestFromGate', $request);
242
243			return false;
244			}
245
246			return true;
247			}
248
249			/**
250			* Validates the required attributes of the found order.
251			* @param Request $request
252			* @param $order
253			* @return bool
254			*/
255			public function validateSearchOrderRequiredAttributes(Request $request, $order)
256			{
257			if (! $order) {
258			$this->eventFillAndSend('webmoneymerchant.error', 'orderNotFound', $request);
259
260			return false;
261			}
262
263			// Checks required found order attributes.
264			$attr = ['WEBMONEY_orderStatus', 'WEBMONEY_orderSum'];
265
266			foreach ($attr as $k => $value) {
267			if (! $order->getAttribute($value)) {
268			$this->eventFillAndSend('webmoneymerchant.error', $value.'Invalid', $request);
269
270			return false;
271			}
272			}
273
274			// Compares order attributes with request params.
275			if ($order->getAttribute('WEBMONEY_orderSum') != $request->input('LMI_PAYMENT_AMOUNT')) {
276			$this->eventFillAndSend('webmoneymerchant.error', $value.'Invalid', $request);
			0 ignored issues – show Bug introduced 2017-02-23 20:59 UTC by Report Bug Copy Issue Report The variable `$value` seems to be defined by a `foreach` iteration on line `266`. Are you sure the iterator is never empty, otherwise this variable is not defined? It seems like you are relying on a variable being defined by an iteration: foreach ($a as $b) { } // $b is defined here only if $a has elements, for example if $a is array() // then $b would not be defined here. To avoid that, we recommend to set a // default value for $b. // Better $b = 0; // or whatever default makes sense in your context foreach ($a as $b) { } // $b is now guaranteed to be defined here. Loading history...
277
278			return false;
279			}
280
281			return true;
282			}
283
284			/**
285			* Calls SearchOrderFilter and check return order params.
286			* @param Request $request
287			* @return bool
288			* @throws InvalidConfiguration
289			*/
290			public function callFilterSearchOrder(Request $request)
291			{
292			$callable = config('webmoney-merchant.searchOrderFilter');
293
294			if (! is_callable($callable)) {
295			throw InvalidConfiguration::searchOrderFilterInvalid();
296			}
297
298			/*
299			* SearchOrderFilter
300			* Searches the order in the database and return the order details.
301			* Must return the array with:
302			*
303			* orderStatus
304			* orderSum
305			*/
306
307			$order = $callable($request, $request->input('LMI_PAYMENT_NO'));
308
309			if (! $this->validateSearchOrderRequiredAttributes($request, $order)) {
310			return false;
311			}
312
313			return $order;
314			}
315
316			/**
317			* Calls PaidOrderFilter if the order is not paid.
318			* @param Request $request
319			* @param $order
320			* @return mixed
321			* @throws InvalidConfiguration
322			*/
323			public function callFilterPaidOrder(Request $request, $order)
324			{
325			$callable = config('webmoney-merchant.paidOrderFilter');
326
327			if (! is_callable($callable)) {
328			throw InvalidConfiguration::orderPaidFilterInvalid();
329			}
330
331			// Unset the custom order attributes; for Eloquent support.
332			unset($order['WEBMONEY_orderSum'], $order['WEBMONEY_orderStatus']);
333
334			// Runs the `PaidOrderFilter` callback.
335			return $callable($request, $order);
336			}
337
338			/** Returns "YES"
339			* @return string
340			*/
341			public function payOrderFromGateOK()
342			{
343			return "YES";
344			}
345
346			/**
347			* Runs WebMoneyMerchant::payOrderFromGate($request) when the request from WebMoney Merchant has been received.
348			* @param Request $request
349			* @return mixed
350			*/
351			public function payOrderFromGate(Request $request)
352			{
353			if (! $request->has('LMI_HASH')) {
354			return "OK";
355			}
356
357			if ($request->has('LMI_PREREQUEST')) {
358			return 'YES';
359			}
360
361			// Validates the request params from the WebMoney Merchant server.
362			if (! $this->validateOrderRequestFromGate($request)) {
363			$this->eventFillAndSend('webmoneymerchant.error', 'validateOrderRequestFromGate', $request);
364
365			return $this->responseError('validateOrderRequestFromGate');
366			}
367
368			// Searches and returns the order
369			$order = $this->callFilterSearchOrder($request);
370
371			if (! $order) {
372			$this->eventFillAndSend('webmoneymerchant.error', 'searchOrderFilter', $request);
373
374			return $this->responseError('searchOrderFilter');
375			}
376
377			// If the current order status is `paid`.
378			// Sends the notification and returns the success response.
379			if (mb_strtolower($order->WEBMONEY_orderStatus) === 'paid') {
380			$this->eventFillAndSend('webmoneymerchant.info', 'The order is already paid', $request);
381
382			return $this->responseError('The order is already paid');
383			}
384
385			// The current order is paid on WebMoney Merchant and not paid in the database.
386
387			$this->eventFillAndSend('webmoneymerchant.success', 'paid order', $request);
388
389			// PaidOrderFilter - update the order into the DB as paid & other actions.
390			// If it returns false, then some error has occurred.
391			if (! $this->callFilterPaidOrder($request, $order)) {
392			$this->eventFillAndSend('webmoneymerchant.error', 'callFilterPaidOrder', $request);
393
394			return $this->responseError('callFilterPaidOrder');
395			}
396
397			// The order is paid on WebMoney Merchant and updated in the database.
398			return $this->responseOK();
399			}
400			}
401

actionm / laravel-webmoney-merchant

Push — master ( 6624a9...525711 )

WebMoneyMerchant B

Complexity

Size/Duplication

Coupling/Cohesion

Importance

19 Methods

How to fix Complexity

Complex Class

Duplication Side-by-Side

Filter issues like