XoopsModules\Newbb\KarmaHandler

Complexity

Total Complexity

7

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	writeUserKarma()	0	2	1
A	readUserKarma()	0	2	1
A	updateUserKarma()	0	2	1
A	calculateUserKarma()	0	9	2
A	getUserKarma()	0	5	2

<?php

namespace XoopsModules\Newbb;

/**
 * NewBB 5.0x,  the forum module for XOOPS project
 *
 * @copyright      XOOPS Project (https://xoops.org)
 * @license        GNU GPL 2 or later (https://www.gnu.org/licenses/gpl-2.0.html)
 * @author         Taiwen Jiang (phppp or D.J.) <phppp@users.sourceforge.net>
 * @since          4.00
 * @package        module::newbb
 */
class KarmaHandler
{
    /**
     * @param null $user

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

     * @return int
     */
    public function getUserKarma($user = null)
    {
        $user = (null === $user) ? $GLOBALS['xoopsUser'] : $user;

The condition null === $user is always true.

        return $this->calculateUserKarma($user);
    }

    /**
     * Placeholder for calculating user karma
     * @param \XoopsUser $user
     * @return int
     */
    public function calculateUserKarma($user)
    {
        if (!\is_object($user)) {
            $user_karma = 0;
        } else {
            $user_karma = $user->getVar('posts') * 50;
        }

        return $user_karma;
    }

    public function updateUserKarma()
    {
    }

    public function writeUserKarma()
    {
    }

    public function readUserKarma()
    {
    }
}