PostController::edit() - Code Metrics - Inspection of "[WIP] Implement adding/editing blog post in yii-de..." - yiisoft/yii-demo - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#132)

unknown

created 2020-09-01 10:24 UTC

PostController::edit() B

↳ Parent: PostController

Complexity

Conditions	6
Paths	20

Size

Total Lines	55
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	6
eloc	31
c	1
b	0
f	0
nc	20
nop	5
dl	0
loc	55
rs	8.8017

How to fix Long Method

<?php

namespace App\Blog\Post;

use App\Blog\Entity\Post;
use App\Entity\User;
use App\ViewRenderer;
use Cycle\ORM\ORMInterface;
use Cycle\ORM\Transaction;
use Psr\Http\Message\ResponseFactoryInterface;
use Psr\Http\Message\ResponseInterface as Response;
use Psr\Http\Message\ServerRequestInterface as Request;
use Yiisoft\Http\Method;
use Yiisoft\Router\UrlGeneratorInterface;
use Yiisoft\Yii\Web\User\User as UserComponent;

final class PostController
{
    private ViewRenderer $viewRenderer;

    public function __construct(ViewRenderer $viewRenderer)
    {
        $this->viewRenderer = $viewRenderer->withControllerName('blog/post');
    }

    public function index(Request $request, PostRepository $postRepository, ResponseFactoryInterface $responseFactory): Response
    {
        $slug = $request->getAttribute('slug', null);
        $item = $postRepository->fullPostPage($slug);
        if ($item === null) {
            return $responseFactory->createResponse(404);
        }

        return $this->viewRenderer->render('index', ['item' => $item]);
    }

    public function add(
        Request $request,
        ORMInterface $orm,
        ResponseFactoryInterface $responseFactory,
        UrlGeneratorInterface $urlGenerator,
        UserComponent $userComponent
    ): Response
    {
        $body = $request->getParsedBody();
        $parameters = [
            'body' => $body,
        ];

        if ($request->getMethod() === Method::POST) {
            $error = '';


            try {
                foreach (['header', 'content'] as $name) {
                    if (empty($body[$name])) {
                        throw new \InvalidArgumentException(ucfirst($name) . ' is required');
                    }
                }

                $post = new Post($body['header'], $body['content']);

                $userRepo = $orm->getRepository(User::class);
                $user = $userRepo->findByPK($userComponent->getId());

                $post->setUser($user);
                $post->setUser(/** @scrutinizer ignore-type */ $user);
                $post->setPublic(true);

                $transaction = new Transaction($orm);
                $transaction->persist($post);

                $transaction->run();

                return $responseFactory
                    ->createResponse(302)
                    ->withHeader(
                        'Location',
                        $urlGenerator->generate('blog/index')
                    );
            } catch (\Throwable $e) {
                $error = $e->getMessage();
            }

            $parameters['error'] = $error;
        }

        $parameters['title'] = 'Add post';
        return $this->viewRenderer->withCsrf()->render('__form', $parameters);
    }

    public function edit(
        Request $request,
        ORMInterface $orm,
        ResponseFactoryInterface $responseFactory,
        UrlGeneratorInterface $urlGenerator,
        PostRepository $postRepository): Response
    {
        $post = $postRepository->fullPostPage($request->getAttribute('slug', null));
        if ($post === null) {
            return $responseFactory->createResponse(404);
        }

        if ($request->getMethod() === Method::POST) {
            try {
                $body = $request->getParsedBody();
                $parameters = [
                    'body' => $body,
                ];

                foreach (['header', 'content'] as $name) {
                    if (empty($body[$name])) {
                        throw new \InvalidArgumentException(ucfirst($name) . ' is required');
                    }
                }

                $post->setTitle($body['header']);
                $post->setContent($body['content']);

                $transaction = new Transaction($orm);
                $transaction->persist($post);

                $transaction->run();

                return $responseFactory
                    ->createResponse(302)
                    ->withHeader(
                        'Location',
                        $urlGenerator->generate('blog/index')
                    );
            } catch (\Throwable $e) {
                $error = $e->getMessage();
            }

            $parameters['error'] = $error;
        } else {
            $parameters = [
                'body' => [
                    'header' => $post->getTitle(),
                    'content' => $post->getContent()
                ]
            ];
        }

        $parameters['title'] = 'Edit post';
        return $this->viewRenderer->withCsrf()->render('__form', $parameters);
    }
}


yiisoft / yii-demo

Pull Request — master (#132)

PostController::edit() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like