ACE Editor: Cache string with LF endings only

Caching the content of the ACE editor with CRLF might cause unintended side effects when restoring the content (since we split by LF in some cases). For now, we adjust the caching strategy accordingly.

Author: MrSerth

app/assets/javascripts/editor/editor.js

@@ -1152,7 +1152,7 @@ var CodeOceanEditor = {
         // Persist the content of the editors in a hidden textarea to enable Turbo caching.
         // In this case, we iterate over _all_ editors, not just writable ones.
         for (const [file_id, editor] of this.editor_for_file) {
-            const file_content = editor.getValue();
+            const file_content = editor.getValue().replace(/\r\n/g, '\n');
             const editorContent = $(`.editor-content[data-file-id='${file_id}']`);
             editorContent.text(file_content);
         }

app/assets/javascripts/exercises.js

@@ -37,7 +37,7 @@ $(document).on('turbo-migration:load', function () {
 
         function unloadTeacherEditor() {
             $(document).off('theme:change:ace');
-            const value = editor.getValue();
+            const value = editor.getValue().replace(/\r\n/g, '\n');
             editor.destroy();
             // "Restore" the editor's content to the original element for caching.
             textarea.val = value;

app/assets/javascripts/request_for_comments.js

@@ -399,7 +399,7 @@ $(document).on('turbo-migration:load', function () {
         $(document).off('theme:change:ace');
         $('.editor').each(function (_, editor) {
             const aceEditor = ace.edit(editor);
-            const value = aceEditor.getValue();
+            const value = aceEditor.getValue().replace(/\r\n/g, '\n');
             aceEditor.destroy();
             // "Restore" the editor's content to the original element for caching.
             editor.textContent = value;