mambax7 / newbb5

class/ReadforumHandler.php

<?php

namespace XoopsModules\Newbb;

//
//  ------------------------------------------------------------------------ //
//                XOOPS - PHP Content Management System                      //
//                  Copyright (c) 2000-2020 XOOPS.org                        //
//                       <https://xoops.org>                             //
//  ------------------------------------------------------------------------ //
//  This program is free software; you can redistribute it and/or modify     //
//  it under the terms of the GNU General Public License as published by     //
//  the Free Software Foundation; either version 2 of the License, or        //
//  (at your option) any later version.                                      //
//                                                                           //
//  You may not change or alter any portion of this comment or credits       //
//  of supporting developers from this source code or any supporting         //
//  source code which is considered copyrighted (c) material of the          //
//  original comment or credit authors.                                      //
//                                                                           //
//  This program is distributed in the hope that it will be useful,          //
//  but WITHOUT ANY WARRANTY; without even the implied warranty of           //
//  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the            //
//  GNU General Public License for more details.                             //
//                                                                           //
//  You should have received a copy of the GNU General Public License        //
//  along with this program; if not, write to the Free Software              //
//  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA //
//  ------------------------------------------------------------------------ //
//  Author: phppp (D.J., infomax@gmail.com)                                  //
//  URL: https://xoops.org                                                    //
//  Project: Article Project                                                 //
//  ------------------------------------------------------------------------ //

use XoopsModules\Newbb;

require_once __DIR__ . '/Read.php';

/**
 * A handler for read/unread handling
 *
 * @package       newbb
 *
 * @author        D.J. (phppp, http://xoopsforge.com)
 * @copyright     copyright (c) 2005 XOOPS.org
 */

/**
 * Class ReadforumHandler
 */
class ReadforumHandler extends Newbb\ReadHandler
{
    /**
     * @param \XoopsDatabase|null $db
     */
    public function __construct(\XoopsDatabase $db = null)
    {
        parent::__construct($db, 'forum');

It seems like $db can also be of type null; however, parameter $db of XoopsModules\Newbb\ReadHandler::__construct() does only seem to accept XoopsDatabase, maybe add an additional type check?

    }

    /**
     * clean orphan items from database
     *
     * @param string $table_link
     * @param string $field_link
     * @param string $field_object
     * @return bool   true on success
     */
    public function cleanOrphan($table_link = '', $field_link = '', $field_object = '') //cleanOrphan()
    {
        parent::cleanOrphan($this->db->prefix('newbb_posts'), 'post_id');

        return parent::cleanOrphan($this->db->prefix('newbb_forums'), 'forum_id', 'read_item');
    }

    /**
     * @param int  $status
     * @param null $uid

Are you sure the doc-type for parameter $uid is correct as it would always require null to be passed?

     * @return bool
     */
    public function setReadItems($status = 0, $uid = null)
    {
        if (empty($this->mode)) {
            return true;
        }

        if (1 == $this->mode) {
            return $this->setReadItemsCookie($status);

The call to XoopsModules\Newbb\ReadforumHandler::setReadItemsCookie() has too few arguments starting with items.

        }

        return $this->setReadItemsDb($status, $uid);
    }

    /**
     * @param $status
     * @param $items
     * @return bool
     */
    public function setReadItemsCookie($status, $items)

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

    {
        $cookie_name = 'LF';
        $items       = [];
        if (!empty($status)) {
            /** @var Newbb\ForumHandler $itemHandler */
            $itemHandler = \XoopsModules\Newbb\Helper::getInstance()->getHandler('Forum');
            $items_id    = $itemHandler->getIds();
            foreach ($items_id as $key) {
                $items[$key] = \time();
            }
        }
        \newbbSetCookie($cookie_name, $items);

        return true;
    }

    /**
     * @param $status
     * @param $uid
     * @return bool
     */
    public function setReadItemsDb($status, $uid)
    {
        if (empty($uid)) {
            if (\is_object($GLOBALS['xoopsUser'])) {
                $uid = $GLOBALS['xoopsUser']->getVar('uid');
            } else {
                return false;
            }
        }
        if (empty($status)) {
            $this->deleteAll(new \Criteria('uid', $uid));

            return true;
        }

        /** @var Newbb\ForumHandler $itemHandler */
        $itemHandler = \XoopsModules\Newbb\Helper::getInstance()->getHandler('Forum');
        $itemsObject = $itemHandler->getAll(null, ['forum_last_post_id']);
        foreach (\array_keys($itemsObject) as $key) {
            $this->setReadDb($key, $itemsObject[$key]->getVar('forum_last_post_id'), $uid);
        }
        unset($itemsObject);

        return true;
    }

    /**
     * @return bool|void
     */
    public function synchronization()
    {
        //        return;
    }
}