DonationProductController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	4	1
A	updatevalue()	0	14	3
B	PurchaseForm()	0	26	2

<?php

use Dynamic\FoxyStripe\Page\ProductPage;
use Dynamic\FoxyStripe\Page\ProductPageController;
use SilverStripe\SiteConfig\SiteConfig;
use SilverStripe\View\Requirements;

class DonationProductController extends ProductPageController
{
    /**
     * @var array
     */
    private static $allowed_actions = [

The private property $allowed_actions is not used, and could be removed.

        'PurchaseForm',
        'updatevalue',
    ];

    /**
     *
     */
    public function init()
    {
        parent::init();
        Requirements::javascript('framework/thirdparty/jquery/jquery.js');
    }

    /**
     * @return FoxyStripePurchaseForm

The type FoxyStripePurchaseForm was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    public function PurchaseForm()
    {
        $form = parent::PurchaseForm();

        $fields = $form->Fields();

        $fields->replaceField(ProductPage::getGeneratedValue(
            $this->Code,

The property Code does not exist on DonationProductController. Since you implemented __get, consider adding a @property annotation.

            'price',
            $this->Price

The property Price does not exist on DonationProductController. Since you implemented __get, consider adding a @property annotation.

        ), $currencyField = CurrencyField::create('price', 'Amount'));

The type CurrencyField was not found. Maybe you did not declare it correctly or list all dependencies?

        $fields->removeByName(array(
            'quantity',
            ProductPage::getGeneratedValue($this->Code, 'weight', $this->Weight),

The property Weight does not exist on DonationProductController. Since you implemented __get, consider adding a @property annotation.

        ));

        if (SiteConfig::current_site_config()->CartValidation) {
            Requirements::javascript('framework/thirdparty/jquery-validate/jquery.validate.js');
            Requirements::javascriptTemplate('foxystripe/javascript/donationProduct.js', [
                'Trigger' => (string) $currencyField->getAttribute('id'),
                'UpdateURL' => Director::absoluteURL($this->Link('updatevalue')),
            ]);
        }

        return $form;

The expression return $form returns the type Dynamic\FoxyStripe\Form\FoxyStripePurchaseForm which is incompatible with the documented return type FoxyStripePurchaseForm.

    }

    /**
     * create new encrypted price value based on user input.
     *
     * @param $request
     *
     * @return string|\SilverStripe\Control\HTTPResponse
     */
    public function updatevalue(\SilverStripe\Control\HTTPRequest $request)
    {
        if ($request->getVar('Price') && SiteConfig::current_site_config()->CartValidation) {
            $vars = $request->getVars();
            $signedPrice = FoxyCart_Helper::fc_hash_value($this->Code, 'price', $vars['Price'], 'name', false);

The property Code does not exist on DonationProductController. Since you implemented __get, consider adding a @property annotation.

            $json = json_encode(['Price' => $signedPrice]);
            $response = new HTTPResponse($json);

The call to HTTPResponse::__construct() has too many arguments starting with $json.

            $response->removeHeader('Content-Type');

The method removeHeader() does not exist on HttpResponse.

            $response->addHeader('Content-Type', 'application/json');

The method addHeader() does not exist on HttpResponse.

            return $response;

The expression return $response returns the type HTTPResponse which is incompatible with the documented return type SilverStripe\Control\HTTPResponse|string.

        }

        return 'false';
    }
}