Template::convertHtmlToAjaxXml() - Code Metrics - Inspection of "Merge pull request #1446 from hemberger/error-hand..." - smrealms/smr - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — main ( 705f36...a28172 )

by Dan

created 2022-10-05 15:49 UTC

Template::convertHtmlToAjaxXml() F

↳ Parent: Template

Complexity

Conditions	20
Paths	769

Size

Total Lines	80
Code Lines	46

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	20
eloc	46
nc	769
nop	2
dl	0
loc	80
rs	0.3207
c	0
b	0
f	0

How to fix Long Method Complexity

<?php declare(strict_types=1);

namespace Smr;

use DOMDocument;
use DOMNode;
use DOMXPath;
use Exception;
use Smr\Container\DiContainer;

class Template {

	/** @var array<string, mixed> */
	private array $data = [];
	private int $nestedIncludes = 0;
	/** @var array<string, mixed> */
	private array $ajaxJS = [];
	/** @var array<string> */
	protected array $jsAlerts = [];
	/** @var array<string> */
	protected array $jsSources = [];

	/**
	 * Defines a listjs_include.js function to call at the end of the HTML body.
	 */
	public ?string $listjsInclude = null;

	/**
	 * Return the Smr\Template in the DI container.
	 * If one does not exist yet, it will be created.
	 * This is the intended way to construct this class.
	 */
	public static function getInstance(): self {
		return DiContainer::get(self::class);
	}

	public function hasTemplateVar(string $var): bool {
		return isset($this->data[$var]);
	}

	public function assign(string $var, mixed $value): void {
		if (!isset($this->data[$var])) {
			$this->data[$var] = $value;
		} else {
			// We insist that template variables not change once they are set
			throw new Exception("Cannot re-assign template variable '$var'!");
		}
	}

	public function unassign(string $var): void {
		unset($this->data[$var]);
	}

	/**
	 * Displays the template HTML. Stores any ajax-enabled elements for future
	 * comparison, and outputs modified elements in XML for ajax if requested.
	 */
	public function display(string $templateName, bool $outputXml = false): void {
		// If we already started output buffering before calling `display`,
		// we may have unwanted content in the buffer that we need to remove
		// before we send the Content-Type headers below.
		// Skip this for debug builds to help discover offending output.
		if (!ENABLE_DEBUG) {
			if (ob_get_length() > 0) {
				ob_clean();
			}
		}
		ob_start();
		$this->includeTemplate($templateName);
		$output = ob_get_clean();
		if ($output === false) {
			throw new Exception('Output buffering is not active!');
		}

		$ajaxEnabled = ($this->data['AJAX_ENABLE_REFRESH'] ?? false) !== false;
		if ($ajaxEnabled) {
			$ajaxXml = $this->convertHtmlToAjaxXml($output, $outputXml);
			if ($outputXml) {
				/* Left out for size: <?xml version="1.0" encoding="ISO-8859-1"?>*/
				$output = '<all>' . $ajaxXml . '</all>';
			}
			$session = Session::getInstance();
			$session->saveAjaxReturns();
		}

		// Now that we are completely done processing, we can output
		if ($outputXml) {
			header('Content-Type: text/xml; charset=utf-8');
		} else {
			header('Content-Type: text/html; charset=utf-8');
		}
		echo $output;
	}


	protected function getTemplateLocation(string $templateName): string {
		if (isset($this->data['ThisAccount'])) {
			$templateDir = $this->data['ThisAccount']->getTemplate() . '/';
		} else {
			$templateDir = 'Default/';
		}
		$templateDirs = array_unique([$templateDir, 'Default/']);
		$gameDirs = array_unique([get_game_dir(), 'Default/']);

		foreach ($gameDirs as $gameDir) {
			foreach ($templateDirs as $templateDir) {
				$filePath = TEMPLATES . $templateDir . 'engine/' . $gameDir . $templateName;
				if (is_file($filePath)) {
					return $filePath;
				}
			}
		}
		foreach ($templateDirs as $templateDir) {
			$filePath = TEMPLATES . $templateDir . $templateName;
			if (is_file($filePath)) {
				return $filePath;
			}
		}
		throw new Exception('No template found for ' . $templateName);
	}

	/**
	 * @param array<string, mixed> $assignVars
	 */
	protected function includeTemplate(string $templateName, array $assignVars = null): void {
		if ($this->nestedIncludes > 15) {
			throw new Exception('Nested more than 15 template includes, is something wrong?');
		}
		foreach ($this->data as $key => $value) {
			$$key = $value;
		}
		if ($assignVars !== null) {
			foreach ($assignVars as $key => $value) {
				$$key = $value;
			}
		}
		$this->nestedIncludes++;
		require($this->getTemplateLocation($templateName));
		$this->nestedIncludes--;
	}

