newbbSynchronization()

Complexity

Conditions	7
Paths	36

Size

Total Lines	34
Code Lines	25

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    function newbbSynchronization($type = null)

The parameter $type is not used and could be removed.

    {
        $allTypes = [
            'category',
            'forum',
            'topic',
            'post',
            'report',
            'rate',
            'moderate',
            'readtopic',
            'readforum',
            'stats',
        ];
        $type     = [];
        $type     = empty($type) ? $allTypes : (is_array($type) ? $type : [$type]);

The condition is_array($type) is always true.

        foreach ($type as $item) {
            $handler = \XoopsModules\Newbb\Helper::getInstance()->getHandler($item);
            if ('stats' !== $item) {
                $handler->synchronization();

The method synchronization() does not exist on XoopsObjectHandler. It seems like you code against a sub-type of XoopsObjectHandler such as XoopsPersistableObjectHandler.

            } else {
                $handler->reset();

The method reset() does not exist on XoopsObjectHandler. It seems like you code against a sub-type of XoopsObjectHandler such as XoopsPersistableObjectHandler.

            }

            if (method_exists($handler, 'cleanExpires')) {
                $handler->cleanExpires();
            }
            if (method_exists($handler, 'clearGarbage')) {
                $handler->clearGarbage();
            }
            unset($handler);
        }

        return true;
    }