escape.php ➔ esc() - Code Metrics - Inspection of "Merge pull request #168 from kfern-github/develop" - ci-bonfire/Sprint - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( e54387...b62a26 )

by Lonnie

created 2016-07-21 17:22 UTC

escape.php ➔ esc() C

↳ Parent: Project

Complexity

Conditions	11
Paths	32

Size

Total Lines	51
Code Lines	30

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	11
eloc	30
nc	32
nop	3
dl	0
loc	51
rs	5.7333
c	0
b	0
f	0

How to fix Long Method Complexity

<?php

use Zend\Escaper\Escaper;

if (! function_exists('esc'))
{
    /**
     * Escapes strings to make them safe for use
     * within HTML templates. Used by the auto-escaping
     * functionality in setVar() and available to
     * use within your views.
     *
     * Uses ZendFramework's Escaper to handle the actual escaping,
     * based on context. Valid contexts are:
     *      - html
     *      - htmlAttr
     *      - js
     *      - css
     *      - url
     *
     * References:
     *  - https://www.owasp.org/index.php/XSS_(Cross_Site_Scripting)_Prevention_Cheat_Sheet
     *  - http://framework.zend.com/manual/current/en/modules/zend.escaper.introduction.html
     *
     * @param $data
     * @param $context
     * @param escaper   // An instance of ZF's Escaper to avoid repeated class instantiation.
     *
     * @return string
     */
    function esc($data, $context='html', $escaper=null)
    {
        if (is_array($data))
        {
            foreach ($data as $key => &$value)
            {
                $value = esc($value, $context);
            }
        }

        $context = strtolower($context);

        if (! is_object($escaper))
        {
            $escaper = new Escaper(config_item('charset'));
        }

        // Valid context?
        if (! in_array($context, ['html', 'htmlattr', 'js', 'css', 'url']))
        {
            throw new \InvalidArgumentException('Invalid Context type: '. $context);
        }

        if (! is_string($data))
        {
            return $data;
        }

        switch ($context)
        {
            case 'html':
                $data = $escaper->escapeHtml($data);
                break;
            case 'htmlattr':
                $data = $escaper->escapeHtmlAttr($data);
                break;
            case 'js':
                $data = $escaper->escapeJs($data);
                break;
            case 'css':
                $data = $escaper->escapeCss($data);
                break;
            case 'url':
                $data = $escaper->escapeUrl($data);
                break;
            default:
                break;
        }

        return $data;
    }
}

1			<?php
2
3			use Zend\Escaper\Escaper;
4
5			if (! function_exists('esc'))
6			{
7			/**
8			* Escapes strings to make them safe for use
9			* within HTML templates. Used by the auto-escaping
10			* functionality in setVar() and available to
11			* use within your views.
12			*
13			* Uses ZendFramework's Escaper to handle the actual escaping,
14			* based on context. Valid contexts are:
15			* - html
16			* - htmlAttr
17			* - js
18			* - css
19			* - url
20			*
21			* References:
22			* - https://www.owasp.org/index.php/XSS_(Cross_Site_Scripting)_Prevention_Cheat_Sheet
23			* - http://framework.zend.com/manual/current/en/modules/zend.escaper.introduction.html
24			*
25			* @param $data
26			* @param $context
27			* @param escaper // An instance of ZF's Escaper to avoid repeated class instantiation.
28			*
29			* @return string
30			*/
31			function esc($data, $context='html', $escaper=null)
32			{
33			if (is_array($data))
34			{
35			foreach ($data as $key => &$value)
36			{
37			$value = esc($value, $context);
38			}
39			}
40
41			$context = strtolower($context);
42
43			if (! is_object($escaper))
44			{
45			$escaper = new Escaper(config_item('charset'));
46			}
47
48			// Valid context?
49			if (! in_array($context, ['html', 'htmlattr', 'js', 'css', 'url']))
50			{
51			throw new \InvalidArgumentException('Invalid Context type: '. $context);
52			}
53
54			if (! is_string($data))
55			{
56			return $data;
57			}
58
59			switch ($context)
60			{
61			case 'html':
62			$data = $escaper->escapeHtml($data);
63			break;
64			case 'htmlattr':
65			$data = $escaper->escapeHtmlAttr($data);
66			break;
67			case 'js':
68			$data = $escaper->escapeJs($data);
69			break;
70			case 'css':
71			$data = $escaper->escapeCss($data);
72			break;
73			case 'url':
74			$data = $escaper->escapeUrl($data);
75			break;
76			default:
77			break;
78			}
79
80			return $data;
81			}
82			}

ci-bonfire / Sprint

GitHub Access Token became invalid

Push — develop ( e54387...b62a26 )

escape.php ➔ esc() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like