PropertyParameterValueUtil::escapeParamValue() - Code Metrics - Inspection of "cleanup parameter bag" - markuspoerschke/iCal - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 1.x ( 8c7a9a...c7de71 )

by Markus

created 2016-04-26 17:09 UTC

PropertyParameterValueUtil::escapeParamValue() B

↳ Parent: PropertyParameterValueUtil

Complexity

Conditions	5
Paths	2

Size

Total Lines	13
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	13
rs	8.8571
cc	5
eloc	8
nc	2
nop	1

<?php

namespace Eluceo\iCal\Util;

abstract class PropertyParameterValueUtil
{
    /**
     * Returns an escaped string for a param value.
     *
     * @param string $value
     *
     * @return string
     */
    public static function escapeParamValue($value)
    {
        $count = 0;
        $value = str_replace('\\', '\\\\', $value);
        $value = str_replace('"', '\"', $value, $count);
        $value = str_replace("\n", '\\n', $value);

        if (false !== strpos($value, ';') || false !== strpos($value, ',') || false !== strpos($value, ':') || $count) {
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
            $value = '"' . $value . '"';
        }

        return $value;
    }
}


1			<?php
2
3			namespace Eluceo\iCal\Util;
4
5			abstract class PropertyParameterValueUtil
6			{
7			/**
8			* Returns an escaped string for a param value.
9			*
10			* @param string $value
11			*
12			* @return string
13			*/
14			public static function escapeParamValue($value)
15			{
16			$count = 0;
17			$value = str_replace('\\', '\\\\', $value);
18			$value = str_replace('"', '\"', $value, $count);
19			$value = str_replace("\n", '\\n', $value);
20
21			if (false !== strpos($value, ';') \|\| false !== strpos($value, ',') \|\| false !== strpos($value, ':') \|\| $count) {
			0 ignored issues – show Bug Best Practice introduced 2015-12-04 09:21 UTC by Report Bug Copy Issue Report The expression `$count` of type `integer\|null` is loosely compared to `true`; this is ambiguous if the integer can be zero. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
22			$value = '"' . $value . '"';
23			}
24
25			return $value;
26			}
27			}
28

markuspoerschke / iCal

Push — 1.x ( 8c7a9a...c7de71 )

PropertyParameterValueUtil::escapeParamValue() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like