AccordionParser

Complexity

Total Complexity

3

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	get_shortcodes()	0	4	1
A	handle_shortcode()	0	11	2

<?php

/**
 * Class AccordionParser
 *
 * Parse the shortcode and return the rendered accordion template
 */
class AccordionParser

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    public static function get_shortcodes()
    {
        return ['code'];
    }

    /**
     * @param string $arguments array with the type
     * @param array $code string of the code to parse
     * @param ShortcodeParser $parser Parser root user.
     * @param string $shortcode
     * @param array $extra
     *
     * @return String of parsed code.
     */
    public static function handle_shortcode($arguments, $code, $parser, $shortcode, $extra = array())

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

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

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

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

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

    {
        // Only if we're on a Page, so the CMS doesn't crash.
        if (Controller::curr() instanceof Page_Controller) {

The class Page_Controller does not exist. Did you forget a USE statement, or did you not list all dependencies?

            $accordionItems = Controller::curr()->dataRecord;
            $template = 'AccordionItems';
            $ssViewer = new SSViewer($template);

            return $ssViewer->process($accordionItems);
        }
    }
}