--- loncom/interface/lonchatfetch.pm	2003/06/23 21:30:42	1.11
+++ loncom/interface/lonchatfetch.pm	2005/06/06 02:29:46	1.18
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Chat Fetching
 #
-# $Id: lonchatfetch.pm,v 1.11 2003/06/23 21:30:42 albertel Exp $
+# $Id: lonchatfetch.pm,v 1.18 2005/06/06 02:29:46 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -29,7 +29,7 @@
 package Apache::lonchatfetch;
 
 use strict;
-use Apache::Constants qw(:common);
+use Apache::Constants qw(:common :http);
 use Apache::lontexconvert;
 use Apache::loncommon;
 use Apache::lonnet;
@@ -37,26 +37,32 @@ use Apache::lonnet;
 sub handler {
     my $r = shift;
 
+    if (! &Apache::lonnet::allowed('pch',$env{'request.course.id'}.
+             ($env{'request.course.sec'}?'/'.$env{'request.course.sec'}:''))
+        ) {
+        return HTTP_NOT_ACCEPTABLE;
+    }
+
     my $loaderror=&Apache::lonnet::overloaderror($r);
     if ($loaderror) { return $loaderror; }
     $loaderror=
        &Apache::lonnet::overloaderror($r,
-         $ENV{'course.'.$ENV{'request.course.id'}.'.home'});
+         $env{'course.'.$env{'request.course.id'}.'.home'});
     if ($loaderror) { return $loaderror; }
 
-    $r->content_type('text/html');
+    &Apache::loncommon::content_type($r,'text/html');
     $r->send_http_header;
     return OK if $r->header_only;
 
 # ------------------------------------------------------------ retrieve entries
 
-    my $cnum=$ENV{'course.'.$ENV{'request.course.id'}.'.num'};
-    my $cdom=$ENV{'course.'.$ENV{'request.course.id'}.'.domain'};
-    my $chome=$ENV{'course.'.$ENV{'request.course.id'}.'.home'};
+    my $cnum=$env{'course.'.$env{'request.course.id'}.'.num'};
+    my $cdom=$env{'course.'.$env{'request.course.id'}.'.domain'};
+    my $chome=$env{'course.'.$env{'request.course.id'}.'.home'};
 
     my @entries=split(/\:/,
        &Apache::lonnet::reply(
-        "chatretr:$cdom:$cnum:$ENV{'user.domain'}:$ENV{'user.name'}",$chome));
+        "chatretr:$cdom:$cnum:$env{'user.domain'}:$env{'user.name'}",$chome));
 # Figure out what the last valid entry-id is
     my ($lastid,$thentime,$idnum);
     foreach (@entries) {
@@ -67,9 +73,9 @@ sub handler {
 	}
     }
 # ----------------------------------------------------------- Can see identity?
-    my $crs='/'.$ENV{'request.course.id'};
-    if ($ENV{'request.course.sec'}) {
-       $crs.='_'.$ENV{'request.course.sec'};
+    my $crs='/'.$env{'request.course.id'};
+    if ($env{'request.course.sec'}) {
+       $crs.='_'.$env{'request.course.sec'};
     }                 
     $crs=~s/\_/\//g;
     my $seeid=&Apache::lonnet::allowed('rin',$crs);
@@ -78,28 +84,30 @@ sub handler {
     my $include=0;
     my $newstuff='';
     my $bottomid='';
-    unless ($ENV{'form.lastid'}) { 
+    unless ($env{'form.lastid'}) { 
        $include=1; 
-       $newstuff='<html><body bgcolor="#FFFFFF">';
+       my $html=&Apache::lonxml::xmlbegin();
+       $newstuff=$html.'<head></head><body bgcolor="#FFFFFF">';
     }
     my @participants=();
     foreach (@entries) {
 	my ($id,$msg,$udom)=split(/\:/,&Apache::lonnet::unescape($_));
  	if ($id eq 'active_participant') {
            chomp($udom);
-	   $participants[$#participants+1]=
-           &Apache::loncommon::nickname($msg,$udom);
+	   my $participant= &Apache::loncommon::nickname($msg,$udom);
+	   unless ($participant=~/\w/) { $participant=$msg.'@'.$udom; }
+	   $participants[$#participants+1]=$participant;
 	} elsif ($include) {
 	    chomp($msg);
 	    my ($msgtime,$msgnum)=split(/\_/,$id);
 	    my ($sdom,$snum,$anon,$contrib)=split(/\:/,
 					     &Apache::lonnet::unescape($msg));
 	    $contrib=&Apache::lonnet::unescape($contrib);
-	    $contrib=~s/\n/\<br \/\>/g;
+	    &Apache::lonfeedback::newline_to_br(\$contrib);
 	    ($contrib,my $errors)=&Apache::lontexconvert::msgtexconverted($contrib);
 	    if ($errors) { $contrib.="[Message not fully displayed due to incorrect embedded TeX]"; }
-	    if ($errors && $snum eq $ENV{'user.name'} &&
-		$sdom eq $ENV{'user.domain'} ) {
+	    if ($errors && $snum eq $env{'user.name'} &&
+		$sdom eq $env{'user.domain'} ) {
 		$contrib.="<br />[TeX error message: $errors]";
 	    }
 	    $contrib=~s/\n/ /g;
@@ -111,14 +119,16 @@ sub handler {
 		if (($nick) && ($nick ne $sender)) {
 		    $sender.=' '.$nick;
 		}
+		unless ($sender) { $sender=$snum.'@'.$sdom; }
 		if ($anon) { $sender.=' [Anon]' };
 	    } elsif (!$anon) {
 		$sender=&Apache::loncommon::nickname($snum,$sdom);
+		unless ($sender) { $sender=$snum.'@'.$sdom; }
 	    } else {
 		$sender=&Apache::loncommon::screenname($snum,$sdom);
 		unless ($sender) { $sender="Anonymous"; }
 	    }
-	    $sender=~s/\'/\"/g;
+	    $sender=~s/\'/\&\#39\;/g;
 	    my $color=$sender;
 	    $color=~tr/a-j/0-9/;
 	    $color=~tr/A-J/0-9/;
@@ -136,12 +146,13 @@ sub handler {
 	    $bottomid=$id;
 	} else {
 	    $_=~/^(\w+)/;
-	    if ($1 eq $ENV{'form.lastid'}) { $include=1; }
+	    if ($1 eq $env{'form.lastid'}) { $include=1; }
 	}
     }
     my $participant_output=join('<br />',sort @participants);
+    my $html=&Apache::lonxml::xmlbegin();
     $r->print(<<ENDDOCUMENT);
-<html>
+$html
 <head>
 <title>The LearningOnline Network with CAPA</title>
   <meta HTTP-EQUIV="Refresh" CONTENT="5; url=/adm/chatfetch?lastid=$lastid">