JiraRestApi\RapidCharts\SprintReportService

Complexity

Total Complexity

2

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getSprintReportData()	0	8	1

<?php

namespace JiraRestApi\RapidCharts;

use JiraRestApi\Configuration\ConfigurationInterface;
use JiraRestApi\GreenHopperTrait;
use Psr\Log\LoggerInterface;

class SprintReportService extends \JiraRestApi\JiraClient
{
    use GreenHopperTrait;

    private $uri = '/rapid/charts/sprintreport';

    public function __construct(ConfigurationInterface $configuration = null, LoggerInterface $logger = null, $path = './')
    {
        parent::__construct($configuration, $logger, $path);
        $this->setupAPIUri();
    }

    public function getSprintReportData($rapidViewId, $sprintId, $paramArray = [])
    {
        $paramArray['rapidViewId'] = $rapidViewId;
        $paramArray['sprintId'] = $sprintId;
        $json = $this->exec($this->uri.'/'.$this->toHttpQueryParameter($paramArray), null);
        $sprintReport = json_decode($json);

It seems like $json can also be of type true; however, parameter $json of json_decode() does only seem to accept string, maybe add an additional type check?

        return $sprintReport;
    }
}