When i include a configuration file, htmltidy adds a complete htmlpage (see appended screenshot) with the content of the configuration file in front of the regular drupal html code.

The content of the config file doesn't matter, happens even if invalid or empty.

I got the latest versions of the htmltidy module and executable (tried both a windows and linux server) and didn't change any configuration options.

I tried playing with the command line parameters for htmltidy, but couldn't figure out where the error comes from, except it is caused by the executable.

CommentFileSizeAuthor
htmltidyerror.png8.53 KBxerxus@drupalcenter.de
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drewish’s picture

i've committed some new tidy code. once the new package is up, download that let me know if it works any better.

xerxus@drupalcenter.de’s picture

yes, since the problem still exists since it's caused by the executable.

it seems, that htmltidy sends the code directly to the browser.
I, for one couldn't figure out whereelse the code should come from.

The temporary page htmltidy returns to the module looks proper.

honestbleeps’s picture

Hi. I believe I've solved this (2+ year old) problem, which is way too late for those who posted it, but....

We got HTMLTidy running on a Drupal site of ours and couldn't get it to work for some reason. We wanted hide-comments active, and there wasn't an option for that in the module, so instead of modifying the module we went the route of a config file. 2 problems:

1) No error message for a nonexistent config file.
2) Broken implementation of using a config file (may not have been broken at the time of release, I'm not sure, but it's broken now!)

The issue, after hours of tracking it down, was the use of --config instead of -config as a commandline parameter by the htmltidy module. I simply removed one of the dashes and voila - tidy works!

annajonna’s picture

How about closing this bug. :)

michaelfavia’s picture

Status: Active » Fixed

Patched in dev will release soon. Thank you for posting this.

Status: Fixed » Closed (fixed)

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