Vivait\AuthBundle\EventListener\ConfigureMenuListener

Complexity

Total Complexity

5

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	2
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	onMenuConfigure()	0	39	4

<?php

namespace Vivait\AuthBundle\EventListener;

use Symfony\Component\Security\Core\SecurityContext;
use Vivait\BootstrapBundle\Event\ConfigureMenuEvent;

class ConfigureMenuListener {
	/**
	 * @var SecurityContext
	 */
	private $security_context;

	/**
	 * @var int
	 */
	private $license_warning;

	function __construct(SecurityContext $security_context, $license_warning = 7) {

It is generally recommended to explicitly declare the visibility for methods.

		$this->security_context = $security_context;
		$this->license_warning  = intval($license_warning);
	}

	/**
	 * @param ConfigureMenuEvent $event
	 */
	public function onMenuConfigure(ConfigureMenuEvent $event) {
		$menu  = $event->getMenu();
		$token = $this->security_context->getToken();

The method Symfony\Component\Security\Core\SecurityContext::getToken() has been deprecated with message: since version 2.6, to be removed in 3.0. Use TokenStorageInterface::getToken() instead. {@inheritdoc}

		$root = $menu->addChild('Auth Menu', [
			'navbar' => true,
		]);

		if ($token === null) {
			return;
		}

		/* @var $user \Vivait\AuthBundle\Entity\User */
		$user = $token->getUser();

		if ($user) {
			if ($user->getLicensedUntil() > new \DateTime('+' . $this->license_warning . ' days')) {
				$licensing = $root->addChild('Licensing Warning', [
					'dropdown' => true,
					'icon'     => 'warning-sign',
					'label'    => '',
					'pull-right' => true
				]);

				$licensing->setAttribute('class', $licensing->getAttribute('class') .' menu-danger');

				$licensing->addChild('This product will expire soon', [
					'dropdown-header' => true
				]);
				$licensing->addChild('Licensed until '. $user->getLicensedUntil()->format('d M Y H:i'), [
					'dropdown-header' => true
				]);
			}

//			$menu->addChild('Tenants', array('route' => 'vivait_auth_tenant'));

70% of this comment could be valid code. Did you maybe forget this after debugging?

//			$menu->addChild('Groups', array('route' => 'vivait_auth_group'));
//			$menu->addChild('Users', array('route' => 'vivait_auth_user'));
		}
	}
}