2.47.2

package.json

@@ -1,6 +1,6 @@
 {
   "name": "suneditor",
-  "version": "2.47.1",
+  "version": "2.47.2",
   "description": "Vanilla javascript based WYSIWYG web editor, with no dependencies",
   "author": "JiHong.Lee",
   "license": "MIT",

sample/html/options.html

@@ -69,7 +69,7 @@
         <div style="background-color: #f3f3f3; padding: 4px 20px 20px 20px;">
             <h4 style="color: #b94a48;">--Value</h4>
             <label><input type="checkbox" id="strictMode" checked>&nbsp;&nbsp;strictMode&nbsp;&nbsp;&nbsp;</label><br><br>
-            <label><input type="checkbox" id="strictHTMLValidation">&nbsp;&nbsp;strictHTMLValidation&nbsp;&nbsp;&nbsp;</label><br><br>
+            <label><input type="checkbox" id="strictHTMLValidation" checked>&nbsp;&nbsp;strictHTMLValidation&nbsp;&nbsp;&nbsp;</label><br><br>
             <label><input type="checkbox" id="defaultTag">&nbsp;&nbsp;defaultTag&nbsp;&nbsp;&nbsp;</label>
             <input type="text" id="defaultTag_value" placeholder="String ('p', 'div'..)"></input>
             <br><br>
@@ -635,7 +635,7 @@ <h2 class="sub-title">Applied options</h2>
 
     var options = {
         strictMode: document.getElementById('strictMode').checked ? undefined : false,
-        strictHTMLValidation: document.getElementById('strictHTMLValidation').checked ? true : undefined,
+        strictHTMLValidation: document.getElementById('strictHTMLValidation').checked ? undefined : false,
         defaultTag: document.getElementById('defaultTag').checked ? document.getElementById('defaultTag_value').value : undefined,
         textTags: document.getElementById('textTags').checked ? {
             bold: 'b',

src/lib/constructor.js

@@ -416,7 +416,7 @@ export default {
         options.plugins = plugins;
         /** Values */
         options.strictMode = options.strictMode !== false;
-        options.strictHTMLValidation = options.strictHTMLValidation === true;
+        options.strictHTMLValidation = options.strictHTMLValidation !== false;
         options.lang = options.lang || _defaultLang;
         options.value = typeof options.value === 'string' ? options.value : null;
         options.allowedClassNames = new util._w.RegExp((options.allowedClassNames && typeof options.allowedClassNames === 'string' ? options.allowedClassNames + '|' : '') + '^__se__|se-|katex');

src/lib/core.js

@@ -5480,12 +5480,12 @@ export default function (context, pluginCallButtons, plugins, lang, options, _re
          * @returns {String}
          */
         cleanHTML: function (html, whitelist, blacklist) {
-            if (!options.strictMode) return html;
+            if (!options.strictMode) return util.htmlCompress(html);
 
             html = this._deleteDisallowedTags(this._parser.parseFromString(util.htmlCompress(html), 'text/html').body.innerHTML).replace(/(<[a-zA-Z0-9\-]+)[^>]*(?=>)/g, this._cleanTags.bind(this, true)).replace(/<br\/?>$/i, '');
             const dom = _d.createRange().createContextualFragment(html);
             try {
-                util._consistencyCheckOfHTML(dom, this._htmlCheckWhitelistRegExp, this._htmlCheckBlacklistRegExp, this._classNameFilter);
+                util._consistencyCheckOfHTML(dom, this._htmlCheckWhitelistRegExp, this._htmlCheckBlacklistRegExp, this._classNameFilter, options.strictHTMLValidation);
             } catch (error) {
                 console.warn('[SUNEDITOR.cleanHTML.consistencyCheck.fail] ' + error);
             }
@@ -5538,16 +5538,13 @@ export default function (context, pluginCallButtons, plugins, lang, options, _re
          * @returns {String}
          */
         convertContentsForEditor: function (contents) {
-            if (!options.strictMode) return contents;
+            if (!options.strictMode) return util.htmlCompress(contents);
+
             contents = this._deleteDisallowedTags(this._parser.parseFromString(util.htmlCompress(contents), 'text/html').body.innerHTML).replace(/(<[a-zA-Z0-9\-]+)[^>]*(?=>)/g, this._cleanTags.bind(this, true));
             const dom = _d.createRange().createContextualFragment(contents);
 
             try {
-                if (this.options.strictHTMLValidation) {
-                    util._consistencyCheckOfHTML(dom, this._htmlCheckWhitelistRegExp, this._htmlCheckBlacklistRegExp, this._classNameFilter);
-                } else {
-                    util._consistencyCheckOfHTML(dom, this._htmlCheckWhitelistRegExp, false);
-                }
+                util._consistencyCheckOfHTML(dom, this._htmlCheckWhitelistRegExp, this._htmlCheckBlacklistRegExp, this._classNameFilter, options.strictHTMLValidation);
             } catch (error) {
                 console.warn('[SUNEDITOR.convertContentsForEditor.consistencyCheck.fail] ' + error);
             }

src/lib/util.js

@@ -1931,9 +1931,10 @@ const util = {
      * @param {RegExp} htmlCheckWhitelistRegExp Editor tags whitelist (core._htmlCheckWhitelistRegExp)
      * @param {RegExp} htmlCheckBlacklistRegExp Editor tags blacklist (core._htmlCheckBlacklistRegExp)
      * @param {Function} classNameFilter Class name filter function
+     * @param {Function} strictHTMLValidation Enforces strict HTML validation based on the editor`s policy
      * @private
      */
-    _consistencyCheckOfHTML: function (documentFragment, htmlCheckWhitelistRegExp, htmlCheckBlacklistRegExp, classNameFilter) {
+    _consistencyCheckOfHTML: function (documentFragment, htmlCheckWhitelistRegExp, htmlCheckBlacklistRegExp, classNameFilter, strictHTMLValidation) {
         /**
          * It is can use ".children(util.getListChildren)" to exclude text nodes, but "documentFragment.children" is not supported in IE.
          * So check the node type and exclude the text no (current.nodeType !== 1)
@@ -1982,7 +1983,7 @@ const util = {
                 else current.removeAttribute('class');
             }
 
-            const result = current.parentNode !== documentFragment && nrtag &&
+            const result = strictHTMLValidation && current.parentNode !== documentFragment && nrtag &&
                 ((this.isListCell(current) && !this.isList(current.parentNode)) ||
                     ((this.isFormatElement(current) || this.isComponent(current)) && !this.isRangeFormatElement(current.parentNode) && !this.getParentElement(current, this.isComponent)));
 

test/dev/suneditor_build_test.js

@@ -988,8 +988,8 @@ let s2 = window.s2 = suneditor.create(document.getElementById('editor2'), {
 
 //     `,
 // imageUploadUrl: 'http://localhost:3000/editor/upload',
-    // strictMode: false,
-    // strictHTMLValidation: true,
+    strictMode: false,
+    strictHTMLValidation: false,
     allowedClassNames: '.',
     width: '100%',
     // fontSizeUnit: 'em',
@@ -1059,9 +1059,10 @@ let s2 = window.s2 = suneditor.create(document.getElementById('editor2'), {
     // addTagsWhitelist: "fld|sort|sortType|lst|lstfld|header|section",
     lineAttrReset: 'class',
     imageAccept: '*/*',
-    value: `<p>Ôtez la <span style="color: rgb(235, 141, 117)">complexité</span>,<br>
-M.Biz vous apporte des solutions utiles<br>
-</p>`,
+    value: `<div style="text-align: center">
+    <p>This text is centered.</p>
+    <p>This text is centered.</p>
+  </div>`,
     // attributesWhitelist: {
     //     all: 'style|class',
     // },