EcommerceTaskReviewSearches

Complexity

Total Complexity

10

Size/Duplication

Total Lines	114
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	run()	0	63	9
A	Link()	0	4	1

<?php

/**
 * @authors: Nicolaas [at] Sunny Side Up .co.nz
 * @package: ecommerce
 * @sub-package: tasks
 **/
class EcommerceTaskReviewSearches extends BuildTask
{
    /**
     * number of days shown.
     *
     * @int
     */
    protected $defaultMaxRows = 999;
    /**
     * number of days shown.
     *
     * @int
     */
    protected $defaultDays = 100;

    /**
     * minimum number of searches for
     * a particular keyword in order to show it at all.
     *
     * @int
     */
    protected $defaultMinimum = 5;

    /**
     * show up to XXX days ago.
     *
     * @int
     */
    protected $endingDaysBack = 0;

    /**
     * Standard (required) SS variable for BuildTasks.
     *
     * @var string
     */
    protected $title = 'Search Statistics';

    /**
     * Standard (required) SS variable for BuildTasks.
     *
     * @var string
     */
    protected $description = '
        What did people search for on the website, you can use the days, min and ago GET variables to query different sets.';

    public function run($request)
    {
        $maxRows = intval(preg_replace('/[^\d.]/', '', $request->getVar('maxrows')));
        $maxRows = intval($maxRows - 0);
        if (!$maxRows) {
            $maxRows = $this->defaultMaxRows;
        }
        $days = intval($request->getVar('days') - 0);
        if (!$days) {
            $days = $this->defaultDays;
        }
        $countMin = intval($request->getVar('min') - 0);
        if (!$countMin) {
            $countMin = $this->defaultMinimum;
        }
        $endingDaysBack = intval($request->getVar('ago') - 0);
        if (!$endingDaysBack) {
            $endingDaysBack = $this->endingDaysBack;
        }
        $field = EcommerceSearchHistoryFormField::create('stats', $this->title)
            ->setNumberOfDays($days)
            ->setMinimumCount($countMin)
            ->setMaxRows($maxRows)
            ->setEndingDaysBack($endingDaysBack);
        echo $field->forTemplate();
        $arrayNumberOfDays = array(30, 365);

$arrayNumberOfDays is not used, you could remove the assignment.

        $fields = FieldList::create(
            HeaderField::create(
                'ShowResultsFor',
                'Show results for ...'
            ),
            NumericField::create(
                'days',
                'Number of days',
                isset($_GET['days']) ? $_GET['days'] : $this->defaultDays
            )->setRightTitle('For example, enter 10 to get results from a 10 day period.'),
            NumericField::create(
                'maxrows',
                'Maximum Number of Rows?',
                isset($_GET['maxrows']) ? $_GET['maxrows'] : $this->defaultMaxRows
            )->setRightTitle('For example, enter 10 to get results from a 10 day period.'),
            NumericField::create(
                'ago',
                'Up to how many days go',
                isset($_GET['ago']) ? $_GET['ago'] : $this->endingDaysBack
            )->setRightTitle('For example, entering 365 days means you get all statistics the specified number of days up to one year ago.'),
            NumericField::create(
                'min',
                'Count treshold',
                isset($_GET['min']) ? $_GET['min'] : $this->defaultMinimum
            )->setRightTitle('Minimum number of searches for it to show up in the statistics. For example, enter five to show only phrases that were searched for at least five times during the specified period.')
        );
        $actions = FieldList::create(FormAction::create("run")->setTitle("show results"));
        $form = Form::create($this, "SearchFields", $fields, $actions, null);
        $form->setAttribute('method', 'get');
        $form->setAttribute('action', $this->Link());
        echo $form->forTemplate();
        echo '<style>
            div.field {margin-bottom: 20px;}
            .right {font-style:italics; color: #555;}
        </style>';
    }

    public function Link($action = null)

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

    {
        return '/dev/tasks/EcommerceTaskReviewSearches/';
    }
}