Seboettg\CiteProc\CiteProc

Complexity

Total Complexity

20

Size/Duplication

Total Lines	156
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	9

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	getContext()	0	4	1
A	setContext()	0	4	1
A	loadStyleSheet()	0	4	1
A	__construct()	0	5	1
C	parse()	0	30	7
A	bibliography()	0	5	1
A	citation()	0	4	1
B	render()	0	33	6
A	init()	0	7	1

<?php
/*
 * citeproc-php
 *
 * @link        http://github.com/seboettg/citeproc-php for the source repository
 * @copyright   Copyright (c) 2016 Sebastian Böttger.
 * @license     https://opensource.org/licenses/MIT
 */

namespace Seboettg\CiteProc;
use Seboettg\CiteProc\Data\DataList;
use Seboettg\CiteProc\Exception\CiteProcException;
use Seboettg\CiteProc\Style\Bibliography;
use Seboettg\CiteProc\Style\Citation;
use Seboettg\CiteProc\Style\Macro;
use Seboettg\CiteProc\Style\Root;


/**
 * Class CiteProc
 * @package Seboettg\CiteProc
 *
 * @author Sebastian Böttger <seboettg@gmail.com>
 */
class CiteProc
{

    /**
     * @var Context
     */
    private static $context;


    /**
     * @return Context
     */
    public static function getContext()
    {
        return self::$context;
    }

    /**
     * @param Context $context
     */
    public static function setContext($context)
    {
        self::$context = $context;
    }

    /**
     * @param $styleName
     * @deprecated
     */
    public static function loadStyleSheet($styleName)
    {
        return StyleSheet::loadStyleSheet($styleName);
    }


    /**
     * @var string
     */
    private $styleSheet;

    /**
     * @var \SimpleXMLElement
     */
    private $styleSheetXml;

    /**
     * CiteProc constructor.
     * @param string $styleSheet xml formatted csl stylesheet
     */
    public function __construct($styleSheet, $lang = "en-US")
    {
        $this->styleSheet = $styleSheet;
        $this->lang = $lang;

The property lang does not exist. Did you maybe forget to declare it?

    }

    /**
     * @param \SimpleXMLElement $style
     */
    private function parse(\SimpleXMLElement $style)
    {
        $root = new Root();
        $root->initInheritableNameAttributes($style);
        self::$context->setRoot($root);

        /** @var \SimpleXMLElement $node */
        foreach ($style as $node) {
            $name = $node->getName();
            switch ($name) {
                case 'info':
                    break;
                case 'locale':
                    self::$context->getLocale()->addXml($node);
                    break;
                case 'macro':
                    $macro = new Macro($node, $root);
                    self::$context->addMacro($macro->getName(), $macro);
                    break;
                case 'bibliography':
                    $bibliography = new Bibliography($node, $root);
                    self::$context->setBibliography($bibliography);
                    break;
                case 'citation':
                    $citation = new Citation($node, $root);
                    self::$context->setCitation($citation);
                    break;
            }
        }
    }

    /**
     * @param array|DataList $data
     * @return string
     */
    protected function bibliography($data)
    {

        return self::$context->getBibliography()->render($data);

$data is of type array|object<Seboettg\CiteProc\Data\DataList>, but the function expects a object<stdClass>.

    }

    /**
     * @param array|DataList $data
     * @return string
     */
    protected function citation($data)
    {
        return self::$context->getCitation()->render($data);

It seems like $data defined by parameter $data on line 128 can also be of type object<Seboettg\CiteProc\Data\DataList>; however, Seboettg\CiteProc\Style\Citation::render() does only seem to accept array|object<Seboettg\CiteProc\Style\DataList>, maybe add an additional type check?

    }

    /**
     * @param array|DataList $data
     * @param string $mode (citation|bibliography)
     * @return string
     * @throws CiteProcException
     */
    public function render($data, $mode = "bibliography")
    {

        if (!in_array($mode, ['citation', 'bibliography'])) {
            throw new \InvalidArgumentException("\"$mode\" is not a valid mode.");
        }

        $this->init(); //initialize

        $res = "";

        if (is_array($data)) {
            $data = new DataList($data);
        } else if (!($data instanceof DataList)) {
            throw new CiteProcException('No valid format for variable data. Either DataList or array expected');
        }

        // set CitationItems to Context
        self::getContext()->setCitationItems($data);

        switch ($mode) {
            case 'bibliography':
                self::$context->setMode($mode);
                $res = $this->bibliography($data);
                break;
            case 'citation':
                self::$context->setMode($mode);
                $res = $this->citation($data);
        }
        self::setContext(null);

null is of type null, but the function expects a object<Seboettg\CiteProc\Context>.

        return $res;
    }

    private function init()
    {
        self::$context = new Context($this);
        self::$context->setLocale(new Locale\Locale($this->lang)); //init locale
        $this->styleSheetXml = new \SimpleXMLElement($this->styleSheet);
        $this->parse($this->styleSheetXml);
    }
}