WebhookController::handleRefund() - Code Metrics - Inspection of "update" - recca0120/laravel-payum - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — refactor (0d2a4d)

by recca

created 2017-04-25 08:39 UTC

WebhookController::handleRefund() A

↳ Parent: WebhookController

Complexity

Conditions	2
Paths	1

Size

Total Lines	14
Code Lines	8

Duplication

Lines	14
Ratio	100 %

Code Coverage

Tests	7
CRAP Score	2.0078

Importance

Changes

Metric	Value
dl	14
loc	14
ccs	7
cts	8
cp	0.875
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	8
nc	1
nop	2
crap	2.0078

<?php

namespace Recca0120\LaravelPayum\Http\Controllers;

use Payum\Core\Payum;
use Illuminate\Http\Request;
use Payum\Core\Request\Sync;
use Payum\Core\Request\Cancel;
use Payum\Core\Request\Notify;
use Payum\Core\Request\Payout;
use Payum\Core\Request\Refund;
use Payum\Core\Request\Capture;
use Payum\Core\Request\Authorize;
use Illuminate\Routing\Controller;
use Payum\Core\Reply\ReplyInterface;
use Illuminate\Contracts\Routing\ResponseFactory;
use Payum\Core\Bridge\Symfony\ReplyToSymfonyResponseConverter;

class WebhookController extends Controller
{
    /**
     * $payum.
     *
     * @var \Payum\Core\Payum
     */
    protected $payum;

    /**
     * $responseFactory.
     *
     * @var \Illuminate\Contracts\Routing\ResponseFactory
     */
    protected $responseFactory;

    /**
     * $replyToSymfonyResponseConverter.
     *
     * @var \Payum\Core\Bridge\Symfony\ReplyToSymfonyResponseConverter
     */
    protected $replyToSymfonyResponseConverter;

    /**
     * __construct.
     *
     * @param \Payum\Core\Payum $payum
     * @param \Illuminate\Contracts\Routing\ResponseFactory $responseFactory
     * @param \Payum\Core\Bridge\Symfony\ReplyToSymfonyResponseConverter $replyToSymfonyResponseConverter
     */
    public function __construct(
        Payum $payum,
        ResponseFactory $responseFactory,
        ReplyToSymfonyResponseConverter $replyToSymfonyResponseConverter
    ) {
        $this->payum = $payum;
        $this->responseFactory = $responseFactory;
        $this->replyToSymfonyResponseConverter = $replyToSymfonyResponseConverter;
    }

    /**
     * handleAuthorize.
     *
     * @param \Illuminate\Http\Request $request
     * @param string $payumToken
     * @return \Illuminate\Http\Response
     */
    public function handleAuthorize(Request $request, $payumToken)

