--- loncom/interface/lonmsg.pm 2006/06/19 09:36:14 1.173.2.2 +++ loncom/interface/lonmsg.pm 2006/01/30 07:32:20 1.174 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # Routines for messaging # -# $Id: lonmsg.pm,v 1.173.2.2 2006/06/19 09:36:14 albertel Exp $ +# $Id: lonmsg.pm,v 1.174 2006/01/30 07:32:20 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -137,7 +137,7 @@ my $interdis; sub packagemsg { my ($subject,$message,$citation,$baseurl,$attachmenturl, - $recuser,$recdomain,$msgid,$type)=@_; + $recuser,$recdomain,$msgid,$type,$crsmsgid)=@_; $message =&HTML::Entities::encode($message,'<>&"'); $citation=&HTML::Entities::encode($citation,'<>&"'); $subject =&HTML::Entities::encode($subject,'<>&"'); @@ -170,11 +170,19 @@ sub packagemsg { $msgid = &buildmsgid($now,$subject,$env{'user.name'},$env{'user.domain'}, $msgcount,$course_context,$$); } - my $result=''.$env{'user.name'}.''. + my $result = ''.$env{'user.name'}.''. ''.$env{'user.domain'}.''. ''.$subject.''. - ''. - ''.$ENV{'SERVER_NAME'}.''. + ''; + if (defined($crsmsgid)) { + $result.= ''.$course_context.''. + ''.$env{'request.course.sec'}.''. + ''.$msgid.''. + ''.$crsmsgid.''. + ''.$message.''; + return ($msgid,$result); + } + $result .= ''.$ENV{'SERVER_NAME'}.''. ''.$ENV{'HTTP_HOST'}.''. ''.$ENV{'REMOTE_ADDR'}.''. ''.$env{'browser.type'}.''. @@ -431,18 +439,6 @@ sub all_url_author_res_msg { return %returnhash; } -# ====================================== Add a comment to the User Notes screen - -sub store_instructor_comment { - my ($msg,$uname,$udom) = @_; - my $cid = $env{'request.course.id'}; - my $cnum = $env{'course.'.$cid.'.num'}; - my $cdom = $env{'course.'.$cid.'.domain'}; - my $subject= &mt('Record').' ['.$uname.':'.$udom.']'; - my $result = &user_normal_msg_raw($cnum,$cdom,$subject,$msg); - return $result; -} - # ================================================== Critical message to a user sub user_crit_msg_raw { @@ -548,7 +544,7 @@ sub user_crit_received { sub user_normal_msg_raw { my ($user,$domain,$subject,$message,$citation,$baseurl,$attachmenturl, - $toperm,$currid,$newid,$sentmessage)=@_; + $toperm,$currid,$newid,$sentmessage,$crsmsgid)=@_; # Check if allowed missing my ($status,$packed_message); my $msgid='undefined'; @@ -558,10 +554,9 @@ sub user_normal_msg_raw { if ($homeserver ne 'no_host') { ($msgid,$packed_message)= &packagemsg($subject,$message,$citation,$baseurl, - $attachmenturl,$user,$domain,$currid); - (undef, my $packed_message_no_citation)= - &packagemsg($subject,$message,undef ,$baseurl, - $attachmenturl,$user,$domain,$currid); + $attachmenturl,$user,$domain,$currid, + undef,$crsmsgid); + # Store in user folder $status=&Apache::lonnet::critical( 'put:'.$domain.':'.$user.':nohist_email:'. @@ -577,6 +572,11 @@ sub user_normal_msg_raw { (&Apache::lonnet::allowed('srm',$env{'request.course.id'}) || &Apache::lonnet::allowed('srm',$env{'request.course.id'}. '/'.$env{'request.course.sec'})))) { + (undef, my $packed_message_no_citation)= + &packagemsg($subject,$message,undef ,$baseurl, + $attachmenturl,$user,$domain,$currid, + undef,$crsmsgid); + $status .= &store_sent_mail($msgid,$packed_message_no_citation); } } else { @@ -712,9 +712,8 @@ sub statuschange { &Apache::lonnet::put('email_status'.$suffix,{$msgid => $newstatus}); } if ($newstatus eq 'deleted') { - return &movemsg($msgid,$folder,'trash'); - } - return ; + &movemsg(&Apache::lonnet::unescape($msgid),$folder,'trash'); + } } # ============================================================= Make new folder @@ -735,53 +734,19 @@ sub movemsg { if ($srcfolder eq 'new') { $srcfolder=''; } my $srcsuffix=&foldersuffix($srcfolder); my $trgsuffix=&foldersuffix($trgfolder); - if ($srcsuffix eq $trgsuffix) { - return (0,&mt('Message not moved, Attempted to move message to the same folder as it already is in.')); - } # Copy message my %message=&Apache::lonnet::get('nohist_email'.$srcsuffix,[$msgid]); - if (!exists($message{$msgid}) || $message{$msgid} eq '') { - if (&Apache::slotrequest::network_error(%message)) { - return (0,&mt('Message not moved, A network error occurred.')); - } else { - return (0,&mt('Message not moved as the message is no longer in the source folder.')); - } - } - - my $result =&Apache::lonnet::put('nohist_email'.$trgsuffix, - {$msgid => $message{$msgid}}); - if (&Apache::slotrequest::network_error($result)) { - return (0,&mt('Message not moved, A network error occurred.')); - } + &Apache::lonnet::put('nohist_email'.$trgsuffix,{$msgid => $message{$msgid}}); # Copy status unless ($trgfolder eq 'trash') { - my %status=&Apache::lonnet::get('email_status'.$srcsuffix,[$msgid]); - # a non-existant status is the mark of an unread msg - if (&Apache::slotrequest::network_error(%status)) { - return (0,&mt('Message copied to new folder but status was not, A network error occurred.')); - } - my $result=&Apache::lonnet::put('email_status'.$trgsuffix, - {$msgid => $status{$msgid}}); - if (&Apache::slotrequest::network_error($result)) { - return (0,&mt('Message copied to new folder but status was not, A network error occurred.')); - } + my %status=&Apache::lonnet::get('email_status'.$srcsuffix,[$msgid]); + &Apache::lonnet::put('email_status'.$trgsuffix,{$msgid => $status{$msgid}}); } - # Delete orginals - my $result_del_msg = - &Apache::lonnet::del('nohist_email'.$srcsuffix,[$msgid]); - my $result_del_stat = - &Apache::lonnet::del('email_status'.$srcsuffix,[$msgid]); - if (&Apache::slotrequest::network_error($result_del_msg)) { - return (0,&mt('Message copied, but unable to delete the original from the source folder.')); - } - if (&Apache::slotrequest::network_error($result_del_stat)) { - return (0,&mt('Message copied, but unable to delete the original status from the source folder.')); - } - - return (1); + &Apache::lonnet::del('nohist_email'.$srcsuffix,[$msgid]); + &Apache::lonnet::del('email_status'.$srcsuffix,[$msgid]); } # ======================================================= Display a course list @@ -1210,9 +1175,7 @@ ENDDISHEADER $description.''.$status.''."\n"); } elsif ($status eq 'deleted') { # purge - my ($result,$msg) = - &movemsg(&Apache::lonnet::unescape($origID),$folder,'trash'); - + &movemsg(&Apache::lonnet::unescape($origID),$folder,'trash'); } } $r->print("\n

