Bug 28525: Dont let TinyMCE do code cleanup
authorLucas Gass <lucas@bywatersolutions.com>
Mon, 7 Jun 2021 22:17:46 +0000 (22:17 +0000)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Wed, 18 Aug 2021 13:08:45 +0000 (15:08 +0200)
If you look at Bug 24764 and Bug 26942 we adjusted the TinyMCE for the News tool so it will not do some types of automatic code clean up.

The TinyMCE editor for system preferences which can be enabled by 'UseWYSIWYGinSystemPreferences' has the same problems.

We should make the configurations the same.

To Test:
1. Turn on 'UseWYSIWYGinSystemPreferences'
2. Go to a system preference like 'RestrictedPageContent'
3. Try entering something like '<i class="fa fa-facebook-official" aria-hidden="true">TEST</i>' in the Source Code window
4. It gets cleaned up by the editor
5. Try something like '<a href="https://www.test.com" referrerpolicy="&quot;no-referrer-when-downgrade">TEST</a>'
6. It's cleaned up by the editor.
7. Try something like '<link href="https://fonts.googleapis.com/css?family=Open+Sans” rel=“stylesheet">'
8. Cleaned up by editor.
9. Apply patch
10. Try step 3, 5, and 7 again.
11. It should not be changed by the editor

Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
koha-tmpl/intranet-tmpl/prog/en/includes/wysiwyg-systempreferences.inc

index f1e8be3..c6875c0 100644 (file)
     }
 
     tinyMCE.init({
+        verify_html: false,
+        force_br_newlines : false,
+        force_p_newlines : false,
+        forced_root_block : '',
+        extended_valid_elements:"style,link[href|rel]",
+        custom_elements:"style,link,~link",
         branding : false,
         relative_urls : false,
         content_css : "[% interface | html %]/[% theme | html %]/css/tinymce.css",