[1.x] Fix case insensitvity and recursiveness in stripImages and stri…

…pIframes (#72)

Co-authored-by: Richard Fath <[email protected]>

Tests/OutputFilterTest.php

@@ -227,6 +227,18 @@ public function testStripImages()
 			$this->object->stripImages('Hello <img src="wave.jpg"> I am waving at you.'),
 			'Should remove img tags'
 		);
+
+		$this->assertEquals(
+			'Hello  I am waving at you.',
+			$this->object->stripImages('Hello <IMG src="wave.jpg"> I am waving at you.'),
+			'Should remove uppercase img tags'
+		);
+
+		$this->assertEquals(
+			'Hello  I am waving at you.',
+			$this->object->stripImages('Hello <<img>IMg src="wave.jpg"> I am waving at you.'),
+			'Should remove nested tags'
+		);
 	}
 
 	/**
@@ -244,5 +256,19 @@ public function testStripIframes()
 				' height="281" frameborder="0" webkitAllowFullScreen mozallowfullscreen allowFullScreen></iframe> I am waving at you.'),
 				'Should remove iFrame tags'
 		);
+
+		$this->assertEquals(
+			'Hello  I am waving at you.',
+			$this->object->stripIframes('Hello <IFrame src="http://player.vimeo.com/video/37576499" width="500"' .
+				' height="281" frameborder="0" webkitAllowFullScreen mozallowfullscreen allowFullScreen></iframe> I am waving at you.'),
+			'Should remove uppercase iFrame tags'
+		);
+
+		$this->assertEquals(
+			'Hello  I am waving at you.',
+			$this->object->stripIframes('Hello <<iframe>iframe src="http://player.vimeo.com/video/37576499" width="500"' .
+				' height="281" frameborder="0" webkitAllowFullScreen mozallowfullscreen allowFullScreen></iframe> I am waving at you.'),
+			'Should remove nested iFrame tags'
+		);
 	}
 }

src/OutputFilter.php

@@ -197,7 +197,12 @@ public static function cleanText(&$text)
 	 */
 	public static function stripImages($string)
 	{
-		return preg_replace('#(<[/]?img.*>)#U', '', $string);
+		while (preg_match('#(<[/]?img.*>)#Ui', $string))
+		{
+			$string = preg_replace('#(<[/]?img.*>)#Ui', '', $string);
+		}
+
+		return $string;
 	}
 
 	/**
@@ -211,6 +216,11 @@ public static function stripImages($string)
 	 */
 	public static function stripIframes($string)
 	{
-		return preg_replace('#(<[/]?iframe.*>)#U', '', $string);
+		while (preg_match('#(<[/]?iframe.*>)#Ui', $string))
+		{
+			$string = preg_replace('#(<[/]?iframe.*>)#Ui', '', $string);
+		}
+
+		return $string;
 	}
 }