StockistsCountryReports_WithoutCountry

Complexity

Total Complexity

6

Size/Duplication

Total Lines	67
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	title()	4	4	1
A	group()	4	4	1
A	sort()	4	4	1
A	sourceRecords()	4	4	1
A	columns()	9	9	1
A	getParameterFields()	4	4	1

<?php

/**
 *
 * @authors: Nicolaas [at] Sunny Side Up .co.nz
 * @sub-package: reports
 * @inspiration: Silverstripe Ltd, Jeremy
 **/

class StockistsCountryReports_WithoutCountry extends SS_Report

This class seems to be duplicated in your project.

{

    /**
     * The class of object being managed by this report.
     * Set by overriding in your subclass.
     */
    protected $dataClass = 'StockistCountryPage';

    /**
     *
     * @return String
     */
    public function title()
    {
        return "STOCKISTS: stockist countries without a country (".$this->sourceRecords()->count().")";
    }

    /**
     * not sure if this is used in SS3
     * @return String
     */
    public function group()
    {
        return "Stockists";
    }

    /**
     *
     * @return INT - for sorting reports
     */
    public function sort()
    {
        return 9000;
    }

    /**
     * working out the items
     * @return DataList
     */
    public function sourceRecords($params = null)

The parameter $params is not used and could be removed.

    {
        return StockistCountryPage::get()->where("Country = '' OR Country IS NULL");
    }

    /**
     * @return Array
     */
    public function columns()
    {
        return array(
            "Title" => array(
                "title" => "FullName",
                "link" => true
            )
        );
    }

    /**
     *
     * @return FieldList
     */
    public function getParameterFields()
    {
        return new FieldList();
    }
}