App\Event\Statistics

Complexity

Total Complexity

15

Size/Duplication

Total Lines	151
Duplicated Lines	25.83 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	1
Bugs	1	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	implementedEvents()	0	10	1
A	newArticleStats()	13	13	2
A	newArticleCommentStats()	13	13	2
A	newArticleLikeStats()	13	13	2
A	newUserStats()	0	19	2
A	updateGroupStats()	0	12	2
A	_writeCache()	0	13	4

<?php
namespace App\Event;

use Cake\Cache\Cache;
use Cake\Event\Event;
use Cake\Event\EventListenerInterface;
use Cake\I18n\Number;
use Cake\ORM\TableRegistry;

class Statistics implements EventListenerInterface
{

    /**
     * ImplementedEvents method.
     *
     * @return array
     */
    public function implementedEvents()
    {
        return [
            'Model.Users.register' => 'newUserStats',
            'Model.Groups.update' => 'updateGroupStats',
            'Model.BlogArticles.new' => 'newArticleStats',
            'Model.BlogArticlesLikes.new' => 'newArticleLikeStats',
            'Model.BlogArticlesComments.new' => 'newArticleCommentStats'
        ];
    }

    /**
     * Count the articles and write it in the Cache.
     *
     * @param \Cake\Event\Event $event The event that was fired.
     *
     * @return array|false
     */
    public function newArticleStats(Event $event)

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

This method seems to be duplicated in your project.

    {
        $this->BlogArticles = TableRegistry::get('BlogArticles');

The property BlogArticles does not exist. Did you maybe forget to declare it?

        $articles = $this->BlogArticles->find()->count();
        $articles = Number::format($articles);

        if ($this->_writeCache($articles, 'Articles')) {
            return $articles;
        }

        return false;
    }

    /**
     * Count the article's comments and write it in the Cache.
     *
     * @param \Cake\Event\Event $event The event that was fired.
     *
     * @return array|false
     */
    public function newArticleCommentStats(Event $event)

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

This method seems to be duplicated in your project.

    {
        $this->BlogArticlesComments = TableRegistry::get('BlogArticlesComments');

The property BlogArticlesComments does not seem to exist. Did you mean BlogArticles?

        $comments = $this->BlogArticlesComments->find()->count();

The property BlogArticlesComments does not seem to exist. Did you mean BlogArticles?

        $comments = Number::format($comments);

        if ($this->_writeCache($comments, 'ArticlesComments')) {
            return $comments;
        }

        return false;
    }

    /**
     * Count the article's likes and write it in the Cache.
     *
     * @param \Cake\Event\Event $event The event that was fired.
     *
     * @return array|false
     */
    public function newArticleLikeStats(Event $event)

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

This method seems to be duplicated in your project.

    {
        $this->BlogArticlesLikes = TableRegistry::get('BlogArticlesLikes');

The property BlogArticlesLikes does not seem to exist. Did you mean BlogArticles?

        $likes = $this->BlogArticlesLikes->find()->count();

The property BlogArticlesLikes does not seem to exist. Did you mean BlogArticles?

        $likes = Number::format($likes);

        if ($this->_writeCache($likes, 'ArticlesLikes')) {
            return $likes;
        }

        return false;
    }

    /**
     * Re-count the number of user and find the latest user and write it in the Cache.
     *
     * @param \Cake\Event\Event $event The event that was fired.
     *
     * @return array|false
     */
    public function newUserStats(Event $event)

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

    {
        $this->Users = TableRegistry::get('Users');

The property Users does not exist. Did you maybe forget to declare it?

        $totalUsers = $this->Users->find()->count();
        $totalUsers = Number::format($totalUsers);

        $lastRegistered = $this->Users->find('short')->order(['Users.created' => 'DESC'])->first();

        $data = [];
        $data['TotalUsers'] = $totalUsers;
        $data['LastRegistered'] = $lastRegistered;

        if ($this->_writeCache($data, 'Users')) {

$data is of type array<string,*,{"LastRegistered":"*"}>, but the function expects a integer|object|string.

            return $data;
        }

        return false;
    }

    /**
     * Get the Groups and write it in the Cache.
     *
     * @param \Cake\Event\Event $event The event that was fired.
     *
     * @return array|false
     */
    public function updateGroupStats(Event $event)

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

    {
        $this->Groups = TableRegistry::get('Groups');

The property Groups does not exist. Did you maybe forget to declare it?

        $groups = $this->Groups->find('translations')->order(['Groups.id' => 'DESC'])->toArray();

        if ($this->_writeCache($groups, 'Groups')) {

$groups is of type array, but the function expects a integer|object|string.

            return $groups;
        }

        return false;
    }

    /**
     * Write the data into the Cache with the passed key.
     *
     * @param int|object|string $data The data to save in the Cache.
     * @param string $key The key to save the data.
     *
     * @return bool
     */
    protected function _writeCache($data, $key)
    {
        if (empty($data) || empty($key)) {
            return true;
        }

        $result = Cache::write($key, $data, 'statistics');
        if ($result) {
            return true;
        }

        return false;
    }
}