lab1521 / NeatyHTML

Cleans up your HTML. Useful for writing content or blogs that accepts HTML markup. Requires the PHP tidy class. Please install this extension first.
MIT License
22 stars 0 forks source link

tidy_parse_string(): Unknown Tidy configuration option "drop-font-tags" #1

Open tricoos opened 3 years ago

tricoos commented 3 years ago

Does not work anymore with Tidy 5.8.0, also even in 5.2.0 the option "drop-font-tags" is marked as deprecated: https://api.html-tidy.org/tidy/quickref_5.2.0.html#drop-font-tags

Would've been easy to circumvent if there was an option to change the config, however that it not possible. I found a workaround in deriving the class as thankfully the $tidyConfig has been set to protected.

sdmarcz commented 5 months ago

Does not work anymore with Tidy 5.8.0, also even in 5.2.0 the option "drop-font-tags" is marked as deprecated: https://api.html-tidy.org/tidy/quickref_5.2.0.html#drop-font-tags

Would've been easy to circumvent if there was an option to change the config, however that it not possible. I found a workaround in deriving the class as thankfully the $tidyConfig has been set to protected.

Thanks @tricoos for looking into this. If there is still an appetite for this library since it's been a long time then I would look into updating this library. I haven't used Laravel for many years now and probably there's already other better ways to handle html validations which probably you already knew for Laravel. Would be good to know if there is one that is actively maintained.