SessionDownloadStrategy::isGranted() - Code Metrics - Inspection of "Merge 3.x into master" - sonata-project/SonataMediaBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( ce34f1...a61f8e )

unknown

created 2016-06-08 00:00 UTC

SessionDownloadStrategy::isGranted() A

↳ Parent: SessionDownloadStrategy

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	1	Features	0

Metric	Value
cc	2
eloc	7
c	2
b	1
f	0
nc	2
nop	2
dl	0
loc	14
rs	9.4285

<?php

/*
 * This file is part of the Sonata Project package.
 *
 * (c) Thomas Rabaix <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Sonata\MediaBundle\Security;

use Sonata\MediaBundle\Model\MediaInterface;
use Symfony\Component\DependencyInjection\ContainerInterface;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Session\SessionInterface;
use Symfony\Component\Translation\TranslatorInterface;

/**
 * Class SessionDownloadStrategy.
 *
 * @author Ahmet Akbana <[email protected]>
 */
class SessionDownloadStrategy implements DownloadStrategyInterface
{
    /**
     * @var ContainerInterface
     *
     * @deprecated Since version 3.x, will be removed in 4.0.
     * NEXT_MAJOR : remove this property
     */
    protected $container;

    /**
     * @var TranslatorInterface
     */
    protected $translator;

    /**
     * @var int
     */
    protected $times;

    /**
     * @var string
     */
    protected $sessionKey = 'sonata/media/session/times';

    /**
     * @var SessionInterface
     */
    private $session;

    /**
     * @param TranslatorInterface                 $translator
     * @param ContainerInterface|SessionInterface $session
     * @param int                                 $times
     */
    public function __construct(TranslatorInterface $translator, $session, $times)
    {
        // NEXT_MAJOR : remove this block and set session from parameter.
        if ($session instanceof ContainerInterface) {
            @trigger_error(
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
                'Using an instance of Symfony\Component\DependencyInjection\ContainerInterface is deprecated since 
                version 3.x and will be removed in 4.0. 
                Use Symfony\Component\HttpFoundation\Session\SessionInterface instead.',
                E_USER_DEPRECATED
            );

            $this->session = $session->get('session');
        } elseif ($session instanceof SessionInterface) {
            $this->session = $session;
        } else {
            throw new \InvalidArgumentException(
                '$session should be an instance of Symfony\Component\HttpFoundation\Session\SessionInterface'
            );
        }

        $this->times = $times;
        $this->translator = $translator;
    }

    /**
     * {@inheritdoc}
     */
    public function isGranted(MediaInterface $media, Request $request)
    {
        $times = $this->session->get($this->sessionKey, 0);

        if ($times >= $this->times) {
            return false;
        }

        ++$times;

        $this->session->set($this->sessionKey, $times);

        return true;
    }

    /**
     * {@inheritdoc}
     */
    public function getDescription()
    {
        return $this->translator->transChoice(
            'description.session_download_strategy',
            $this->times,
            array('%times%' => $this->times),
            'SonataMediaBundle'
        );
    }
}


1			<?php
2
3			/*
4			* This file is part of the Sonata Project package.
5			*
6			* (c) Thomas Rabaix <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Sonata\MediaBundle\Security;
13
14			use Sonata\MediaBundle\Model\MediaInterface;
15			use Symfony\Component\DependencyInjection\ContainerInterface;
16			use Symfony\Component\HttpFoundation\Request;
17			use Symfony\Component\HttpFoundation\Session\SessionInterface;
18			use Symfony\Component\Translation\TranslatorInterface;
19
20			/**
21			* Class SessionDownloadStrategy.
22			*
23			* @author Ahmet Akbana <[email protected]>
24			*/
25			class SessionDownloadStrategy implements DownloadStrategyInterface
26			{
27			/**
28			* @var ContainerInterface
29			*
30			* @deprecated Since version 3.x, will be removed in 4.0.
31			* NEXT_MAJOR : remove this property
32			*/
33			protected $container;
34
35			/**
36			* @var TranslatorInterface
37			*/
38			protected $translator;
39
40			/**
41			* @var int
42			*/
43			protected $times;
44
45			/**
46			* @var string
47			*/
48			protected $sessionKey = 'sonata/media/session/times';
49
50			/**
51			* @var SessionInterface
52			*/
53			private $session;
54
55			/**
56			* @param TranslatorInterface $translator
57			* @param ContainerInterface\|SessionInterface $session
58			* @param int $times
59			*/
60			public function __construct(TranslatorInterface $translator, $session, $times)
61			{
62			// NEXT_MAJOR : remove this block and set session from parameter.
63			if ($session instanceof ContainerInterface) {
64			@trigger_error(
			0 ignored issues – show Security Best Practice introduced 2016-06-08 00:04 UTC by Report Bug Copy Issue Report It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended. If you suppress an error, we recommend checking for the error condition explicitly: // For example instead of @mkdir($dir); // Better use if (@mkdir($dir) === false) { throw new \RuntimeException('The directory '.$dir.' could not be created.'); } Loading history...
65			'Using an instance of Symfony\Component\DependencyInjection\ContainerInterface is deprecated since
66			version 3.x and will be removed in 4.0.
67			Use Symfony\Component\HttpFoundation\Session\SessionInterface instead.',
68			E_USER_DEPRECATED
69			);
70
71			$this->session = $session->get('session');
72			} elseif ($session instanceof SessionInterface) {
73			$this->session = $session;
74			} else {
75			throw new \InvalidArgumentException(
76			'$session should be an instance of Symfony\Component\HttpFoundation\Session\SessionInterface'
77			);
78			}
79
80			$this->times = $times;
81			$this->translator = $translator;
82			}
83
84			/**
85			* {@inheritdoc}
86			*/
87			public function isGranted(MediaInterface $media, Request $request)
88			{
89			$times = $this->session->get($this->sessionKey, 0);
90
91			if ($times >= $this->times) {
92			return false;
93			}
94
95			++$times;
96
97			$this->session->set($this->sessionKey, $times);
98
99			return true;
100			}
101
102			/**
103			* {@inheritdoc}
104			*/
105			public function getDescription()
106			{
107			return $this->translator->transChoice(
108			'description.session_download_strategy',
109			$this->times,
110			array('%times%' => $this->times),
111			'SonataMediaBundle'
112			);
113			}
114			}
115

sonata-project / SonataMediaBundle

Push — master ( ce34f1...a61f8e )

SessionDownloadStrategy::isGranted() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like