FincludeBlock::display() - Code Metrics - Inspection of "Merge remote-tracking branch 'origin/1.4'" - zikula/core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 5d604f...33b193 )

by Craig

created 2017-01-10 13:02 UTC

FincludeBlock::display() B

↳ Parent: FincludeBlock

Complexity

Conditions	5
Paths	5

Size

Total Lines	23
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
eloc	17
nc	5
nop	1
dl	0
loc	23
rs	8.5906
c	0
b	0
f	0

<?php

/*
 * This file is part of the Zikula package.
 *
 * Copyright Zikula Foundation - http://zikula.org/
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Zikula\BlocksModule\Block;

use Zikula\BlocksModule\AbstractBlockHandler;

/**
 * Block to display the contents of a file given a path.
 */
class FincludeBlock extends AbstractBlockHandler
{
    public function display(array $properties)
    {
        if (!$this->hasPermission('fincludeblock::', "$properties[title]::", ACCESS_READ)) {
            return '';
        }

        switch ($properties['typo']) {
            case 0: // Html
                return file_get_contents($properties['filo']);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case 1: // Text
                return htmlspecialchars(file_get_contents($properties['filo']));
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case 2: // PHP
                ob_start();
                include \DataUtil::formatForOS($properties['filo']);

                return ob_get_clean();
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            default:
                return '';
        }
    }

    public function getFormClassName()
    {
        return 'Zikula\BlocksModule\Block\Form\Type\FincludeBlockType';
    }

    public function getFormOptions()
    {
        return ['translator' => $this->getTranslator()];
    }

    public function getType()
    {
        return $this->__("File Include");
    }
}


1			<?php
2
3			/*
4			* This file is part of the Zikula package.
5			*
6			* Copyright Zikula Foundation - http://zikula.org/
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 Zikula\BlocksModule\Block;
13
14			use Zikula\BlocksModule\AbstractBlockHandler;
15
16			/**
17			* Block to display the contents of a file given a path.
18			*/
19			class FincludeBlock extends AbstractBlockHandler
20			{
21			public function display(array $properties)
22			{
23			if (!$this->hasPermission('fincludeblock::', "$properties[title]::", ACCESS_READ)) {
24			return '';
25			}
26
27			switch ($properties['typo']) {
28			case 0: // Html
29			return file_get_contents($properties['filo']);
30			break;
			0 ignored issues – show Unused Code introduced 2016-12-17 15:07 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
31			case 1: // Text
32			return htmlspecialchars(file_get_contents($properties['filo']));
33			break;
			0 ignored issues – show Unused Code introduced 2016-12-17 15:07 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
34			case 2: // PHP
35			ob_start();
36			include \DataUtil::formatForOS($properties['filo']);
37
38			return ob_get_clean();
39			break;
			0 ignored issues – show Unused Code introduced 2016-12-17 15:07 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
40			default:
41			return '';
42			}
43			}
44
45			public function getFormClassName()
46			{
47			return 'Zikula\BlocksModule\Block\Form\Type\FincludeBlockType';
48			}
49
50			public function getFormOptions()
51			{
52			return ['translator' => $this->getTranslator()];
53			}
54
55			public function getType()
56			{
57			return $this->__("File Include");
58			}
59			}
60

zikula / core

Push — master ( 5d604f...33b193 )

FincludeBlock::display() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like