XoopsModules\Publisher\Form\ItemForm::buildMainTab()

Complexity

Conditions	13
Paths	768

Size

Total Lines	94
Code Lines	55

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    private function buildMainTab(Publisher\Item $obj, ContainerInterface $mainTab): void
    {
        $xoops = Xoops::getInstance();
        $helper = Helper::getInstance();

        // Category
        $category_select = new Select(_CO_PUBLISHER_CATEGORY, 'categoryid', $obj->getVar('categoryid', 'e'));
        $category_select->setDescription(_CO_PUBLISHER_CATEGORY_DSC);
        $category_select->addOptionArray($helper->getCategoryHandler()->getCategoriesForSubmit());
        $mainTab->addElement($category_select);

        // ITEM TITLE
        $mainTab->addElement(new Text(_CO_PUBLISHER_TITLE, 'title', 50, 255, $obj->getVar('title', 'e')), true);

It seems like $obj->getVar('title', 'e') can also be of type string[]; however, parameter $value of Xoops\Form\Text::__construct() does only seem to accept string, maybe add an additional type check?

        // SUBTITLE
        if ($this->isGranted(\_PUBLISHER_SUBTITLE)) {
            $mainTab->addElement(new Text(_CO_PUBLISHER_SUBTITLE, 'subtitle', 50, 255, $obj->getVar('subtitle', 'e')));
        }

        // SHORT URL
        if ($this->isGranted(\_PUBLISHER_ITEM_SHORT_URL)) {
            $text_short_url = new Text(_CO_PUBLISHER_ITEM_SHORT_URL, 'item_short_url', 50, 255, $obj->getVar('short_url', 'e'));
            $text_short_url->setDescription(_CO_PUBLISHER_ITEM_SHORT_URL_DSC);
            $mainTab->addElement($text_short_url);
        }

        // TAGS
        if ($xoops->isActiveModule('tag') && $this->isGranted(\_PUBLISHER_ITEM_TAG)) {
            require_once $xoops->path('modules/tag/include/formtag.php');
            $text_tags = new \Tag('item_tag', 60, 255, $obj->getVar('item_tag', 'e'), 0);

The type Tag was not found. Maybe you did not declare it correctly or list all dependencies?

            $mainTab->addElement($text_tags);
        }

        $this->buildEditors($obj, $mainTab);
        $this->buildTSOptions($obj, $mainTab);

        // Available pages to wrap
        if ($this->isGranted(\_PUBLISHER_AVAILABLE_PAGE_WRAP)) {
            $wrap_pages = XoopsLists::getHtmlListAsArray(Publisher\Utils::getUploadDir(true, 'content'));
            $available_wrap_pages_text = [];
            foreach ($wrap_pages as $page) {
                $available_wrap_pages_text[] = "<span onclick='publisherPageWrap(\"body\", \"[pagewrap=$page] \");'" . " onmouseover='style.cursor=\"pointer\"'>$page</span>";
            }
            $available_wrap_pages = new Label(_CO_PUBLISHER_AVAILABLE_PAGE_WRAP, \implode(', ', $available_wrap_pages_text));
            $available_wrap_pages->setDescription(_CO_PUBLISHER_AVAILABLE_PAGE_WRAP_DSC);
            $mainTab->addElement($available_wrap_pages);
        }

        // Uid
        /*  We need to retrieve the users manually because for some reason, on the frxoops.org server,
         the method users::getobjects encounters a memory error
         */
        // Trabis : well, maybe is because you are getting 6000 objects into memory , no??? LOL
        if ($this->isGranted(\_PUBLISHER_UID)) {
            $uid_select = new SelectUser(_CO_PUBLISHER_UID, 'uid', true, [$obj->getVar('uid', 'e')], 1, false);
            $uid_select->setDescription(_CO_PUBLISHER_UID_DSC);
            $mainTab->addElement($uid_select);
        }

        // Author Alias
        if ($this->isGranted(\_PUBLISHER_AUTHOR_ALIAS)) {
            $element = new Text(_CO_PUBLISHER_AUTHOR_ALIAS, 'author_alias', 50, 255, $obj->getVar('author_alias', 'e'));
            $element->setDescription(_CO_PUBLISHER_AUTHOR_ALIAS_DSC);
            $mainTab->addElement($element);
            unset($element);
        }

        // STATUS
        if ($this->isGranted(\_PUBLISHER_STATUS)) {
            $options = [
                \_PUBLISHER_STATUS_PUBLISHED => _CO_PUBLISHER_PUBLISHED,
                \_PUBLISHER_STATUS_OFFLINE => _CO_PUBLISHER_OFFLINE,
                \_PUBLISHER_STATUS_SUBMITTED => _CO_PUBLISHER_SUBMITTED,
                \_PUBLISHER_STATUS_REJECTED => _CO_PUBLISHER_REJECTED,
            ];
            $status_select = new Select(_CO_PUBLISHER_STATUS, 'status', $obj->getVar('status'));
            $status_select->addOptionArray($options);
            $status_select->setDescription(_CO_PUBLISHER_STATUS_DSC);
            $mainTab->addElement($status_select);
            unset($status_select);
        }

        // Datesub
        if ($this->isGranted(\_PUBLISHER_DATESUB)) {
            $datesub = (0 == $obj->getVar('datesub')) ? \time() : $obj->getVar('datesub');
            $datesub_datetime = new DateTimeSelect(_CO_PUBLISHER_DATESUB, 'datesub', $datesub);
            $datesub_datetime->setDescription(_CO_PUBLISHER_DATESUB_DSC);
            $mainTab->addElement($datesub_datetime);
        }

        // NOTIFY ON PUBLISH
        if ($this->isGranted(\_PUBLISHER_NOTIFY)) {
            $notify_radio = new RadioYesNo(_CO_PUBLISHER_NOTIFY, 'notify', $obj->getVar('notifypub'));

It seems like $obj->getVar('notifypub') can also be of type string[]; however, parameter $value of Xoops\Form\RadioYesNo::__construct() does only seem to accept null|string, maybe add an additional type check?

            $mainTab->addElement($notify_radio);
        }