Add feedback from nickvergessen 07.11.17

Author: Crizz0

src/Phpbb/TranslationValidator/Validator/LangKeyValidator.php

@@ -675,26 +675,27 @@ public function validateHtml($file, $key, $sourceString, $originString)
 				$ignoreAdditional = true;
 			}
 
-			// missing closingTag allowed in: <br>, <hr>
+			// missing closingTag allowed in: <br>, <hr>, <img>
 			$allowedMissingClosingTag = array(
 				'br',
 				'hr',
+				'img',
 			);
 
 			$tag = (strpos($possibleHtml, ' ') !== false) ? substr($possibleHtml, 1, strpos($possibleHtml, ' ') - 1) : substr($possibleHtml, 1, strpos($possibleHtml, '>') - 1);
 			$tag = ($openingTag) ? $tag : substr($tag, 1);
 
 			if ($openingTag)
 			{
-				if (in_array($tag, $openTags) && !in_array($tag, $allowedMissingClosingTag))
+				if (in_array($tag, $openTags))
 				{
 					if (!$failedUnclosed)
 					{
 						$this->output->addMessage(Output::FATAL, 'String is missing closing tag for html: ' . $tag, $file, $key);
 					}
 					$failedUnclosed = true;
 				}
-				else if (substr($possibleHtml, -3) !== ' />')
+				else if ((substr($possibleHtml, -3) !== ' />') && !in_array($tag, $allowedMissingClosingTag))
 				{
 					$openTags[] = $tag;
 				}
@@ -755,7 +756,7 @@ public function validateHtml($file, $key, $sourceString, $originString)
 			}
 		}
 
-		if (!empty($openTags) && !$failedUnclosed && !in_array($openTags[0], $allowedMissingClosingTag))
+		if (!empty($openTags) && !$failedUnclosed)
 		{
 			$this->output->addMessage(Output::FATAL, 'String is missing closing tag for html: ' . $openTags[0], $file, $key);
 		}