--- loncom/interface/lonmsg.pm 2004/06/02 20:40:07 1.102 +++ loncom/interface/lonmsg.pm 2004/09/10 08:42:21 1.107 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # Routines for messaging # -# $Id: lonmsg.pm,v 1.102 2004/06/02 20:40:07 raeburn Exp $ +# $Id: lonmsg.pm,v 1.107 2004/09/10 08:42:21 www Exp $ # # Copyright Michigan State University Board of Trustees # @@ -201,10 +201,12 @@ sub unpackagemsg { # ======================================================= Get info out of msgid sub unpackmsgid { - my $msgid=&Apache::lonnet::unescape(shift); + my ($msgid,$folder)=@_; + $msgid=&Apache::lonnet::unescape($msgid); + my $suffix=&foldersuffix($folder); my ($sendtime,$shortsubj,$fromname,$fromdomain)=split(/\:/, &Apache::lonnet::unescape($msgid)); - my %status=&Apache::lonnet::get('email_status',[$msgid]); + my %status=&Apache::lonnet::get('email_status'.$suffix,[$msgid]); if ($status{$msgid}=~/^error\:/) { $status{$msgid}=''; } unless ($status{$msgid}) { $status{$msgid}='new'; } return ($sendtime,$shortsubj,$fromname,$fromdomain,$status{$msgid}); @@ -219,10 +221,14 @@ sub sendemail { my $msg = new Mail::Send; $msg->to($to); $msg->subject('[LON-CAPA] '.$subject); + my %oldENV=%ENV; + undef(%ENV); if (my $fh = $msg->open()) { print $fh $body; $fh->close; } + %ENV=%oldENV; + undef(%oldENV); } # ==================================================== Send notification emails @@ -494,20 +500,77 @@ sub user_normal_msg { } +# ============================================================ List all folders + +sub folderlist { + my $folder=shift; + my @allfolders=&Apache::lonnet::getkeys('email_folders'); + if ($allfolders[0]=~/^error:/) { @allfolders=(); } + return '
'. + ''. + &Apache::loncommon::select_form($folder,'folder', + ('' => &mt('INBOX'),'trash' => &mt('TRASH'), + 'sent' => &mt('Sent Messages'), + map { $_ => $_ } @allfolders)). + ''. + &mt('View Critical Messages').''. + '
'; +} +# =============================================================== Folder suffix + +sub foldersuffix { + my $folder=shift; + unless ($folder) { return ''; } + return '_'.&Apache::lonnet::escape($folder); +} + # =============================================================== Status Change sub statuschange { - my ($msgid,$newstatus)=@_; - my %status=&Apache::lonnet::get('email_status',[$msgid]); + my ($msgid,$newstatus,$folder)=@_; + my $suffix=&foldersuffix($folder); + my %status=&Apache::lonnet::get('email_status'.$suffix,[$msgid]); if ($status{$msgid}=~/^error\:/) { $status{$msgid}=''; } unless ($status{$msgid}) { $status{$msgid}='new'; } unless (($status{$msgid} eq 'replied') || ($status{$msgid} eq 'forwarded')) { - &Apache::lonnet::put('email_status',{$msgid => $newstatus}); + &Apache::lonnet::put('email_status'.$suffix,{$msgid => $newstatus}); } if (($newstatus eq 'deleted') || ($newstatus eq 'new')) { - &Apache::lonnet::put('email_status',{$msgid => $newstatus}); + &Apache::lonnet::put('email_status'.$suffix,{$msgid => $newstatus}); + } +} + +# ============================================================= Make new folder + +sub makefolder { + my ($newfolder)=@_; + &Apache::lonnet::put('email_folders',{$newfolder => time}); +} + +# ======================================================== Move between folders + +sub movemsg { + my ($msgid,$srcfolder,$trgfolder)=@_; + my $unmsgid=&Apache::lonnet::unescape($msgid); + my $srcsuffix=&foldersuffix($srcfolder); + my $trgsuffix=&foldersuffix($trgfolder); + +# Copy message + my %message=&Apache::lonnet::get('nohist_email'.$srcsuffix,[$msgid]); + &Apache::lonnet::put('nohist_email'.$trgsuffix,{$msgid => $message{$msgid}}); + +# Copy status + my %status=&Apache::lonnet::get('email_status'.$srcsuffix,[$unmsgid]); + &Apache::lonnet::put('email_status'.$trgsuffix,{$unmsgid => $status{$unmsgid}}); +# See if was deleted -> becomes "read" in trash + my $currentstatus=(&unpackmsgid($status{$unmsgid})); + if ($currentstatus eq 'deleted') { + &statuschange($msgid,'read',$trgfolder); } +# Delete orginals + &Apache::lonnet::del('nohist_email'.$srcsuffix,[$msgid]); + &Apache::lonnet::del('email_status'.$srcsuffix,[$unmsgid]); } # ======================================================= Display a course list @@ -607,13 +670,13 @@ sub discrit { my %content=&unpackagemsg($what{$_}); next if ($content{'senderdomain'} eq ''); $content{'message'}=~s/\n/\/g; - $result.='
'.&mt('From').': '. + $result.='
'.&mt('From').': '. &Apache::loncommon::aboutmewrapper( &Apache::loncommon::plainname($content{'sendername'},$content{'senderdomain'}),$content{'sendername'},$content{'senderdomain'}).' ('. $content{'sendername'}.'@'. $content{'senderdomain'}.') '.$content{'time'}. - '
'.&mt('Subject').': '.$content{'subject'}. - '
'. + '
'.&mt('Subject').': '.$content{'subject'}. + '
'. &Apache::lontexconvert::msgtexconverted($content{'message'}). '
'. &mt('You have to confirm that you received this message. After confirmation, this message will be moved to your regular inbox'). @@ -625,17 +688,19 @@ $content{'sendername'}.'@'. # Check to see if there were any messages. if ($result eq '') { $result = "

