[Bug 877] Changed - Cleaning style attribute for HTML documents [patch]
bugs@bugs.horde.org
bugs@bugs.horde.org
Mon, 18 Feb 2002 10:16:28 -0400
http://bugs.horde.org/show_bug.cgi?id=877
*** shadow/877 Mon Feb 18 06:39:25 2002
--- shadow/877.tmp.32531 Mon Feb 18 10:16:28 2002
***************
*** 3,9 ****
Version: 2.2 Stable
Platform: PHP Code
OS/Version: All
! Status: CLOSED
Resolution: INVALID
Severity: minor
Priority: P2
--- 3,9 ----
Version: 2.2 Stable
Platform: PHP Code
OS/Version: All
! Status: REOPENED
Resolution: INVALID
Severity: minor
Priority: P2
***************
*** 47,49 ****
--- 47,73 ----
No, IMP cleans out STYLE tags not STYLE attributes. And this is important
because the whole page gets messed up if we use style sheet definitions from
html emails that overwrite the Horde style sheets.
+
+ ------- Additional Comments From dexter@top.pl 02/18/02 10:16 -------
+ So my patch is not correct, but the problem still exists.
+
+ Example HTML code:
+
+ [ <a style="text-decoration: none;"
+ href="http://bbc.internetia.pl/news.html?info=21889">more
+ online...</a> ]
+
+ The IMP generates:
+
+ [ <cleaned_tag style="text-decoration: none;"
+ href="http://bbc.internetia.pl/news.html?info=21889">more
+ online...</a> ]
+
+ It should generate:
+
+ [ <a cleaned="text-decoration: none;"
+ href="http://bbc.internetia.pl/news.html?info=21889">more
+ online...</a> ]
+
+
+