PostingsController::meta() - Code Metrics - Schlaefer/Saito - Measure and Improve Code Quality continuously with Scrutinizer

PostingsController::meta() B
last analyzed 2020-10-20 13:08 UTC

↳ Parent: PostingsController
Complexity

Conditions	8
Paths	23
Size

Total Lines	52
Code Lines	30
Duplication

Lines	0
Ratio	0 %
Importance

Changes	1
Bugs	0	Features	0
Metric	Value
cc	8
eloc	30
c	1
b	0
f	0
nc	23
nop	1
dl	0
loc	52
rs	8.1954
How to fix Long Method

<?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 App\Controller;

use Api\Controller\ApiAppController;
use App\Controller\Component\PostingComponent;
use App\Model\Entity\Entry;
use App\Model\Table\EntriesTable;
use Cake\Core\Configure;
use Cake\Http\Exception\BadRequestException;
use Cake\Http\Exception\NotFoundException;
use Saito\Exception\SaitoForbiddenException;
use Saito\Posting\PostingInterface;

/**
 * Endpoint for adding/POST and editing/PUT posting
 *
 * @property EntriesTable $Entries
 * @property PostingComponent $Posting
 */
class PostingsController extends ApiAppController
{
    /**
     * {@inheritDoc}
     */
    public function initialize()
    {
        parent::initialize();
        $this->loadModel('Entries');
        $this->loadComponent('Posting');
    }

    /**
     * Add a a new posting
     *
     * @return void
     */
    public function add(): void
    {
        $data = $this->getRequest()->getData();
        $allowedFields = ['category_id', 'edited', 'edited_by', 'pid', 'subject', 'text'];
        $data = array_intersect_key($data, array_fill_keys($allowedFields, 1));
        $data = array_intersect_key(/** @scrutinizer ignore-type */ $data, array_fill_keys($allowedFields, 1));

        $data += [
            'name' => $this->CurrentUser->get('username'),
            'user_id' => $this->CurrentUser->getId(),
        ];

        /** @var Entry */
        $posting = $this->Posting->create($data, $this->CurrentUser);

        if (empty($posting)) {
            throw new BadRequestException();
        }

        $errors = $posting->getErrors();

        if (!count($errors)) {
            $this->set(compact('posting'));

            return;
        }

        $this->set(compact('errors'));
        $this->viewBuilder()->setTemplate('/Error/json/entityValidation');
    }

    /**
     * Edit an existing posting
     *
     * @param string $id Unused in favor of request-data.
     * @return void
     */
    public function edit(string $id): void
    public function edit(/** @scrutinizer ignore-unused */ string $id): void
    {
        $id = $this->getRequest()->getData('id', null);

        if (empty($id)) {
            throw new BadRequestException('No posting-id provided.');
        }

        $posting = $this->Entries->get($id);
        $posting = $this->Entries->get(/** @scrutinizer ignore-type */ $id);
        if (!$posting) {
            throw new NotFoundException('Posting not found.');
        }

        $data = $this->getRequest()->getData();
        $allowedFields = ['category_id', 'edited', 'edited_by', 'subject', 'text'];
        $data = array_intersect_key($data, array_fill_keys($allowedFields, 1));
        $data = array_intersect_key(/** @scrutinizer ignore-type */ $data, array_fill_keys($allowedFields, 1));

        $data += [
            'edited' => bDate(),
            'edited_by' => $this->CurrentUser->get('username'),
        ];

        $updatedPosting = $this->Posting->update($posting, $data, $this->CurrentUser);
        $updatedPosting = $this->Posting->update(/** @scrutinizer ignore-type */ $posting, $data, $this->CurrentUser);

        if (!$updatedPosting) {

            throw new BadRequestException('Posting could not be saved.');
        }

        if (!$updatedPosting->hasErrors()) {
            $this->set('posting', $updatedPosting);
            $this->render('/Postings/json/add');

            return;
        }

        $errors = $updatedPosting->getErrors();
        $this->set(compact('errors'));
        $this->viewBuilder()->setTemplate('/Error/json/entityValidation');
    }

    /**
     * Serves meta information required to add or edit a posting
     *
     * @param string|null $id ID of the posting (send on edit)
     * @return void
     */
    public function meta(?string $id = null): void
    {
        $id = (int)$id;
        $isEdit = !empty($id);
        $pid = $this->getRequest()->getQuery('pid', null);
        $isAnswer = !empty($pid);

        if ($isAnswer) {
            /** @var PostingInterface */
            $parent = $this->Entries->get($pid)->toPosting()->withCurrentUser($this->CurrentUser);
            $parent = $this->Entries->get(/** @scrutinizer ignore-type */ $pid)->toPosting()->withCurrentUser($this->CurrentUser);

            // Don't leak content of forbidden categories
            if ($parent->isAnsweringForbidden()) {
                throw new SaitoForbiddenException(
                    'Access to parent in PostingsController:meta() forbidden.',
                    ['CurrentUser' => $this->CurrentUser]
                );
            }

            $this->set('parent', $parent);
        }

        if ($isEdit) {
            /** @var PostingInterface */
            $posting = $this->Entries->get($id)->toPosting()->withCurrentUser($this->CurrentUser);
            if (!$posting->isEditingAllowed()) {
                throw new SaitoForbiddenException(
                    'Access to posting in PostingsController:meta() forbidden.',
                    ['CurrentUser' => $this->CurrentUser]
                );
            }
            $this->set('posting', $posting);
        } else {
            /// We currently don't save drafts for edits
            $where = ['user_id' => $this->CurrentUser->getId()];
            if (is_numeric($pid)) {
                $where['pid'] = $pid;
            }
            $draft = $this->Entries->Drafts->find()->where($where)->first();

            if ($draft) {
                $this->set('draft', $draft);
            }
        }

        $settings = Configure::read('Saito.Settings');

        $this->set(compact('isAnswer', 'isEdit', 'settings'));

        $action = $isAnswer ? 'answer' : 'thread';
        $categories = $this->CurrentUser->getCategories()->getAll($action, 'list');
        $this->set('categories', $categories);
    }
}

Schlaefer / Saito

PostingsController::meta() B last analyzed 2020-10-20 13:08 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like

PostingsController::meta() B
last analyzed 2020-10-20 13:08 UTC