	/**
	 * Check if the HTML includes input elements where the user is able to
	 * input data (i.e. we don't want to AJAX update a form that they may
	 * have already started filling out).
	 */
	protected function checkDisableAJAX(string $html): bool {
		return preg_match('/<input (?![^>]*(submit|hidden|image))/i', $html) != 0;
	}

	protected function doDamageTypeReductionDisplay(int &$damageTypes): void {
		if ($damageTypes == 3) {
			echo ', ';
		} elseif ($damageTypes == 2) {
			echo ' and ';
		}
		$damageTypes--;
	}

	protected function doAn(string $wordAfter): string {
		$char = strtoupper($wordAfter[0]);
		return str_contains('AEIOU', $char) ? 'an' : 'a';
	}

	/*
	 * EVAL is special (well, will be when needed and implemented in the javascript).
	 */
	public function addJavascriptForAjax(string $varName, mixed $obj): string {
		if ($varName == 'EVAL') {
			if (!isset($this->ajaxJS['EVAL'])) {
				return $this->ajaxJS['EVAL'] = $obj;
			}
			return $this->ajaxJS['EVAL'] .= ';' . $obj;
		}

		if (isset($this->ajaxJS[$varName])) {
			throw new Exception('Trying to set javascript val twice: ' . $varName);
		}
		return $this->ajaxJS[$varName] = json_encode($obj);
	}

	protected function addJavascriptAlert(string $string): void {
		$session = Session::getInstance();
		if (!$session->addAjaxReturns('ALERT:' . $string, $string)) {
			$this->jsAlerts[] = $string;
		}
	}

	/**
	 * Registers a JS target for inclusion at the end of the HTML body.
	 */
	protected function addJavascriptSource(string $src): void {
		$this->jsSources[] = $src;
	}

	protected function convertHtmlToAjaxXml(string $str, bool $returnXml): string {
		if (empty($str)) {
			return '';
		}

		$session = Session::getInstance();

		$getInnerHTML = function(DOMNode $node): string {
			$innerHTML = '';
			foreach ($node->childNodes as $child) {
				$innerHTML .= $child->ownerDocument->saveHTML($child);
			}
			return $innerHTML;
		};

		// Helper function to canonicalize making an XML element,
		// with its inner content properly escaped.
		$xmlify = function(string $id, string $str): string {
			return '<' . $id . '>' . htmlspecialchars($str, ENT_XML1, 'utf-8') . '</' . $id . '>';
		};

		$xml = '';
		$dom = new DOMDocument();
		$dom->loadHTML($str);
		$xpath = new DOMXPath($dom);

		// Use relative xpath selectors so that they can be reused when we
		// pass the middle panel as the xpath query's context node.
		$ajaxSelectors = ['.//span[@id]', './/*[contains(@class,"ajax")]'];

		foreach ($ajaxSelectors as $selector) {
			$matchNodes = $xpath->query($selector);
			foreach ($matchNodes as $node) {
				$id = $node->getAttribute('id');
				$inner = $getInnerHTML($node);
				if (!$session->addAjaxReturns($id, $inner) && $returnXml) {
					$xml .= $xmlify($id, $inner);
				}
			}
		}

		// Determine if we should do ajax updates on the middle panel div
		$mid = $dom->getElementById('middle_panel');
		$doAjaxMiddle = true;
		if ($mid === null) {
			// Skip if there is no middle_panel.
			$doAjaxMiddle = false;
		} else {
			// Skip if middle_panel has ajax-enabled children.
			foreach ($ajaxSelectors as $selector) {
				if (count($xpath->query($selector, $mid)) > 0) {
					$doAjaxMiddle = false;
					break;
				}
			}
		}

		if ($doAjaxMiddle) {
			$inner = $getInnerHTML($mid);
			if (!$this->checkDisableAJAX($inner)) {
				$id = $mid->getAttribute('id');
				if (!$session->addAjaxReturns($id, $inner) && $returnXml) {
					$xml .= $xmlify($id, $inner);
				}
			}
		}

		$js = '';
		foreach ($this->ajaxJS as $varName => $JSON) {
			if (!$session->addAjaxReturns('JS:' . $varName, $JSON) && $returnXml) {
				$js .= $xmlify($varName, $JSON);
			}
		}
		if ($returnXml && count($this->jsAlerts) > 0) {
			$js = '<ALERT>' . json_encode($this->jsAlerts) . '</ALERT>';
		}
		if (strlen($js) > 0) {
			$xml .= '<JS>' . $js . '</JS>';
		}
		return $xml;
	}

}


