PriceListPage

Complexity

Total Complexity

13

Size/Duplication

Total Lines	113
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	5

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
D	currentFinalProducts()	0	35	10

<?php

/**
 *@author nicolaas[at]
 *@description
 *
 * Provides a price list
 *
 *
 *
 *
 **/

class PriceListPage extends ProductGroup

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

{

    /**
     * Standard SS variable.
     */
    private static $singular_name = "Price List Page";

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("ProductGroup.PRICELISTPAGE", "Price List Page");
    }

    /**
     * Standard SS variable.
     */
    private static $plural_name = "Price List Pages";

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("ProductGroup.PRICELISTPAGES", "Price List Pages");
    }

    /**
     * standard SS variable
     * @static Array | String
     *
     */
    private static $icon = "ecommerce_complex_pricing/images/treeicons/PriceListPage";

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
     * @static Array
     *
     */
    private static $db = array(

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

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

        "NumberOfLevelsToHide" => "Int"
    );

    /**
     * standard SS variable
     * @static Array
     *
     */
    private static $defaults = array(

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

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

        "LevelOfProductsToShow" => -1,
        "NumberOfProductsPerPage" => 100,
        "NumberOfLevelsToHide" => 1
    );


    /**
     * standard SS variable
     * @static Array
     *
     */
    private static $allowed_children = "none";

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

    /**
     * standard SS Method
     * return FieldSet
     *
     */
    public function getCMSFields()
    {
        $fields = parent::getCMSFields();
        $fields->addFieldToTab('Root.ProductDisplay', new NumericField("NumberOfLevelsToHide", _t("PriceListPage.NUMBEROFLEVELSTOHIDE", "Number of levels to hide from the top down (e.g. set to one to hide main (top) product group holder). To hide all the parent product groups you can set this variable to something like 999.")));
        return $fields;
    }


    /**
     * Retrieve a set of products, based on the given parameters.
     * Add Parent Group Pages to diplay within list.
     *
     * Note that you can hide the "top level"
     * @return DataObjectSet | Null

Should the return type not be DataList|null?

     */
    public function currentFinalProducts($alternativeSort = null)
    {
        $products = parent::currentFinalProducts($alternativeSort);
        if ($products) {
            foreach ($products as $product) {
                $product->ParentSegments = null;
                if ($this->NumberOfLevelsToHide < 20) {
                    $segmentArray = array();
                    $item = $product;
                    while ($item && $item->ParentID) {
                        $item = ProductGroup::get()->byID($item->ParentID);
                        if ($item) {
                            $segmentArray[] = array(
                                "URLSegment" => $item->URLSegment,
                                "ID" => $item->ID,
                                "ClassName" => $item->ClassName,
                                "Title" => $item->Title,
                                "Link" => $item->Link()
                            );
                        }
                    }
                    if (count($segmentArray)) {
                        $product->ParentSegments = new ArrayList();
                        $segmentArray = array_reverse($segmentArray);
                        foreach ($segmentArray as $key => $segment) {
                            if ($key > $this->NumberOfLevelsToHide) {
                                $product->ParentSegments->push(new ArrayData($segment));
                            }
                        }
                    }
                }
            }
        }
        return $products;
    }
}

class PriceListPage_Controller extends ProductGroup_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.

{
    public function init()
    {
        parent::init();
        Requirements::themedCSS("PriceListPage", "ecommerce_complex_pricing");
    }
}