App\Controller\BlogController

Complexity

Total Complexity

45

Size/Duplication

Total Lines	798
Duplicated Lines	24.56 %

Coupling/Cohesion

Components	1
Dependencies	12

Importance

Changes

0

14 Methods

Rating	Name	Duplication	Size	Complexity
A	initialize()	0	6	1
A	beforeFilter()	0	6	1
B	index()	26	26	1
B	category()	5	46	2
B	article()	0	145	6
A	quote()	29	56	3
A	go()	48	48	3
B	archive()	28	28	1
B	search()	10	43	3
B	articleLike()	16	77	5
B	articleUnlike()	8	52	4
B	deleteComment()	5	51	5
B	getEditComment()	16	54	5
B	editComment()	5	48	5

<?php
namespace App\Controller;

use App\Event\Badges;
use App\Event\Statistics;
use Cake\Core\Configure;
use Cake\Event\Event;
use Cake\Network\Exception\NotFoundException;
use Cake\Routing\Router;

class BlogController extends AppController
{

    /**
     * Initialization hook method.
     *
     * @return void
     */
    public function initialize()
    {
        parent::initialize();

        $this->loadComponent('RequestHandler');
    }

    /**
     * BeforeFilter handle.
     *
     * @param Event $event The beforeFilter event that was fired.
     *
     * @return void
     */
    public function beforeFilter(Event $event)
    {
        parent::beforeFilter($event);

        $this->Auth->allow(['index', 'category', 'article', 'go', 'archive', 'search']);
    }

    /**
     * Display all Articles.
     *
     * @return void
     */
    public function index()

This method seems to be duplicated in your project.

    {
        $this->loadModel('BlogArticles');
        $this->paginate = [
            'maxLimit' => Configure::read('Blog.article_per_page')
        ];

        $articles = $this->BlogArticles

The property BlogArticles does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'BlogCategories',
                'Users' => function ($q) {
                    return $q->find('short');
                }
            ])
            ->order([
                'BlogArticles.created' => 'desc'
            ])
            ->where([
                'BlogArticles.is_display' => 1
            ]);

        $articles = $this->paginate($articles);

        $this->set(compact('articles'));
    }

    /**
     * Display a specific category with all its articles.
     *
     * @return \Cake\Network\Response|void
     */
    public function category()
    {
        $this->loadModel('BlogCategories');

        $category = $this->BlogCategories

The property BlogCategories does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'BlogCategories.id' => $this->request->id
            ])
            ->contain([
                'BlogArticles'
            ])
            ->first();

        //Check if the category is found.
        if (empty($category)) {

This code seems to be duplicated across your project.

            $this->Flash->error(__('This category doesn\'t exist or has been deleted.'));

            return $this->redirect(['action' => 'index']);
        }

        //Paginate all Articles.
        $this->loadModel('BlogArticles');
        $this->paginate = [
            'maxLimit' => Configure::read('Blog.article_per_page')
        ];

        $articles = $this->BlogArticles

The property BlogArticles does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Users' => function ($q) {
                    return $q->find('short');
                }
            ])
            ->where([
                'BlogArticles.category_id' => $category->id,
                'BlogArticles.is_display' => 1
            ])
            ->order([
                'BlogArticles.created' => 'desc'
            ]);

        $articles = $this->paginate($articles);

        $this->set(compact('category', 'articles'));
    }

    /**
     * Display a specific article.
     *
     * @return \Cake\Network\Response|void
     */
    public function article()
    {
        $this->loadModel('BlogArticles');

        $article = $this->BlogArticles

The property BlogArticles does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'BlogArticles.id' => $this->request->id,
                'BlogArticles.is_display' => 1
            ])
            ->contain([
                'BlogCategories',
                'BlogAttachments',
                'Users' => function ($q) {
                    return $q->find('full');
                },
                'Polls',
                'Polls.PollsAnswers',
                'Polls.PollsAnswers.Polls' => function ($q) {
                    return $q->select(['id', 'user_count']);
                },
                'Polls.PollsUsers'
            ])
            ->first();

        $this->loadModel('PollsUsers');
        $hasVoted = $this->PollsUsers

