From 2953d1b68f716ca095a6dbe7c96f292dea205b96 Mon Sep 17 00:00:00 2001
From: scyrma <scyrma>
Date: Mon, 4 Aug 2008 06:26:19 +0000
Subject: [PATCH] MDL-15688: Fix to make it work with IE (it was already doing
 fine in Firefox, Safari and Opera).

---
 lib/editor/tinymce/strings.php | 26 +++++++++++++++++++-------
 1 file changed, 19 insertions(+), 7 deletions(-)

diff --git a/lib/editor/tinymce/strings.php b/lib/editor/tinymce/strings.php
index 251c377b74..a6123802ee 100644
--- a/lib/editor/tinymce/strings.php
+++ b/lib/editor/tinymce/strings.php
@@ -28,11 +28,18 @@ if (!is_null($contexturl)) {
 
     $selectedkeys = preg_grep('/^'. $moduletype .'\/'. $modulename . $dialogpostfix .':/', $keys);
    
-    $output = "tinyMCE.addI18n('$lang". ($modulename ? '.'.$modulename:'') ."$dialogpostfix',{\n";
+    $output = "tinyMCE.addI18n('$lang". ($modulename ? '.'.$modulename:'') ."$dialogpostfix',{\r\n";
+    $i = count($selectedkeys);
     foreach($selectedkeys as $key) {
-        $output .= substr($key, strpos($key, ':')+1) .':"'. addslashes_js(get_string($key, 'tinymce')) ."\",\n";
+        $i--;
+        $output .= substr($key, strpos($key, ':')+1) .':"'. addslashes_js(get_string($key, 'tinymce')) .'"';
+        if ($i > 0) {
+            $output .= ","; // must not add commas at the last element - breaks in IE 6 and 7.
+        }
+        $output .= "\r\n";
     }
     $output .= "});";
+    
 
 } else {
     $output = "tinyMCE.addI18n({". $lang .":{";
@@ -44,19 +51,24 @@ if (!is_null($contexturl)) {
         $section = $subkey[0];
         $string = $subkey[1];
         if ($section != $currentsection) {
+            $output .= "\r\n";
             if ($firstiteration) {
                 $firstiteration = false;
-                $output .= "\n";
             } else {
-                $output .= "},\n"; 
+                $output .= "},\r\n"; 
             }
             $currentsection = $section;
-            $output .= $currentsection .":{\n";
+            $output .= $currentsection .":{\r\n";
+        } else {
+            $output .= ",\r\n"; 
         }
-        $output .= $string .':"'. addslashes_js(get_string($key, 'tinymce')) ."\",\n";
+
+        $output .= $string .':"'. addslashes_js(get_string($key, 'tinymce')) .'"';
     } 
-    $output .= "}}});";
+    $output .= "\r\n}}});";
+    
 }
+
 $lifetime = '86400';
 @header('Content-type: text/javascript; charset=utf-8');
 @header('Content-length: '.strlen($output));
-- 
2.39.5