regen_xoStats()

Complexity

Conditions	6
Paths	32

Size

Total Lines	78
Code Lines	55

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
function regen_xoStats()
{
    global $xoopsUser, $xoopsModule;

    $stats = array();

    /** @var \XoopsMemberHandler $memberHandler */
    $memberHandler = xoops_getHandler('member');

    // Getting Total Online Users
    /** @var \XoopsOnlineHandler $online_handler */
    $online_handler = xoops_getHandler('online');
    // set gc probabillity to 10% for now..
    if (mt_rand(1, 100) < 11) {
        $online_handler->gc(300);
    }
    if (is_object($xoopsUser)) {
        $uid   = $xoopsUser->getVar('uid');
        $uname = $xoopsUser->getVar('uname');
    } else {
        $uid   = 0;
        $uname = '';
    }

    $requestIp = IPAddress::fromRequest()->asReadable();
    $requestIp = (false === $requestIp) ? '0.0.0.0' : $requestIp;

The condition false === $requestIp is always false.

    if (is_object($xoopsModule)) {
        $online_handler->write($uid, $uname, time(), $xoopsModule->getVar('mid'), $requestIp);
    } else {
        $online_handler->write($uid, $uname, time(), 0, $requestIp);
    }
    $onlines = $online_handler->getAll();
    if (empty($onlines)) {
        $stats['totalonline'] = 0;
    } else {
        $totalonline   = count($onlines);
        $stats['totalonline'] = $totalonline;
    }

    //Getting Total Registered Users
    $level_criteria = new \Criteria('level', 0, '>');
    $criteria = new \CriteriaCompo($level_criteria);
    $criteria24 = new \CriteriaCompo($level_criteria);
    $criteria48 = new \CriteriaCompo($level_criteria);
    $totaluser = $memberHandler->getUserCount($level_criteria);
    $stats['totaluser'] = $totaluser;

    //Getting User Registration Statistics
    $users_reg_24 = $memberHandler->getUserCount($criteria24->add(new \Criteria('user_regdate', (mktime(0, 0, 0) - (24 * 3600)), '>=')), 'AND');

The call to XoopsMemberHandler::getUserCount() has too many arguments starting with 'AND'.

    $users_reg_48 = $memberHandler->getUserCount($criteria48->add(new \Criteria('user_regdate', (mktime(0, 0, 0) - (48 * 3600)), '>=')), 'AND');
    $todayregister       = $users_reg_24;
    $yesterdayregister   = $users_reg_48;
    $stats['totalregisteredtoday'] = $todayregister;
    $stats['totalregisteredyesterday'] = $yesterdayregister;

    // Getting Last Registered Member
    $limit        = 1;
    $criteria->setOrder('DESC');
    $criteria->setSort('user_regdate');
    $criteria->setLimit($limit);
    $lastmembers = $memberHandler->getUsers($criteria);
    $lastusername = $lastmembers[0]->getVar('uname');
    $lastrealname = $lastmembers[0]->getVar('name');
    $latestuid = $lastmembers[0]->getVar('uid');
    $latestmembername = $lastrealname;
    $latestmemberuname = $lastusername;
    $stats['latestmembername'] = $latestmembername;
    $stats['latestmemberuname'] = $latestmemberuname;
    $stats['latestuid'] = $latestuid;

    //Total Post Count
    $sql = 'SELECT SUM(posts) AS totalpost FROM ' . $GLOBALS['xoopsDB']->prefix('users') . ' WHERE level > 0';
    $result = $GLOBALS['xoopsDB']->query($sql);
    $myrow = $GLOBALS['xoopsDB']->fetchArray($result);
    $totalpost = $myrow['totalpost'];
    $stats['totalpost'] = $totalpost;

    return $stats;
}