From: tjhunt <tjhunt>
Date: Mon, 30 Mar 2009 06:11:32 +0000 (+0000)
Subject: help.php: related to MDL-13816. Clean up the URLs used for help files by removing... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=e79079deaef9ece876dcc89519cbee1bb4540867;p=moodle.git

help.php: related to MDL-13816. Clean up the URLs used for help files by removing unused parameters.
---

diff --git a/help.php b/help.php
index 3c9844de81..cbe2b22534 100644
--- a/help.php
+++ b/help.php
@@ -103,7 +103,7 @@ echo '<p class="helpindex"><a href="help.php?file=index.html">'. get_string('hel
 
 // Offer a link to the alternative help file language
 $currentlang = current_language();
-if ($helpfound && ($foundlang != 'en_utf8' || ($forcelang == 'en_utf8' && current_language() != 'en_utf8'))) {
+if ($file && $helpfound && ($foundlang != 'en_utf8' || ($forcelang == 'en_utf8' && current_language() != 'en_utf8'))) {
     $url = new moodle_url();
     if ($foundlang != 'en_utf8') {
         $url->param('forcelang', 'en_utf8');
diff --git a/lib/weblib.php b/lib/weblib.php
index be16aeffbf..ce82070c59 100644
--- a/lib/weblib.php
+++ b/lib/weblib.php
@@ -5948,13 +5948,6 @@ function helpbutton($page, $title, $module='moodle', $image=true, $linktext=fals
         debugging('Warning: it\'s not recommended to use $text parameter in helpbutton ($page=' . $page . ', $module=' . $module . ') function', DEBUG_DEVELOPER);
     }
 
-    // fix for MDL-7734
-    if (!empty($COURSE->lang)) {
-        $forcelang = $COURSE->lang;
-    } else {
-        $forcelang = '';
-    }
-
     // Catch references to the old text.html and emoticons.html help files that
     // were renamed in MDL-13233.
     if (in_array($page, array('text', 'emoticons', 'richtext'))) {
@@ -5996,14 +5989,17 @@ function helpbutton($page, $title, $module='moodle', $image=true, $linktext=fals
 
     // fix for MDL-7734
     if ($text) {
-        $url = '/help.php?module='. $module .'&amp;text='. s(urlencode($text).'&amp;forcelang='.$forcelang);
+        $url = '/help.php?text='. s(urlencode($text));
     } else {
-        $url = '/help.php?module='. $module .'&amp;file='. $page .'.html&amp;forcelang='.$forcelang;
+        $url = '/help.php?module='. $module .'&amp;file='. $page .'.html';
+        // fix for MDL-7734
+        if (!empty($COURSE->lang)) {
+            $url .= '&amp;forcelang=' . $COURSE->lang;
+        }
     }
 
-    $link = '<span class="helplink">'.
-            link_to_popup_window ($url, 'popup', $linkobject, 400, 500, $tooltip, 'none', true).
-            '</span>';
+    $link = '<span class="helplink">' . link_to_popup_window($url, 'popup',
+            $linkobject, 400, 500, $tooltip, 'none', true) . '</span>';
 
     if ($return) {
         return $link;