    {
        return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {
            $gateway->execute(new Authorize($token));
            $httpRequestVerifier->invalidate($token);

            return $this->responseFactory->redirectTo($token->getAfterUrl());
        });
    }

    /**
     * handleCancel.
     *
     * @param \Illuminate\Http\Request $request
     * @param string $payumToken
     * @return \Illuminate\Http\Response
     */
    public function handleCancel(Request $request, $payumToken)

    {
        return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {
            $gateway->execute(new Cancel($token));
            $httpRequestVerifier->invalidate($token);
            $afterUrl = $token->getAfterUrl();
            if (empty($afterUrl) === false) {
                return $this->responseFactory->redirectTo($afterUrl);
            }

            return $this->responseFactory->make(null, 204);
        });
    }

    /**
     * handleCapture.
     *
     * @param \Illuminate\Http\Request $request
     * @param string $payumToken
     * @return \Illuminate\Http\Response
     */
    public function handleCapture(Request $request, $payumToken = null)

    {
        return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {
            $gateway->execute(new Capture($token));
            $httpRequestVerifier->invalidate($token);

            return $this->responseFactory->redirectTo($token->getAfterUrl());
        });
    }

    /**
     * handleNotify.
     *
     * @param \Illuminate\Http\Request $request
     * @param string $payumToken
     * @return \Illuminate\Http\Response
     */
    public function handleNotify(Request $request, $payumToken)
    {
        return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {

            $gateway->execute(new Notify($token));

            return $this->responseFactory->make(null, 204);
        });
    }

    /**
     * handleNotifyUnsafe.
     *
     * @param string $gatewayName
     * @return \Illuminate\Http\Response
     */
    public function handleNotifyUnsafe($gatewayName)
    {
        try {
            $gateway = $this->getPayum()->getGateway($gatewayName);
            $gateway->execute(new Notify(null));

            return $this->responseFactory->make(null, 204);
        } catch (ReplyInterface $reply) {
            return $this->convertReply($reply);
        }
    }

    /**
     * handleRefund.
     *
     * @param \Illuminate\Http\Request $request
     * @param string $payumToken
     * @return \Illuminate\Http\Response
     */
    public function handleRefund(Request $request, $payumToken)

    {
        return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {
            $gateway->execute(new Refund($token));
            $httpRequestVerifier->invalidate($token);
            $afterUrl = $token->getAfterUrl();

            if (empty($afterUrl) === false) {
                return $this->responseFactory->redirectTo($afterUrl);
            }

            return $this->responseFactory->make(null, 204);
        });
    }

    /**
     * handlePayout.
     *
     * @param \Illuminate\Http\Request $request
     * @param string $payumToken
     * @return \Illuminate\Http\Response
     */
    public function handlePayout(Request $request, $payumToken)

    {
        return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {
            $gateway->execute(new Payout($token));
            $httpRequestVerifier->invalidate($token);

            return $this->responseFactory->redirectTo($token->getAfterUrl());
        });
    }

    /**
     * handleSync.
     *
     * @param \Illuminate\Http\Request $request
     * @param string $payumToken
     * @return \Illuminate\Http\Response
     */
    public function handleSync(Request $request, $payumToken)

    {
        return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {
            $gateway->execute(new Sync($token));
            $httpRequestVerifier->invalidate($token);

            return $this->responseFactory->redirectTo($token->getAfterUrl());
        });
    }

    /**
     * handleReceived.
     *
     * @param \Illuminate\Http\Request $request
     * @param string $payumToken
     * @param callable $callback
     * @return \Illuminate\Http\Response
     */
    protected function handleReceived(Request $request, $payumToken, callable $callback)
    {
        if (is_null($payumToken) === true) {
            $payumToken = $request->session()->remove('payum_token');
        }

        $request->merge(['payum_token' => $payumToken]);
        $httpRequestVerifier = $this->getPayum()->getHttpRequestVerifier();
        $token = $httpRequestVerifier->verify($request);
        $gateway = $this->getPayum()->getGateway($token->getGatewayName());

        try {
            return $callback($gateway, $token, $httpRequestVerifier, $request);
        } catch (ReplyInterface $reply) {
            $session = $request->session();
            $method = method_exists($session, 'set') === true ? 'set' : 'put';
            call_user_func_array([$session, $method], ['payum_token', $payumToken]);

            return $this->convertReply($reply);
        }
    }

    protected function getPayum()
    {
        return $this->payum;
    }

    protected function convertReply($reply)
    {
        return $this->replyToSymfonyResponseConverter->convert($reply);
    }
}


