diff --git a/docs/spec.txt b/docs/spec.txt index 2b4684b2..e4a60d42 100644 --- a/docs/spec.txt +++ b/docs/spec.txt @@ -63,10 +63,6 @@ hold everything in an associative $info array, if it's set, it's valid, and we can include. Otherwise zap it, or attempt to figure out what they meant. ? A misspelling of ! This feature may be too sugary though. -While we're at it, we can change the Processing Instructions and Java/ASP -Instructions into data blocks, scratch comment blocks, change CharacterData -into Data (although I don't see why we can't do that at the start). - One last thing: the remove foreign elements has to do the element transformations, from FONT to SPAN, etc. diff --git a/library/HTMLPurifier/AttrDef/ID.php b/library/HTMLPurifier/AttrDef/ID.php index a88386ba..bc3015f6 100644 --- a/library/HTMLPurifier/AttrDef/ID.php +++ b/library/HTMLPurifier/AttrDef/ID.php @@ -6,7 +6,7 @@ require_once 'HTMLPurifier/IDAccumulator.php'; class HTMLPurifier_AttrDef_ID extends HTMLPurifier_AttrDef { - function validate($id, $accumulator) { + function validate($id, &$accumulator) { $id = @ (string) $id; // sanity check diff --git a/tests/HTMLPurifier/Strategy/CompositeTest.php b/tests/HTMLPurifier/Strategy/CompositeTest.php index b2ef7a06..769971f7 100644 --- a/tests/HTMLPurifier/Strategy/CompositeTest.php +++ b/tests/HTMLPurifier/Strategy/CompositeTest.php @@ -29,7 +29,7 @@ class HTMLPurifier_Strategy_CompositeTest extends UnitTestCase // setup the object - $strategies = array($mock_1, $mock_2, $mock_3); + $strategies = array(&$mock_1, &$mock_2, &$mock_3); $composite = new HTMLPurifier_Strategy_Composite_Test($strategies); // setup expectations