scrutinizer

Author: mambax7

class/OnlineHandler.php

@@ -257,6 +257,7 @@ public function write($uid, $uname, $time, $forum_id, $ip, $topic_id)
             return false;
         }
 
+        /** @var \XoopsOnlineHandler $xoopsOnlineHandler */
         $xoopsOnlineHandler = \xoops_getHandler('online');
         $xoopsOnlineTable    = $xoopsOnlineHandler->table;
 
@@ -311,7 +312,7 @@ public function getAll($criteria = null)
         $ret   = [];
         $limit = $start = 0;
         $sql   = 'SELECT * FROM ' . $this->db->prefix('newbb_online');
-        if (\is_object($criteria) && is_subclass_of($criteria,  \CriteriaElement::class)) {
+        if (\is_object($criteria) && ($criteria instanceof \CriteriaCompo || $criteria instanceof \Criteria)) {
             $sql   .= ' ' . $criteria->renderWhere();
             $limit = $criteria->getLimit();
             $start = $criteria->getStart();