Closed (fixed)
Project:
HTML Purifier
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
24 Dec 2007 at 15:11 UTC
Updated:
28 Jan 2008 at 21:10 UTC
I get the following error in the log when an unauthenticated user is viewing content:
user warning: Value not supported, valid values are: Serializer in /kast/www2/ideed.anttila.ee/sites/all/modules/htmlpurifier/library/HTMLPurifier/Config.php on line 232.
Comments
Comment #1
yhager commentedComment #2
yhager commentedI am also getting this same error. Could it be relating to setting the cache incorrectly?
Comment #3
yhager commentedI managed to get rid of the error above by setting full permissions on the cache directory (as noted at http://htmlpurifier.org/download.html#toclink0), and also change the function htmlpurifier_create_purifier() to read:
I am not recommending this for production, since it doesn't use Drupal's caching like intended, but it works for the moment.
Module author: please step in to provide the correct solution. Thanks.
Comment #4
kripaludas commentedI was getting this error as well.
On the page http://htmlpurifier.org/ there are two downloads: HTML Purifier 2.1.3 zip and HTML Purifier 2.1.3 for PHP 5 zip. Because I have PHP 5 installed, naturally I tried using that version. However that version does not have Serializer and hence the errors.
I went back to http://htmlpurifier.org/ and got the first zip instead (the non-PHP5 version) [http://htmlpurifier.org/releases/htmlpurifier-2.1.3.zip]. I extracted the library directory from it, set owner and permissions for apache 755, and now no error. It seems to work fine.
Comment #5
icalvo commentedIt seems that the Drupal module does not extend the HTMLPurifier_ConfigSchema singleton to have it allow Drupal as a valid Definition Cache. I have inserted this line at
htmlpurifier.module:Just before:
which is the line that causes this error. So far, it is working.
Comment #6
ezyang commentedI need to figure out how to make all bug reports mail to me. My apologies for the late response. icalvo, your solution is correct and will be added to the CVS repository.
Comment #7
ezyang commentedClosed as fixed. If anyone would like to test the fix on another configuration, that would be greatly appreciated.