The property PollsUsers does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Polls' => function ($q) {
                    return $q->select(['id']);
                },
                'PollsAnswers'
            ])
            ->where([
                'PollsUsers.user_id' => 1, //$this->Auth->user('id'),

78% of this comment could be valid code. Did you maybe forget this after debugging?

                'Polls.id' => $article->poll->id
            ])
            ->first();

        //Check if the article is found.
        if (is_null($article)) {
            $this->Flash->error(__('This article doesn\'t exist or has been deleted.'));

            return $this->redirect(['action' => 'index']);
        }

        $this->loadModel('BlogArticlesComments');

        //A comment has been posted.
        if ($this->request->is('post')) {
            //Check if the user is connected.
            if (!$this->Auth->user()) {
                return $this->Flash->error(__('You must be connected to post a comment.'));
            }

            $this->request->data['article_id'] = $article->id;
            $this->request->data['user_id'] = $this->Auth->user('id');

            $newComment = $this->BlogArticlesComments->newEntity($this->request->data, ['validate' => 'create']);

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            //Attach Event.
            $this->BlogArticlesComments->eventManager()->attach(new Badges($this));

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            if ($insertComment = $this->BlogArticlesComments->save($newComment)) {

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

                $this->eventManager()->attach(new Statistics());

new \App\Event\Statistics() is of type object<App\Event\Statistics>, but the function expects a callable.

The method Cake\Event\EventManager::attach() has been deprecated with message: 3.0.0 Use on() instead.

                $event = new Event('Model.BlogArticlesComments.new');
                $this->eventManager()->dispatch($event);

                $this->Flash->success(__('Your comment has been posted successfully !'));
                //Redirect the user to the last page of the article.
                $this->redirect([
                    'action' => 'go',
                    $insertComment->id
                ]);
            }
        }

        //Paginate all comments related to the article.
        $this->paginate = [
            'maxLimit' => Configure::read('Blog.comment_per_page')
        ];

        $comments = $this->BlogArticlesComments

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'BlogArticlesComments.article_id' => $article->id
            ])
            ->contain([
                'Users' => function ($q) {
                    return $q->find('medium');
                }
            ])
            ->order([
                'BlogArticlesComments.created' => 'asc'
            ]);

        $comments = $this->paginate($comments);

        //Select the like for the current auth user.
        $this->loadModel('BlogArticlesLikes');
        $like = $this->BlogArticlesLikes

The property BlogArticlesLikes does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'user_id' => ($this->Auth->user()) ? $this->Auth->user('id') : null,
                'article_id' => $article->id
            ])
            ->first();

        //Build the newEntity for the comment form.
        $formComments = $this->BlogArticlesComments->newEntity();

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

        //Search related articles
        $keywords = preg_split("/([\s,\W])+/", $article->title);

        $articles = $this->BlogArticles

The property BlogArticles does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'BlogCategories',
            ])
            ->where([
                'BlogArticles.is_display' => 1,
                'BlogArticles.id !=' => $article->id
            ])
            ->andWhere([
                'BlogArticles.title RLIKE' => rtrim(implode('|', $keywords), '|')
            ]);

        //Current user.
        $this->loadModel('Users');
        $currentUser = $this->Users

The property Users does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Groups' => function ($q) {
                    return $q->select(['id', 'is_staff']);
                }
            ])
            ->where([
                'Users.id' => $this->Auth->user('id')
            ])
            ->select(['id', 'group_id'])
            ->first();

        $this->set(compact('article', 'formComments', 'comments', 'like', 'articles', 'currentUser', 'hasVoted'));
    }

    /**
     * Quote a message.
     *
     * @param int $articleId Id of the article where is the message to quote.
     * @param int $commentId Id of the message to quote.
     *
     * @throws \Cake\Network\Exception\NotFoundException
     *
     * @return mixed
     */
    public function quote($articleId = null, $commentId = null)
    {
        if (!$this->request->is('ajax')) {
            throw new NotFoundException();
        }

        $this->loadModel('BlogArticlesComments');

        $comment = $this->BlogArticlesComments

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'BlogArticlesComments.article_id' => $articleId,
                'BlogArticlesComments.id' => $commentId
            ])
            ->contain([
                'Users' => function ($q) {
                        return $q->find('short');
                }
            ])
            ->first();

        $json = [];

        if (!is_null($comment)) {

This code seems to be duplicated across your project.

            $comment->toArray();

            $url = Router::url(['action' => 'go', $comment->id]);
            $text = __("has said :");

            //Build the quote.
            $json['comment'] = <<<EOT
<div>
     <div>
        <a href="{$url}">
            <strong>{$comment->user->full_name} {$text}</strong>
        </a>
    </div>
    <blockquote>
        $comment->content
    </blockquote>
</div><p>&nbsp;</p><p>&nbsp;</p>
EOT;

            $json['error'] = false;

            $this->set(compact('json'));
        } else {
            $json['comment'] = __("This comment doesn't exist.");
            $json['error'] = true;

            $this->set(compact('json'));
        }

        //Send response in JSON.
        $this->set('_serialize', 'json');
    }

    /**
     * Redirect an user to an article, page and comment.
     *
     * @param int $commentId Id of the comment.
     *
     * @return \Cake\Network\Response
     */
    public function go($commentId = null)

