EcommercePayment_TestSuccess

Complexity

Total Complexity

5

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	i18n_singular_name()	0	4	1
A	i18n_plural_name()	0	4	1
A	processPayment()	0	8	1
A	getPaymentFormFields()	0	6	1
A	getPaymentFormRequirements()	0	4	1

<?php


/**
 * Payment object representing a TEST = SUCCESS.
 */
class EcommercePayment_TestSuccess extends EcommercePayment_Test

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{


    /**
     * standard SS variable.
     *
     * @Var String
     */
    private static $singular_name = 'Ecommerce Test Success Payment';

Consider using a different property name as you override a private property of the parent class.

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

    public function i18n_singular_name()

The return type could not be reliably inferred; please add a @return annotation.

    {
        return $this->Config()->get('singular_name');
    }

    /**
     * standard SS variable.
     *
     * @Var String
     */
    private static $plural_name = 'Ecommerce Test Success Payments';

Consider using a different property name as you override a private property of the parent class.

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

    public function i18n_plural_name()

The return type could not be reliably inferred; please add a @return annotation.

    {
        return $this->Config()->get('plural_name');
    }

    /**
     * @param array     $data The form request data - see OrderForm
     * @param OrderForm $form The form object submitted on
     *
     * @return EcommercePayment_Result
     */
    public function processPayment($data, $form)
    {
        $this->Status = 'Success';

The property Status does not exist on object<EcommercePayment_TestSuccess>. Since you implemented __set, maybe consider adding a @property annotation.

        $this->Message = '<div>PAYMENT TEST: SUCCESS</div>';

The property Message does not exist on object<EcommercePayment_TestSuccess>. Since you implemented __set, maybe consider adding a @property annotation.

        $this->write();

        return new EcommercePayment_Success();
    }

    public function getPaymentFormFields()
    {
        return new FieldList(
            new LiteralField('SuccessBlurb', '<div>SUCCESSFUL PAYMENT TEST</div>')
        );
    }

    public function getPaymentFormRequirements()
    {
        return;
    }
}