wggithub_search()

Complexity

Conditions	12
Paths	192

Size

Total Lines	58
Code Lines	41

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php

function wggithub_search($queryarray, $andor, $limit, $offset, $userid)

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

{

    $ret = [];

    $helper = \XoopsModules\Wggithub\Helper::getInstance();

    $repositoriesHandler = $helper->getHandler('Repositories');

    $directoriesHandler  = $helper->getHandler('Directories');


    $directoriesAll = $directoriesHandler->getAll();

    foreach (\array_keys($directoriesAll) as $i) {

        $directories[$directoriesAll[$i]->getVar('dir_name')] = $directoriesAll[$i]->getVar('dir_id');

    }

    unset ($directoriesAll);


    // search in table repositories

    // search keywords

    $elementCount = 0;

    if (\is_array($queryarray)) {

        $elementCount = \count($queryarray);

    }

    if ($elementCount > 0) {

        $crKeywords = new \CriteriaCompo();

The type CriteriaCompo was not found. Maybe you did not declare it correctly or list all dependencies?

        for ($i = 0; $i  <  $elementCount; $i++) {

            $crKeyword = new \CriteriaCompo();

            unset($crKeyword);

        }

    }

    // search user(s)

    if ($userid && \is_array($userid)) {

        $userid = array_map('intval', $userid);

        $crUser = new \CriteriaCompo();

        $crUser->add(new \Criteria('repo_submitter', '(' . \implode(',', $userid) . ')', 'IN'), 'OR');

The type Criteria was not found. Maybe you did not declare it correctly or list all dependencies?

    } elseif (is_numeric($userid) && $userid > 0) {

        $crUser = new \CriteriaCompo();

        $crUser->add(new \Criteria('repo_submitter', $userid), 'OR');

    }

    $crSearch = new \CriteriaCompo();

    if (isset($crKeywords)) {

        $crSearch->add($crKeywords, 'AND');

    }

    if (isset($crUser)) {

        $crSearch->add($crUser, 'AND');

    }

    $crSearch->setStart($offset);

    $crSearch->setLimit($limit);

    $crSearch->setSort('repo_datecreated');

    $crSearch->setOrder('DESC');

    $repositoriesAll = $repositoriesHandler->getAll($crSearch);

    foreach (\array_keys($repositoriesAll) as $i) {

        $ret[] = [

            'image'  => 'assets/icons/16/github.png',

            'link'   => 'index.php?op=show&menu=' . $directories[$repositoriesAll[$i]->getVar('repo_user')],

The variable $directories seems to be defined by a foreach iteration on line 44. Are you sure the iterator is never empty, otherwise this variable is not defined?

            'title'  => $repositoriesAll[$i]->getVar('repo_name'),

            'time'   => $repositoriesAll[$i]->getVar('repo_datecreated')

        ];

    }

    unset($crKeywords, $crKeyword, $crUser, $crSearch, $repositoriesAll);


    return $ret;