Broarm\EventTickets\TicketPayment

Complexity

Total Complexity

7

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	onAuthorized()	0	8	4
A	onCaptured()	0	7	3

<?php

namespace Broarm\EventTickets;

use DataExtension;
use SilverStripe\Omnipay\Service\ServiceResponse;

/**
 * Class TicketPayment
 *
 * @author Bram de Leeuw
 * @property TicketPayment|\Payment $owner
 *
 * @property int                    ReservationID
 */
class TicketPayment extends DataExtension
{
    private static $has_one = array(

The property $has_one is not used and could be removed.

        'Reservation' => 'Broarm\EventTickets\Reservation',
    );

    /**
     * Fix issue manual gateway doesn't call onCaptured hook
     *
     * @param ServiceResponse $response
     */
    public function onAuthorized(ServiceResponse $response)
    {
        if ($response->getPayment()->Gateway === 'Manual') {
            if (($reservation = Reservation::get()->byID($this->owner->ReservationID)) && $reservation->exists()) {
                $reservation->complete();
            }
        }
    }

    /**
     * Complete the order on a successful transaction
     *
     * @param ServiceResponse $response
     *
     * @throws \ValidationException
     */
    public function onCaptured(ServiceResponse $response)

The parameter $response is not used and could be removed.

    {
        /** @var Reservation $reservation */
        if (($reservation = Reservation::get()->byID($this->owner->ReservationID)) && $reservation->exists()) {
            $reservation->complete();
        }
    }
}