Broarm\EventTickets\WaitingListRegistrationForm

Complexity

Total Complexity

2

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	7

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	17	1
A	doRegister()	0	7	1

<?php
/**
 * CheckInForm.php
 *
 * @author Bram de Leeuw
 * Date: 07/04/17
 */

namespace Broarm\EventTickets;

use FieldList;
use Form;
use FormAction;
use RequiredFields;
use TextField;

class WaitingListRegistrationForm extends Form
{
    public function __construct($controller, $name = 'CheckInForm')
    {
        $fields = FieldList::create(
            TextField::create('Title', _t('WaitingListRegistration.Name', 'Name')),
            TextField::create('Email', _t('WaitingListRegistration.Email', 'Email')),
            TextField::create('Telephone', _t('WaitingListRegistration.Telephone', 'Telephone'))
        );

        $actions = FieldList::create(
            FormAction::create('doRegister', _t('WaitingListRegistrationForm.REGISTER', 'Register'))
        );

        $required = new RequiredFields(array('Title', 'Email'));

        parent::__construct($controller, $name, $fields, $actions, $required);
        $this->extend('updateWaitingListRegistrationForm');
    }

    /**
     * Sets the person on the waiting list
     *
     * @param                             $data
     * @param WaitingListRegistrationForm $form
     */
    public function doRegister($data, WaitingListRegistrationForm $form)

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

    {
        $form->saveInto($registration = WaitingListRegistration::create());
        $registration->EventID = $this->getController()->ID;

The property EventID does not exist on object<Broarm\EventTickets\WaitingListRegistration>. Since you implemented __set, maybe consider adding a @property annotation.

        $this->getController()->WaitingList()->add($registration);
        $this->getController()->redirect($this->getController()->Link('?waitinglist=1'));

The call to Controller::Link() has too many arguments starting with '?waitinglist=1'.

    }

}