Eccube\Application\ApplicationTrait

Complexity

Total Complexity

19

Size/Duplication

Total Lines	123
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

62.22%

Importance

Changes

0

14 Methods

Rating	Name	Duplication	Size	Complexity
A	addSuccess()	0	4	1
A	addError()	0	4	1
A	addWarning()	0	4	1
A	addDanger()	0	4	1
A	setLoginTargetPath()	0	8	2
A	isAdminRequest()	0	4	2
A	isFrontRequest()	0	4	2
A	forwardChain()	0	5	1
A	overwrite()	0	8	2
A	addInfo()	0	4	1
A	forward()	0	18	2
A	clearMessage()	0	4	1
A	deleteMessage()	0	5	1
A	addRequestError()	0	4	1

<?php

namespace Eccube\Application;

use Eccube\Event\TemplateEvent;
use Monolog\Logger;
use Symfony\Component\Form\FormBuilder;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpFoundation\StreamedResponse;
use Symfony\Component\HttpKernel\HttpKernelInterface;
use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
use Symfony\Component\Security\Core\User\UserInterface;

trait ApplicationTrait
{
    /**

Doc comment for parameter "$message" missing

Doc comment for parameter "$namespace" missing

     * Application Shortcut Methods
     */
    public function addSuccess($message, $namespace = 'front')
    {
        $this['session']->getFlashBag()->add('eccube.' . $namespace . '.success', $message);

Concat operator must not be surrounded by spaces

    }

    public function addError($message, $namespace = 'front')

Missing function doc comment

    {
        $this['session']->getFlashBag()->add('eccube.' . $namespace . '.error', $message);

Concat operator must not be surrounded by spaces

    }

    public function addDanger($message, $namespace = 'front')

Missing function doc comment

    {
        $this['session']->getFlashBag()->add('eccube.' . $namespace . '.danger', $message);

Concat operator must not be surrounded by spaces

    }

    public function addWarning($message, $namespace = 'front')

Missing function doc comment

    {
        $this['session']->getFlashBag()->add('eccube.' . $namespace . '.warning', $message);

Concat operator must not be surrounded by spaces

    }

    public function addInfo($message, $namespace = 'front')

Missing function doc comment

    {
        $this['session']->getFlashBag()->add('eccube.' . $namespace . '.info', $message);

Concat operator must not be surrounded by spaces

    }

    public function addRequestError($message, $namespace = 'front')

Missing function doc comment

    {
        $this['session']->getFlashBag()->set('eccube.' . $namespace . '.request.error', $message);

Concat operator must not be surrounded by spaces

    }

    public function clearMessage()

Missing function doc comment

    {
        $this['session']->getFlashBag()->clear();
    }

    public function deleteMessage()

Missing function doc comment

    {
        $this->clearMessage();
        $this->addWarning('admin.delete.warning', 'admin');
    }

    public function setLoginTargetPath($targetPath, $namespace = null)

Missing function doc comment

    {
        if (is_null($namespace)) {
            $this['session']->getFlashBag()->set('eccube.login.target.path', $targetPath);
        } else {
            $this['session']->getFlashBag()->set('eccube.' . $namespace . '.login.target.path', $targetPath);

Concat operator must not be surrounded by spaces

        }
    }

    public function isAdminRequest()

Missing function doc comment

    {
        return isset($this['admin']) ? $this['admin'] : null;
    }

    public function isFrontRequest()

Missing function doc comment

    {
        return isset($this['front']) ? $this['front'] : null;
    }

    /**
     * 他のコントローラにリクエストをフォワードします.
     *
     * @param string $path フォワード先のパス

Expected 14 spaces after parameter name; 1 found

     * @param array $requestParameters

Expected 2 spaces after parameter type; 1 found

     * @return Response
     */
    public function forward($path, array $requestParameters = [])
    {
        $request = $this['request_stack']->getCurrentRequest();

        $subRequest = Request::create(
            $path,
            $request->getMethod(),
            $requestParameters,
            $request->cookies->all(),
            [],
            $request->server->all()
        );
        if ($request->getSession()) {
            $subRequest->setSession($request->getSession());
        }

        return $this->handle($subRequest, HttpKernelInterface::SUB_REQUEST, false);

It seems like handle() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * フォワードをチェーンでつなげます.
     *
     * @param string $path フォワード先のパス

Expected 3 spaces after parameter type; 1 found

Expected 14 spaces after parameter name; 1 found

     * @param array $requestParameters

Expected 4 spaces after parameter type; 1 found

     * @param Response $response
     * @return Application
     */
    public function forwardChain($path, array $requestParameters = [], Response &$response = null)
    {
        $response = $this->forward($path, $requestParameters);
        return $this;

Missing blank line before return statement

    }

    /**

Doc comment for parameter "$key" missing

Doc comment for parameter "$service" missing

     * コンテナに登録済のサービスを上書きする.
     * Pimple標準では再登録を行うと, `RuntimeException: Cannot override frozen service`が投げられるため,一度unsetしてから再登録を行う.
     * config系の変更程度に利用はとどめること
     *
     * @param $key

Missing parameter name

     * @param $service

Missing parameter name

     * @throws \InvalidArgumentException keyが存在しない場合.
     */
    public function overwrite($key, $service)
    {
        if (!$this->offsetExists($key)) {

It seems like offsetExists() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            throw new \InvalidArgumentException();
        }
        $this->offsetUnset($key);

It seems like offsetUnset() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this[$key] = $service;
    }
}