ElectronicDownloadProduct

Complexity

Total Complexity

4

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	i18n_singular_name()	0	4	1
A	i18n_plural_name()	0	4	1
A	getCMSFields()	0	6	1
A	QuantityDecimals()	0	4	1

<?php

class ElectronicDownloadProduct extends Product

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

{

    /**
     * Standard SS variable.
     */
    private static $many_many = array(

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

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

        'DownloadFiles' => 'File'
    );

    /**
     * Standard SS variable.
     */
    private static $icon = 'ecommerce_delivery_electronic/images/icons/ElectronicDownloadProduct';

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

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

    /**
     * Standard SS variable.
     * @var String
     */
    private static $description = "A product can be downloaded.";

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

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

    /**
     * Standard SS variable.
     */
    private static $singular_name = "Electronic Download Product";

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()
    {
        return _t("ElectronicDownloadProduct.SINGULAR_NAME", "Electronic Download Product");
    }

    /**
     * Standard SS variable.
     */
    private static $plural_name = "Electronic Download Products";

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()
    {
        return _t("ElectronicDownloadProduct.PLURAL_NAME", "Electronic Download Products");
    }


    /**
     * Standard SS Method
     */
    public function getCMSFields()

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

    {
        $fields = parent::getCMSFields();
        $fields->addFieldToTab('Root.Details', new UploadField('DownloadFiles', _t('ElectronicDownloadProduct.DOWNLOADFILES', 'Download Files')));
        return $fields;
    }

    /**
     * This is used when you add a product to your cart
     * if you set it to 1 then you can add 0.1 product to cart.
     * If you set it to -1 then you can add 10, 20, 30, etc.. products to cart.
     *
     * @return Int
     **/
    public function QuantityDecimals()
    {
        return 0;
    }
}


class ElectronicDownloadProduct_Controller extends Product_Controller

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.

{
}