--- loncom/interface/lonsyllabus.pm	2006/10/23 18:48:47	1.55
+++ loncom/interface/lonsyllabus.pm	2008/01/16 17:13:57	1.63
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Syllabus
 #
-# $Id: lonsyllabus.pm,v 1.55 2006/10/23 18:48:47 albertel Exp $
+# $Id: lonsyllabus.pm,v 1.63 2008/01/16 17:13:57 bisitz Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -66,9 +66,6 @@ sub handler {
 
 # ------------------------------------------------------------ Print the screen
 
-    my $rss_link =
-	&Apache::lonrss::rss_link('http://'.$ENV{'HTTP_HOST'}.'/public/'.$cdom.'/'.$cnum.'/Course_Announcements.rss');
-
     if ($target eq 'tex') {
 	$r->print(&Apache::lonprintout::print_latex_header($env{'form.latex_type'}));
     } 
@@ -112,6 +109,7 @@ sub handler {
        
 # ----------------------------------------------------------------- Make header 
     if ($target ne 'tex') {
+	my $rss_link = &Apache::lonrss::rss_link($cnum,$cdom);
 	my $start_page = 
 	    &Apache::loncommon::start_page("Syllabus", $rss_link,
 					   {'function'       => $forcestudent,
@@ -120,10 +118,10 @@ sub handler {
 						$env{'form.register'},});
 
 	$r->print($start_page.'<h1>'.$courseenv{'description'}.'</h1><h3>'.
-                 $Apache::lonnet::domaindescription{$cdom}.'</h3>');
+		  &Apache::lonnet::domain($cdom,'description').'</h3>');
     } else {
 	$r->print('\noindent{\large\textbf{'.$courseenv{'description'}.'}}\\\\\\\\\textbf{'.
-                 $Apache::lonnet::domaindescription{$cdom}.'}\\\\');
+		  &Apache::lonnet::domain($cdom,'description').'}\\\\');
     }
 # -------------------------------------------------------------- Announcements?
     my $day = &Apache::lonannounce::showday(time,2,
@@ -137,13 +135,14 @@ sub handler {
 # -------------------------------------------------------- Get course personnel
     my %coursepersonnel=&Apache::lonnet::get_course_adv_roles($cdom.'/'.$cnum);
     if ($target ne 'tex') {
-	$r->print('<table border="2">');
+	$r->print(&Apache::loncommon::start_data_table());
     } else {
 	$r->print('\begin{tabular}{|p{0.45\textwidth}|p{0.45\textwidth}|}\hline');
     }
     foreach my $element (sort keys %coursepersonnel) {
 	if ($target ne 'tex') {
-	    $r->print('<tr><td>'.$element.'</td><td>');
+	    $r->print(&Apache::loncommon::start_data_table_row().
+		      '<td>'.$element.'</td><td>');
 	} else {
 	    $r->print(' '.&Apache::lonxml::xmlparse($r,'tex',$element).' & '); 
 	}
@@ -159,13 +158,13 @@ sub handler {
 	    }
 	}
 	if ($target ne 'tex') {
-	    $r->print('</td></tr>');
+	    $r->print('</td>'.&Apache::loncommon::end_data_table_row());
 	} else {
 	    $r->print('\\\\ \hline');
 	}
     }
     if ($target ne 'tex') {
-	$r->print('</table>');
+	$r->print(&Apache::loncommon::end_data_table());
     } else {
 	$r->print('\end{tabular}\\\\');
     }
@@ -190,9 +189,9 @@ sub handler {
        }
        if ($allowed) {
           $r->print('<p>'.
-&Apache::loncommon::help_open_topic('Uploaded_Templates_TextBoxes','Help with filling in text boxes').'</p><p>'.&mt('This syllabus can be publicly viewed at')
+&Apache::loncommon::help_open_topic('Uploaded_Templates_TextBoxes',&mt('Help with filling in text boxes')).'</p><p>'.&mt('This syllabus can be publicly viewed at')
 		    .' <tt>http://'.
-		    $Apache::lonnet::hostname{$homeserver}.$r->uri.'</tt>'.
+		    &Apache::lonnet::hostname($homeserver).$r->uri.'</tt>'.
                &Apache::loncommon::help_open_topic('Syllabus_ExtLink').'</p>'.
           '<p>'.&mt('You can specify an external URL as Syllabus in the [_1].','<a href="/adm/parmset?action=crsenv">'.&mt('Course Parameters').'</a>').'</p>'.
 	  '<p><a href="'.$r->uri.'?forcestudent=1"><font size="+1">'.
@@ -225,9 +224,10 @@ sub handler {
 			  my $host = $1;
                           my $remainder=$2;
 # remove the hostname from internal URLs
-		          foreach my $possible_host (keys(%Apache::lonnet::hostname)) {
-                              if ($possible_host =~
-				     /\Q$Apache::lonnet::hostname{$host}\E/i) {
+			  my $hostname = &Apache::lonnet::hostname($host);
+			  my %all_hostnames = &Apache::lonnet::all_hostnames();
+		          foreach my $possible_host (keys(%all_hostnames)) {
+                              if ($possible_host =~ /\Q$hostname\E/i) {
 			         $url=$remainder;
 			      }
 		          }
@@ -260,12 +260,16 @@ sub handler {
                      $syllabus{'uploaded.domain'});
        if ($target ne 'tex') {
 	   $r->print('<table><tr><td>'.&mt('Last updated').':</td><td>'.
-		     $lastmod.'</td><td>'.&mt('by').' '.$who.
+		     $lastmod.'</td><td>'.
+		     ($who ? &mt('by').' '.$who
+                           : '' ).
 		     '</td></tr></table><p>');
        } else {
-	   $r->print('\\\\ '.&mt('Last updated').': '.$lastmod.' '.&mt('by').'\\\\ '.
-		     &Apache::loncommon::plainname($syllabus{'uploaded.name'},
-                     $syllabus{'uploaded.domain'}).'\\\\');
+	   $r->print('\\\\ '.&mt('Last updated').': '.$lastmod.' '.
+		     ($who? &mt('by').'\\\\ '.
+		            &Apache::loncommon::plainname($syllabus{'uploaded.name'},$syllabus{'uploaded.domain'})
+		          :'')
+		     .'\\\\');
        }
        if ($allowed) {
 	   $r->print('<form method="post">'.
@@ -306,7 +310,7 @@ sub handler {
 		   } 
 	       } else {
 		   &Apache::lonfeedback::newline_to_br(\$message);
-		   $message =~s|(https*://[^\s]+)|<a href="$1"><tt>$1</tt></a>/g;
+		   $message =~s|(https*://[^\s]+)|<a href="$1"><tt>$1</tt></a>|g;
 		   if ($allowed) {
 		       $message=&Apache::lonspeller::markeduptext($message);
 		   }
@@ -323,7 +327,7 @@ sub handler {
 	       if ($allowed) {
 		   $r->print('<br /><textarea cols="80" rows="12" name="'.$field.'" id="'.$field.'">'.
 			     &HTML::Entities::encode($syllabus{$field},'"&<>').
-			     '</textarea> <input type="submit" name="storesyl" value="Store" />');
+			     '</textarea> <input type="submit" name="storesyl" value="'.&mt('Save').'" />');
 	       }
 	   }
        }
@@ -334,7 +338,7 @@ sub handler {
        if ($target ne 'tex') {$r->print('</p>');} else {$r->print('\\\\');}
     } else {
 	if ($target ne 'tex') {$r->print('<p>');} else {$r->print('\par ');} 
-	$r->print('No syllabus information provided.');
+	$r->print(&mt('No syllabus information provided.'));
 	if ($target ne 'tex') {$r->print('</p>');}
     }
     if ($target ne 'tex') {