Method getAllRejected has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function getAllRejected($limit = 0, $start = 0, $categoryid = -1, $sort = 'datesub', $order = 'DESC', $notNullFields = '', $asObject = true, $idKey = 'none')

Method getAllOffline has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function getAllOffline($limit = 0, $start = 0, $categoryid = -1, $sort = 'datesub', $order = 'DESC', $notNullFields = '', $asObject = true, $idKey = 'none')

Method getAllSubmitted has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function getAllSubmitted($limit = 0, $start = 0, $categoryid = -1, $sort = 'datesub', $order = 'DESC', $notNullFields = null, $asObject = true, $idKey = 'none')

Method isBlockCloned has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function isBlockCloned(int $bid, string $bside, string $bweight, string $bvisible, string $bcachetime, ?array $bmodule, ?array $options, ?array $groups): void

Method getAllExpired has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function getAllExpired($limit = 0, $start = 0, $categoryid = -1, $sort = 'datesub', $order = 'DESC', $notNullFields = null, $asObject = true, $idKey = 'none')

Function apply has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    apply: function () {
        for (h = 0; sheet = Behaviour.list[h]; h++) {
            for (selector in sheet) {
                list = document.getElementsBySelector(selector);

• Read up
  Read up

Function calcDocRoot has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    protected function calcDocRoot()
    {
        $docRoot = @$_SERVER['DOCUMENT_ROOT'];
        if (defined('LOCAL_FILE_BASE_DIRECTORY')) {
            $docRoot = LOCAL_FILE_BASE_DIRECTORY;

• Read up
  Read up

Function recurseCopy has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    public static function recurseCopy(string $src, string $dst): void
    {
        $dir = \opendir($src);
        if (!\mkdir($dst) && !\is_dir($dst)) {
            throw new \RuntimeException(\sprintf('Directory "%s" was not created', $dst));

• Read up
  Read up

Function getItemsCriteria has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    private function getItemsCriteria($categoryid = -1, $status = '', $notNullFields = null, $criteriaPermissions = null)
    {
        //        $notNullFields = (null !== $notNullFields) ?: '';
        //        global $publisherIsAdmin;
        //        $ret = 0;

• Read up
  Read up

Function rcopy has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    public static function rcopy($src, $dest)
    {
        // Only continue if user is a 'global' Admin
        if (!($GLOBALS['xoopsUser'] instanceof \XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
            return false;

• Read up
  Read up

Function insert has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    public function insert(\XoopsObject $obj, $force = false)// insert($obj, $force = false)
    {
        // Make sure object is of correct type
        if (0 != \strcasecmp($this->className, \get_class($obj))) {
            return false;

• Read up
  Read up

Function rmove has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    public static function rmove($src, $dest)
    {
        // Only continue if user is a 'global' Admin
        if (!($GLOBALS['xoopsUser'] instanceof \XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
            return false;

• Read up
  Read up

Function store has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    public function store($force = true)
    {
        $isNew = $this->isNew();
        if (!$this->helper->getHandler('Item')
                          ->insert($this, $force)) {

• Read up
  Read up

Method setOrder has 7 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function setOrder(string $bid, string $title, string $weight, string $visible, string $side, string $bcachetime, ?array $bmodule = null): void

Method getAllFiles has 7 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function getAllFiles($itemId = 0, $status = -1, $limit = 0, $start = 0, $sort = 'datesub', $order = 'DESC', $category = [])

Avoid deeply nested control flow statements.

Open

                if (file_exists($filename)) {
                    if (copy($filename, $GLOBALS['xoops']->path('uploads/publisher/' . $arrFile['filerealname']))) {
                        /** @var File $fileObj */
                        $fileObj = $helper->getHandler('File')
                                          ->create();

Avoid deeply nested control flow statements.

Open

                if (file_exists($pagewrap_filename)) {
                    if (copy($pagewrap_filename, PUBLISHER_UPLOAD_PATH . '/content/' . $arrArticle['htmlpage'])) {
                        $itemObj->setVar('body', '[pagewrap=' . $arrArticle['htmlpage'] . ']');
                        echo sprintf('&nbsp;&nbsp;&nbsp;&nbsp;' . _AM_PUBLISHER_IMPORT_ARTICLE_WRAP, $arrArticle['htmlpage']) . '<br>';
                    }

Avoid deeply nested control flow statements.

Open

                    if ($fileObj->storeUpload('item_upload_file', $allowedMimetypes, $errors)) {
                        unlink($oldfile);
                    }

Avoid deeply nested control flow statements.

Open

                            if (is_file($templateFolder . $v)) {
                                unlink($templateFolder . $v);
                            }

Avoid deeply nested control flow statements.

Open

                if (file_exists($filename)) {
                    if (copy($filename, $GLOBALS['xoops']->path('uploads/publisher/' . $arrFile['filerealname']))) {
                        /** @var File $fileObj */
                        $fileObj = $helper->getHandler('File')
                                          ->create();