mambax7/tdmdownloads

File category.php has 345 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php

declare(strict_types=1);

use Xmf\Module\Admin;

Consider simplifying this complex logical expression.

Open

        if (\Xmf\Request::hasVar('ok', 'REQUEST') && 1 == \Xmf\Request::getInt('ok', 0, 'REQUEST')) {
            if (!$GLOBALS['xoopsSecurity']->check()) {
                redirect_header('category.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
            }
            // supression des téléchargements de la catégorie

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                if (1 == $helper->getConfig('usetag') && class_exists(LinkHandler::class)) {
                    /** @var \XoopsModules\Tag\LinkHandler $linkHandler */
                    $linkHandler = TagHelper::getInstance()->getHandler('Link');
                    $criteria    = new \CriteriaCompo();
                    $criteria->add(new \Criteria('tag_itemid', $downloadsArray[$i]->getVar('lid')));

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                    if (1 == $helper->getConfig('usetag') && class_exists(LinkHandler::class)) {
                        /** @var \XoopsModules\Tag\LinkHandler $linkHandler */
                        $linkHandler = TagHelper::getInstance()->getHandler('Link');
                        $criteria    = new \CriteriaCompo();
                        $criteria->add(new \Criteria('tag_itemid', $downloadsArray[$j]->getVar('lid')));

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    case 'new_cat':
        //Affichage de la partie haute de l'administration de Xoops
        xoops_cp_header();
        $adminObject = Admin::getInstance();
        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation(basename(__FILE__)));

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        if ($uploader->fetchMedia($_POST['xoops_upload_file'][0])) {
            $uploader->setPrefix('downloads_');
            $uploader->fetchMedia($_POST['xoops_upload_file'][0]);
            if ($uploader->upload()) {
                $obj->setVar('cat_imgurl', $uploader->getSavedFileName());

• Read up
  Read up

Identical blocks of code found in 5 locations. Consider refactoring.

Open

if (is_object($helper->getModule())) {
    $pathModIcon16 = $helper->getModule()->getInfo('modicons16');
    $pathModIcon32 = $helper->getModule()->getInfo('modicons32');
    $GLOBALS['xoopsTpl']->assign('pathModIcon16', XOOPS_URL . '/modules/' . $moduleDirName . '/' . $pathModIcon16);
    $GLOBALS['xoopsTpl']->assign('pathModIcon32', $pathModIcon32);

• Read up
  Read up