".&mt('You have no critical messages.')."

". - ''.&mt('Select a course').''; + ''.&mt('Select a course').'
'. + ''.&mt('Communicate').''; } else { $r->print($header); } $r->print($result); - $r->print(''); + $r->print(''); } sub sortedmessages { - my ($blocked,$startblock,$endblock,$numblocked) = @_; - my @messages = &Apache::lonnet::getkeys('nohist_email'); + my ($blocked,$startblock,$endblock,$numblocked,$folder) = @_; + my $suffix=&foldersuffix($folder); + my @messages = &Apache::lonnet::getkeys('nohist_email'.$suffix); #unpack the varibles and repack into temp for sorting my @temp; foreach (@messages) { @@ -691,7 +756,7 @@ sub sortedmessages { # ======================================================== Display all messages sub disall { - my $r=shift; + my ($r,$folder)=@_; my %blocked = (); my %setters = (); my $startblock; @@ -719,9 +784,10 @@ sub disall { } ENDDISHEADER - $r->print('

'.&mt('Display All Messages').'

'. - ''. ''); if ($counter > 0){ $r->print('
 '); + $r->print('

'.&mt('Display All Messages').'

'. + &folderlist($folder). + ''. + ''); } else { @@ -752,7 +818,7 @@ ENDDISHEADER $r->print(''.&mt('Status').''); } $r->print(''); - my @temp=sortedmessages(\%blocked,$startblock,$endblock,\$numblocked); + my @temp=sortedmessages(\%blocked,$startblock,$endblock,\$numblocked,$folder); foreach (@temp){ my ($sendtime,$shortsubj,$fromname,$fromdomain,$status,$origID)= @$_; if (($status ne 'deleted') && defined($sendtime) && $sendtime!~/error/) { @@ -765,21 +831,35 @@ ENDDISHEADER } else { $r->print(''); } - $r->print(''. + $r->print(''. ''); + } elsif ($status eq 'deleted') { +# purge + &movemsg($origID,$folder,'trash'); } } $r->print('
 '); if ($ENV{'form.sortedby'} eq "revdate") { $r->print(''.&mt('Date').'