This method seems to be duplicated in your project.

    {
        $this->loadModel('BlogArticlesComments');

        $comment = $this->BlogArticlesComments

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'BlogArticles'
            ])
            ->where([
                'BlogArticlesComments.id' => $commentId
            ])
            ->first();

        if (is_null($comment)) {
            $this->Flash->error(__("This comment doesn't exist or has been deleted."));

            return $this->redirect(['action' => 'index']);
        }

        $comment->toArray();

        //Count the number of message before this message.
        $messagesBefore = $this->BlogArticlesComments

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'BlogArticlesComments.article_id' => $comment->article_id,
                'BlogArticlesComments.created <' => $comment->created
            ])
            ->count();

        //Get the number of messages per page.
        $messagesPerPage = Configure::read('Blog.comment_per_page');

        //Calculate the page.
        $page = floor($messagesBefore / $messagesPerPage) + 1;

        $page = ($page > 1) ? $page : 1;

        //Redirect the user.
        return $this->redirect([
            '_name' => 'blog-article',
            'slug' => $comment->blog_article->title,
            'id' => $comment->blog_article->id,
            '?' => ['page' => $page],
            '#' => 'comment-' . $commentId
        ]);
    }

    /**
     * Get all articles by a date formatted to "m-Y".
     *
     * @param string $date The date of the archive.
     *
     * @return void
     */
    public function archive($date = null)

This method seems to be duplicated in your project.

    {
        $this->loadModel('BlogArticles');

        $this->paginate = [
            'maxLimit' => Configure::read('Blog.article_per_page')
        ];

        $archives = $this->BlogArticles

The property BlogArticles does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'DATE_FORMAT(BlogArticles.created,\'%m-%Y\')' => $date,
                'BlogArticles.is_display' => 1
            ])
            ->contain([
                'BlogCategories',
                'Users' => function ($q) {
                        return $q->find('short');
                }
            ])
            ->order([
                'BlogArticles.created' => 'desc'
            ]);

        $articles = $this->paginate($archives);

        $this->set(compact('articles', 'date'));
    }

    /**
     * Search articles.
     *
     * @return void
     */
    public function search()
    {
        $this->loadModel('BlogArticles');

        //Check the keyword to search. (For pagination)
        if (!empty($this->request->data['search'])) {

This code seems to be duplicated across your project.

            $keyword = $this->request->data['search'];
            $this->request->session()->write('Search.Blog.Keyword', $keyword);
        } else {
            if ($this->request->session()->read('Search.Blog.Keyword')) {

The expression $this->request->session()->read('Search.Blog.Keyword') of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

                $keyword = $this->request->session()->read('Search.Blog.Keyword');
            } else {
                $keyword = '';
            }
        }

        //Pagination
        $this->paginate = [
            'maxLimit' => Configure::read('Blog.article_per_page')
        ];

        $articles = $this->BlogArticles

The property BlogArticles does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Users' => function ($q) {
                    return $q->find('short');
                }
            ])
            ->where([
                'BlogArticles.is_display' => 1
            ])
            ->andWhere(function ($q) use ($keyword) {
                return $q
                    ->like('title', "%$keyword%");
            })
            ->order([
                'BlogArticles.created' => 'desc'
            ]);

        $articles = $this->paginate($articles);

        $this->set(compact('articles', 'keyword'));
    }

    /**
     * Like an article.
     *
     * @param int $articleId Id of the article to like.
     *
     * @throws \Cake\Network\Exception\NotFoundException When it's not an AJAX request.
     *
     * @return void
     */
    public function articleLike($articleId = null)
    {
        if (!$this->request->is('ajax')) {
            throw new NotFoundException();
        }

        //Check if the user hasn't already liked this article.
        $this->loadModel('BlogArticlesLikes');
        $checkLike = $this->BlogArticlesLikes

The property BlogArticlesLikes does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'BlogArticlesLikes.user_id' => $this->Auth->user('id'),
                'BlogArticlesLikes.article_id' => $articleId
            ])
            ->first();

        $json = [];

        if (!is_null($checkLike)) {

This code seems to be duplicated across your project.

            $json['message'] = __('You already like this article !');
            $json['error'] = true;

            $this->set(compact('json'));

            $this->set('_serialize', 'json');
        }

        //Check if the article exist.
        $this->loadModel('BlogArticles');
        $checkArticle = $this->BlogArticles

