--- loncom/interface/lonchatfetch.pm 2003/03/29 01:39:10 1.9 +++ loncom/interface/lonchatfetch.pm 2005/04/07 06:56:22 1.17 @@ -1,7 +1,7 @@ # The LearningOnline Network # Chat Fetching # -# $Id: lonchatfetch.pm,v 1.9 2003/03/29 01:39:10 www Exp $ +# $Id: lonchatfetch.pm,v 1.17 2005/04/07 06:56:22 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,32 +37,45 @@ 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)); - my ($lastid)=($entries[$#entries]=~/^(\w+)/); - my ($thentime,$idnum)=split(/\_/,$lastid); + "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) { + $_=~/^(\w+)/; + if ($1 ne 'active_participant') { + $lastid=$1; + ($thentime,$idnum)=split(/\_/,$lastid); + } + } # ----------------------------------------------------------- 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); @@ -71,17 +84,19 @@ sub handler { my $include=0; my $newstuff=''; my $bottomid=''; - unless ($ENV{'form.lastid'}) { + unless ($env{'form.lastid'}) { $include=1; - $newstuff='
'; + my $html=&Apache::lonxml::xmlbegin(); + $newstuff=$html.''; } 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); @@ -89,7 +104,12 @@ sub handler { &Apache::lonnet::unescape($msg)); $contrib=&Apache::lonnet::unescape($contrib); $contrib=~s/\n/\