Issues in ganon.php - Fixed Issues - Inspection of "some improvements" - tburry/pquery - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#16)

unknown

created 2018-02-27 17:38 UTC

ganon.php (1 issue)

Labels

Bug 1

Severity

Major 1

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php
/**
 * @author Niels A.D.
 * @author Todd Burry <[email protected]>
 * @copyright 2010 Niels A.D., 2014 Todd Burry
 * @license http://opensource.org/licenses/LGPL-2.1 LGPL-2.1
 * @package pQuery
 */

use pQuery\Html5Parser;
use pQuery\HtmlFormatter;

/**
 * Returns HTML DOM from string
 * @param string $str
 * @param bool $return_root Return root node or return parser object
 * @return Html5Parser|DomNode
 */
function str_get_dom($str, $return_root = true) {
	$a = new Html5Parser($str);
	return (($return_root) ? $a->root : $a);

}

/**
 * Returns HTML DOM from file/website
 * @param string $str
 * @param bool $return_root Return root node or return parser object
 * @param bool $use_include_path Use include path search in file_get_contents
 * @param resource $context Context resource used in file_get_contents (PHP >= 5.0.0)
 * @return Html5Parser|DomNode
 */
function file_get_dom($file, $return_root = true, $use_include_path = false, $context = null) {
	if (version_compare(PHP_VERSION, '5.0.0', '>='))
		$f = file_get_contents($file, $use_include_path, $context);
	else {
		if ($context !== null)
			trigger_error('Context parameter not supported in this PHP version');
		$f = file_get_contents($file, $use_include_path);
	}

	return (($f === false) ? false : str_get_dom($f, $return_root));
}

/**
 * Format/beautify DOM
 * @param DomNode $root
 * @param array $options Extra formatting options {@link Formatter::$options}
 * @return bool
 */
function dom_format(&$root, $options = array()) {
	$formatter = new HtmlFormatter($options);
	return $formatter->format($root);
}

if (version_compare(PHP_VERSION, '5.0.0', '<')) {
	/**
	 * PHP alternative to str_split, for backwards compatibility
	 * @param string $string
	 * @return string
	 */
	function str_split($string) {
		$res = array();
		$size = strlen($string);
		for ($i = 0; $i < $size; $i++) {
			$res[] = $string[$i];
		}

		return $res;
	}
}

if (version_compare(PHP_VERSION, '5.2.0', '<')) {
	/**
	 * PHP alternative to array_fill_keys, for backwards compatibility
	 * @param array $keys
	 * @param mixed $value
	 * @return array
	 */
	function array_fill_keys($keys, $value) {
		$res = array();
		foreach($keys as $k) {
			$res[$k] = $value;
		}

		return $res;
	}
}

#!! <- Ignore when converting to single file
if (!defined('GANON_NO_INCLUDES')) {
	define('GANON_NO_INCLUDES', true);
    include_once('IQuery.php');
	include_once('gan_tokenizer.php');
	include_once('gan_parser_html.php');
	include_once('gan_node_html.php');
	include_once('gan_selector_html.php');
	include_once('gan_formatter.php');
}
#!

?>

1			<?php
2			/**
3			* @author Niels A.D.
4			* @author Todd Burry <[email protected]>
5			* @copyright 2010 Niels A.D., 2014 Todd Burry
6			* @license http://opensource.org/licenses/LGPL-2.1 LGPL-2.1
7			* @package pQuery
8			*/
9
10			use pQuery\Html5Parser;
11			use pQuery\HtmlFormatter;
12
13			/**
14			* Returns HTML DOM from string
15			* @param string $str
16			* @param bool $return_root Return root node or return parser object
17			* @return Html5Parser\|DomNode
18			*/
19			function str_get_dom($str, $return_root = true) {
20			$a = new Html5Parser($str);
21			return (($return_root) ? $a->root : $a);
			0 ignored issues – show Bug introduced 2016-01-14 17:15 UTC by Report Bug Copy Issue Report Show Similar Issues like this The property `root` cannot be accessed from this context as it is declared `private` in class `pQuery\HtmlParser`. This check looks for access to properties that are not accessible from the current context. If you need to make a property accessible to another context you can either raise its visibility level or provide an accessible getter in the defining class. Loading history...
22			}
23
24			/**
25			* Returns HTML DOM from file/website
26			* @param string $str
27			* @param bool $return_root Return root node or return parser object
28			* @param bool $use_include_path Use include path search in file_get_contents
29			* @param resource $context Context resource used in file_get_contents (PHP >= 5.0.0)
30			* @return Html5Parser\|DomNode
31			*/
32			function file_get_dom($file, $return_root = true, $use_include_path = false, $context = null) {
33			if (version_compare(PHP_VERSION, '5.0.0', '>='))
34			$f = file_get_contents($file, $use_include_path, $context);
35			else {
36			if ($context !== null)
37			trigger_error('Context parameter not supported in this PHP version');
38			$f = file_get_contents($file, $use_include_path);
39			}
40
41			return (($f === false) ? false : str_get_dom($f, $return_root));
42			}
43
44			/**
45			* Format/beautify DOM
46			* @param DomNode $root
47			* @param array $options Extra formatting options {@link Formatter::$options}
48			* @return bool
49			*/
50			function dom_format(&$root, $options = array()) {
51			$formatter = new HtmlFormatter($options);
52			return $formatter->format($root);
53			}
54
55			if (version_compare(PHP_VERSION, '5.0.0', '<')) {
56			/**
57			* PHP alternative to str_split, for backwards compatibility
58			* @param string $string
59			* @return string
60			*/
61			function str_split($string) {
62			$res = array();
63			$size = strlen($string);
64			for ($i = 0; $i < $size; $i++) {
65			$res[] = $string[$i];
66			}
67
68			return $res;
69			}
70			}
71
72			if (version_compare(PHP_VERSION, '5.2.0', '<')) {
73			/**
74			* PHP alternative to array_fill_keys, for backwards compatibility
75			* @param array $keys
76			* @param mixed $value
77			* @return array
78			*/
79			function array_fill_keys($keys, $value) {
80			$res = array();
81			foreach($keys as $k) {
82			$res[$k] = $value;
83			}
84
85			return $res;
86			}
87			}
88
89			#!! <- Ignore when converting to single file
90			if (!defined('GANON_NO_INCLUDES')) {
91			define('GANON_NO_INCLUDES', true);
92			include_once('IQuery.php');
93			include_once('gan_tokenizer.php');
94			include_once('gan_parser_html.php');
95			include_once('gan_node_html.php');
96			include_once('gan_selector_html.php');
97			include_once('gan_formatter.php');
98			}
99			#!
100
101			?>

tburry / pquery

Pull Request — master (#16)

ganon.php (1 issue)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like