The property BlogArticles does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'BlogArticles.id' => $articleId,
                'BlogArticles.is_display' => 1
            ])
            ->first();

        if (is_null($checkArticle)) {

This code seems to be duplicated across your project.

            $json['message'] = __("This article doesn't exist !");
            $json['error'] = true;

            $this->set(compact('json'));

            $this->set('_serialize', 'json');
        }

        //Prepare data to be saved.
        $data = [];
        $data['BlogArticlesLikes']['user_id'] = $this->Auth->user('id');
        $data['BlogArticlesLikes']['article_id'] = $articleId;

        $like = $this->BlogArticlesLikes->newEntity($data);

The property BlogArticlesLikes does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

        if ($this->BlogArticlesLikes->save($like)) {

The property BlogArticlesLikes does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            //Update the Statistics
            $this->eventManager()->attach(new Statistics());

new \App\Event\Statistics() is of type object<App\Event\Statistics>, but the function expects a callable.

The method Cake\Event\EventManager::attach() has been deprecated with message: 3.0.0 Use on() instead.

            $event = new Event('Model.BlogArticlesLikes.new');
            $this->eventManager()->dispatch($event);

            $json['message'] = __('Thanks for {0} this article ! ', "<i class='fa fa-heart text-danger'></i>");
            $json['title'] = __('You {0} this article.', "<i class='fa fa-heart text-danger'></i>");
            $json['url'] = Router::url(
                [
                    'action' => 'articleUnlike',
                    $articleId
                ]
            );
            $json['error'] = false;
        } else {
            $json['message'] = __('An error occurred, please try again later.');
            $json['error'] = true;
        }

        $this->set(compact('json'));

        $this->set('_serialize', 'json');
    }

    /**
     * Unlike an article.
     *
     * @param int|null $articleId Id of the article to like.
     *
     * @throws \Cake\Network\Exception\NotFoundException When it's not an AJAX request.
     *
     * @return void
     */
    public function articleUnlike($articleId = null)
    {
        if (!$this->request->is('ajax')) {
            throw new NotFoundException();
        }

        //Check if the user like this article.
        $this->loadModel('BlogArticlesLikes');
        $like = $this->BlogArticlesLikes

The property BlogArticlesLikes does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'BlogArticles'
            ])
            ->where([
                'BlogArticlesLikes.user_id' => $this->Auth->user('id'),
                'BlogArticlesLikes.article_id' => $articleId,
                'BlogArticles.is_display' => 1
            ])
            ->first();

        $json = [];

        if (is_null($like)) {

This code seems to be duplicated across your project.

            $json['message'] = __("You don't like this article !");
            $json['error'] = true;

            $this->set(compact('json'));

            $this->set('_serialize', 'json');
        }

        if ($this->BlogArticlesLikes->delete($like)) {

The property BlogArticlesLikes does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            //Update the Statistics
            $this->eventManager()->attach(new Statistics());

new \App\Event\Statistics() is of type object<App\Event\Statistics>, but the function expects a callable.

The method Cake\Event\EventManager::attach() has been deprecated with message: 3.0.0 Use on() instead.

            $event = new Event('Model.BlogArticlesLikes.new');
            $this->eventManager()->dispatch($event);

            $json['url'] = Router::url([
                                'action' => 'articleLike',
                                $articleId
                            ]);
            $json['title'] = __('Like {0}', "<i class='fa fa-heart text-danger'></i>");
            $json['error'] = false;
        } else {
            $json['message'] = __('An error occurred, please try again later.');
            $json['error'] = true;
        }

        $this->set(compact('json'));

        $this->set('_serialize', 'json');
    }

    /**
     * Delete a comment.
     *
     * @param int $id Id of the comment to delete.
     *
     * @return \Cake\Network\Response
     */
    public function deleteComment($id = null)
    {
        $this->loadModel('BlogArticlesComments');

        $comment = $this->BlogArticlesComments

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'BlogArticles'
            ])
            ->where([
                'BlogArticlesComments.id' => $id
            ])
            ->first();

        if (is_null($comment)) {
            $this->Flash->error(__("This comment doesn't exist or has been deleted !"));

            return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Network\Request>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

        }

        //Current user.
        $this->loadModel('Users');
        $currentUser = $this->Users

