include/functions.php ➔ news_updateCache()

Complexity

Conditions	5
Paths	5

Size

Total Lines	26
Code Lines	17

Duplication

Lines	26
Ratio	100 %

Importance

Changes

0

<?php
function news_updateCache()

This function seems to be duplicated in your project.

{
    global $xoopsModule;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

    $folder  = $xoopsModule->getVar('dirname');
    $tpllist = [];

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

    require_once XOOPS_ROOT_PATH . '/class/xoopsblock.php';
    require_once XOOPS_ROOT_PATH . '/class/template.php';
    $tplfileHandler = xoops_getHandler('tplfile');
    $tpllist        = $tplfileHandler->find(null, null, null, $folder);
    $xoopsTpl       = new XoopsTpl();

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

    xoops_template_clear_module_cache($xoopsModule->getVar('mid')); // Clear module's blocks cache

    // Remove cache for each page.
    foreach ($tpllist as $onetemplate) {
        if ('module' === $onetemplate->getVar('tpl_type')) {
            // Note, I've been testing all the other methods (like the one of Smarty) and none of them run, that's why I have used this code
            $files_del = [];

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

            $files_del = glob(XOOPS_CACHE_PATH . '/*' . $onetemplate->getVar('tpl_file') . '*');
            if (count($files_del) > 0) {
                foreach ($files_del as $one_file) {
                    unlink($one_file);
                }
            }
        }
    }
}