[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:
+ 
+ [&nbsp;<a style="text-decoration: none;"
+ href="http://bbc.internetia.pl/news.html?info=21889">more
+ online...</a>&nbsp;]
+ 
+ The IMP generates:
+ 
+ [&nbsp;<cleaned_tag style="text-decoration: none;"
+ href="http://bbc.internetia.pl/news.html?info=21889">more
+ online...</a>&nbsp;]
+ 
+ It should generate:
+ 
+ [&nbsp;<a cleaned="text-decoration: none;"
+ href="http://bbc.internetia.pl/news.html?info=21889">more
+ online...</a>&nbsp;]
+ 
+ 
+