If HTML Purifier (advanced) is selected, then one of the advanced options, 'ExtractStyleBlocks' prompts use of CSS Tidy module

CSS Tidy is no longer maintained and does not have a drupal 7 branch:
http://drupal.org/project/csstidy

Comments

ezyang’s picture

Status: Active » Needs work

Aw, that's a shame. HTML Purifier doesn't actually need the CSS Tidy Drupal module per se, but just the CSS Tidy module. So if someone makes a stub project which just exports that library, that should probably get things working again.

heddn’s picture

With a stable build coming soon, should we remove the mention of css tidy so we can get out the door and re-add (if/when) said project is stood up?

ezyang’s picture

The mention seems fairly harmless... fix it or leave it alone, I'd say.

heddn’s picture

Title: ExtractStyleBlocks requires invallid module » ExtractStyleBlocks requires invalid module
StatusFileSize
new7.41 KB

Ok, here's a patch to try. If someone else will test it this week, we can probably still get it into the stable release. Otherwise, I'll wait and do some thorough testing later. However, it would seem that it doesn't adversely affect anything.

ezyang’s picture

I don't think removing this functionality is the right way. It's perfectly valid functionality, the only trouble is we need to stop depending on the csstidy Drupal module.

heddn’s picture

Version: 7.x-1.0-rc1 » 7.x-2.x-dev
Status: Needs work » Needs review
heddn’s picture

Status: Needs review » Fixed

I'm going to say this is fixed, unless someone disagrees.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.