mambax7 / wflinks

admin/altcat.php

<?php
/**
 * Module: WF
 * Version: v1.0.3
 * Release Date: 21 June 2005
 * Developer: John N
 * Team: WF-Projects
 * Licence: GNU
 */

use Xmf\Request;
use XoopsModules\Wflinks;

require_once __DIR__ . '/admin_header.php';

global $xoopsModuleConfig;

$op  = \Xmf\Request::getString('op', '');
$lid = \Xmf\Request::getInt('lid', 0);

/**
 * @param Wflinks\Tree $xt
 * @param              $itemid
 * @param              $title
 * @param              $checks
 */
function makeTreeCheckTable(Wflinks\Tree $xt, $itemid, $title, $checks)
{
    global $myts;

    echo "<div style='text-align: left;'>\n";
    echo "<form name='altcat' method='post' action='" . xoops_getenv('SCRIPT_NAME') . "'>\n";
    echo "<table width='100%' callspacing='1' class='outer'>\n";
    $sql = 'SELECT ' . $xt->id . ', ' . $title . ' FROM ' . $xt->table . ' WHERE ' . $xt->pid . '=0 ORDER BY ' . $title;

    $result = $xt->db->query($sql);

    while (list($cid, $name) = $xt->db->fetchRow($result)) {
        $checked  = array_key_exists($cid, $checks) ? 'checked' : '';
        $disabled = ($cid == Request::getInt('cid', 0, 'GET')) ? "disabled='yes'" : '';
        $level    = 1;
        echo "
        <tr style='text-align: left;'>
         <td width='30%' class='head'>$name</td>
         <td class='head'>
             <input type='checkbox' name='cid-" . (int)$cid . "' value='0' " . $checked . ' ' . $disabled . ">
         </td>
        </tr>\n";
        $arr = $xt->getChildTreeArray((int)$cid, $title);

        foreach ($arr as $cat) {
            $cat['prefix'] = str_replace('.', '-', $cat['prefix']);
            $catpath       = $cat['prefix'] . '&nbsp;' . htmlspecialchars($cat[$title], ENT_QUOTES | ENT_HTML5) . '&nbsp;';
            $checked       = array_key_exists($cat['cid'], $checks) ? 'checked' : '';
            $disabled      = ($cat['cid'] == Request::getInt('cid', 0, 'GET')) ? "disabled='yes'" : '';
            $level         = mb_substr_count($cat['prefix'], '-') + 1;

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

            //            echo "<tr><td>" . $catpath . "<input type='checkbox' name='cid-" . $cat['cid'] . "' value='0' " . $checked . " " . $disabled . "></td></tr>\n";
            echo "
        <tr style='text-align: left;'>
         <td width='30%' class='even'>$catpath</td>
         <td class='even'>
             <input type='checkbox' name='cid-" . $cat['cid'] . "' value='0' " . $checked . ' ' . $disabled . ">
         </td>
        </tr>\n";
        }
    }
    echo "<tr>
           <td width='30%' class='head'></td>
           <td class='even' style='text-align: left;'>
            <input type='submit' class='mainbutton' value='save'>
            <input type='hidden' name='op' value='save'>
            <input type='hidden' name='lid' value='" . $itemid . "'>
            </td>
          </tr>";
    echo "</table></form></div>\n";
}

switch (mb_strtolower($op)) {
    case 'save':
        // first delete all alternate categories for this topic
        $sql = 'DELETE FROM ' . $xoopsDB->prefix('wflinks_altcat') . ' WHERE lid=' . $lid;
        if (!$result = $xoopsDB->query($sql)) {
            /** @var \XoopsLogger $logger */
            $logger = \XoopsLogger::getInstance();
            $logger->handleError(E_USER_WARNING, $sql, __FILE__, __LINE__);

            return false;
        }

        $k = array_keys($_REQUEST);
        foreach ($k as $sid) {
            if (preg_match('/cid-([0-9]*)/', $sid, $cid)) {
                $sql = 'INSERT INTO ' . $xoopsDB->prefix('wflinks_altcat') . "(cid, lid) VALUES('" . $cid[1] . "','" . $lid . "')";
                if (!$result = $xoopsDB->query($sql)) {
                    /** @var \XoopsLogger $logger */
                    $logger = \XoopsLogger::getInstance();
                    $logger->handleError(E_USER_WARNING, $sql, __FILE__, __LINE__);

                    return false;
                }
            }
        }
        redirect_header('index.php', 1, _AM_WFL_ALTCAT_CREATED);
        break;
    case 'main':
    default:
        xoops_cp_header();

        echo "
            <fieldset style='border: #e8e8e8 1px solid;'><legend style='display: inline; font-weight: bold; color: #0A3760;'>" . _AM_WFL_ALTCAT_MODIFYF . "</legend>\n
            <div style='padding: 8px;'>" . _AM_WFL_ALTCAT_INFOTEXT . "</div>\n
            </fieldset>\n
        ";

        echo "<div style='text-align: left; font-size: larger;'><h4>" . htmlspecialchars(trim($_GET['title']), ENT_QUOTES | ENT_HTML5) . '</h4></div>';
        // Get an array of all alternate categories for this topic
        $sql     = $xoopsDB->query('SELECT cid FROM ' . $xoopsDB->prefix('wflinks_altcat') . ' WHERE lid=' . $lid . ' ORDER BY lid');
        $altcats = [];
        while (false !== ($altcat = $xoopsDB->fetchArray($sql))) {
            $altcats[$altcat['cid']] = true;
        }
        $mytree = new Wflinks\Tree($xoopsDB->prefix('wflinks_cat'), 'cid', 'pid');

        makeTreeCheckTable($mytree, $lid, 'title', $altcats);
        require_once __DIR__ . '/admin_footer.php';
}