". @@ -1473,17 +1436,29 @@ sub disfacetoface { } elsif ($content{'subject'}=~/^Broadcast/) { $result .='

'.&mt('Broadcast Message').'

'; if ($content{'subject'}=~/^Broadcast\./) { - %content=&unpackagemsg($content{'message'}); - $content{'message'}= + if (defined($content{'coursemsgid'})) { + my $crsmsgid = &Apache::lonnet::escape($content{'coursemsgid'}); + my $broadcast_message = &general_message($crsmsgid); + $content{'message'} = ''.&mt('Subject').': '.$content{'message'}.'
'.$broadcast_message; + } else { + %content=&unpackagemsg($content{'message'}); + $content{'message'} = ''.&mt('Subject').': '.$content{'subject'}.'
'. $content{'message'}; + } } } else { $result.='

'.&mt('Critical Message').'

'; - %content=&unpackagemsg($content{'message'}); - $content{'message'}= + if (defined($content{'coursemsgid'})) { + my $crsmsgid=&Apache::lonnet::escape($content{'coursemsgid'}); + my $critical_message = &general_message($crsmsgid); + $content{'message'} = ''.&mt('Subject').': '.$content{'message'}.'
'.$critical_message; + } else { + %content=&unpackagemsg($content{'message'}); + $content{'message'}= ''.&mt('Subject').': '.$content{'subject'}.'
'. $content{'message'}; + } } $result.=&mt('By').': '. &Apache::loncommon::aboutmewrapper( @@ -1506,6 +1481,18 @@ $content{'sendername'}.'@'. } } +sub general_message { + my ($crsmsgid) = @_; + my %general_content; + if ($crsmsgid) { + my %course_content = &Apache::lonnet::get('nohist_email',[$crsmsgid], + $env{'course.'.$env{'request.course.id'}.'.domain'}, + $env{'course.'.$env{'request.course.id'}.'.num'}); + %general_content = &unpackagemsg($course_content{$crsmsgid}); + } + return $general_content{'message'}; +} + # ---------------------------------------------------------------- Face to face sub facetoface { @@ -1556,9 +1543,13 @@ ENDTREC ($env{'form.recdomain'}) && ($env{'form.recuname'})) { chomp($env{'form.newrecord'}); if ($env{'form.newrecord'}) { - &Apache::lonmsg::store_instructor_comment($env{'form.newrecord'}, - $env{'form.recuname'}, - $env{'form.recdomain'}); + my $recordtxt = $env{'form.newrecord'}; + &user_normal_msg_raw( + $env{'course.'.$env{'request.course.id'}.'.num'}, + $env{'course.'.$env{'request.course.id'}.'.domain'}, + &mt('Record'). + ' ['.$env{'form.recuname'}.':'.$env{'form.recdomain'}.']', + $recordtxt); } $r->print('

