admin/main.php ➔ importCats()

Complexity

Conditions	8
Paths	11

Size

Total Lines

49

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
function importCats()
{
    global $xoopsDB, $xoopsModule, $xoopsConfig, $myts;

    $ok = MylinksUtility::mylinks_cleanVars($_POST, 'ok', 0, 'int', array('min' => 0, 'max' => 1));

'ok' is of type string, but the function expects a object<unknown_type>.

    if (1 == $ok) {
        if (file_exists(XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/language/' . $xoopsConfig['language'] . '/sql/mylinks_cat.dat')) {
            $importFile = XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/language/' . $xoopsConfig['language'] . '/sql/mylinks_cat.dat';
        } else {
            $importFile = XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/language/english/sql/mylinks_cat.dat';
        }

        if (file_exists($importFile)) {
            /* the following will not work on some shared servers even though it's the most efficient
            $sql = "LOAD DATA INFILE '{$importFile}' INTO TABLE " . $xoopsDB->prefix('mylinks_cat') . " FIELDS TERMINATED BY ',' IGNORE 1 LINES";
            $result = $xoopsDB->query($sql);
            */

            if (($handle = fopen($importFile, 'r')) !== false) {
                // set 1000 to 0 in the following line if input line is truncated
                while (($data = fgetcsv($handle, 1000, ',')) !== false) {
                    $sql    = sprintf("INSERT INTO %s (cid, pid, title, imgurl) VALUES (%u, %u, '%s', '%s')", $xoopsDB->prefix('mylinks_cat'), (int)$data[0], (int)$data[1], $myts->addSlashes($data[2]), $myts->addSlashes($data[3]));
                    $result = $xoopsDB->query($sql);
                    if (!$result) {
                        MylinksUtility::show_message(_MD_MYLINKS_NORECORDFOUND);

The method MylinksUtility::show_message() has been deprecated.

                        exit();
                    }
                }
                fclose($handle);
                redirect_header('index.php', 2, _MD_MYLINKS_CATSIMPORTED);
            } else {
                // problem importing categories
                $mylinksCatHandler = xoops_getModuleHandler('category', $xoopsModule->getVar('dirname'));
                $result            = $mylinksCatHandler->getAll();
                if (count($result)) {
                    $result = $mylinksCatHandler->deleteAll();  // empty the dB table from partial import

$result is not used, you could remove the assignment.

                }
                redirect_header('index.php', 2, _MD_MYLINKS_CATSNOTIMPORTED);
            }
        } else {  //exit somewhat gracefully if import file not found
            redirect_header('index.php', 2, sprintf(_MD_MYLINKS_IMPORTFILENOTFOUND, $importFile));
        }
        exit();
    } else {
        xoops_cp_header();
        xoops_confirm(array('op' => 'importCats', 'ok' => 1), 'main.php', _MD_MYLINKS_CATWARNING);
        include __DIR__ . '/admin_footer.php';
    }
}