The property Users does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Groups' => function ($q) {
                    return $q->select(['id', 'is_staff']);
                }
            ])
            ->where([
                'Users.id' => $this->Auth->user('id')
            ])
            ->select(['id', 'group_id'])
            ->first();

        if ($comment->user_id != $this->Auth->user('id') && !$currentUser->group->is_staff) {

This code seems to be duplicated across your project.

            $this->Flash->error(__("You don't have the authorization to delete this comment !"));

            return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Network\Request>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

        }

        if ($this->BlogArticlesComments->delete($comment)) {

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->eventManager()->attach(new Statistics());

new \App\Event\Statistics() is of type object<App\Event\Statistics>, but the function expects a callable.

The method Cake\Event\EventManager::attach() has been deprecated with message: 3.0.0 Use on() instead.

            $event = new Event('Model.BlogArticlesComments.new');
            $this->eventManager()->dispatch($event);

            $this->Flash->success(__("This comment has been deleted successfully !"));
        }

        return $this->redirect(['_name' => 'blog-article', 'slug' => $comment->blog_article->title, 'id' => $comment->blog_article->id, '?' => ['page' => $comment->blog_article->last_page]]);
    }

    /**
     * Get the form to edit a comment.
     *
     * @throws \Cake\Network\Exception\NotFoundException When it's not an AJAX request.
     *
     * @return void
     */
    public function getEditComment()
    {
        if (!$this->request->is('ajax')) {
            throw new NotFoundException();
        }

        $this->loadModel('BlogArticlesComments');
        $comment = $this->BlogArticlesComments

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->where([
                'BlogArticlesComments.id' => $this->request->data['id']
            ])
            ->first();

        $json = [
            'error' => false,
            'errorMessage' => ''
        ];

        if (is_null($comment)) {

This code seems to be duplicated across your project.

            $json['error'] = true;
            $json['errorMessage'] = __("This comment doesn't exist or has been deleted !");

            $this->set(compact('json'));

            return;
        }

        //Current user.
        $this->loadModel('Users');
        $currentUser = $this->Users

The property Users does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Groups' => function ($q) {
                    return $q->select(['id', 'is_staff']);
                }
            ])
            ->where([
                'Users.id' => $this->Auth->user('id')
            ])
            ->select(['id', 'group_id'])
            ->first();

        if ($comment->user_id != $this->Auth->user('id') && !$currentUser->group->is_staff) {

This code seems to be duplicated across your project.

            $json['error'] = true;
            $json['errorMessage'] = __("You don't have the authorization to edit this comment !");

            $this->set(compact('json'));

            return;
        }

        $this->set(compact('json', 'comment'));
    }

    /**
     * Edit a comment.
     *
     * @param int $id Id of the comment.
     *
     * @return \Cake\Network\Response
     */
    public function editComment($id = null)
    {
        $this->loadModel('BlogArticlesComments');

        $comment = $this->BlogArticlesComments

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'BlogArticles'
            ])
            ->where([
                'BlogArticlesComments.id' => $id
            ])
            ->first();

        if (is_null($comment)) {
            $this->Flash->error(__("This comment doesn't exist or has been deleted !"));

            return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Network\Request>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

        }

        //Current user.
        $this->loadModel('Users');
        $currentUser = $this->Users

The property Users does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            ->find()
            ->contain([
                'Groups' => function ($q) {
                    return $q->select(['id', 'is_staff']);
                }
            ])
            ->where([
                'Users.id' => $this->Auth->user('id')
            ])
            ->select(['id', 'group_id'])
            ->first();

        if ($comment->user_id != $this->Auth->user('id') && !$currentUser->group->is_staff) {

This code seems to be duplicated across your project.

            $this->Flash->error(__("You don't have the authorization to edit this comment !"));

            return $this->redirect($this->referer());

It seems like $this->referer() targeting Cake\Controller\Controller::referer() can also be of type object<Cake\Network\Request>; however, Cake\Controller\Controller::redirect() does only seem to accept string|array, maybe add an additional type check?

        }

        $this->BlogArticlesComments->patchEntity($comment, $this->request->data());

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

        if ($this->BlogArticlesComments->save($comment)) {

The property BlogArticlesComments does not exist on object<App\Controller\BlogController>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->Flash->success(__("This comment has been edited successfully !"));
        }

        return $this->redirect(['action' => 'go', $comment->id]);
    }
}