entryEdit()

Complexity

Conditions	27
Paths	> 20000

Size

Total Lines	209
Code Lines	131

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

<?php
function entryEdit($entryID = '')
{
    global $xoopsUser, $xoopsConfig, $xoopsDB, $xoopsModule, $init;
    $helper = Helper::getInstance();

    $myts    = \MyTextSanitizer::getInstance();

The assignment to $myts is dead and can be removed.

    $utility = new Utility();
    /**
     * Clear all variables before we start
     */
    if (!isset($block)) {

The variable $block seems to never exist and therefore isset should always be false.

        $block = 1;
    }
    if (!isset($html)) {

The variable $html seems to never exist and therefore isset should always be false.

        $html = 1;
    }
    if (!isset($smiley)) {

The variable $smiley does not exist. Did you maybe mean $smiley_checkbox?

        $smiley = 1;
    }
    if (!isset($xcodes)) {

The variable $xcodes seems to never exist and therefore isset should always be false.

        $xcodes = 1;
    }
    if (!isset($breaks)) {

The variable $breaks seems to never exist and therefore isset should always be false.

        $breaks = 1;
    }
    if (!isset($offline)) {

The variable $offline does not exist. Did you maybe mean $offline_radio?

        $offline = 0;
    }
    if (!isset($submit)) {

The variable $submit seems to never exist and therefore isset should always be false.

        $submit = 0;
    }
    if (!isset($request)) {

The variable $request seems to never exist and therefore isset should always be false.

        $request = 0;

The assignment to $request is dead and can be removed.

    }
    if (!isset($notifypub)) {

The variable $notifypub seems to never exist and therefore isset should always be false.

        $notifypub = 1;
    }
    if (!isset($categoryID)) {

The variable $categoryID seems to never exist and therefore isset should always be false.

        $categoryID = 1;
    }
    if (!isset($term)) {

The variable $term seems to never exist and therefore isset should always be false.

        $term = '';
    }
    if (!isset($init)) {
        $init = '';
    }
    if (!isset($definition)) {

The variable $definition seems to never exist and therefore isset should always be false.

        $definition = _AM_LEXIKON_WRITEHERE;
    }
    if (!isset($ref)) {

The variable $ref seems to never exist and therefore isset should always be false.

        $ref = '';
    }
    if (!isset($url)) {

The variable $url seems to never exist and therefore isset should always be false.

        $url = '';
    }
    if (!isset($datesub)) {

The variable $datesub seems to never exist and therefore isset should always be false.

        $datesub = 0;

The assignment to $datesub is dead and can be removed.

    }

    // If there is a parameter, and the id exists, retrieve data: we're editing an entry
    if ($entryID) {
        $result = $xoopsDB->query(
            '
                                     SELECT categoryID, term, init, definition, ref, url, uid, submit, datesub, html, smiley, xcodes, breaks, block, offline, notifypub, request
                                     FROM ' . $xoopsDB->prefix('lxentries') . "
                                     WHERE entryID = '$entryID'"
        );
        [$categoryID, $term, $init, $definition, $ref, $url, $uid, $submit, $datesub, $html, $smiley, $xcodes, $breaks, $block, $offline, $notifypub, $request] = $xoopsDB->fetchRow($result);

        if (!$xoopsDB->getRowsNum($result)) {
            redirect_header('index.php', 1, _AM_LEXIKON_NOENTRYTOEDIT);
        }
        $term = (htmlspecialchars($term, ENT_QUOTES | ENT_HTML5));

        echo "<strong style='color: #2F5376; margin-top:6px; font-size:medium'>" . _AM_LEXIKON_ADMINENTRYMNGMT . '</strong>';
        $sform = new \XoopsThemeForm(_AM_LEXIKON_MODENTRY . ": $term", 'op', xoops_getenv('SCRIPT_NAME'), 'post', true);
    } else { // there's no parameter, so we're adding an entry
        $result01 = $xoopsDB->query('SELECT COUNT(*) FROM ' . $xoopsDB->prefix('lxcategories') . ' ');
        [$totalcats] = $xoopsDB->fetchRow($result01);
        if (0 == $totalcats && 1 == $helper->getConfig('multicats')) {
            redirect_header('index.php', 1, _AM_LEXIKON_NEEDONECOLUMN);
        }
        $uid = $xoopsUser->getVar('uid');
        echo "<strong style='color: #2F5376; margin-top:6px; font-size:medium'>" . _AM_LEXIKON_ADMINENTRYMNGMT . '</strong>';
        $sform = new \XoopsThemeForm(_AM_LEXIKON_NEWENTRY, 'op', xoops_getenv('SCRIPT_NAME'), 'post', true);
    }

    $sform->setExtra('enctype="multipart/form-data"');
    // Category selector
    if (1 == $helper->getConfig('multicats')) {
        $mytree         = new LexikonTree($xoopsDB->prefix('lxcategories'), 'categoryID', '0');
        $categoryselect = new \XoopsFormSelect(_AM_LEXIKON_CATNAME, 'categoryID', $categoryID);
        $tbl            = [];

The assignment to $tbl is dead and can be removed.

        $tbl            = $mytree->getChildTreeArray(0, 'name');
        foreach ($tbl as $oneline) {
            if ('.' === $oneline['prefix']) {
                $oneline['prefix'] = '';
            }
            $oneline['prefix'] = str_replace('.', '-', $oneline['prefix']);
            $categoryselect->addOption($oneline['categoryID'], $oneline['prefix'] . ' ' . $oneline['name']);
        }
        $sform->addElement($categoryselect, true);
    }

    // Author selector
    ob_start();
    $utility::getUserForm((int)$uid);
    $sform->addElement(new \XoopsFormLabel(_AM_LEXIKON_AUTHOR, ob_get_clean()));

    // Initial selector
    ob_start();
    lx_getinit((int)$init);
    $sform->addElement(new \XoopsFormLabel(_AM_LEXIKON_INIT, ob_get_clean()));

    // Term, definition, reference and related URL
    $sform->addElement(new \XoopsFormText(_AM_LEXIKON_ENTRYTERM, 'term', 50, 80, $term), true);

    // set editor according to the module's option "form_options"
    $editor = $utility::getWysiwygForm(_AM_LEXIKON_ENTRYDEF, 'definition', $definition, 15, 60);
    if (_MD_LEXIKON_WRITEHERE == $definition) {
        $editor->setExtra('onfocus="this.select()"');
    }
    $sform->addElement($editor, true);
    unset($editor);

    $sform->addElement(new \XoopsFormTextArea(_AM_LEXIKON_ENTRYREFERENCE, 'ref', $ref, 5, 60), false);
    $sform->addElement(new \XoopsFormText(_AM_LEXIKON_ENTRYURL, 'url', 50, 80, $url), false);

    // tags of this term - for module 'Tag'
    /** @var \XoopsModuleHandler $moduleHandler */
    $moduleHandler = xoops_getHandler('module');
    $tagsModule    = $moduleHandler->getByDirname('tag');

The assignment to $tagsModule is dead and can be removed.

//    if (is_object($tagsModule)) {
//        require_once XOOPS_ROOT_PATH . '/modules/tag/include/formtag.php';
//        $sform->addElement(new \TagFormTag('item_tag', 60, 255, $entryID, $catid = 0));
//    }

//    if (class_exists('TagFormTag')) {
//        $formobj['tags'] = new FormTag('tags', 60, 255, $xcontent['xcontent']->getVar('storyid'), $xcontent['xcontent']->getVar('catid'));
//    } else {
//        $formobj['tags'] = new \XoopsFormHidden('tags', $xcontent['xcontent']->getVar('tags'));
//    }


    // Code to take entry offline, for maintenance purposes
    $offline_radio = new \XoopsFormRadioYN(_AM_LEXIKON_SWITCHOFFLINE, 'offline', $offline, ' ' . _AM_LEXIKON_YES . '', ' ' . _AM_LEXIKON_NO . '');
    $sform->addElement($offline_radio);

    // Code to put entry in block
    $block_radio = new \XoopsFormRadioYN(_AM_LEXIKON_BLOCK, 'block', $block, ' ' . _AM_LEXIKON_YES . '', ' ' . _AM_LEXIKON_NO . '');
    $sform->addElement($block_radio);

    // VARIOUS OPTIONS
    $options_tray = new \XoopsFormElementTray(_AM_LEXIKON_OPTIONS, '<br>');
    if ($submit) {
        $notify_checkbox = new \XoopsFormCheckBox('', 'notifypub', $notifypub);
        $notify_checkbox->addOption(1, _AM_LEXIKON_NOTIFYPUBLISH);
        $options_tray->addElement($notify_checkbox);
    } else {
        $notifypub = 0;

The assignment to $notifypub is dead and can be removed.

    }
    $html_checkbox = new \XoopsFormCheckBox('', 'html', $html);
    $html_checkbox->addOption(1, _AM_LEXIKON_DOHTML);
    $options_tray->addElement($html_checkbox);

    $smiley_checkbox = new \XoopsFormCheckBox('', 'smiley', $smiley);
    $smiley_checkbox->addOption(1, _AM_LEXIKON_DOSMILEY);
    $options_tray->addElement($smiley_checkbox);

    $xcodes_checkbox = new \XoopsFormCheckBox('', 'xcodes', $xcodes);
    $xcodes_checkbox->addOption(1, _AM_LEXIKON_DOXCODE);
    $options_tray->addElement($xcodes_checkbox);

    $breaks_checkbox = new \XoopsFormCheckBox('', 'breaks', $breaks);
    $breaks_checkbox->addOption(1, _AM_LEXIKON_BREAKS);
    $options_tray->addElement($breaks_checkbox);

    $sform->addElement($options_tray);

    $sform->addElement(new \XoopsFormHidden('entryID', $entryID));

    $buttonTray = new \XoopsFormElementTray('', '');
    $hidden     = new \XoopsFormHidden('op', 'addentry');
    $buttonTray->addElement($hidden);

    if (!$entryID) { // there's no entryID? Then it's a new entry
        $butt_create = new \XoopsFormButton('', '', _AM_LEXIKON_CREATE, 'submit');
        $butt_create->setExtra('onclick="this.form.elements.op.value=\'addentry\'"');
        $buttonTray->addElement($butt_create);

        $butt_clear = new \XoopsFormButton('', '', _AM_LEXIKON_CLEAR, 'reset');
        $buttonTray->addElement($butt_clear);

        $butt_cancel = new \XoopsFormButton('', '', _AM_LEXIKON_CANCEL, 'button');
        $butt_cancel->setExtra('onclick="history.go(-1)"');
        $buttonTray->addElement($butt_cancel);
    } else { // else, we're editing an existing entry
        $butt_create = new \XoopsFormButton('', '', _AM_LEXIKON_MODIFY, 'submit');
        $butt_create->setExtra('onclick="this.form.elements.op.value=\'addentry\'"');
        $buttonTray->addElement($butt_create);

        $butt_cancel = new \XoopsFormButton('', '', _AM_LEXIKON_CANCEL, 'button');
        $butt_cancel->setExtra('onclick="history.go(-1)"');
        $buttonTray->addElement($butt_cancel);
    }

    $sform->addElement($buttonTray);
    $sform->display();
    unset($hidden);
}