b_waiting_tdmdownloads()

Complexity

Conditions	4
Paths	8

Size

Total Lines	34
Code Lines	25

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
function b_waiting_tdmdownloads()
{
    /** @var \XoopsMySQLDatabase $xoopsDB */
    $xoopsDB       = \XoopsDatabaseFactory::getDatabaseConnection();
    $moduleDirName = basename(dirname(__DIR__, 2));
    $ret           = [];
    // TDMdownloads waiting
    $block  = [];
    $result = $xoopsDB->query('SELECT COUNT(*) FROM ' . $xoopsDB->prefix('tdmdownloads_downloads') . ' WHERE status=0');
    if ($result) {
        $block['adminlink'] = XOOPS_URL . "/modules/$moduleDirName/admin/downloads.php?op=liste&statut_display=0";
        [$block['pendingnum']] = $xoopsDB->fetchRow($result);

It seems like $result can also be of type true; however, parameter $result of XoopsMySQLDatabase::fetchRow() does only seem to accept mysqli_result, maybe add an additional type check?

        $block['lang_linkname'] = _PI_WAITING_WAITINGS;

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

    }
    $ret[] = $block;
    // TDMDownloads broken
    $block  = [];
    $result = $xoopsDB->query('SELECT COUNT(*) FROM ' . $xoopsDB->prefix('tdmdownloads_broken'));
    if ($result) {
        $block['adminlink'] = XOOPS_URL . "/modules/$moduleDirName/admin/broken.php";
        [$block['pendingnum']] = $xoopsDB->fetchRow($result);
        $block['lang_linkname'] = _PI_WAITING_BROKENS;

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

    }
    $ret[] = $block;
    // TDMDownloads modreq
    $block  = [];
    $result = $xoopsDB->query('SELECT COUNT(*) FROM ' . $xoopsDB->prefix('tdmdownloads_mod'));
    if ($result) {
        $block['adminlink'] = XOOPS_URL . "/modules/$moduleDirName/admin/modified.php";
        [$block['pendingnum']] = $xoopsDB->fetchRow($result);
        $block['lang_linkname'] = _PI_WAITING_MODREQS;

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

    }
    $ret[] = $block;
    return $ret;
}