Merge pull request #344 from ajaxorg/issue/1790_settings_cdata

added multiple replace of ]]> for ide.settings in ide.tmpl.html
mattpardee · Oct 25, 2011 · 6333e37 · 6333e37
2 parents 5c3df61 + e898efb
commit 6333e37
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/server/cloud9/ide.js b/server/cloud9/ide.js
@@ -211,7 +211,7 @@ Ide.DEFAULT_PLUGINS = [
             }
             else {
                 settingsPlugin.loadSettings(user, function(err, settings) {
-                    replacements.settingsXml = err || !settings ? "defaults" : settings.replace("]]>", "]]&gt;");
+                    replacements.settingsXml = err || !settings ? "defaults" : settings.replace(/]]>/g, '&#093;&#093;&gt;');
                     index = template.fill(index, replacements);
                     res.end(index);
                 });