Oc\GlobalContext\GlobalContextFactory

Complexity

Total Complexity

3

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	createFromRequest()	0	7	1
A	createFromRequestStack()	0	6	1

<?php

namespace Oc\GlobalContext;

use Oc\GlobalContext\Provider\LanguageProvider;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\RequestStack;

/**
 * Class GlobalContextFactory
 */
class GlobalContextFactory
{
    /**
     * @var LanguageProvider
     */
    private $languageProvider;

    /**
     * GlobalContextFactory constructor.
     *
     * @param LanguageProvider $languageProvider
     */
    public function __construct(LanguageProvider $languageProvider)
    {
        $this->languageProvider = $languageProvider;
    }

    /**
     * Creates a global context from given request.
     *
     * @param Request $request
     *
     * @return GlobalContext
     */
    public function createFromRequest(Request $request)
    {
        return new GlobalContext(
            $this->languageProvider->getDefaultLanguage(),
            $this->languageProvider->getPreferredLanguage($request)
        );
    }

    /**
     * Creates a global context from request stack.
     *
     * @param RequestStack $requestStack
     *
     * @return GlobalContext
     */
    public function createFromRequestStack(RequestStack $requestStack)
    {
        return $this->createFromRequest(
            $requestStack->getMasterRequest()

It seems like $requestStack->getMasterRequest() can be null; however, createFromRequest() does not accept null, maybe add an additional type check?

        );
    }
}