From: scyrma <scyrma>
Date: Thu, 3 Jul 2008 07:12:03 +0000 (+0000)
Subject: MDL-14741:
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=f8c9bb6beeb1d62dcad327e62e6e0fad7c256ceb;p=moodle.git

MDL-14741:
* use a better name for a variable
* remove extra browse button and simplify code. Fix by Mauno K.
---

diff --git a/lib/editor/tinymce.js.php b/lib/editor/tinymce.js.php
index 72c63fd971..6a104003d6 100644
--- a/lib/editor/tinymce.js.php
+++ b/lib/editor/tinymce.js.php
@@ -2,7 +2,7 @@
 
 require_once('../../config.php');
 
-$course = optional_param('course', 0, PARAM_INT);
+$courseid = optional_param('course', 0, PARAM_INT);
 
 /*
  *
@@ -177,7 +177,7 @@ echo <<<EOF
         theme_advanced_buttons3: "media,advhr,emoticons,dragmath,spellchecker,search,code,fullscreen,bullist,numlist,outdent,indent,forecolor,backcolor,link,unlink,anchor,image,charmap,insertlayer,table",
         theme_advanced_buttons3_add: "media,emoticons,charmap,dragmath,search,code,fullscreen",
         theme_advanced_fonts: "Trebuchet=Trebuchet MS,Verdana,Arial,Helvetica,sans-serif;Arial=arial,helvetica,sans-serif;Courier New=courier new,courier,monospace;Georgia=georgia,times new roman,times,serif;Tahoma=tahoma,arial,helvetica,sans-serif;Times New Roman=times new roman,times,serif;Verdana=verdana,arial,helvetica,sans-serif;Impact=impact;Wingdings=wingdings", 
-        moodleimage_course_id: $course,
+        moodleimage_course_id: $courseid,
         theme_advanced_resize_horizontal: true,
         theme_advanced_resizing: true,
         theme_advanced_toolbar_location : "top",
@@ -198,7 +198,7 @@ echo <<<EOF
     }
     function moodlefilemanager(field_name, url, type, win) {
         tinyMCE.activeEditor.windowManager.open({
-            file: "{$CFG->httpswwwroot}/lib/editor/tinymce/jscripts/tiny_mce/plugins/moodlelink/link.php?id={$course}",
+            file: "{$CFG->httpswwwroot}/lib/editor/tinymce/jscripts/tiny_mce/plugins/moodlelink/link.php?id={$courseid}",
             width: 480,  
             height: 380,
             resizable: "yes",
diff --git a/lib/editor/tinymce/jscripts/tiny_mce/plugins/moodlelink/link.php b/lib/editor/tinymce/jscripts/tiny_mce/plugins/moodlelink/link.php
index 67f9ae7950..e36cac5c7c 100644
--- a/lib/editor/tinymce/jscripts/tiny_mce/plugins/moodlelink/link.php
+++ b/lib/editor/tinymce/jscripts/tiny_mce/plugins/moodlelink/link.php
@@ -44,11 +44,6 @@ tinyMCEPopup.onInit.add(FileBrowserDialogue.init, FileBrowserDialogue);
 
 //<![CDATA[
 
-function onCancel() {
-  window.close();
-  return false;
-}
-
 function checkvalue(elm,formname) {
     var el = document.getElementById(elm);
     if(!el.value) {
@@ -142,7 +137,7 @@ form { margin-bottom: 0px; margin-top: 0px; }
     </table>
   </td>
   <td>
-    <button type="button" name="close" onclick="return onCancel();"><?php print_string("close","editor");?></button>
+    <button type="button" name="close" onclick="tinyMCEPopup.close();"><?php print_string("close","editor");?></button>
   </td>
   </tr>
   </table>
@@ -166,8 +161,6 @@ form { margin-bottom: 0px; margin-top: 0px; }
           <input type="hidden" name="sesskey" value="<?php p($USER->sesskey) ?>" />
           <input type="file" name="userfile" id="userfile" size="35" />
           <input name="save" type="submit" id="save" onclick="return checkvalue('userfile','uploader');" value="<?php print_string("upload","editor");?>" />
-          
-          <input name="save" type="submit" id="save" onclick="return checkvalue('userfile','uploader');" value="Browse" />
           </form>
           <?php
           } else {