phpffcms / ffcms

Apps/Controller/Front/Profile/ActionIgnore.php

<?php

namespace Apps\Controller\Front\Profile;

use Apps\ActiveRecord\Blacklist;
use Apps\Model\Front\Profile\FormIgnoreAdd;
use Ffcms\Core\App;
use Ffcms\Core\Arch\View;
use Ffcms\Core\Exception\ForbiddenException;
use Ffcms\Core\Network\Request;
use Ffcms\Core\Network\Response;

/**
 * Trait ActionIgnore
 * @package Apps\Controller\Front\Profile
 * @property View $view
 * @property Request $request
 * @property Response $response
 */
trait ActionIgnore
{
    /**
     * Show users in blacklist and allow add new users
     * @return string
     * @throws \Ffcms\Core\Exception\SyntaxException
     * @throws ForbiddenException
     */
    public function ignore(): ?string
    {
        // check if not auth
        if (!App::$User->isAuth()) {
            throw new ForbiddenException();
        }

        // get user object and init model of it
        $user = App::$User->identity();
        $model = new FormIgnoreAdd($user);

It seems like $user can also be of type null; however, parameter $user of Apps\Model\Front\Profile\FormIgnoreAdd::__construct() does only seem to accept Ffcms\Core\Interfaces\iUser, maybe add an additional type check?

        // set user id from ?id= get param if form not sended
        if (!$model->send()) {
            $uid = (int)$this->request->query->get('id');
            if ($uid > 0) {
                $model->id = $uid;
            }
        }

        // sended new block add?
        if ($model->send() && $model->validate()) {
            if ($model->save()) {
                App::$Session->getFlashBag()->add('success', __('User was successful blocked!'));
            } else {
                App::$Session->getFlashBag()->add('error', __('This user is always in ignore list'));
            }
        }

        // get blocked users
        $query = Blacklist::where('user_id', '=', $user->getId());

        // prepare pagination data
        $page = (int)$this->request->query->get('page');
        $offset = $page * static::BLOCK_PER_PAGE;

The constant Apps\Controller\Front\Profile\ActionIgnore::BLOCK_PER_PAGE was not found. Maybe you did not declare it correctly or list all dependencies?

        $totalCount = $query->count();

        // get records as object
        $records = $query->with(['targetUser', 'targetUser.profile'])
            ->skip($offset)
            ->take(static::BLOCK_PER_PAGE)
            ->get();

        // render output view
        return $this->view->render('profile/ignore', [
            'records' => $records,
            'model' => $model,
            'pagination' => [
                'total' => $totalCount,
                'page' => $page,
                'step' => static::BLOCK_PER_PAGE
            ]
        ]);
    }
}