DirectCreditPayment_TESTFAILURE

Complexity

Total Complexity

3

Size/Duplication

Total Lines	45
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	processPayment()	7	7	1
A	getPaymentFormFields()	7	7	1
A	getPaymentFormRequirements()	4	4	1

<?php

/**
 * Payment object representing a DirectCredit payment.
 * @author Nicolaas [at] sunnysideup.co.nz
 * @package payment
 */
class DirectCreditPayment extends EcommercePayment

This class seems to be duplicated in your project.

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

{

    /**
     * Message shown before payment is made
     * @var String
     */
    private static $before_payment_message = "";

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

    /**
     * Message shown after payment is made
     * @var String
     */
    private static $after_payment_message = "";

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

    /**
     * Default Status for Payment
     * @var String
     */
    private static $default_status = "Pending";

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

    /**
     * Process the DirectCredit payment method
     */
    public function processPayment($data, $form)
    {
        $this->Status = Config::inst()->get("DirectCreditPayment", "default_status");

The property Status does not seem to exist. Did you mean default_status?

        $this->Message = Config::inst()->get("DirectCreditPayment", "after_payment_message");

The property Message does not seem to exist. Did you mean before_payment_message?

        $this->write();
        return EcommercePayment_Success::create();
    }

    public function getPaymentFormFields()
    {
        return new FieldList(
            new LiteralField($this->ClassName.'_BeforeMessage', '<div id="'.$this->ClassName.'_BeforeMessage">' . Config::inst()->get("DirectCreditPayment", "before_payment_message") . '</div>'),
            new HiddenField($this->ClassName, $this->ClassName, 0)
        );
    }

    public function getPaymentFormRequirements()
    {
        return null;
    }
}

class DirectCreditPayment_ViaCreditCart extends EcommercePayment

This class seems to be duplicated in your project.

PSR1 recommends that each class should be in its own file to aid autoloaders.

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

{

    /**
     * Message shown before payment is made
     * @var String
     */
    private static $before_payment_message = "";

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

    /**
     * Message shown after payment is made
     * @var String
     */
    private static $after_payment_message = "";

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

    /**
     * Default Status for Payment
     * @var String
     */
    private static $default_status = "Pending";

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

    /**
     * Process the DirectCredit payment method
     */
    public function processPayment($data, $form)
    {
        $this->Status = Config::inst()->get("DirectCreditPayment_ViaCreditCart", "default_status");

The property Status does not seem to exist. Did you mean default_status?

        $this->Message = Config::inst()->get("DirectCreditPayment_ViaCreditCart", "after_payment_message");

The property Message does not seem to exist. Did you mean before_payment_message?

        $this->write();
        return EcommercePayment_Success::create();
    }

    public function getPaymentFormFields()
    {
        return new FieldList(
            new LiteralField($this->ClassName.'_BeforeMessage', '<div id="'.$this->ClassName.'_BeforeMessage">' . Config::inst()->get("DirectCreditPayment_ViaCreditCart", "before_payment_message") . '</div>'),
            new HiddenField($this->ClassName, $this->ClassName, 0)
        );
    }

    public function getPaymentFormRequirements()
    {
        return null;
    }
}

class DirectCreditPayment_ViaCheque extends EcommercePayment

This class seems to be duplicated in your project.

PSR1 recommends that each class should be in its own file to aid autoloaders.

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

{

    /**
     * Message shown before payment is made
     * @var String
     */
    private static $before_payment_message = "";

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

    /**
     * Message shown after payment is made
     * @var String
     */
    private static $after_payment_message = "";

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

    /**
     * Default Status for Payment
     * @var String
     */
    private static $default_status = "Pending";

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

    /**
     * Process the DirectCredit payment method
     */
    public function processPayment($data, $form)
    {
        $this->Status = Config::inst()->get("DirectCreditPayment_ViaCheque", "default_status");

The property Status does not seem to exist. Did you mean default_status?

        $this->Message = Config::inst()->get("DirectCreditPayment_ViaCheque", "after_payment_message");

The property Message does not seem to exist. Did you mean before_payment_message?

        $this->write();
        return EcommercePayment_Success::create();
    }

    public function getPaymentFormFields()
    {
        return new FieldList(
            new LiteralField($this->ClassName.'_BeforeMessage', '<div id="'.$this->ClassName.'_BeforeMessage">' . Config::inst()->get("DirectCreditPayment_ViaCheque", "before_payment_message") . '</div>'),
            new HiddenField($this->ClassName, $this->ClassName, 0)
        );
    }

    public function getPaymentFormRequirements()
    {
        return null;
    }
}

class DirectCreditPayment_TESTSUCCESS extends EcommercePayment

This class seems to be duplicated in your project.

PSR1 recommends that each class should be in its own file to aid autoloaders.

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

{

    /**
     * Message shown before payment is made
     * @var String
     */
    private static $before_payment_message = "This is for testing purposes only";

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

    /**
     * Message shown after payment is made
     * @var String
     */
    private static $after_payment_message = "Payment is always successful";

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

    /**
     * Default Status for Payment
     * @var String
     */
    private static $default_status = "Success";

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

    /**
     * Process the DirectCredit payment method
     */
    public function processPayment($data, $form)
    {
        $this->Status = Config::inst()->get("DirectCreditPayment_TESTSUCCESS", "default_status");

The property Status does not seem to exist. Did you mean default_status?

        $this->Message = Config::inst()->get("DirectCreditPayment_TESTSUCCESS", "after_payment_message");

The property Message does not seem to exist. Did you mean before_payment_message?

        $this->write();
        return EcommercePayment_Success::create();
    }

    public function getPaymentFormFields()
    {
        return new FieldList(
            new LiteralField($this->ClassName.'_BeforeMessage', '<div id="'.$this->ClassName.'_BeforeMessage">' . Config::inst()->get("DirectCreditPayment_TESTSUCCESS", "before_payment_message") . '</div>'),
            new HiddenField($this->ClassName, $this->ClassName, 0)
        );
    }

    public function getPaymentFormRequirements()
    {
        return null;
    }
}


class DirectCreditPayment_TESTFAILURE extends EcommercePayment

This class seems to be duplicated in your project.

PSR1 recommends that each class should be in its own file to aid autoloaders.

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

{

    /**
     * Message shown before payment is made
     * @var String
     */
    private static $before_payment_message = "This is for testing purposes only";

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

    /**
     * Message shown after payment is made
     * @var String
     */
    private static $after_payment_message = "Payment is always unsuccessful";

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

    /**
     * Default Status for Payment
     * @var String
     */
    private static $default_status = "Failure";

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

    /**
     * Process the DirectCredit payment method
     */
    public function processPayment($data, $form)
    {
        $this->Status = Config::inst()->get("DirectCreditPayment_TESTFAILURE", "default_status");

The property Status does not seem to exist. Did you mean default_status?

        $this->Message = Config::inst()->get("DirectCreditPayment_TESTFAILURE", "after_payment_message");

The property Message does not seem to exist. Did you mean before_payment_message?

        $this->write();
        return EcommercePayment_Success::create();
    }

    public function getPaymentFormFields()
    {
        return new FieldList(
            new LiteralField($this->ClassName.'_BeforeMessage', '<div id="'.$this->ClassName.'_BeforeMessage">' . Config::inst()->get("DirectCreditPayment_TESTFAILURE", "before_payment_message") . '</div>'),
            new HiddenField($this->ClassName, $this->ClassName, 0)
        );
    }

    public function getPaymentFormRequirements()
    {
        return null;
    }
}