'.&mt('Open').''.&mt('Delete').''.&mt('Open').''. + ($folder ne 'trash'?''.&mt('Delete'):' ').''.&Apache::lonlocal::locallocaltime($sendtime).''. $fromname.''.$fromdomain.''. &Apache::lonnet::unescape($shortsubj).''. $status.'

'. - ''.&mt('Check All').' '. - ''.&mt('Uncheck All').'

'. - ''. - ''. - ''); + ''.&mt('Check All').' '. + ''.&mt('Uncheck All').'

'. + ''); + if ($folder ne 'trash') { + $r->print( + '

'); + } +$r->print('

'); + my @allfolders=&Apache::lonnet::getkeys('email_folders'); + if ($allfolders[0]=~/^error:/) { @allfolders=(); } + $r->print( + &Apache::loncommon::select_form('','movetofolder', + ( map { $_ => $_ } @allfolders)) + ); + $r->print(''); if ($numblocked > 0) { my $beginblock = &Apache::lonlocal::locallocaltime($startblock); my $finishblock = &Apache::lonlocal::locallocaltime($endblock); @@ -787,7 +867,6 @@ ENDDISHEADER $numblocked.' '.&mt('message(s) is/are not viewable because display of LON-CAPA messages sent to you by other students between').' '.$beginblock.' '.&mt('and').' '.$finishblock.' '.&mt('is currently being blocked because of online exams.')); &build_block_table($r,$startblock,$endblock,\%setters); } - $r->print(''); } # ============================================================== Compose output @@ -857,8 +936,8 @@ sub compout { if ($replying) { %message=&Apache::lonnet::get('nohist_email',[$replying]); %content=&unpackagemsg($message{$replying}); - $dispcrit.=''; + $dispcrit.=''; $func=&mt('Replying to'); $dissub=&mt('Reply').': '.$content{'subject'}; @@ -945,6 +1024,31 @@ ENDUPLOAD # ---------------------------------------------------- Display all face to face +sub retrieve_instructor_comments { + my ($user,$domain)=@_; + my $target=$ENV{'form.grade_target'}; + if (! $ENV{'request.course.id'}) { return; } + if (! &Apache::lonnet::allowed('srm',$ENV{'request.course.id'})) { + return; + } + my %records=&Apache::lonnet::dump('nohist_email', + $ENV{'course.'.$ENV{'request.course.id'}.'.domain'}, + $ENV{'course.'.$ENV{'request.course.id'}.'.num'}, + '%255b'.$user.'%253a'.$domain.'%255d'); + my $result=''; + foreach (sort(keys(%records))) { + my %content=&unpackagemsg($records{$_}); + next if ($content{'senderdomain'} eq ''); + next if ($content{'subject'} !~ /^Record/); + # $content{'message'}=~s/\n/\/g; + $result.='Recorded by '. + $content{'sendername'}.'@'.$content{'senderdomain'}."\n"; + $result.= + &Apache::lontexconvert::msgtexconverted($content{'message'})."\n"; + } + return $result; +} + sub disfacetoface { my ($r,$user,$domain)=@_; my $target=$ENV{'form.grade_target'}; @@ -1433,7 +1537,8 @@ END # ----------------------------------------------------------- Display a message sub displaymessage { - my ($r,$msgid)=@_; + my ($r,$msgid,$folder)=@_; + my $suffix=&foldersuffix($folder); my %blocked = (); my %setters = (); my $startblock = 0; @@ -1441,16 +1546,17 @@ sub displaymessage { my $numblocked = 0; # info to generate "next" and "previous" buttons and check if message is blocked &blockcheck(\%setters,\$startblock,\$endblock); - my @messages=&sortedmessages(\%blocked,$startblock,$endblock,\$numblocked); + my @messages=&sortedmessages(\%blocked,$startblock,$endblock,\$numblocked,$folder); if ( $blocked{$msgid} eq 'ON' ) { &printheader($r,'/adm/email',&mt('Display a Message')); $r->print(&mt('You attempted to display a message that is currently blocked because you are enrolled in one or more courses for which there is an ongoing online exam.')); &build_block_table($r,$startblock,$endblock,\%setters); return; } - &statuschange($msgid,'read'); - my %message=&Apache::lonnet::get('nohist_email',[$msgid]); + &statuschange($msgid,'read',$folder); + my %message=&Apache::lonnet::get('nohist_email'.$suffix,[$msgid]); my %content=&unpackagemsg($message{$msgid}); + my $counter=0; $r->print('

