Broarm\EventTickets\TicketPayment

Complexity

Total Complexity

3

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
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',
    );

    /**
     * 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();
        }
    }
}