XoopsModules25x / xfguestbook

include/waiting.plugin.php

<?php

/**
 * @return array
 */
function b_waiting_xfguestbook()
{
    /** @var \XoopsMySQLDatabase $xoopsDB */
    $xoopsDB = \XoopsDatabaseFactory::getDatabaseConnection();
    $block   = [];

    $result = $xoopsDB->query('SELECT count(*) FROM ' . $xoopsDB->prefix('xfguestbook_msg') . ' WHERE moderate = 1');
    if ($result) {
        $block['adminlink'] = XOOPS_URL . '/modules/xfguestbook/admin/index.php?action=waiting';
        [$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;
    }

    return $block;
}