XoopsModules\Smartobject\Form\Elements\SmartFormRadioElement

Complexity

Total Complexity

9

Size/Duplication

Total Lines	53
Duplicated Lines	64.15 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	__construct()	34	45	9

<?php namespace XoopsModules\Smartobject\Form\Elements;

/**
 * Contains the SmartObjectControl class
 *
 * @license    GNU
 * @author     marcan <marcan@smartfactory.ca>
 * @link       http://smartfactory.ca The SmartFactory
 * @package    SmartObject
 * @subpackage SmartObjectForm
 */

use XoopsModules\Smartobject;

/**
 * Class SmartFormRadioElement
 * @package XoopsModules\Smartobject\Form\Elements
 */
class SmartFormRadioElement extends \XoopsFormRadio
{
    /**
     * SmartFormRadioElement constructor.
     * @param string $object
     * @param string $key
     */
    public function __construct($object, $key)
    {
        $var = $object->vars[$key];

        parent::__construct($var['form_caption'], $key, $object->getVar($key, 'e'));

The method getVar cannot be called on $object (of type string).

        // Adding the options inside this SelectBox
        // If the custom method is not from a module, than it's from the core
        $control = $object->getControl($key);

The method getControl cannot be called on $object (of type string).

        if (isset($control['options'])) {

This code seems to be duplicated across your project.

            $this->addOptionArray($control['options']);
        } else {
            // let's find if the method we need to call comes from an already defined object
            if (isset($control['object'])) {
                if (method_exists($control['object'], $control['method'])) {
                    if ($option_array = $control['object']->{$control['method']}()) {
                        // Adding the options array to the XoopsFormSelect
                        $this->addOptionArray($option_array);
                    }
                }
            } else {
                // finding the itemHandler; if none, let's take the itemHandler of the $object
                if (isset($control['itemHandler'])) {
                    if (!$control['module']) {
                        // Creating the specified core object handler
                        $controlHandler = xoops_getHandler($control['itemHandler']);
                    } else {
                        $controlHandler = xoops_getModuleHandler($control['itemHandler'], $control['module']);
                    }
                } else {
                    $controlHandler = $object->handler;
                }

                // Checking if the specified method exists
                if (method_exists($controlHandler, $control['method'])) {
                    // TODO: How could I pass the parameters in the following call ...
                    if ($option_array = $controlHandler->{$control['method']}()) {
                        // Adding the options array to the XoopsFormSelect
                        $this->addOptionArray($option_array);
                    }
                }
            }
        }
    }
}