smtech\GradingAnalytics\Toolbox

Complexity

Total Complexity

11

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Coupling/Cohesion

Components	4
Dependencies	2

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getGenerator()	0	17	1
A	loadConfiguration()	0	8	2
A	graphWidth()	0	7	2
A	graphHeight()	0	7	2
A	getHistory()	0	7	2
A	getSnapshot()	0	7	2

<?php

namespace smtech\GradingAnalytics;

use smtech\LTI\Configuration\Option;
use Battis\HierarchicalSimpleCache;

class Toolbox extends \smtech\StMarksReflexiveCanvasLTI\Toolbox
{
    const TOOL_CANVAS_EXTERNAL_TOOL_ID = 'TOOL_CANVAS_EXTERNAL_TOOL_ID';

    protected $history;

    protected $snapshots;

    public function getGenerator()
    {
        parent::getGenerator();

        $this->generator->setOptionProperty(
            Option::COURSE_NAVIGATION(),
            'visibility',
            'admins'
        );
        $this->generator->setOptionProperty(
            Option::ACCOUNT_NAVIGATION(),
            'visibility',
            'admins'
        );

        return $this->generator;
    }

    protected function loadConfiguration($configFilePath, $forceRecache = false)
    {
        parent::loadConfiguration($configFilePath, $forceRecache);

        if ($forceRecache) {
            $this->clearConfig(self::TOOL_CANVAS_EXTERNAL_TOOL_ID);
        }
    }

    private $GRAPH_DATA_COUNT = 0;
    public function graphWidth($dataCount = false)
    {
        if ($dataCount) {
            $this->GRAPH_DATA_COUNT = $dataCount;

The property $GRAPH_DATA_COUNT was declared of type integer, but $dataCount is of type boolean. Maybe add a type cast?

        }
        return max(GRAPH_MIN_WIDTH, $this->GRAPH_DATA_COUNT * GRAPH_BAR_WIDTH);
    }

    public function graphHeight($dataCount = false)
    {
        if ($dataCount) {
            $this->GRAPH_DATA_COUNT = $dataCount;

The property $GRAPH_DATA_COUNT was declared of type integer, but $dataCount is of type boolean. Maybe add a type cast?

        }
        return $this->graphWidth() * GRAPH_ASPECT_RATIO;
    }

    public function getHistory($courseId)
    {
        if (!is_a($this->history, History::class)) {
            $this->history = new History($this, $courseId);
        }
        return $this->history->getHistory();
    }

    public function getSnapshot($courseOrDepartmentId, Domain $domain, $isCourseId = true, $teacherFilter = false)
    {
        if (empty($this->snapshots[$domain])) {
            $this->snapshots[$domain] = new Snapshot($this, $domain, $courseOrDepartmentId, $isCourseId);
        }
        return $this->snapshots[$domain]->getSnapshot($teacherFilter);
    }
}