1			<?php declare(strict_types=1);
2
3			namespace Smr;
4
5			use DOMDocument;
6			use DOMNode;
7			use DOMXPath;
8			use Exception;
9			use Smr\Container\DiContainer;
10
11			class Template {
12
13			/** @var array<string, mixed> */
14			private array $data = [];
15			private int $nestedIncludes = 0;
16			/** @var array<string, mixed> */
17			private array $ajaxJS = [];
18			/** @var array<string> */
19			protected array $jsAlerts = [];
20			/** @var array<string> */
21			protected array $jsSources = [];
22
23			/**
24			* Defines a listjs_include.js function to call at the end of the HTML body.
25			*/
26			public ?string $listjsInclude = null;
27
28			/**
29			* Return the Smr\Template in the DI container.
30			* If one does not exist yet, it will be created.
31			* This is the intended way to construct this class.
32			*/
33			public static function getInstance(): self {
34			return DiContainer::get(self::class);
35			}
36
37			public function hasTemplateVar(string $var): bool {
38			return isset($this->data[$var]);
39			}
40
41			public function assign(string $var, mixed $value): void {
42			if (!isset($this->data[$var])) {
43			$this->data[$var] = $value;
44			} else {
45			// We insist that template variables not change once they are set
46			throw new Exception("Cannot re-assign template variable '$var'!");
47			}
48			}
49
50			public function unassign(string $var): void {
51			unset($this->data[$var]);
52			}
53
54			/**
55			* Displays the template HTML. Stores any ajax-enabled elements for future
56			* comparison, and outputs modified elements in XML for ajax if requested.
57			*/
58			public function display(string $templateName, bool $outputXml = false): void {
59			// If we already started output buffering before calling `display`,
60			// we may have unwanted content in the buffer that we need to remove
61			// before we send the Content-Type headers below.
62			// Skip this for debug builds to help discover offending output.
63			if (!ENABLE_DEBUG) {
64			if (ob_get_length() > 0) {
65			ob_clean();
66			}
67			}
68			ob_start();
69			$this->includeTemplate($templateName);
70			$output = ob_get_clean();
71			if ($output === false) {
72			throw new Exception('Output buffering is not active!');
73			}
74
75			$ajaxEnabled = ($this->data['AJAX_ENABLE_REFRESH'] ?? false) !== false;
76			if ($ajaxEnabled) {
77			$ajaxXml = $this->convertHtmlToAjaxXml($output, $outputXml);
78			if ($outputXml) {
79			/* Left out for size: <?xml version="1.0" encoding="ISO-8859-1"?>*/
80			$output = '<all>' . $ajaxXml . '</all>';
81			}
82			$session = Session::getInstance();
83			$session->saveAjaxReturns();
84			}
85
86			// Now that we are completely done processing, we can output
87			if ($outputXml) {
88			header('Content-Type: text/xml; charset=utf-8');
89			} else {
90			header('Content-Type: text/html; charset=utf-8');
91			}
92			echo $output;
93			}
94
95
96			protected function getTemplateLocation(string $templateName): string {
97			if (isset($this->data['ThisAccount'])) {
98			$templateDir = $this->data['ThisAccount']->getTemplate() . '/';
99			} else {
100			$templateDir = 'Default/';
101			}
102			$templateDirs = array_unique([$templateDir, 'Default/']);
103			$gameDirs = array_unique([get_game_dir(), 'Default/']);
104
105			foreach ($gameDirs as $gameDir) {
106			foreach ($templateDirs as $templateDir) {
107			$filePath = TEMPLATES . $templateDir . 'engine/' . $gameDir . $templateName;
108			if (is_file($filePath)) {
109			return $filePath;
110			}
111			}
112			}
113			foreach ($templateDirs as $templateDir) {
114			$filePath = TEMPLATES . $templateDir . $templateName;
115			if (is_file($filePath)) {
116			return $filePath;
117			}
118			}
119			throw new Exception('No template found for ' . $templateName);
120			}
121
122			/**
123			* @param array<string, mixed> $assignVars
124			*/
125			protected function includeTemplate(string $templateName, array $assignVars = null): void {
126			if ($this->nestedIncludes > 15) {
127			throw new Exception('Nested more than 15 template includes, is something wrong?');
128			}
129			foreach ($this->data as $key => $value) {
130			$$key = $value;
131			}
132			if ($assignVars !== null) {
133			foreach ($assignVars as $key => $value) {
134			$$key = $value;
135			}
136			}
137			$this->nestedIncludes++;
138			require($this->getTemplateLocation($templateName));
139			$this->nestedIncludes--;
140			}
141
142			/**
143			* Check if the HTML includes input elements where the user is able to
144			* input data (i.e. we don't want to AJAX update a form that they may
145			* have already started filling out).
146			*/
147			protected function checkDisableAJAX(string $html): bool {
148			return preg_match('/<input (?![^>]*(submit\|hidden\|image))/i', $html) != 0;
149			}
150
151			protected function doDamageTypeReductionDisplay(int &$damageTypes): void {
152			if ($damageTypes == 3) {
153			echo ', ';
154			} elseif ($damageTypes == 2) {
155			echo ' and ';
156			}
157			$damageTypes--;
158			}
159
160			protected function doAn(string $wordAfter): string {
161			$char = strtoupper($wordAfter[0]);
162			return str_contains('AEIOU', $char) ? 'an' : 'a';
163			}
164
165			/*
166			* EVAL is special (well, will be when needed and implemented in the javascript).
167			*/
168			public function addJavascriptForAjax(string $varName, mixed $obj): string {
169			if ($varName == 'EVAL') {
170			if (!isset($this->ajaxJS['EVAL'])) {
171			return $this->ajaxJS['EVAL'] = $obj;
172			}
173			return $this->ajaxJS['EVAL'] .= ';' . $obj;
174			}
175
176			if (isset($this->ajaxJS[$varName])) {
177			throw new Exception('Trying to set javascript val twice: ' . $varName);
178			}
179			return $this->ajaxJS[$varName] = json_encode($obj);
180			}
181
182			protected function addJavascriptAlert(string $string): void {
183			$session = Session::getInstance();
184			if (!$session->addAjaxReturns('ALERT:' . $string, $string)) {
185			$this->jsAlerts[] = $string;
186			}
187			}
188
189			/**
190			* Registers a JS target for inclusion at the end of the HTML body.
191			*/
192			protected function addJavascriptSource(string $src): void {
193			$this->jsSources[] = $src;
194			}
195
196			protected function convertHtmlToAjaxXml(string $str, bool $returnXml): string {
197			if (empty($str)) {
198			return '';
199			}
200
201			$session = Session::getInstance();
202
203			$getInnerHTML = function(DOMNode $node): string {
204			$innerHTML = '';
205			foreach ($node->childNodes as $child) {
206			$innerHTML .= $child->ownerDocument->saveHTML($child);
207			}
208			return $innerHTML;
209			};
210
211			// Helper function to canonicalize making an XML element,
212			// with its inner content properly escaped.
213			$xmlify = function(string $id, string $str): string {
214			return '<' . $id . '>' . htmlspecialchars($str, ENT_XML1, 'utf-8') . '</' . $id . '>';
215			};
216
217			$xml = '';
218			$dom = new DOMDocument();
219			$dom->loadHTML($str);
220			$xpath = new DOMXPath($dom);
221
222			// Use relative xpath selectors so that they can be reused when we
223			// pass the middle panel as the xpath query's context node.
224			$ajaxSelectors = ['.//span[@id]', './/*[contains(@class,"ajax")]'];
225
226			foreach ($ajaxSelectors as $selector) {
227			$matchNodes = $xpath->query($selector);
228			foreach ($matchNodes as $node) {
229			$id = $node->getAttribute('id');
230			$inner = $getInnerHTML($node);
231			if (!$session->addAjaxReturns($id, $inner) && $returnXml) {
232			$xml .= $xmlify($id, $inner);
233			}
234			}
235			}
236
237			// Determine if we should do ajax updates on the middle panel div
238			$mid = $dom->getElementById('middle_panel');
239			$doAjaxMiddle = true;
240			if ($mid === null) {
241			// Skip if there is no middle_panel.
242			$doAjaxMiddle = false;
243			} else {
244			// Skip if middle_panel has ajax-enabled children.
245			foreach ($ajaxSelectors as $selector) {
246			if (count($xpath->query($selector, $mid)) > 0) {
247			$doAjaxMiddle = false;
248			break;
249			}
250			}
251			}
252
253			if ($doAjaxMiddle) {
254			$inner = $getInnerHTML($mid);
255			if (!$this->checkDisableAJAX($inner)) {
256			$id = $mid->getAttribute('id');
257			if (!$session->addAjaxReturns($id, $inner) && $returnXml) {
258			$xml .= $xmlify($id, $inner);
259			}
260			}
261			}
262
263			$js = '';
264			foreach ($this->ajaxJS as $varName => $JSON) {
265			if (!$session->addAjaxReturns('JS:' . $varName, $JSON) && $returnXml) {
266			$js .= $xmlify($varName, $JSON);
267			}
268			}
269			if ($returnXml && count($this->jsAlerts) > 0) {
270			$js = '<ALERT>' . json_encode($this->jsAlerts) . '</ALERT>';
271			}
272			if (strlen($js) > 0) {
273			$xml .= '<JS>' . $js . '</JS>';
274			}
275			return $xml;
276			}
277
278			}
279

smrealms / smr

Scrutinizer GitHub App not installed

Push — main ( 705f36...a28172 )

Template::convertHtmlToAjaxXml() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like