SixtyEightPublishers\Application\Storage\SessionProfileStorage

Complexity

Total Complexity

8

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	setProfile()	0	23	5
A	getProfile()	0	4	1
A	persist()	0	8	1

<?php

namespace SixtyEightPublishers\Application\Storage;

use Nette\Http\Session;
use Nette\SmartObject;
use SixtyEightPublishers\Application\ActiveProfile;
use SixtyEightPublishers\Application\IProfile;
use SixtyEightPublishers\Application\IProfileStorage;
use SixtyEightPublishers\Application\ProfileConfigurationException;

class SessionProfileStorage implements IProfileStorage
{
	use SmartObject;

	const SESSION_SECTION = 'SixtyEightPublishers.Application';

	/** @var null|ActiveProfile */
	private $profile;

	/** @var \Nette\Http\SessionSection  */
	private $session;

	/** @var null|callable */
	public $onPersist;

	/** @var null|callable */
	public $onProfileSet;

	/**
	 * @param \Nette\Http\Session $session
	 */
	public function __construct(Session $session)
	{
		$this->session = $session->getSection(self::SESSION_SECTION);
	}


	/***************** interface \SixtyEightPublishers\Application\IProfileStorage *****************/

	/**
	 * {@inheritdoc}
	 */
	public function setProfile(IProfile $profile)
	{
		$this->profile = new ActiveProfile($profile, $this);

		if ($this->session['profileName'] === $this->profile->getName()) {
			try {
				$this->profile->changeCountry($this->session['profileCountry'], FALSE);
			} catch (ProfileConfigurationException $e) {

Consider adding a comment why this CATCH block is empty.

			}
			try {
				$this->profile->changeLanguage($this->session['profileLanguage'], FALSE);
			} catch (ProfileConfigurationException $e) {

Consider adding a comment why this CATCH block is empty.

			}
			try {
				$this->profile->changeCurrency($this->session['profileCurrency'], FALSE);
			} catch (ProfileConfigurationException $e) {

Consider adding a comment why this CATCH block is empty.

			}
		} else {
			$this->session['profileName'] = $this->profile->getName();
		}

		$this->onProfileSet($this->profile);

The method onProfileSet does not exist on object<SixtyEightPublishers\Application\Storage\SessionProfileStorage>? Since you implemented __call, maybe consider adding a @method annotation.

	}

	/**
	 * @return \SixtyEightPublishers\Application\ActiveProfile
	 */
	public function getProfile() : ActiveProfile
	{
		return $this->profile;
	}

	/**
	 * {@inheritdoc}
	 */
	public function persist()
	{
		$this->onPersist($this->profile);

The method onPersist() does not exist on SixtyEightPublishers\Application\Storage\SessionProfileStorage. Did you maybe mean persist()?

		$this->session['profileName'] = $this->profile->getName();
		$this->session['profileCountry'] = $this->profile->getCountry();
		$this->session['profileLanguage'] = $this->profile->getLanguage();
		$this->session['profileCurrency'] = $this->profile->getCurrency();
	}
}