--- loncom/interface/lonhelp.pm	2006/05/30 12:46:09	1.27
+++ loncom/interface/lonhelp.pm	2006/07/14 16:46:36	1.30
@@ -1,6 +1,6 @@
 # The LearningOnline Network with CAPA
 #
-# $Id: lonhelp.pm,v 1.27 2006/05/30 12:46:09 www Exp $
+# $Id: lonhelp.pm,v 1.30 2006/07/14 16:46:36 www Exp $
 #
 # .tex help system web server handler
 #
@@ -58,7 +58,7 @@ sub processLabelName {
 # Serve out the text
 sub servetext {
     my ($r,$text) = @_;
-    my $bugs=&Apache::loncommon::help_open_bug('Documentation');
+    my $bugs=&Apache::loncommon::help_open_bug('Documentation','Report a documentation bug');
     my $start_page=
 	&Apache::loncommon::start_page('LON-CAPA Help',undef,
 				       {'only_body' => 1,});
@@ -68,7 +68,7 @@ sub servetext {
     $r->print(<<HEADER);
     $start_page
     <h3 style="font: sans-serif"><img align="right" alt="help logo"
-    src="/adm/help/gif/lonhelpheader.gif"/>$header</h3><hr />$bugs
+    src="/adm/help/gif/lonhelpheader.gif"/>$header</h3><hr />
     <!-- BEGIN -->
 HEADER
 
@@ -85,6 +85,7 @@ $search: <input type="text" name="search
 </form>
 <br />
 <a href="/adm/help/abouthelp.html">$about</a>
+$bugs
 FOOTER
  
     }
@@ -111,11 +112,19 @@ sub render {
     # absolute paths for use with help.loncapa.org
     $tex =~ s|  \\ref\{([^}]*)\}
              |
-              if (not(exists($fragmentLabels{$1}))) {
-	          &Apache::lonnet::logthis("ERROR: $1 not a valid help label");
-              };
+              my $label=$1;
+              if ($1!~/\.hlp$/) {
+                  if ((!exists($fragmentLabels{$1})) && ($1!~/\.hlp$/)) {
+	             &Apache::lonnet::logthis("ERROR: $1 not a valid help label");
+                     $label='Error';
+                  } else {
+                     $label=substr($fragmentLabels{$1}, 0, -4);
+		  }
+	      } else {
+		  $label=~s/\.hlp$//;
+	      }
              '\\begin{html}<a href="http://' . $serverroot ."/adm/help/".
-              substr($fragmentLabels{$1}, 0, -4) .
+              $label .
               '.hlp#' . &processLabelName($1) . 
              '"><img src="http://' . $serverroot . '/adm/help/gif/smallHelp.gif" border="0" /></a>' .
              '\\end{html}'
@@ -164,9 +173,9 @@ sub listmatches {
 		    $line=~s/\{//gs;
 		    $line=~s/\}//gs;
 		    $line=~s/\\/ /gs;
-		    $line=~s/(\Q$term\E)/\<b\>$1\<\/b\>/gsi;
                     $line=~s/\</\&lt\;/gs;
                     $line=~s/\>/\&gt\;/gs;
+		    $line=~s/(\Q$term\E)/\<b\>$1\<\/b\>/gsi;
 		    $quote.='<br />...'.$line.'...';
 		}
 	    }
@@ -191,9 +200,9 @@ sub handler {
      my $serverroot = $ENV{'HTTP_HOST'};
 
      &Apache::lonlocal::get_language_handle($r);
+     &Apache::loncommon::content_type($r,"text/html");
      my $text='';
      if ($env{'form.searchterm'}=~/\w/) {
-	 &Apache::loncommon::content_type($r,"text/html");
 	 ($text,my $matches)=&listmatches($docroot,$env{'form.searchterm'},&Apache::lonlocal::current_language().'/');
          if ($matches) {
              my ($englishresult,$englishmatches)=&listmatches($docroot,$env{'form.searchterm'});
@@ -233,24 +242,7 @@ sub handler {
 	         or return HTTP_NOT_FOUND;
 	     $tex .= join('', <$file>);
 	 }
-	 
-	 if ($env{'browser.mathml'}) {
-	     &Apache::loncommon::content_type($r,'text/xml');
-	     &tth::ttminit();
-	     if ($env{'browser.unicode'}) {
-		 &tth::ttmoptions('-L -u1');
-	     } else {
-		 &tth::ttmoptions('-L -u0');
-	     }
-	 } else {
-	     &Apache::loncommon::content_type($r,"text/html");
-	     &tth::tthinit();
-	     if ($env{'browser.unicode'}) {
-		 &tth::tthoptions('-L -u1');
-	     } else {
-		 &tth::tthoptions('-L -u0');
-	     }
-	 }
+
 	 $text = &render($tex, $docroot, $serverroot);
      }