');
     my $escmsgid=&Apache::lonnet::escape($msgid);
@@ -1476,6 +1582,7 @@ sub displaymessage {
 	      '
Delete'.&mt('Display all Messages').'12,bug=>'Communication Tools',}); +# ------------------------------------------------------------------ Get Folder + + my $folder=$ENV{'form.folder'}; + unless ($folder) { + $folder=''; + } else { + $sqs='&folder='.&Apache::lonnet::escape($folder); + } + # --------------------------------------------------------------- Render Output if ($ENV{'form.display'}) { - &displaymessage($r,$ENV{'form.display'}); + &displaymessage($r,$ENV{'form.display'},$folder); } elsif ($ENV{'form.replyto'}) { &compout($r,'',$ENV{'form.replyto'}); } elsif ($ENV{'form.confirm'}) { @@ -1587,8 +1703,20 @@ sub handler { &compout($r,$ENV{'form.forward'}); } elsif ($ENV{'form.markdel'}) { &printheader($r,'','Deleted Message'); - &statuschange($ENV{'form.markdel'},'deleted'); - &disall($r); + &statuschange($ENV{'form.markdel'},'deleted',$folder); + &disall($r,$folder); + } elsif ($ENV{'form.markedmove'}) { + my $total=0; + foreach (keys %ENV) { + if ($_=~/^form\.delmark_(.*)$/) { + &movemsg(&Apache::lonnet::unescape($1),$folder, + $ENV{'form.movetofolder'}); + $total++; + } + } + &printheader($r,'','Moved Messages'); + $r->print('Moved '.$total.' message(s)

'); + &disall($r,$folder); } elsif ($ENV{'form.markeddel'}) { my $total=0; foreach (keys %ENV) { @@ -1599,11 +1727,11 @@ sub handler { } &printheader($r,'','Deleted Messages'); $r->print('Deleted '.$total.' message(s)

'); - &disall($r); + &disall($r,$folder); } elsif ($ENV{'form.markunread'}) { &printheader($r,'','Marked Message as Unread'); &statuschange($ENV{'form.markunread'},'new'); - &disall($r); + &disall($r,$folder); } elsif ($ENV{'form.compose'}) { &compout($r,'','',$ENV{'form.compose'}); } elsif ($ENV{'form.recordftf'}) { @@ -1625,6 +1753,12 @@ sub handler { $ENV{'form.message'}.="\n\n-- Forwarded message --\n\n". $content{'message'}; } + if ($ENV{'form.replyid'}) { + my $msgid=$ENV{'form.replyid'}; + my %message=&Apache::lonnet::get('nohist_email',[$msgid]); + %content=&unpackagemsg($message{$msgid},1); + &statuschange($msgid,'replied'); + } my %toaddr=(); undef %toaddr; if ($ENV{'form.sendmode'} eq 'group') { @@ -1696,9 +1830,13 @@ sub handler { &mt('Please use the browser "Back" button and correct the recipient addresses') ); } + } elsif ($ENV{'form.newfolder'}) { + &printheader($r,'','New Folder'); + &makefolder($ENV{'form.newfolder'}); + &disall($r,$ENV{'form.newfolder'}); } else { &printheader($r,'','Display All Messages'); - &disall($r); + &disall($r,$folder); } $r->print(''); return OK;