From 47dcc5411ec51b6e06fa3ae68e3fd0375b88df08 Mon Sep 17 00:00:00 2001
From: alec <alec@208e9e7b-5314-0410-a742-e7e81cd9613c>
Date: Tue, 19 Apr 2011 08:23:31 +0000
Subject: - Fix regression in html conditional comments handling by washtml
 class

git-svn-id: https://svn.roundcube.net/trunk@4667 208e9e7b-5314-0410-a742-e7e81cd9613c
---
 roundcubemail/tests/mailfunc.php | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

(limited to 'roundcubemail/tests/mailfunc.php')

diff --git a/roundcubemail/tests/mailfunc.php b/roundcubemail/tests/mailfunc.php
index dc25f0ce4..5bf77b1df 100644
--- a/roundcubemail/tests/mailfunc.php
+++ b/roundcubemail/tests/mailfunc.php
@@ -121,7 +121,7 @@ class rcube_test_mailfunc extends UnitTestCase
   function test_mailto()
   {
     $part = $this->get_html_part('src/mailto.txt');
-    
+
     // render HTML in normal mode
     $html = rcmail_html4inline(rcmail_print_body($part, array('safe' => false)), 'foo');
 
@@ -131,4 +131,18 @@ class rcube_test_mailfunc extends UnitTestCase
     $this->assertPattern('|'.preg_quote($mailto, '|').'|', $html, "Extended mailto links");
   }
 
+  /**
+   * Test the elimination of HTML comments
+   */
+  function test_html_comments()
+  {
+    $part = $this->get_html_part('src/htmlcom.txt');
+    $washed = rcmail_print_body($part, array('safe' => true));
+
+    // #1487759
+    $this->assertPattern('|<p>test1</p>|', $washed, "Buggy HTML comments");
+    // but conditional comments (<!--[if ...) should be removed
+    $this->assertNoPattern('|<p>test2</p>|', $washed, "Conditional HTML comments");
+  }
+
 }
-- 
cgit v1.2.3