Fix for #7842

Benjamin Renard
1 parent b9a2fc5c
Showing 3 changed files with 8 additions and 5 deletions Show diff stats
src/InputOutput/IHMImpl/Params/IHMInputOutputParamsAbstractClass.php
src/Request/ParamsRequestImpl/Nodes/NodeClass.php
src/Request/ParamsRequestImpl/ParamsRequestClass.php
@@ -54,7 +54,9 @@ abstract class IHMInputOutputParamsAbstractClass implements InputOutputInterface
 	 */
 	protected function getTask($input)
 	{
-		return $input->nodeType;
+		if (isset($input->nodeType))
+			return $input->nodeType;
+		return NULL;
 	}
  
 	/*
@@ -116,7 +116,7 @@ class NodeClass
 		$xmlNode = $doc->createElement($this->getName());
  
 		if ($this->getValue() != "")
-			$xmlNode->nodeValue = $this->getValue();
+			$xmlNode->nodeValue = htmlspecialchars($this->getValue());
  
 		foreach ($this->attributes as $key => $value)
 			$xmlNode->setAttribute($key,$value);
@@ -233,9 +233,10 @@ class ParamsRequestClass extends ProcessRequestClass
 	public function run()
 	{
 		$amdaStat = new AmdaStats(IHMConfigClass::getUserName());
-		
-		if (count($this->requestData->getDatasetForStat()) > 0)
-			$amdaStat->addTask(IHMConfigClass::getUserName(), $this->requestData->getTask(), $this->requestData->getDatasetForStat());
+
+		$task = $this->requestData->getTask();
+		if (isset($task) && (count($this->requestData->getDatasetForStat()) > 0))
+			$amdaStat->addTask(IHMConfigClass::getUserName(), $task, $this->requestData->getDatasetForStat());
  
 		return parent::run();
 	}