Saito\Posting\UserPosting\UserPostingTrait

Complexity

Total Complexity

21

Size/Duplication

Total Lines	151
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	getCurrentUser()	0	4	1
A	setCurrentUser()	0	4	1
A	isAnsweringForbidden()	0	9	2
A	isBookmarked()	0	4	1
A	isEditingAllowed()	0	4	1
A	isEditingAsUserAllowed()	0	7	1
B	_isEditingAllowed()	0	28	8
A	isIgnored()	0	4	1
A	isUnread()	0	11	2
A	hasNewAnswers()	0	15	3

<?php

declare(strict_types=1);

/**
 * Saito - The Threaded Web Forum
 *
 * @copyright Copyright (c) the Saito Project Developers
 * @link https://github.com/Schlaefer/Saito
 * @license http://opensource.org/licenses/MIT
 */

namespace Saito\Posting\UserPosting;

use Cake\Core\Configure;
use Saito\Posting\Basic\BasicPostingInterface;
use Saito\User\CurrentUser\CurrentUserInterface;

/**
 * Implements UserPostingInterface
 */
trait UserPostingTrait
{
    /**
     * @var array
     */
    protected $_cache = [];

    /**
     * @var CurrentUserInterface
     */
    protected $_CurrentUser;

    /**
     * Get current-user.
     *
     * @return CurrentUserInterface
     */
    public function getCurrentUser()
    {
        return $this->_CurrentUser;
    }

    /**
     * Set current user.
     *
     * @param CurrentUserInterface $CurrentUser  current user
     * @return void
     */
    public function setCurrentUser($CurrentUser)
    {
        $this->_CurrentUser = $CurrentUser;
    }

    /**
     * {@inheritDoc}
     */
    public function isAnsweringForbidden()
    {
        if ($this->isLocked()) {

It seems like isLocked() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return 'locked';
        }
        $permission = $this->_CurrentUser->getCategories()->permission('answer', $this->get('category'));

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return !$permission;
    }

    /**
     * {@inheritDoc}
     */
    public function isBookmarked(): bool
    {
        return $this->_CurrentUser->hasBookmarked($this->get('id'));

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * {@inheritDoc}
     */
    public function isEditingAllowed(): bool
    {
        return $this->_isEditingAllowed($this, $this->_CurrentUser);
    }

    /**
     * {@inheritDoc}
     */
    public function isEditingAsUserAllowed(): bool
    {
        $MockedUser = clone $this->_CurrentUser;
        $MockedUser->set('user_type', 'user');

        return $this->_isEditingAllowed($this, $MockedUser);
    }

    /**
     * Check if editing on the posting is forbidden.
     *
     * @param BasicPostingInterface $posting The posting.
     * @param CurrentUserInterface $User The user.
     * @return bool|string string if a reason is available.
     */
    protected function _isEditingAllowed(BasicPostingInterface $posting, CurrentUserInterface $User)
    {
        if ($User->isLoggedIn() !== true) {
            return false;
        }

        if ($User->permission('saito.core.posting.edit.unrestricted')) {
            return true;
        }

        $editPeriod = Configure::read('Saito.Settings.edit_period') * 60;
        $timeLimit = $editPeriod + ($posting->get('time')->format('U'));
        $isOverTime = time() > $timeLimit;

        $isOwn = $User->getId() === $posting->get('user_id');

        if (!$isOverTime && $isOwn && !$this->isLocked()) {

It seems like isLocked() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return true;
        }

        if ($User->permission('saito.core.posting.edit.restricted')) {
            if ($posting->isPinned()) {
                return true;
            }
        }

        return false;
    }

    /**
     * {@inheritDoc}
     */
    public function isIgnored(): bool
    {
        return $this->_CurrentUser->ignores($this->get('user_id'));

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * {@inheritDoc}
     */
    public function isUnread(): bool
    {
        if (!isset($this->_cache['isUnread'])) {
            $id = $this->get('id');

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $time = $this->get('time');

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->_cache['isUnread'] = !$this->getCurrentUser()
                ->getReadPostings()->isRead($id, $time);
        }

        return $this->_cache['isUnread'];
    }

    /**
     * {@inheritDoc}
     */
    public function hasNewAnswers(): bool
    {
        if (!$this->isRoot()) {

It seems like isRoot() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            throw new \RuntimeException(
                'Posting with id ' . $this->get('id') . ' is no root posting.'

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            );
        }
        if (!$this->_CurrentUser->get('last_refresh')) {
            return false;
        }

        return $this->_CurrentUser->get('last_refresh_unix') < strtotime(
            $this->get('last_answer')

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        );
    }
}