From: moodler <moodler>
Date: Sun, 18 Jun 2006 08:10:52 +0000 (+0000)
Subject: Merged helplink fix from stable
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=a6320cc0b88f1fc2238f943d7cd580d14e2ebb97;p=moodle.git

Merged helplink fix from stable
---

diff --git a/lib/weblib.php b/lib/weblib.php
index 6dadc56f50..9fc171dad8 100644
--- a/lib/weblib.php
+++ b/lib/weblib.php
@@ -4142,12 +4142,13 @@ function helpbutton ($page, $title='', $module='moodle', $image=true, $linktext=
         $module = 'moodle';
     }
 
-    $linkobject = '<span class="helplink">';
 
     //Accessibility: prefix the alt text/title with 'Help with', strip distracting dots '...'
     // PLEASE DO NOT CHANGE. ('...' is VERY distracting for non-visual users)
     $tooltip = get_string('helpprefix', '', trim($title, ". \t"));
     
+    $linkobject = '';
+
     if ($image) {
         if ($imagetext == '') {
             $imagetext = '<img alt="'.$tooltip.'" src="'.
@@ -4163,15 +4164,15 @@ function helpbutton ($page, $title='', $module='moodle', $image=true, $linktext=
         $linkobject .= $tooltip;
     }
 
-    $linkobject .= '</span>';
-
     if ($text) {
         $url = '/help.php?module='. $module .'&amp;text='. s(urlencode($text));
     } else {
         $url = '/help.php?module='. $module .'&amp;file='. $page .'.html';
     }
 
-    $link = link_to_popup_window ($url, 'popup', $linkobject, 400, 500, $tooltip, 'none', true);
+    $link = '<span class="helplink">'. 
+            link_to_popup_window ($url, 'popup', $linkobject, 400, 500, $tooltip, 'none', true).
+            '</span>';
 
     if ($return) {
         return $link;