[BUGFIX] Prevent duplicate ckeditor instances when moving inline fields

Resolves: #99410
Releases: 11.5
Change-Id: I354ed27532f7f07ef070488866e685762fe14165
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/77426


Tested-by: core-ci <typo3@b13.com>
Tested-by: Oliver Bartsch <bo@cedev.de>
Reviewed-by: Oliver Bartsch <bo@cedev.de>
Tested-by: Stefan Bürk <stefan@buerk.tech>
Reviewed-by: Stefan Bürk <stefan@buerk.tech>

Build/Sources/TypeScript/rte_ckeditor/Resources/Public/TypeScript/FormEngineInitializer.ts

@@ -65,11 +65,9 @@ export class FormEngineInitializer {
           }
         });
         document.addEventListener('inline:sorting-changed', () => {
-          instance.destroy();
           CKEDITOR.replace(fieldId, options.configuration);
         });
         document.addEventListener('formengine:flexform:sorting-changed', () => {
-          instance.destroy();
           CKEDITOR.replace(fieldId, options.configuration);
         });
       });

typo3/sysext/rte_ckeditor/Resources/Public/JavaScript/FormEngineInitializer.js

@@ -10,4 +10,4 @@
  *
  * The TYPO3 project - inspiring people to share!
  */
-var __importDefault=this&&this.__importDefault||function(e){return e&&e.__esModule?e:{default:e}};define(["require","exports","jquery","TYPO3/CMS/Backend/FormEngine"],(function(e,i,t,a){"use strict";Object.defineProperty(i,"__esModule",{value:!0}),i.FormEngineInitializer=void 0,t=__importDefault(t);i.FormEngineInitializer=class{static initializeCKEditor(i){e(["ckeditor"],e=>{e.timestamp+="-"+i.configurationHash,i.externalPlugins.forEach(i=>e.plugins.addExternal(i.name,i.resource,"")),(0,t.default)(()=>{const n=i.fieldId,o="#"+t.default.escapeSelector(n);e.replace(n,i.configuration);const d=e.instances[n];d.on("change",e=>{let i=e.sender.commands;d.updateElement(),a.Validation.validateField((0,t.default)(o)),a.Validation.markFieldAsChanged((0,t.default)(o)),void 0!==i.maximize&&1===i.maximize.state&&d.on("maximize",e=>{(0,t.default)(this).off("maximize"),a.Validation.markFieldAsChanged((0,t.default)(o))})}),d.on("mode",e=>{if("source"===e.editor.mode){const e=d.editable();e.attachListener(e,"change",()=>{a.Validation.markFieldAsChanged((0,t.default)(o))})}}),document.addEventListener("inline:sorting-changed",()=>{d.destroy(),e.replace(n,i.configuration)}),document.addEventListener("formengine:flexform:sorting-changed",()=>{d.destroy(),e.replace(n,i.configuration)})})})}}}));
\ No newline at end of file
+var __importDefault=this&&this.__importDefault||function(e){return e&&e.__esModule?e:{default:e}};define(["require","exports","jquery","TYPO3/CMS/Backend/FormEngine"],(function(e,i,t,a){"use strict";Object.defineProperty(i,"__esModule",{value:!0}),i.FormEngineInitializer=void 0,t=__importDefault(t);i.FormEngineInitializer=class{static initializeCKEditor(i){e(["ckeditor"],e=>{e.timestamp+="-"+i.configurationHash,i.externalPlugins.forEach(i=>e.plugins.addExternal(i.name,i.resource,"")),(0,t.default)(()=>{const n=i.fieldId,o="#"+t.default.escapeSelector(n);e.replace(n,i.configuration);const d=e.instances[n];d.on("change",e=>{let i=e.sender.commands;d.updateElement(),a.Validation.validateField((0,t.default)(o)),a.Validation.markFieldAsChanged((0,t.default)(o)),void 0!==i.maximize&&1===i.maximize.state&&d.on("maximize",e=>{(0,t.default)(this).off("maximize"),a.Validation.markFieldAsChanged((0,t.default)(o))})}),d.on("mode",e=>{if("source"===e.editor.mode){const e=d.editable();e.attachListener(e,"change",()=>{a.Validation.markFieldAsChanged((0,t.default)(o))})}}),document.addEventListener("inline:sorting-changed",()=>{e.replace(n,i.configuration)}),document.addEventListener("formengine:flexform:sorting-changed",()=>{e.replace(n,i.configuration)})})})}}}));
\ No newline at end of file