Orkhanahmadov\LaravelGoldenpay\Http\Controllers\GoldenpayController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	14	1
A	checkPaymentResult()	0	6	1
A	fireEvent()	0	16	2

<?php

namespace Orkhanahmadov\LaravelGoldenpay\Http\Controllers;

use Illuminate\Contracts\Config\Repository;
use Illuminate\Contracts\Events\Dispatcher;
use Illuminate\Contracts\Foundation\Application;
use Illuminate\Http\Request;
use Orkhanahmadov\LaravelGoldenpay\Goldenpay;
use Orkhanahmadov\LaravelGoldenpay\Models\Payment;

abstract class GoldenpayController
{
    /**
     * @var Application
     */
    protected $application;
    /**
     * @var Goldenpay
     */
    protected $goldenpay;
    /**
     * @var Dispatcher
     */
    protected $dispatcher;
    /**
     * @var Repository
     */
    protected $config;
    /**
     * @var Payment
     */
    protected $payment;

    /**
     * Controller constructor.
     *
     * @param Application $application
     * @param Repository $config
     * @param Request $request
     * @param Goldenpay $goldenpay
     * @param Dispatcher $dispatcher
     */
    public function __construct(
        Application $application,
        Repository $config,
        Request $request,
        Goldenpay $goldenpay,
        Dispatcher $dispatcher
    ) {
        $this->application = $application;
        $this->config = $config;
        $this->goldenpay = $goldenpay;
        $this->dispatcher = $dispatcher;

        $this->checkPaymentResult($request);
    }

    /**
     * Checks payment result with "payment_key" query parameter.
     *
     * @param Request $request
     */
    final protected function checkPaymentResult(Request $request): void
    {
        $this->payment = $this->goldenpay->paymentResult($request->query('payment_key'));

It seems like $request->query('payment_key') targeting Illuminate\Http\Concerns\InteractsWithInput::query() can also be of type array or null; however, Orkhanahmadov\LaravelGoldenpay\Goldenpay::paymentResult() does only seem to accept object<Orkhanahmadov\LaravelGoldenpay\Models\Payment>|string, maybe add an additional type check?

        $this->fireEvent($this->payment);
    }

    /**
     * Fires event based on payment status.
     *
     * @param Payment $payment
     */
    private function fireEvent(Payment $payment): void
    {
        if ($payment->status === 1) {
            $event = $this->application->make(
                $this->config->get('goldenpay.events.payment_successful'),
                [$payment]
            );
        } else {
            $event = $this->application->make(
                $this->config->get('goldenpay.events.payment_failed'),
                [$payment]
            );
        }

        $this->dispatcher->dispatch($event);
    }
}