UserPostingTrait::_isEditingForbidden() - Code Metrics - Inspection of "Finish phpstanLevel3" - Schlaefer/Saito - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — develop (263ba5)

by Schlaefer

created 2019-07-27 11:59 UTC

UserPostingTrait::_isEditingForbidden() B

↳ Parent: UserPostingTrait

Complexity

Conditions	10
Paths	8

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	10
nc	8
nop	2
dl	0
loc	32
rs	7.6666
c	0
b	0
f	0

How to fix Complexity

<?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\PostingInterface;
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()) {
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            return 'locked';
        }
        $permission = $this->_CurrentUser->getCategories()->permission('answer', $this->get('category'));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        return !$permission;
    }

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

    /**
     * {@inheritDoc}
     */
    public function isEditingAsCurrentUserForbidden()
    {
        return $this->_isEditingForbidden($this, $this->_CurrentUser);
    }

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

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

    /**
     * Check if editing on the posting is forbidden.
     *
     * @param PostingInterface $posting The posting.
     * @param CurrentUserInterface $User The user.
     * @return bool|string string if a reason is available.
     */
    protected function _isEditingForbidden(PostingInterface $posting, CurrentUserInterface $User)
    {
        if ($User->isLoggedIn() !== true) {
            return true;
        } elseif ($User->permission('saito.core.posting.edit.unrestricted')) {
            return false;
        }

        $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 ($User->permission('saito.core.posting.edit.restricted')) {
            if ($isOwn && $isOverTime && !$posting->isPinned()) {
                return 'time';
            } else {
                return false;
            }
        }

        if (!$isOwn) {
            return 'user';
        } elseif ($isOverTime) {
            return 'time';
        } elseif ($this->isLocked()) {
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            return 'locked';
        }

        return false;
    }

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

    /**
     * {@inheritDoc}
     */
    public function isUnread(): bool
    {
        if (!isset($this->_cache['isUnread'])) {
            $id = $this->get('id');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $time = $this->get('time');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $this->_cache['isUnread'] = !$this->getCurrentUser()
                ->getReadPostings()->isRead($id, $time);
        }

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

    /**
     * {@inheritDoc}
     */
    public function hasNewAnswers(): bool
    {
        if (!$this->isRoot()) {
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            throw new \RuntimeException(
                'Posting with id ' . $this->get('id') . ' is no root posting.'
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            );
        }
        if (!$this->_CurrentUser->get('last_refresh')) {
            return false;
        }

        return $this->_CurrentUser->get('last_refresh_unix') < strtotime(
            $this->get('last_answer')
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        );
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/**
6			* Saito - The Threaded Web Forum
7			*
8			* @copyright Copyright (c) the Saito Project Developers
9			* @link https://github.com/Schlaefer/Saito
10			* @license http://opensource.org/licenses/MIT
11			*/
12
13			namespace Saito\Posting\UserPosting;
14
15			use Cake\Core\Configure;
16			use Saito\Posting\PostingInterface;
17			use Saito\User\CurrentUser\CurrentUserInterface;
18
19			/**
20			* Implements UserPostingInterface
21			*/
22			trait UserPostingTrait
23			{
24			/**
25			* @var array
26			*/
27			protected $_cache = [];
28
29			/**
30			* @var CurrentUserInterface
31			*/
32			protected $_CurrentUser;
33
34			/**
35			* Get current-user.
36			*
37			* @return CurrentUserInterface
38			*/
39			public function getCurrentUser()
40			{
41			return $this->_CurrentUser;
42			}
43
44			/**
45			* Set current user.
46			*
47			* @param CurrentUserInterface $CurrentUser current user
48			* @return void
49			*/
50			public function setCurrentUser($CurrentUser)
51			{
52			$this->_CurrentUser = $CurrentUser;
53			}
54
55			/**
56			* {@inheritDoc}
57			*/
58			public function isAnsweringForbidden()
59			{
60			if ($this->isLocked()) {
			0 ignored issues – show Bug introduced 2019-05-16 13:57 UTC by Report Bug Copy Issue Report It seems like `isLocked()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
61			return 'locked';
62			}
63			$permission = $this->_CurrentUser->getCategories()->permission('answer', $this->get('category'));
			0 ignored issues – show Bug introduced 2019-05-16 13:57 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
64
65			return !$permission;
66			}
67
68			/**
69			* {@inheritDoc}
70			*/
71			public function isBookmarked(): bool
72			{
73			return $this->_CurrentUser->hasBookmarked($this->get('id'));
			0 ignored issues – show Bug introduced 2019-05-16 13:57 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
74			}
75
76			/**
77			* {@inheritDoc}
78			*/
79			public function isEditingAsCurrentUserForbidden()
80			{
81			return $this->_isEditingForbidden($this, $this->_CurrentUser);
82			}
83
84			/**
85			* {@inheritDoc}
86			*/
87			public function isEditingWithRoleUserForbidden()
88			{
89			$MockedUser = clone $this->_CurrentUser;
90			$MockedUser->set('user_type', 'user');
91
92			return $this->_isEditingForbidden($this, $MockedUser);
93			}
94
95			/**
96			* Check if editing on the posting is forbidden.
97			*
98			* @param PostingInterface $posting The posting.
99			* @param CurrentUserInterface $User The user.
100			* @return bool\|string string if a reason is available.
101			*/
102			protected function _isEditingForbidden(PostingInterface $posting, CurrentUserInterface $User)
103			{
104			if ($User->isLoggedIn() !== true) {
105			return true;
106			} elseif ($User->permission('saito.core.posting.edit.unrestricted')) {
107			return false;
108			}
109
110			$editPeriod = Configure::read('Saito.Settings.edit_period') * 60;
111			$timeLimit = $editPeriod + ($posting->get('time')->format('U'));
112			$isOverTime = time() > $timeLimit;
113
114			$isOwn = $User->getId() === $posting->get('user_id');
115
116			if ($User->permission('saito.core.posting.edit.restricted')) {
117			if ($isOwn && $isOverTime && !$posting->isPinned()) {
118			return 'time';
119			} else {
120			return false;
121			}
122			}
123
124			if (!$isOwn) {
125			return 'user';
126			} elseif ($isOverTime) {
127			return 'time';
128			} elseif ($this->isLocked()) {
			0 ignored issues – show Bug introduced 2019-05-16 13:57 UTC by Report Bug Copy Issue Report It seems like `isLocked()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
129			return 'locked';
130			}
131
132			return false;
133			}
134
135			/**
136			* {@inheritDoc}
137			*/
138			public function isIgnored(): bool
139			{
140			return $this->_CurrentUser->ignores($this->get('user_id'));
			0 ignored issues – show Bug introduced 2019-05-16 13:57 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
141			}
142
143			/**
144			* {@inheritDoc}
145			*/
146			public function isUnread(): bool
147			{
148			if (!isset($this->_cache['isUnread'])) {
149			$id = $this->get('id');
			0 ignored issues – show Bug introduced 2019-05-16 13:57 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
150			$time = $this->get('time');
			0 ignored issues – show Bug introduced 2019-05-16 13:57 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
151			$this->_cache['isUnread'] = !$this->getCurrentUser()
152			->getReadPostings()->isRead($id, $time);
153			}
154
155			return $this->_cache['isUnread'];
156			}
157
158			/**
159			* {@inheritDoc}
160			*/
161			public function hasNewAnswers(): bool
162			{
163			if (!$this->isRoot()) {
			0 ignored issues – show Bug introduced 2019-05-16 13:57 UTC by Report Bug Copy Issue Report It seems like `isRoot()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
164			throw new \RuntimeException(
165			'Posting with id ' . $this->get('id') . ' is no root posting.'
			0 ignored issues – show Bug introduced 2019-05-16 13:57 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
166			);
167			}
168			if (!$this->_CurrentUser->get('last_refresh')) {
169			return false;
170			}
171
172			return $this->_CurrentUser->get('last_refresh_unix') < strtotime(
173			$this->get('last_answer')
			0 ignored issues – show Bug introduced 2019-05-16 13:57 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
174			);
175			}
176			}
177

Schlaefer / Saito

Branch — develop (263ba5)

UserPostingTrait::_isEditingForbidden() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like