TinyMCE 'Verify HTML' setting ignored.

Committed the patch below to all major branches. The -dev snapshots will be updated within 12h and this will be part of the next official releases.

Thanks!

+++ editors/tinymce.inc
@@ -178,7 +178,7 @@ function wysiwyg_tinymce_settings($editor, $config, $theme) {
     $settings['remove_linebreaks'] = $config['remove_linebreaks'];
   }
   if (isset($config['verify_html'])) {
-    $settings['verify_html'] = $config['verify_html'];
+    $settings['verify_html'] = (bool) $config['verify_html'];

Does this only apply to verify_html and not to the other settings?

If so, we should add an inline comment, so no one mistakenly removes the (bool) in the future.

Powered by Dreditor.

What I could tell from the code, only verify_html is compared using ===, the rest just use if (setting).

+++ editors/tinymce.inc
@@ -178,6 +178,7 @@ function wysiwyg_tinymce_settings($editor, $config, $theme) {
+    // TinyMCE tests "=== false", so ensure it is a Boolean.
     $settings['verify_html'] = (bool) $config['verify_html'];

A super-tiny hint that the type-agnostic comparison (btw, calling things properly is better than code in a sentence ;)) only exists for this setting would be just lovely :)

E.g.,

"TinyMCE performs a type-agnostic comparison on this particular setting."

(or similar)

Powered by Dreditor.

I just copied the comment from CKEditor. or wherever it was from. I'm not at home now, mind taking care of it?

How about this?
Changed the CKEditor comment to match.

Thanks for reporting, reviewing, and testing! Committed to all branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.