'.&Apache::loncommon::plainname($env{'form.recuname'}, $env{'form.recdomain'}).'

'); @@ -2205,6 +2196,7 @@ sub sendoffmail { my $savemsg; my $msgtype; my %sentmessage; + my $msgsubj=&Apache::lonfeedback::clear_out_html($env{'form.subject'}); if ((($env{'form.critmsg'}) || ($env{'form.sendbck'})) && (&Apache::lonnet::allowed('srm',$env{'request.course.id'}) || &Apache::lonnet::allowed('srm',$env{'request.course.id'}. @@ -2226,15 +2218,12 @@ sub sendoffmail { || &Apache::lonnet::allowed('srm',$env{'request.course.id'}. '/'.$env{'request.course.sec'}))) { $r->print(&mt('Sending critical message').' '.$recuname.'@'.$recdomain.': '); - $thismsg=&user_crit_msg($recuname,$recdomain, - &Apache::lonfeedback::clear_out_html($env{'form.subject'}), - $msgtxt, - $env{'form.sendbck'},$env{'form.permanent'},\$sentmessage{$_}); + $thismsg=&user_crit_msg($recuname,$recdomain,$msgsubj,$msgtxt, + $env{'form.sendbck'},$env{'form.permanent'}, + \$sentmessage{$_}); } else { $r->print(&mt('Sending').' '.$recuname.'@'.$recdomain.': '); - $thismsg=&user_normal_msg($recuname,$recdomain, - &Apache::lonfeedback::clear_out_html($env{'form.subject'}), - $msgtxt, + $thismsg=&user_normal_msg($recuname,$recdomain,$msgsubj,$msgtxt, $content{'citation'},undef,undef,$env{'form.permanent'},\$sentmessage{$_}); } if (($env{'request.course.id'}) && (($msgtype eq 'critical') || @@ -2270,7 +2259,7 @@ sub sendoffmail { my $record_sent; my @recusers = (); my @recudoms = (); - my ($stamp,$msgsubj,$msgname,$msgdom,$msgcount,$context,$pid) = + my ($stamp,$crssubj,$msgname,$msgdom,$msgcount,$context,$pid) = split(/\:/,&Apache::lonnet::unescape($specialmsgid)); foreach my $recipient (sort(keys(%toaddr))) { if ($specialmsg_status{$recipient} eq 'ok') { @@ -2278,8 +2267,8 @@ sub sendoffmail { my $usermsgid = &buildmsgid($stamp,$usersubj,$msgname, $msgdom,$msgcount,$context,$pid); &user_normal_msg_raw($cnum,$cdom,$subj_prefix. - ' ['.$recipient.']',$sentmessage{$recipient}, - undef,undef,undef,undef,$usermsgid); + ' ['.$recipient.']',$msgsubj,undef, + undef,undef,undef,$usermsgid,undef,undef,$specialmsgid); my ($uname,$udom) = split/:/,$recipient; push(@recusers,$uname); push(@recudoms,$udom); @@ -2288,14 +2277,12 @@ sub sendoffmail { if (@recusers) { my $specialmessage; my $sentsubj = $subj_prefix.' ('.$numspecial.' sent) '. - &Apache::lonfeedback::clear_out_html($env{'form.subject'}); + $msgsubj; $sentsubj = &HTML::Entities::encode($sentsubj,'<>&"'); my $sentmsgid = &buildmsgid($stamp,$sentsubj,$msgname, $msgdom,$msgcount,$context,$pid); - ($specialmsgid,$specialmessage) = - &packagemsg(&Apache::lonfeedback::clear_out_html( - $env{'form.subject'}),$savemsg,undef,undef,undef, - \@recusers,\@recudoms,$sentmsgid); + ($specialmsgid,$specialmessage) = &packagemsg($msgsubj,$savemsg, + undef,undef,undef,\@recusers,\@recudoms,$sentmsgid); $record_sent = &store_sent_mail($specialmsgid,$specialmessage); } } else { @@ -2423,63 +2410,31 @@ sub handler { &compout($r,$env{'form.forward'},undef,undef,undef,$folder); } elsif ($env{'form.markdel'}) { &printheader($r,'','Deleted Message'); - my ($result,$msg) = - &statuschange($env{'form.markdel'},'deleted',$folder); - if (!$result) { - $r->print('