1			<?php
2
3			namespace Recca0120\LaravelPayum\Http\Controllers;
4
5			use Payum\Core\Payum;
6			use Illuminate\Http\Request;
7			use Payum\Core\Request\Sync;
8			use Payum\Core\Request\Cancel;
9			use Payum\Core\Request\Notify;
10			use Payum\Core\Request\Payout;
11			use Payum\Core\Request\Refund;
12			use Payum\Core\Request\Capture;
13			use Payum\Core\Request\Authorize;
14			use Illuminate\Routing\Controller;
15			use Payum\Core\Reply\ReplyInterface;
16			use Illuminate\Contracts\Routing\ResponseFactory;
17			use Payum\Core\Bridge\Symfony\ReplyToSymfonyResponseConverter;
18
19			class WebhookController extends Controller
20			{
21			/**
22			* $payum.
23			*
24			* @var \Payum\Core\Payum
25			*/
26			protected $payum;
27
28			/**
29			* $responseFactory.
30			*
31			* @var \Illuminate\Contracts\Routing\ResponseFactory
32			*/
33			protected $responseFactory;
34
35			/**
36			* $replyToSymfonyResponseConverter.
37			*
38			* @var \Payum\Core\Bridge\Symfony\ReplyToSymfonyResponseConverter
39			*/
40			protected $replyToSymfonyResponseConverter;
41
42			/**
43			* __construct.
44			*
45			* @param \Payum\Core\Payum $payum
46			* @param \Illuminate\Contracts\Routing\ResponseFactory $responseFactory
47			* @param \Payum\Core\Bridge\Symfony\ReplyToSymfonyResponseConverter $replyToSymfonyResponseConverter
48			*/
49	10		public function __construct(
50			Payum $payum,
51			ResponseFactory $responseFactory,
52			ReplyToSymfonyResponseConverter $replyToSymfonyResponseConverter
53			) {
54	10		$this->payum = $payum;
55	10		$this->responseFactory = $responseFactory;
56	10		$this->replyToSymfonyResponseConverter = $replyToSymfonyResponseConverter;
57	10		}
58
59			/**
60			* handleAuthorize.
61			*
62			* @param \Illuminate\Http\Request $request
63			* @param string $payumToken
64			* @return \Illuminate\Http\Response
65			*/
66	1	View Code Duplication	public function handleAuthorize(Request $request, $payumToken)
			0 ignored issues – show Duplication introduced 2017-04-25 08:49 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
67			{
68			return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {
69	1		$gateway->execute(new Authorize($token));
70	1		$httpRequestVerifier->invalidate($token);
71
72	1		return $this->responseFactory->redirectTo($token->getAfterUrl());
73	1		});
74			}
75
76			/**
77			* handleCancel.
78			*
79			* @param \Illuminate\Http\Request $request
80			* @param string $payumToken
81			* @return \Illuminate\Http\Response
82			*/
83	1	View Code Duplication	public function handleCancel(Request $request, $payumToken)
			0 ignored issues – show Duplication introduced 2017-04-25 08:12 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
84			{
85			return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {
86	1		$gateway->execute(new Cancel($token));
87	1		$httpRequestVerifier->invalidate($token);
88	1		$afterUrl = $token->getAfterUrl();
89	1		if (empty($afterUrl) === false) {
90	1		return $this->responseFactory->redirectTo($afterUrl);
91			}
92
93			return $this->responseFactory->make(null, 204);
94	1		});
95			}
96
97			/**
98			* handleCapture.
99			*
100			* @param \Illuminate\Http\Request $request
101			* @param string $payumToken
102			* @return \Illuminate\Http\Response
103			*/
104	2	View Code Duplication	public function handleCapture(Request $request, $payumToken = null)
			0 ignored issues – show Duplication introduced 2017-04-25 08:49 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
105			{
106			return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {
107	2		$gateway->execute(new Capture($token));
108	1		$httpRequestVerifier->invalidate($token);
109
110	1		return $this->responseFactory->redirectTo($token->getAfterUrl());
111	2		});
112			}
113
114			/**
115			* handleNotify.
116			*
117			* @param \Illuminate\Http\Request $request
118			* @param string $payumToken
119			* @return \Illuminate\Http\Response
120			*/
121	1		public function handleNotify(Request $request, $payumToken)
122			{
123			return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {
			0 ignored issues – show Unused Code introduced 2017-04-25 08:12 UTC by Report Bug Copy Issue Report The parameter `$httpRequestVerifier` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
124	1		$gateway->execute(new Notify($token));
125
126	1		return $this->responseFactory->make(null, 204);
127	1		});
128			}
129
130			/**
131			* handleNotifyUnsafe.
132			*
133			* @param string $gatewayName
134			* @return \Illuminate\Http\Response
135			*/
136	2		public function handleNotifyUnsafe($gatewayName)
137			{
138			try {
139	2		$gateway = $this->getPayum()->getGateway($gatewayName);
140	1		$gateway->execute(new Notify(null));
141
142	1		return $this->responseFactory->make(null, 204);
143	1		} catch (ReplyInterface $reply) {
144	1		return $this->convertReply($reply);
145			}
146			}
147
148			/**
149			* handleRefund.
150			*
151			* @param \Illuminate\Http\Request $request
152			* @param string $payumToken
153			* @return \Illuminate\Http\Response
154			*/
155	1	View Code Duplication	public function handleRefund(Request $request, $payumToken)
			0 ignored issues – show Duplication introduced 2017-04-25 08:12 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
