XoopsModules\Songlist\Utf8map

Complexity

Total Complexity

6

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	getForm()	0	3	1
A	toArray()	0	15	4

<?php declare(strict_types=1);

namespace XoopsModules\Songlist;

use XoopsObject;

\defined('XOOPS_ROOT_PATH') || exit('Restricted access');

require_once \dirname(__DIR__) . '/include/songlist.object.php';
// require_once \dirname(__DIR__) . '/include/songlist.form.php';
use  XoopsModules\Songlist\Form\FormController;

/**
 * Class Utf8map
 */
class Utf8map extends XoopsObject
{
    /**
     * Utf8map constructor.
     * @param null $fid

Are you sure the doc-type for parameter $fid is correct as it would always require null to be passed?

     */
    public function __construct($fid = null)

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

    {
        $this->initVar('utfid', \XOBJ_DTYPE_INT, 0, false);
        $this->initVar('from', \XOBJ_DTYPE_TXTBOX, null, false, 2);
        $this->initVar('to', \XOBJ_DTYPE_TXTBOX, null, false, 2);
        $this->initVar('created', \XOBJ_DTYPE_INT, 0, false);
        $this->initVar('updated', \XOBJ_DTYPE_INT, 0, false);
    }

    /**
     * @param bool $as_array
     * @return array|string
     */
    public function getForm($as_array = false)
    {
        return FormController::getFormUtf8map($this, $as_array);
    }

    /**
     * @return array
     */
    public function toArray(): array
    {
        $ret  = parent::toArray();
        $form = $this->getForm(true);
        foreach ($form as $key => $element) {
            $ret['form'][$key] = $element->render();
        }
        foreach (['created', 'updated'] as $key) {
            if ($this->getVar($key) > 0) {
                $ret['form'][$key] = \date(_DATESTRING, $this->getVar($key));
                $ret[$key]         = \date(_DATESTRING, $this->getVar($key));
            }
        }

        return $ret;
    }
}