Failed to delete the message.

'. - '

'.$msg."

\n"); - } + &statuschange($env{'form.markdel'},'deleted',$folder); &Apache::loncommunicate::menu($r); &disall($r,($folder?$folder:$dismode)); } elsif ($env{'form.markedmove'}) { - my ($total,$failed,@failed_msg)=(0,0); - foreach my $key (keys(%env)) { - if ($key=~/^form\.delmark_(.*)$/) { - my ($result,$msg) = - &movemsg(&Apache::lonnet::unescape($1),$folder, - $env{'form.movetofolder'}); - if ($result) { - $total++; - } else { - $failed++; - push(@failed_msg,$msg); - } + my $total=0; + foreach (keys %env) { + if ($_=~/^form\.delmark_(.*)$/) { + &movemsg(&Apache::lonnet::unescape($1),$folder, + $env{'form.movetofolder'}); + $total++; } } &printheader($r,'','Moved Messages'); - if ($failed) { - $r->print('

- Failed to move '.$failed.' message(s)

'); - $r->print('

'. - join("

\n

",@failed_msg). - "

\n"); - } $r->print('Moved '.$total.' message(s)

'); &Apache::loncommunicate::menu($r); &disall($r,($folder?$folder:$dismode)); } elsif ($env{'form.markeddel'}) { - my ($total,$failed,@failed_msg)=(0,0); - foreach my $key (keys(%env)) { - if ($key=~/^form\.delmark_(.*)$/) { - my ($result,$msg) = - &statuschange(&Apache::lonnet::unescape($1),'deleted', - $folder); - if ($result) { - $total++; - } else { - $failed++; - push(@failed_msg,$msg); - } + my $total=0; + foreach (keys %env) { + if ($_=~/^form\.delmark_(.*)$/) { + &statuschange(&Apache::lonnet::unescape($1),'deleted',$folder); + $total++; } } &printheader($r,'','Deleted Messages'); - if ($failed) { - $r->print('

- Failed to delete '.$failed.' message(s)

'); - $r->print('

'. - join("

\n

",@failed_msg). - "

\n"); - } $r->print('Deleted '.$total.' message(s)

'); &Apache::loncommunicate::menu($r); &disall($r,($folder?$folder:$dismode));