156			{
157			return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {
158	1		$gateway->execute(new Refund($token));
159	1		$httpRequestVerifier->invalidate($token);
160	1		$afterUrl = $token->getAfterUrl();
161
162	1		if (empty($afterUrl) === false) {
163	1		return $this->responseFactory->redirectTo($afterUrl);
164			}
165
166			return $this->responseFactory->make(null, 204);
167	1		});
168			}
169
170			/**
171			* handlePayout.
172			*
173			* @param \Illuminate\Http\Request $request
174			* @param string $payumToken
175			* @return \Illuminate\Http\Response
176			*/
177	1	View Code Duplication	public function handlePayout(Request $request, $payumToken)
			0 ignored issues – show Duplication introduced 2017-04-25 08:49 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
178			{
179			return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {
180	1		$gateway->execute(new Payout($token));
181	1		$httpRequestVerifier->invalidate($token);
182
183	1		return $this->responseFactory->redirectTo($token->getAfterUrl());
184	1		});
185			}
186
187			/**
188			* handleSync.
189			*
190			* @param \Illuminate\Http\Request $request
191			* @param string $payumToken
192			* @return \Illuminate\Http\Response
193			*/
194		View Code Duplication	public function handleSync(Request $request, $payumToken)
			0 ignored issues – show Duplication introduced 2017-04-25 08:49 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
195			{
196	1		return $this->handleReceived($request, $payumToken, function ($gateway, $token, $httpRequestVerifier) {
197	1		$gateway->execute(new Sync($token));
198	1		$httpRequestVerifier->invalidate($token);
199
200	1		return $this->responseFactory->redirectTo($token->getAfterUrl());
201	1		});
202			}
203
204			/**
205			* handleReceived.
206			*
207			* @param \Illuminate\Http\Request $request
208			* @param string $payumToken
209			* @param callable $callback
210			* @return \Illuminate\Http\Response
211			*/
212	8		protected function handleReceived(Request $request, $payumToken, callable $callback)
213			{
214	8		if (is_null($payumToken) === true) {
215	1		$payumToken = $request->session()->remove('payum_token');
216			}
217
218	8		$request->merge(['payum_token' => $payumToken]);
219	8		$httpRequestVerifier = $this->getPayum()->getHttpRequestVerifier();
220	8		$token = $httpRequestVerifier->verify($request);
221	8		$gateway = $this->getPayum()->getGateway($token->getGatewayName());
222
223			try {
224	8		return $callback($gateway, $token, $httpRequestVerifier, $request);
225	1		} catch (ReplyInterface $reply) {
226	1		$session = $request->session();
227	1		$method = method_exists($session, 'set') === true ? 'set' : 'put';
228	1		call_user_func_array([$session, $method], ['payum_token', $payumToken]);
229
230	1		return $this->convertReply($reply);
231			}
232			}
233
234	10		protected function getPayum()
235			{
236	10		return $this->payum;
237			}
238
239	2		protected function convertReply($reply)
240			{
241	2		return $this->replyToSymfonyResponseConverter->convert($reply);
242			}
243			}
244

recca0120 / laravel-payum

Branch — refactor (0d2a4d)

WebhookController::handleRefund() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like