[1.0.1] Fixed rejection of inline style declarations that had lots of extra space in them. This manifested in TinyMCE.

git-svn-id: http://htmlpurifier.org/svnroot/htmlpurifier/trunk@382 48356398-32a2-884e-a903-53898d9a118a

NEWS

@@ -9,6 +9,8 @@ NEWS ( CHANGELOG and HISTORY )                                     HTMLPurifier
 - Fixed slight bug in DOMLex attribute parsing
 - Fixed rejection of case-insensitive configuration values when there is a
   set of allowed values.  This manifested in %Core.Encoding.
+- Fixed rejection of inline style declarations that had lots of extra
+  space in them.  This manifested in TinyMCE.
 
 1.0.0, released 2006-09-01
 - Fixed broken numeric entity conversion

library/HTMLPurifier/AttrDef/CSS.php

@@ -28,6 +28,8 @@ class HTMLPurifier_AttrDef_CSS extends HTMLPurifier_AttrDef
             if (!$declaration) continue;
             if (!strpos($declaration, ':')) continue;
             list($property, $value) = explode(':', $declaration, 2);
+            $property = trim($property);
+            $value    = trim($value);
             if (!isset($definition->info[$property])) continue;
             // inefficient call, since the validator will do this again
             if (strtolower(trim($value)) !== 'inherit') {

tests/HTMLPurifier/AttrDef/CSSTest.php

@@ -92,6 +92,10 @@ class HTMLPurifier_AttrDef_CSSTest extends HTMLPurifier_AttrDefHarness
         $this->assertDef('position:absolute;', false);
         $this->assertDef('background-image:url(javascript:alert\(\));', false);
         
+        // airy input
+        $this->assertDef(' font-weight : bold; color : #ff0000',
+                         'font-weight:bold;color:#ff0000;');
+        
     }
     
 }

tests/HTMLPurifier/Test.php

@@ -8,7 +8,8 @@ class HTMLPurifier_Test extends UnitTestCase
 {
     var $purifier;
     
-    function assertPurification($input, $expect) {
+    function assertPurification($input, $expect = null) {
+        if ($expect === null) $expect = $input;
         $result = $this->purifier->purify($input);
         $this->assertIdentical($expect, $result);
     }