From: tjhunt <tjhunt>
Date: Fri, 11 Aug 2006 10:33:29 +0000 (+0000)
Subject: Tidy up code flow in helpbutton.
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=712c25b10bf40b5bc17cc785fb3d4d7ded3930b8;p=moodle.git

Tidy up code flow in helpbutton.
---

diff --git a/lib/weblib.php b/lib/weblib.php
index 632f548b77..e9f705c963 100644
--- a/lib/weblib.php
+++ b/lib/weblib.php
@@ -4307,6 +4307,7 @@ function print_error ($errorcode, $module='', $link='', $a=NULL) {
  *           'Help with' (or the language equivalent) will be prefixed and '...' will be stripped.
  * @param string $module Which module is the page defined in
  * @param mixed $image Use a help image for the link?  (true/false/"both")
+ * @param boolean $linktext If true, display the title next to the help icon.
  * @param string $text If defined then this text is used in the page, and
  *           the $page variable is ignored.
  * @param boolean $return If true then the output is returned as a string, if false it is printed to the current page.
@@ -4330,15 +4331,16 @@ function helpbutton ($page, $title='', $module='moodle', $image=true, $linktext=
     $linkobject = '';
 
     if ($image) {
-        if ($imagetext == '') {
-            $imagetext = '<img alt="'.$tooltip.'" src="'.
-                          $CFG->pixpath .'/help.gif" />';
-        }
         if ($linktext) {
             $linkobject .= $title.'&nbsp;';
         }
 
-        $linkobject .= $imagetext;
+        if ($imagetext) {
+            $linkobject .= $imagetext;
+        } else {
+            $linkobject .= '<img alt="'.$tooltip.'" src="'.
+                    $CFG->pixpath .'/help.gif" />';
+        }
 
     } else {
         $linkobject .= $tooltip;