[BACK]Return to lonfeedback.pm CVS log [TXT] [DIR] Up to [LON-CAPA] / loncom / interface

Diff for /loncom/interface/lonfeedback.pm between versions 1.111 and 1.122

version 1.111, 2004/08/01 16:05:14 version 1.122, 2004/08/31 22:27:10
Line 36  use Apache::loncommon(); Line 36  use Apache::loncommon();
 use Apache::lontexconvert();  use Apache::lontexconvert();
 use Apache::lonlocal; # must not have ()  use Apache::lonlocal; # must not have ()
 use Apache::lonhtmlcommon();  use Apache::lonhtmlcommon();
   use HTML::LCParser();
 use Apache::lonspeller();  use Apache::lonspeller();
   use Cwd;
   
 sub discussion_open {  sub discussion_open {
     my ($status)=@_;      my ($status,$symb)=@_;
     if (defined($status) &&      if (defined($status) &&
  !($status eq 'CAN_ANSWER' || $status eq 'CANNOT_ANSWER'   !($status eq 'CAN_ANSWER' || $status eq 'CANNOT_ANSWER'
   || $status eq 'OPEN')) {    || $status eq 'OPEN')) {
  return 0;   return 0;
     }      }
     my $close=&Apache::lonnet::EXT('resource.0.discussend');      my $close=&Apache::lonnet::EXT('resource.0.discussend',$symb);
     if (defined($close) && $close ne '' && $close < time) {      if (defined($close) && $close ne '' && $close < time) {
  return 0;   return 0;
     }      }
Line 57  sub discussion_visible { Line 59  sub discussion_visible {
     if (not &discussion_open($status)) {      if (not &discussion_open($status)) {
  my $hidden=&Apache::lonnet::EXT('resource.0.discusshide');   my $hidden=&Apache::lonnet::EXT('resource.0.discusshide');
  if (lc($hidden) eq 'yes' or $hidden eq '' or !defined($hidden))  {   if (lc($hidden) eq 'yes' or $hidden eq '' or !defined($hidden))  {
     return 0;      if (!$ENV{'request.role.adv'}) { return 0; }
  }   }
     }      }
     return 1;      return 1;
Line 66  sub discussion_visible { Line 68  sub discussion_visible {
 sub list_discussion {  sub list_discussion {
     my ($mode,$status,$symb)=@_;      my ($mode,$status,$symb)=@_;
     my $outputtarget=$ENV{'form.grade_target'};      my $outputtarget=$ENV{'form.grade_target'};
       if (defined($ENV{'form.export'})) {
    if($ENV{'form.export'}) {
               $outputtarget = 'export';
           }
       }    
     if (not &discussion_visible($status)) { return ''; }      if (not &discussion_visible($status)) { return ''; }
     my @bgcols = ("#cccccc","#eeeeee");      my @bgcols = ("#cccccc","#eeeeee");
     my $discussiononly=0;      my $discussiononly=0;
Line 195  sub list_discussion { Line 202  sub list_discussion {
     my @discussionitems=();      my @discussionitems=();
     my %shown = ();      my %shown = ();
     my @posteridentity=();      my @posteridentity=();
     my %contrib=&Apache::lonnet::restore($ressymb,$ENV{'request.course.id'},  
   $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},      my $current=0;
   $ENV{'course.'.$ENV{'request.course.id'}.'.num'});  
     my $visible=0;      my $visible=0;
     my @depth=();      my @depth=();
     my @original=();      my @replies = ();
     my @index=();  
     my @replies=();  
     my %alldiscussion=();      my %alldiscussion=();
       my %imsitems=();
       my %imsfiles=();
     my %notshown = ();      my %notshown = ();
     my %newitem = ();      my %newitem = ();
     my $maxdepth=0;      my $maxdepth=0;
Line 218  sub list_discussion { Line 224  sub list_discussion {
     $discinfo{$visitkey} = $visit;      $discinfo{$visitkey} = $visit;
   
     &Apache::lonnet::put('nohist_'.$ENV{'request.course.id'}.'_discuss',\%discinfo,$ENV{'user.domain'},$ENV{'user.name'});      &Apache::lonnet::put('nohist_'.$ENV{'request.course.id'}.'_discuss',\%discinfo,$ENV{'user.domain'},$ENV{'user.name'});
       &build_posting_display(\%usernamesort,\%subjectsort,\%namesort,\%notshown,\%newitem,\%dischash,\%shown,\%alldiscussion,\%imsitems,\%imsfiles,\%roleinfo,\@discussionitems,\@replies,\@depth,\@posters,\$maxdepth,\$visible,\$newpostsflag,\$current,$status,$viewgrades,$seeid,$prevread,$sortposts,$ressymb,$target,$readkey,$showunmark,$showonlyunread,$totposters,$rolefilter,$sectionpick,$statusfilter,$toggkey,$outputtarget);
     if ($contrib{'version'}) {  
         my $oldest = $contrib{'1:timestamp'};  
         if ($prevread eq '0') {  
             $prevread = $oldest-1;  
         }  
  for (my $id=1;$id<=$contrib{'version'};$id++) {  
     my $idx=$id;  
             my $posttime = $contrib{$idx.':timestamp'};  
             if ($prevread <= $posttime) {  
                 $newpostsflag = 1;  
             }  
     my $hidden=($contrib{'hidden'}=~/\.$idx\./);  
             my $studenthidden=($contrib{'studenthidden'}=~/\.$idx\./);  
     my $deleted=($contrib{'deleted'}=~/\.$idx\./);  
     my $origindex='0.';  
             my $numoldver=0;  
     if ($contrib{$idx.':replyto'}) {  
                 if ( (($ENV{'environment.threadeddiscussion'}) && (($sortposts eq '') || ($sortposts eq 'ascdate'))) || ($sortposts eq 'thread')) {  
 # this is a follow-up message  
     $original[$idx]=$original[$contrib{$idx.':replyto'}];  
     $depth[$idx]=$depth[$contrib{$idx.':replyto'}]+1;  
     $origindex=$index[$contrib{$idx.':replyto'}];  
     if ($depth[$idx]>$maxdepth) { $maxdepth=$depth[$idx]; }  
                 } else {  
                     $original[$idx]=0;  
                     $depth[$idx]=0;  
                 }  
     } else {  
 # this is an original message  
  $original[$idx]=0;  
  $depth[$idx]=0;  
     }  
     if ($replies[$depth[$idx]]) {  
  $replies[$depth[$idx]]++;  
     } else {  
  $replies[$depth[$idx]]=1;  
     }  
     unless ((($hidden) && (!$seeid)) || ($deleted)) {  
  $visible++;  
                 if ($contrib{$idx.':history'}) {  
                     if ($contrib{$idx.':history'} =~ /:/) {  
                         my @oldversions = split/:/,$contrib{$idx.':history'};  
                         $numoldver = @oldversions;  
                     } else {  
                         $numoldver = 1;  
                     }   
                 }  
  my ($message,$subject);  
           if ($idx > 0) {  
                     if ($contrib{$idx.':message'} =~ /.*::::\Q$numoldver\E::::(.+?)$/si) {  
                         $message = $1;  
                     } else {  
                         $message = $contrib{$idx.':message'};  
                     }  
                 } else {  
                     $message=$contrib{$idx.':message'};  
                 }  
                 my $attachmenturls = $contrib{$idx.':attachmenturl'};   
  $message=~s/\n/\<br \/\>/g;  
  $message=&Apache::lontexconvert::msgtexconverted($message);  
           if ($idx > 0) {  
                     if ($contrib{$idx.':subject'} =~ /.*::::\Q$numoldver\E::::(.+?)$/si) {  
                         $subject = $1;  
                     } else {  
                         $subject = $contrib{$idx.':subject'};  
                     }  
                 } else {  
                     $subject=$contrib{$idx.':subject'};  
                 }  
                 if (defined($subject)) {  
                     $subject=~s/\n/\<br \/\>/g;  
                     $subject=&Apache::lontexconvert::msgtexconverted($subject);  
                 }  
  if ($attachmenturls) {  
                     my @attachments = ();  
                     my %currattach = ();  
                     &extract_attachments($attachmenturls,$idx,$numoldver,\$message,\@attachments,\%currattach);  
  }  
  if ($message) {  
     if ($hidden) {  
  $message='<font color="#888888">'.$message.'</font>';  
                         if ($studenthidden) {  
                             $message .='<br /><br />Deleted by poster (student).';  
                         }  
     }  
     my $screenname=&Apache::loncommon::screenname(  
     $contrib{$idx.':sendername'},  
     $contrib{$idx.':senderdomain'});  
     my $plainname=&Apache::loncommon::nickname(  
     $contrib{$idx.':sendername'},  
     $contrib{$idx.':senderdomain'});  
       
     my $sender=&mt('Anonymous');  
 # Set up for sorting by subject  
                     if ($contrib{$idx.':subject'} eq '') {  
                         if (defined($subjectsort{'__No subject'})) {  
                             push @{$subjectsort{'__No subject'}}, $idx;  
                         } else {  
                             @{$subjectsort{'__No subject'}} = ("$idx");  
                         }  
                     } else {  
                         if (defined($subjectsort{$contrib{$idx.':subject'}})) {  
                             push @{$subjectsort{$contrib{$idx.':subject'}}}, $idx;  
                         } else {  
                             @{$subjectsort{$contrib{$idx.':subject'}}} = ("$idx");  
                         }  
                     }  
     if ((!$contrib{$idx.':anonymous'}) || ($seeid)) {  
  $sender=&Apache::loncommon::aboutmewrapper(  
  $plainname,  
  $contrib{$idx.':sendername'},  
  $contrib{$idx.':senderdomain'}).' ('.  
  $contrib{$idx.':sendername'}.' at '.  
  $contrib{$idx.':senderdomain'}.')';  
  if ($contrib{$idx.':anonymous'}) {  
     $sender.=' ['.&mt('anonymous').'] '.  
  $screenname;  
  }  
 # Set up for sorting by domain, then username  
                         unless (defined($usernamesort{$contrib{$idx.':senderdomain'}})) {  
                             %{$usernamesort{$contrib{$idx.':senderdomain'}}} = ();  
                         }  
                         if (defined($usernamesort{$contrib{$idx.':senderdomain'}}{$contrib{$idx.':sendername'}})) {  
                             push @{$usernamesort{$contrib{$idx.':senderdomain'}}{$contrib{$idx.':sendername'}}}, $idx;  
                         } else {  
                             @{$usernamesort{$contrib{$idx.':senderdomain'}}{$contrib{$idx.':sendername'}}} = ("$idx");  
                         }  
 # Set up for sorting by last name, then first name  
                         my %names = &Apache::lonnet::get('environment',['firstname','lastname'],  
                                   $contrib{$idx.':senderdomain'},$contrib{$idx.':sendername'});  
                         my $lastname = $names{'lastname'};  
                         my $firstname = $names{'firstname'};  
                         if ($lastname eq '') {  
                             $lastname = '_';  
                         }  
                         if ($firstname eq '') {  
                             $firstname = '_';  
                         }  
                         unless (defined($namesort{$lastname})) {  
                             %{$namesort{$lastname}} = ();  
                         }  
                         if (defined($namesort{$lastname}{$firstname})) {  
                             push @{$namesort{$lastname}{$firstname}}, $idx;  
                         } else {  
                             @{$namesort{$lastname}{$firstname}} = ("$idx");  
                         }  
                         if ($ENV{"course.$cid.allow_discussion_post_editing"} =~ m/yes/i) {  
                             if (($ENV{'user.domain'} eq $contrib{$idx.':senderdomain'}) && ($ENV{'user.name'} eq $contrib{$idx.':sendername'})) {  
                                 $sender.=' <a href="/adm/feedback?editdisc='.  
                                     $ressymb.':::'.$idx;  
                                 if ($newpostsflag) {  
                                     $sender .= '&previous='.$prevread;  
                                 }  
                                 $sender .= '" '.$target.'>'.&mt('Edit').'</a>';                                      unless ($seeid) {  
                                     $sender.=" <a href=\"javascript:studentdelete('$ressymb','$idx','$newpostsflag','$prevread')";  
                                     $sender .= '">'.&mt('Delete').'</a>';  
                                 }  
                             }  
                         }  
  if ($seeid) {  
     if ($hidden) {  
                                 unless ($studenthidden) {  
     $sender.=' <a href="/adm/feedback?unhide='.  
         $ressymb.':::'.$idx;  
                                     if ($newpostsflag) {  
                                         $sender .= '&previous='.$prevread;  
                                     }  
                                     $sender .= '">'.&mt('Make Visible').'</a>';  
                                 }  
     } else {  
  $sender.=' <a href="/adm/feedback?hide='.  
     $ressymb.':::'.$idx;  
                                 if ($newpostsflag) {  
                                     $sender .= '&previous='.$prevread;  
                                 }  
                                 $sender .= '">'.&mt('Hide').'</a>';  
     }                       
     $sender.=' <a href="/adm/feedback?deldisc='.  
     $ressymb.':::'.$idx;  
                             if ($newpostsflag) {  
                                 $sender .= '&previous='.$prevread;  
                             }  
                             $sender .= '">'.&mt('Delete').'</a>';  
  }  
     } else {  
  if ($screenname) {  
     $sender='<i>'.$screenname.'</i>';  
  }  
 # Set up for sorting by domain, then username for anonymous  
                         unless (defined($usernamesort{'__anon'})) {  
                             %{$usernamesort{'__anon'}} = ();  
                         }  
                         if (defined($usernamesort{'__anon'}{'__anon'})) {  
                             push @{$usernamesort{'__anon'}{'__anon'}}, $idx;  
                         } else {  
                             @{$usernamesort{'__anon'}{'__anon'}} = ("$idx");  
                         }  
 # Set up for sorting by last name, then first name for anonymous  
                         unless (defined($namesort{'__anon'})) {  
                             %{$namesort{'__anon'}} = ();  
                         }  
                         if (defined($namesort{'__anon'}{'__anon'})) {  
                             push @{$namesort{'__anon'}{'__anon'}}, $idx;  
                         } else {  
                             @{$namesort{'__anon'}{'__anon'}} = ("$idx");  
                         }  
     }  
     if (&discussion_open($status) &&  
  &Apache::lonnet::allowed('pch',  
  $ENV{'request.course.id'}.  
  ($ENV{'request.course.sec'}?'/'.$ENV{'request.course.sec'}:''))) {  
  $sender.=' <a href="/adm/feedback?replydisc='.  
     $ressymb.':::'.$idx;  
                         if ($newpostsflag) {  
                             $sender .= '&previous='.$prevread;  
                         }  
                         $sender .= '" '.$target.'>'.&mt('Reply').'</a>';  
     }  
     my $vgrlink;  
     if ($viewgrades) {  
  $vgrlink=&Apache::loncommon::submlink('Submissions',  
             $contrib{$idx.':sendername'},$contrib{$idx.':senderdomain'},$symb);  
     }  
                     my $ctlink;  
                     if ($dischash{$readkey}=~/\.$idx\./) {   
                         $ctlink = '<b>'.&mt('Mark unread').'?</b>&nbsp;<input type="checkbox" name="postunread_'.$idx.'" />';  
                     } else {  
                         $ctlink = '<b>'.&mt('Mark read').'?</b>&nbsp;<input type="checkbox" name="postread_'.$idx.'" />';  
                     }  
 #figure out at what position this needs to print  
     my $thisindex=$idx;  
     if ( (($ENV{'environment.threadeddiscussion'}) && (($sortposts eq '') || ($sortposts eq 'ascdate'))) || ($sortposts eq 'thread')) {  
  $thisindex=$origindex.substr('00'.$replies[$depth[$idx]],-2,2);  
     }  
     $alldiscussion{$thisindex}=$idx;  
                     $shown{$idx} = 0;  
                     $index[$idx]=$thisindex;  
                     my $spansize = 2;  
                     if ($showonlyunread && $prevread > $posttime) {  
                         $notshown{$idx} = 1;  
                     } elsif ($showunmark && $dischash{$readkey}=~/\.$idx\./) {  
                         $notshown{$idx} = 1;  
                     } else {  
 # apply filters  
                         my $uname = $contrib{$idx.':sendername'};  
                         my $udom = $contrib{$idx.':senderdomain'};  
                         my $poster = $uname.':'.$udom;  
                         my $rolematch = '';  
                         my $skiptest = 1;  
                         if ($totposters > 0) {  
                             if (grep/^$poster$/,@posters) {  
                                 $shown{$idx} = 1;  
                             }  
                         } else {  
                             if ($rolefilter) {  
                                 if ($rolefilter eq 'all') {  
                                     $rolematch = '([^:]+)';  
                                 } else {  
                                     $rolematch = $rolefilter;  
                                     $skiptest = 0;  
                                 }  
                             }  
                             if ($sectionpick) {  
                                 if ($sectionpick eq 'all') {  
                                     $rolematch .= ':([^:]*)';  
                                 } else {  
                                     $rolematch .= ':'.$sectionpick;  
                                     $skiptest = 0;  
                                 }  
                             }  
                             if ($statusfilter) {  
                                 if ($statusfilter eq 'all') {  
                                     $rolematch .= ':([^:]+)';  
                                 } else {  
                                     $rolematch .= ':'.$statusfilter;  
                                     $skiptest = 0;  
                                 }  
                             }  
                             if ($skiptest) {  
                                 $shown{$idx} = 1;  
                             } else {  
                                 foreach my $role (@{$roleinfo{$poster}}) {  
                                     if ($role =~ m/^$rolematch$/) {  
                                         $shown{$idx} = 1;  
                                         last;  
                                     }  
                                 }  
                             }  
                         }  
                     }  
                     unless ($notshown{$idx} == 1) {  
                         if ($prevread > 0 && $prevread <= $posttime) {  
                             $newitem{$idx} = 1;  
                             $discussionitems[$idx] .= '  
                              <p><table border="0" width="100%">  
                               <tr><td align="left"><font color="#FF0000"><b>NEW</b></font></td>';  
                         } else {  
                             $newitem{$idx} = 0;  
                             $discussionitems[$idx] .= '  
                              <p><table border="0" width="100%">  
                               <tr><td align="left">&nbsp;</td>';  
                         }  
                         $discussionitems[$idx] .= '<td align ="left">&nbsp;&nbsp;'.  
                             '<b>'.$subject.'</b>&nbsp;&nbsp;'.  
                             $sender.'</b> '.$vgrlink.' ('.  
                             &Apache::lonlocal::locallocaltime($posttime).')</td>';  
                         if ($dischash{$toggkey}) {  
                             $discussionitems[$idx].='<td align="right">&nbsp;&nbsp;'.  
                               $ctlink.'</td>';  
                         }  
                         $discussionitems[$idx].= '</tr></table><blockquote>'.$message.'</blockquote></p>';  
                         if ($contrib{$idx.':history'}) {  
                             my @postversions = ();  
                             $discussionitems[$idx] .= '<br />'.&mt('This post has been edited by the author.');  
                             if ($seeid) {  
                                 $discussionitems[$idx] .= '&nbsp;&nbsp;<a href="/adm/feedback?allversions='.$ressymb.':::'.$idx.'">'.&mt('Display all versions').'</a>';  
                             }  
                             $discussionitems[$idx].='<br/>'.&mt('Earlier version(s) were posted on: ');  
                             if ($contrib{$idx.':history'} =~ m/:/) {  
                                 @postversions = split/:/,$contrib{$idx.':history'};  
                             } else {  
                                 @postversions = ("$contrib{$idx.':history'}");  
                             }  
                             for (my $i=0; $i<@postversions; $i++) {  
                                 my $version = $i+1;  
                                 $discussionitems[$idx] .= '<b>'.$version.'.</b> - '.&Apache::lonlocal::locallocaltime($postversions[$i]).'  ';  
                             }  
                         }  
                     }  
                 }  
             }  
  }  
     }  
   
     my $discussion='';      my $discussion='';
       my $manifestfile;
       my $manifestok=0;
       my $tempexport;
       my $imsresources;
       my $copyresult;
   
     my $function = &Apache::loncommon::get_users_function();      my $function = &Apache::loncommon::get_users_function();
     my $color = &Apache::loncommon::designparm($function.'.tabbg',      my $color = &Apache::loncommon::designparm($function.'.tabbg',
Line 622  sub list_discussion { Line 300  sub list_discussion {
   
     if ($visible) {      if ($visible) {
 # Print the discusssion  # Print the discusssion
  if ($outputtarget ne 'tex') {          if ($outputtarget eq 'tex') {
               $discussion.='<tex>\vskip 0 mm\noindent\makebox[2 cm][b]{\hrulefill}'.
                            '\textbf{DISCUSSIONS}\makebox[2 cm][b]{\hrulefill}'.
                            '\vskip 0 mm\noindent\textbf{'.$lt{'cuse'}.'}:\vskip 0 mm'.
                            '\noindent\textbf{'.$lt{'disa'}.'}: \textit{'.$currdisp.'}\vskip 0 mm'.
                            '\noindent\textbf{'.$lt{'npce'}.'}: \textit{'.$currmark.'}</tex>';
           } elsif ($outputtarget eq 'export') {
   # Create temporary directory if this is an export
               my $now = time;
               $tempexport = $Apache::lonnet::perlvar{'lonDaemons'}.'/tmp/ims_exports';
               if (!-e $tempexport) {
                   mkdir($tempexport,0700);
               }
               $tempexport .= '/'.$now;
               if (!-e $tempexport) {
                   mkdir($tempexport,0700);
               }
               $tempexport .= '/'.$ENV{'user.domain'}.'_'.$ENV{'user.name'};
               if (!-e $tempexport) {
                   mkdir($tempexport,0700);
               }
   # open manifest file
               my $manifest = '/imsmanifest.xml';
               my $manifestfilename = $tempexport.$manifest;
               print STDERR "manifestfilename is $manifestfilename\n";
               if ($manifestfile = Apache::File->new('>'.$manifestfilename)) {
                   $manifestok=1;
                   print $manifestfile qq|
   <?xml version="1.0" encoding="UTF-8"?>
   <manifest xmlns="http://www.imsglobal.org/xsd/imscp_v1p1" xmlns:imsmd="http://www.imsglobal.org/xsd/imsmd_v1p2" 
   xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" 
   identifier="MANIFEST-$ressymb" xsi:schemaLocation="http://www.imsglobal.org/xsd/imscp_v1p1 
   imscp_v1p1.xsd http://www.imsglobal.org/xsd/imsmd_v1p2 imsmd_v1p2p2.xsd">
     <organizations default="$ressymb">
       <organization identifier="$ressymb">
         <title>Discussion for $ressymb</title>\n|;
               } else {
                   $discussion .= 'An error occurred opening the manifest file.<br />';
               }
    } else {
             my $colspan=$maxdepth+1;              my $colspan=$maxdepth+1;
             $discussion.= qq|              $discussion.= qq|
 <script>  <script>
Line 676  sub list_discussion { Line 393  sub list_discussion {
  $discussion .= '<td>&nbsp;</td>';   $discussion .= '<td>&nbsp;</td>';
     }      }
     $discussion .= '</tr></table></td></tr>';      $discussion .= '</tr></table></td></tr>';
  } else {  
     $discussion.='\vskip 0 mm\noindent\makebox[2 cm][b]{\hrulefill}'.              my $numhidden = keys %notshown;
                          '\textbf{DISCUSSIONS}\makebox[2 cm][b]{\hrulefill}'.              if ($numhidden > 0) {
                          '\vskip 0 mm\noindent\textbf{'.$lt{'cuse'}.'}:\vskip 0 mm'.                  my $colspan = $maxdepth+1;
                          '\noindent\textbf{'.$lt{'disa'}.'}: \textit{'.$currdisp.'}\vskip 0 mm'.                  $discussion.="\n".'<tr><td bgcolor="#CCCCCC" colspan="'.$colspan.'">'.
                          '\noindent\textbf{'.$lt{'npce'}.'}: \textit{'.$currmark.'}';  
  }  
         my $numhidden = keys %notshown;  
         if ($numhidden > 0) {  
             my $colspan = $maxdepth+1;  
             $discussion.="\n".'<tr><td bgcolor="#CCCCCC" colspan="'.$colspan.'">'.  
                          '<a href="/adm/feedback?allposts='.$ressymb;                           '<a href="/adm/feedback?allposts='.$ressymb;
             if ($newpostsflag) {                  if ($newpostsflag) {
                 $discussion .= '&previous='.$prevread;                      $discussion .= '&previous='.$prevread;
             }                  }
             $discussion .= '">'.&mt('Show all posts').'</a> '.&mt('to display').' '.                  $discussion .= '">'.&mt('Show all posts').'</a> '.&mt('to display').' '.
                          $numhidden.' ';                           $numhidden.' ';
             if ($showunmark) {                  if ($showunmark) {
                 $discussion .= &mt('posts previously marked read');                      $discussion .= &mt('posts previously marked read');
             } else {                  } else {
                 $discussion .= &mt('previously viewed posts');                      $discussion .= &mt('previously viewed posts');
                   }
                   $discussion .= '<br/></td></tr>';
             }              }
             $discussion .= '<br/></td></tr>';  
         }          }
   
 # Choose sort mechanism  # Choose sort mechanism
Line 727  sub list_discussion { Line 439  sub list_discussion {
             $sortposts = 'ascdate';              $sortposts = 'ascdate';
             @showposts =  (sort { $a <=> $b } keys %alldiscussion);              @showposts =  (sort { $a <=> $b } keys %alldiscussion);
         }          }
           my $currdepth = 0;
           my $firstidx = $alldiscussion{$showposts[0]};
         foreach (@showposts) {          foreach (@showposts) {
             unless (($sortposts eq 'thread') || ($sortposts eq 'ascdate' && $ENV{'environment.threadeddiscussion'})) {              unless (($sortposts eq 'thread') || ($sortposts eq 'ascdate' && $ENV{'environment.threadeddiscussion'}) || ($outputtarget eq 'export')) {
                 $alldiscussion{$_} = $_;                  $alldiscussion{$_} = $_;
             }              }
             unless ( ($notshown{$alldiscussion{$_}} eq '1') || ($shown{$alldiscussion{$_}} == 0) ) {              unless ( ($notshown{$alldiscussion{$_}} eq '1') || ($shown{$alldiscussion{$_}} == 0) ) {
                 if ($outputtarget ne 'tex') {                  if ($outputtarget ne 'tex' && $outputtarget ne 'export') {
     $discussion.="\n<tr>";      $discussion.="\n<tr>";
  } else {  
     $discussion.='\vskip 0 mm\noindent\makebox[2 cm][b]{\hrulefill}';  
  }   }
         my $thisdepth=$depth[$alldiscussion{$_}];          my $thisdepth=$depth[$alldiscussion{$_}];
                 if ($outputtarget ne 'tex') {                  if ($outputtarget ne 'tex' && $outputtarget ne 'export') {
     for (1..$thisdepth) {      for (1..$thisdepth) {
  $discussion.='<td>&nbsp;&nbsp;&nbsp;</td>';   $discussion.='<td>&nbsp;&nbsp;&nbsp;</td>';
     }      }
  }   }
         my $colspan=$maxdepth-$thisdepth+1;          my $colspan=$maxdepth-$thisdepth+1;
                 if ($outputtarget ne 'tex') {                  if ($outputtarget eq 'tex') {
     $discussion.='<td  bgcolor="'.$bgcols[$newitem{$alldiscussion{$_}}].'" colspan="'.$colspan.'">'.  
                              $discussionitems[$alldiscussion{$_}].  
                      '</td></tr>';  
  } else {  
     #cleanup block      #cleanup block
     $discussionitems[$alldiscussion{$_}]=~s/<table([^>]*)>/<table TeXwidth="90 mm">/;      $discussionitems[$alldiscussion{$_}]=~s/<table([^>]*)>/<table TeXwidth="90 mm">/;
     $discussionitems[$alldiscussion{$_}]=~s/<tr([^>]*)><td([^>]*)>/<tr><td TeXwidth="20 mm" align="left">/;      $discussionitems[$alldiscussion{$_}]=~s/<tr([^>]*)><td([^>]*)>/<tr><td TeXwidth="20 mm" align="left">/;
Line 759  sub list_discussion { Line 467  sub list_discussion {
     $discussionitems[$alldiscussion{$_}]=~s/<\/td><td([^>]*)>/$threadinsert<\/td><td TeXwidth="65 mm" align="left">/;      $discussionitems[$alldiscussion{$_}]=~s/<\/td><td([^>]*)>/$threadinsert<\/td><td TeXwidth="65 mm" align="left">/;
     $discussionitems[$alldiscussion{$_}]=~s/<a([^>]+)>(Edit|Hide|Delete|Reply|Submissions)<\/a>//g;      $discussionitems[$alldiscussion{$_}]=~s/<a([^>]+)>(Edit|Hide|Delete|Reply|Submissions)<\/a>//g;
                     $discussionitems[$alldiscussion{$_}]=~s/(<b>|<\/b>|<\/a>|<a([^>]+)>)//g;                      $discussionitems[$alldiscussion{$_}]=~s/(<b>|<\/b>|<\/a>|<a([^>]+)>)//g;
       
                     #FIXME xmlparse can't be safely called from inside xmlparse      $discussionitems[$alldiscussion{$_}]='<tex>\vskip 0 mm\noindent\makebox[2 cm][b]{\hrulefill}</tex>'.$discussionitems[$alldiscussion{$_}];
                     #   due to the global variables that are use, the safe      $discussion.=$discussionitems[$alldiscussion{$_}];
                     #   space etc. I expect this has unforseen issues that   } elsif ($outputtarget eq 'export') {
                     #   need resolving.                      my $postfilename = $alldiscussion{$_}.'-'.$imsitems{$alldiscussion{$_}}{'timestamp'}.'.html';
                           if ($manifestok) {
                     $discussion.=&Apache::lonxml::xmlparse('','tex',$discussionitems[$alldiscussion{$_}]);                          if (($depth[$alldiscussion{$_}] <= $currdepth) && ($alldiscussion{$_} != $firstidx)) {
  }                              print STDERR "depth is $depth[$alldiscussion{$_}], currdepth is $currdepth, idx is $alldiscussion{$_}, firstidx is $firstidx\n";
                               print $manifestfile '  </item>'."\n";
                           }
                           $currdepth = $depth[$alldiscussion{$_}];
                           print $manifestfile "\n". 
         '<item identifier="ITEM-'.$ressymb.'-'.$alldiscussion{$_}.'" isvisible="'.
           $imsitems{$alldiscussion{$_}}{'isvisible'}.'" identifieref="RES-'.$ressymb.'-'.$alldiscussion{$_}.'">'.
           '<title>'.$imsitems{$alldiscussion{$_}}{'title'}.'</title>';
                           $imsresources .= "\n".
       '<resource identifier="RES-'.$ressymb.'-'.$alldiscussion{$_}.'" type="webcontent" href="'.$postfilename.'">'.
         '<file href="'.$alldiscussion{$_}.'.html">'."\n".
         $imsfiles{$alldiscussion{$_}}{$imsitems{$alldiscussion{$_}}{'currversion'}}."\n".
       '</resource>';
                       }
                       my $postingfile;
                       my $postingfilename = $tempexport.'/'.$postfilename;
                       if ($postingfile = Apache::File->new('>'.$postingfilename)) {
                           print $postingfile '<html><head><title>Discussion Post</title></head><body>'.
                                              $imsitems{$alldiscussion{$_}}{'title'}.' '.
                                              $imsitems{$alldiscussion{$_}}{'sender'}.
                                              $imsitems{$alldiscussion{$_}}{'timestamp'}.'<br /><br />'.
                                              $imsitems{$alldiscussion{$_}}{'message'}.'<br />'.
                                              $imsitems{$alldiscussion{$_}}{'attach'}.'</body></html>'."\n"; 
                           close($postingfile);
                       } else {
                           $discussion .= 'An error occurred opening the export file for posting '.$alldiscussion{$_}.'<br />';
                       }
                       $copyresult.=&replicate_attachments($imsitems{$alldiscussion{$_}}{'allattachments'},$tempexport);
                   } else {
                       $discussion.='<td  bgcolor="'.$bgcols[$newitem{$alldiscussion{$_}}].
                          '" colspan="'.$colspan.'">'. $discussionitems[$alldiscussion{$_}].
                          '</td></tr>';
                   }
     }      }
         }          }
  if ($outputtarget ne 'tex') {   unless ($outputtarget eq 'tex' || $outputtarget eq 'export') {
             my $colspan=$maxdepth+1;              my $colspan=$maxdepth+1;
             $discussion .= <<END;              $discussion .= <<END;
             <tr bgcolor="#FFFFFF">              <tr bgcolor="#FFFFFF">
Line 824  END Line 564  END
            </table>             </table>
            <br /><br /></form>             <br /><br /></form>
 END  END
  }          } 
           if ($outputtarget eq 'export') {
               if ($manifestok) {
                   while ($currdepth > 0) {
                       print $manifestfile "    </item>\n";
                       $currdepth --;
                   }
                   print $manifestfile qq|
       </organization>
     </organizations>
     <resources>
       $imsresources
     </resources>
   </manifest>
                   |;
                   close($manifestfile);
   
   #Create zip file in prtspool
   
                   my $imszipfile = '/prtspool/'.
                   $ENV{'user.name'}.'_'.$ENV{'user.domain'}.'_'.
                        time.'_'.rand(1000000000).'.zip';
       # zip can cause an sh launch which can pass along all of %ENV
       # which can be too large for /bin/sh to handle
                   my %oldENV=%ENV;
                   undef(%ENV);
                   my $cwd = &getcwd(); 
                   my $imszip = '/home/httpd/'.$imszipfile;
                   chdir $tempexport;
                   open(OUTPUT, "zip -r $imszip *  2> /dev/null |");
                   close(OUTPUT);
                   chdir $cwd;
                   %ENV=%oldENV;
                   undef(%oldENV);
                   $discussion .= 'Download the zip file from <a href="'.$imszipfile.'">Discussion Posting Archive</a><br />';
                   if ($copyresult) {
                       $discussion .= 'The following errors occurred during export - '.$copyresult;
                   }
               } else {
                   $discussion .= '<br />Unfortunately you will not be able to retrieve an archive of the discussion posts at this time, because there was a problem creating a manifest file.<br />';
               }
               return $discussion;
           }
     }      }
     if ($discussiononly) {      if ($discussiononly) {
         my $now = time;          my $now = time;
Line 844  END Line 626  END
                 $attachnum += @currnewattach;                  $attachnum += @currnewattach;
             }              }
         }          }
  $discussion.=(<<ENDDISCUSS);   if (&discussion_open($status)) {
       $discussion.=(<<ENDDISCUSS);
 <form action="/adm/feedback" method="post" name="mailform" enctype="multipart/form-data">  <form action="/adm/feedback" method="post" name="mailform" enctype="multipart/form-data">
 <input type="submit" name="discuss" value="Post Discussion" />  <input type="submit" name="discuss" value="Post Discussion" />
 <input type="submit" name="anondiscuss" value="Post Anonymous Discussion" />  <input type="submit" name="anondiscuss" value="Post Anonymous Discussion" />
Line 883  ENDDISCUSS Line 666  ENDDISCUSS
             $discussion.=$newattachmsg;              $discussion.=$newattachmsg;
     $discussion.=&generate_preview_button();      $discussion.=&generate_preview_button();
  }   }
     } else {   }
           } else {
  if (&discussion_open($status) &&   if (&discussion_open($status) &&
     &Apache::lonnet::allowed('pch',      &Apache::lonnet::allowed('pch',
    $ENV{'request.course.id'}.     $ENV{'request.course.id'}.
Line 896  ENDDISCUSS Line 680  ENDDISCUSS
     }      }
  }   }
     }      }
    return $discussion;      return $discussion;
   }
   
   sub build_posting_display {
       my ($usernamesort,$subjectsort,$namesort,$notshown,$newitem,$dischash,$shown,$alldiscussion,$imsitems,$imsfiles,$roleinfo,$discussionitems,$replies,$depth,$posters,$maxdepth,$visible,$newpostsflag,$current,$status,$viewgrades,$seeid,$prevread,$sortposts,$ressymb,$target,$readkey,$showunmark,$showonlyunread,$totposters,$rolefilter,$sectionpick,$statusfilter,$toggkey,$outputtarget) = @_;
   
       my @original=();
       my @index=();
   
       my %contrib=&Apache::lonnet::restore($ressymb,$ENV{'request.course.id'},
     $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
     $ENV{'course.'.$ENV{'request.course.id'}.'.num'});
   
       if ($contrib{'version'}) {
           my $oldest = $contrib{'1:timestamp'};
           if ($prevread eq '0') {
               $prevread = $oldest-1;
           }
    for (my $id=1;$id<=$contrib{'version'};$id++) {
       my $idx=$id;
               my $posttime = $contrib{$idx.':timestamp'};
               if ($prevread <= $posttime) {
                   $$newpostsflag = 1;
               }
       my $hidden=($contrib{'hidden'}=~/\.$idx\./);
               my $studenthidden=($contrib{'studenthidden'}=~/\.$idx\./);
       my $deleted=($contrib{'deleted'}=~/\.$idx\./);
       my $origindex='0.';
               my $numoldver=0;
       if ($contrib{$idx.':replyto'}) {
                   if ( (($ENV{'environment.threadeddiscussion'}) && (($sortposts eq '') || ($sortposts eq 'ascdate'))) || ($sortposts eq 'thread') || ($outputtarget eq 'export')) {
   # this is a follow-up message
       $original[$idx]=$original[$contrib{$idx.':replyto'}];
       $$depth[$idx]=$$depth[$contrib{$idx.':replyto'}]+1;
       $origindex=$index[$contrib{$idx.':replyto'}];
       if ($$depth[$idx]>$$maxdepth) { $$maxdepth=$$depth[$idx]; }
                   } else {
                       $original[$idx]=0;
                       $$depth[$idx]=0;
                   }
       } else {
   # this is an original message
    $original[$idx]=0;
    $$depth[$idx]=0;
       }
       if ($$replies[$$depth[$idx]]) {
    $$replies[$$depth[$idx]]++;
       } else {
    $$replies[$$depth[$idx]]=1;
       }
       unless ((($hidden) && (!$seeid)) || ($deleted)) {
    $$visible++;
                   if ($contrib{$idx.':history'}) {
                       if ($contrib{$idx.':history'} =~ /:/) {
                           my @oldversions = split/:/,$contrib{$idx.':history'};
                           $numoldver = @oldversions;
                       } else {
                           $numoldver = 1;
                       } 
                   }
                   $$current = $numoldver;
    my %messages = ();
                   my %subjects = ();
                   my %attachtxt = ();
                   my %allattachments = ();
                   my ($screenname,$plainname);
                   my $sender = &mt('Anonymous');
                   my ($message,$subject,$vgrlink,$ctlink);
                   &get_post_contents(\%contrib,$idx,$seeid,$outputtarget,\%messages,\%subjects,\%allattachments,\%attachtxt,$imsfiles,\$screenname,\$plainname,$numoldver);
   
   
   # Set up for sorting by subject
                   unless ($outputtarget eq 'export') {
                       $message=$messages{$numoldver};
                       $message.=$attachtxt{$numoldver};
                       $subject=$subjects{$numoldver};
                       if ($message) {
             if ($hidden) {
       $message='<font color="#888888">'.$message.'</font>';
                               if ($studenthidden) {
                                   $message .='<br /><br />Deleted by poster (student).';
                               }
           }
   
                           if ($subject eq '') {
                              if (defined($$subjectsort{'__No subject'})) {
                                  push @{$$subjectsort{'__No subject'}}, $idx;
                              } else {
                                  @{$$subjectsort{'__No subject'}} = ("$idx");
                              }
                           } else {
                               if (defined($$subjectsort{$subject})) {
                                  push @{$$subjectsort{$subject}}, $idx;
                               } else {
                                  @{$$subjectsort{$subject}} = ("$idx");
                               }
                           }
           if ((!$contrib{$idx.':anonymous'}) || ($seeid)) {
       $sender=&Apache::loncommon::aboutmewrapper(
    $plainname,
    $contrib{$idx.':sendername'},
    $contrib{$idx.':senderdomain'}).' ('.
    $contrib{$idx.':sendername'}.' at '.
    $contrib{$idx.':senderdomain'}.')';
       if ($contrib{$idx.':anonymous'}) {
           $sender.=' ['.&mt('anonymous').'] '.
       $screenname;
       }
   
   # Set up for sorting by domain, then username
                               unless (defined($$usernamesort{$contrib{$idx.':senderdomain'}})) {
                                   %{$$usernamesort{$contrib{$idx.':senderdomain'}}} = ();
                               }
                               if (defined($$usernamesort{$contrib{$idx.':senderdomain'}}{$contrib{$idx.':sendername'}})) {
                                   push @{$$usernamesort{$contrib{$idx.':senderdomain'}}{$contrib{$idx.':sendername'}}}, $idx;
                               } else {
                                   @{$$usernamesort{$contrib{$idx.':senderdomain'}}{$contrib{$idx.':sendername'}}} = ("$idx");
                               }
   # Set up for sorting by last name, then first name
                               my %names = &Apache::lonnet::get('environment',
                                    ['firstname','lastname'],$contrib{$idx.':senderdomain'},
                                     ,$contrib{$idx.':sendername'});
                               my $lastname = $names{'lastname'};
                               my $firstname = $names{'firstname'};
                               if ($lastname eq '') {
                                   $lastname = '_';
                               }
                               if ($firstname eq '') {
                                   $firstname = '_';
                               }
                               unless (defined($$namesort{$lastname})) {
                                   %{$$namesort{$lastname}} = ();
                               }
                               if (defined($$namesort{$lastname}{$firstname})) {
                                   push @{$$namesort{$lastname}{$firstname}}, $idx;
                               } else {
                                   @{$$namesort{$lastname}{$firstname}} = ("$idx");
                               }
                               if ($ENV{'course.'.$ENV{'request.course.id'}.'.allow_discussion_post_editing'} =~ m/yes/i) {
                                   if (($ENV{'user.domain'} eq $contrib{$idx.':senderdomain'}) && ($ENV{'user.name'} eq $contrib{$idx.':sendername'})) {
                                       $sender.=' <a href="/adm/feedback?editdisc='.
                                            $ressymb.':::'.$idx;
                                       if ($newpostsflag) {
                                           $sender .= '&previous='.$prevread;
                                       }
                                       $sender .= '" '.$target.'>'.&mt('Edit').'</a>';                                             
                                       unless ($seeid) {
                                           $sender.=" <a href=\"javascript:studentdelete('$ressymb','$idx','$newpostsflag','$prevread')";
                                           $sender .= '">'.&mt('Delete').'</a>';
                                       }
                                   }
                               } 
       if ($seeid) {
           if ($hidden) {
                                       unless ($studenthidden) {
                   $sender.=' <a href="/adm/feedback?unhide='.
                   $ressymb.':::'.$idx;
                                           if ($newpostsflag) {
                                                $sender .= '&previous='.$prevread;
                                           }
                                           $sender .= '">'.&mt('Make Visible').'</a>';
                                       }
           } else {
       $sender.=' <a href="/adm/feedback?hide='.
           $ressymb.':::'.$idx;
                                       if ($newpostsflag) {
                                           $sender .= '&previous='.$prevread;
                                       }
                                       $sender .= '">'.&mt('Hide').'</a>';
           }                     
           $sender.=' <a href="/adm/feedback?deldisc='.
           $ressymb.':::'.$idx;
                                   if ($newpostsflag) {
                                       $sender .= '&previous='.$prevread;
                                   }
                                   $sender .= '">'.&mt('Delete').'</a>';
                               }
           } else {
       if ($screenname) {
           $sender='<i>'.$screenname.'</i>';
       }
   # Set up for sorting by domain, then username for anonymous
                               unless (defined($$usernamesort{'__anon'})) {
                                   %{$$usernamesort{'__anon'}} = ();
                               }
                               if (defined($$usernamesort{'__anon'}{'__anon'})) {
                                   push @{$$usernamesort{'__anon'}{'__anon'}}, $idx;
                               } else {
                                   @{$$usernamesort{'__anon'}{'__anon'}} = ("$idx");
                               }
   # Set up for sorting by last name, then first name for anonymous
                               unless (defined($$namesort{'__anon'})) {
                                   %{$$namesort{'__anon'}} = ();
                               }
                               if (defined($$namesort{'__anon'}{'__anon'})) {
                                   push @{$$namesort{'__anon'}{'__anon'}}, $idx;
                               } else {
                                   @{$$namesort{'__anon'}{'__anon'}} = ("$idx");
                               }
           }
           if (&discussion_open($status) &&
           &Apache::lonnet::allowed('pch',
    $ENV{'request.course.id'}.
    ($ENV{'request.course.sec'}?'/'.$ENV{'request.course.sec'}:''))) {
       $sender.=' <a href="/adm/feedback?replydisc='.
               $ressymb.':::'.$idx;
                               if ($newpostsflag) {
                                       $sender .= '&previous='.$prevread;
                               }
                               $sender .= '" '.$target.'>'.&mt('Reply').'</a>';
                           }
           if ($viewgrades) {
           $vgrlink=&Apache::loncommon::submlink('Submissions',
                               $contrib{$idx.':sendername'},$contrib{$idx.':senderdomain'},$ressymb);
           }
                           if ($$dischash{$readkey}=~/\.$idx\./) { 
                               $ctlink = '<b>'.&mt('Mark unread').'?</b>&nbsp;<input type="checkbox" name="postunread_'.$idx.'" />';
                           } else {
                               $ctlink = '<b>'.&mt('Mark read').'?</b>&nbsp;<input type="checkbox" name="postread_'.$idx.'" />';
                           }
                       }
   #figure out at what position this needs to print
                   }
                   if ($outputtarget eq 'export' || $message) {
       my $thisindex=$idx;
       if ( (($ENV{'environment.threadeddiscussion'}) && (($sortposts eq '') || ($sortposts eq 'ascdate'))) || ($sortposts eq 'thread') || ($outputtarget eq 'export')) {
    $thisindex=$origindex.substr('00'.$$replies[$$depth[$idx]],-2,2);
       }
       $$alldiscussion{$thisindex}=$idx;
                       $$shown{$idx} = 0;
                       $index[$idx]=$thisindex;
                   }
                   if ($outputtarget eq 'export') {
                       %{$$imsitems{$idx}} = ();
                       $$imsitems{$idx}{'isvisible'}='true';
                       if ($hidden) {
                           $$imsitems{$idx}{'isvisible'}='false';
                       }
                       $$imsitems{$idx}{'title'}=$subjects{$numoldver};
                       $$imsitems{$idx}{'message'}=$messages{$numoldver};
                       $$imsitems{$idx}{'attach'}=$attachtxt{$numoldver};
                       $$imsitems{$idx}{'timestamp'}=$contrib{$idx.':timestamp'};
                       $$imsitems{$idx}{'sender'}=$plainname.' ('.
                                            $contrib{$idx.':sendername'}.' at '.
                                            $contrib{$idx.':senderdomain'}.')';
                       $$imsitems{$idx}{'isanonymous'}='false';
                       if ($contrib{$idx.':anonymous'}) {
                           $$imsitems{$idx}{'isanonymous'}='true';
                       }
                       $$imsitems{$idx}{'currversion'}=$numoldver;
                       %{$$imsitems{$idx}{'allattachments'}}=%allattachments;
                       unless ($messages{$numoldver} eq '' && $attachtxt{$numoldver} eq '') {
                           $$shown{$idx} = 1;
                       }
                   } else {
                       if ($message) {
                           my $spansize = 2;
                           if ($showonlyunread && $prevread > $posttime) {
                               $$notshown{$idx} = 1;
                           } elsif ($showunmark && $$dischash{$readkey}=~/\.$idx\./) {
                               $$notshown{$idx} = 1;
                           } else {
   # apply filters
                               my $uname = $contrib{$idx.':sendername'};
                               my $udom = $contrib{$idx.':senderdomain'};
                               my $poster = $uname.':'.$udom;
                               my $rolematch = '';
                               my $skiptest = 1;
                               if ($totposters > 0) {
                                   if (grep/^$poster$/,@{$posters}) {
                                       $$shown{$idx} = 1;
                                   }
                               } else {
                                   if ($rolefilter) {
                                       if ($rolefilter eq 'all') {
                                           $rolematch = '([^:]+)';
                                       } else {
                                           $rolematch = $rolefilter;
                                           $skiptest = 0;
                                       }
                                   }
                                   if ($sectionpick) {
                                       if ($sectionpick eq 'all') {
                                           $rolematch .= ':([^:]*)';
                                       } else {
                                           $rolematch .= ':'.$sectionpick;
                                           $skiptest = 0;
                                       }
                                   }
                                   if ($statusfilter) {
                                       if ($statusfilter eq 'all') {
                                           $rolematch .= ':([^:]+)';
                                       } else {
                                           $rolematch .= ':'.$statusfilter;
                                           $skiptest = 0;
                                       }
                                   }
                                   if ($skiptest) {
                                       $$shown{$idx} = 1;
                                   } else {
                                       foreach my $role (@{$$roleinfo{$poster}}) {
                                           if ($role =~ m/^$rolematch$/) {
                                               $$shown{$idx} = 1;
                                               last;
                                           }
                                       }
                                   }
                               }
                           }
                           unless ($$notshown{$idx} == 1) {
                               if ($prevread > 0 && $prevread <= $posttime) {
                                   $$newitem{$idx} = 1;
                                   $$discussionitems[$idx] .= '
                                    <p><table border="0" width="100%">
                                     <tr><td align="left"><font color="#FF0000"><b>NEW</b></font></td>';
                               } else {
                                   $$newitem{$idx} = 0;
                                   $$discussionitems[$idx] .= '
                                    <p><table border="0" width="100%">
                                     <tr><td align="left">&nbsp;</td>';
                               }
                               $$discussionitems[$idx] .= '<td align ="left">&nbsp;&nbsp;'.
                                   '<b>'.$subject.'</b>&nbsp;&nbsp;'.
                                   $sender.'</b> '.$vgrlink.' ('.
                                   &Apache::lonlocal::locallocaltime($posttime).')</td>';
                               if ($$dischash{$toggkey}) {
                                   $$discussionitems[$idx].='<td align="right">&nbsp;&nbsp;'.
                                     $ctlink.'</td>';
                               }
                               $$discussionitems[$idx].= '</tr></table><blockquote>'.
                                       $message.'</blockquote></p>';
                               if ($contrib{$idx.':history'}) {
                                   my @postversions = ();
                                   $$discussionitems[$idx] .= &mt('This post has been edited by the author.');
                                   if ($seeid) {
                                       $$discussionitems[$idx] .= '&nbsp;&nbsp;<a href="/adm/feedback?allversions='.$ressymb.':::'.$idx.'">'.&mt('Display all versions').'</a>';
                                   }
                                   $$discussionitems[$idx].='<br/>'.&mt('Earlier version(s) were posted on: ');
                                   if ($contrib{$idx.':history'} =~ m/:/) {
                                       @postversions = split/:/,$contrib{$idx.':history'};
                                   } else {
                                       @postversions = ("$contrib{$idx.':history'}");
                                   }
                                   for (my $i=0; $i<@postversions; $i++) {
                                       my $version = $i+1;
                                       $$discussionitems[$idx] .= '<b>'.$version.'.</b> - '.&Apache::lonlocal::locallocaltime($postversions[$i]).'  ';
                                   }
                               }
                           }
                       }
                   }
               }
    }
       }
   }
   
   sub get_post_contents {
       my ($contrib,$idx,$seeid,$type,$messages,$subjects,$allattachments,$attachtxt,$imsfiles,$screenname,$plainname,$numver) = @_;
       my $discussion = '';
       my $start=$numver;
       my $end=$numver + 1;
       %{$$imsfiles{$idx}}=();
       if ($type eq 'allversions') {
          unless($seeid) {
              $discussion=&mt('You do not have privileges to view all versions of posts.').&mt('Please select a different role');
              return $discussion;
          } 
       }
       $$screenname=&Apache::loncommon::screenname(
                                           $$contrib{$idx.':sendername'},
                                           $$contrib{$idx.':senderdomain'});
       $$plainname=&Apache::loncommon::nickname(
                                           $$contrib{$idx.':sendername'},
                                           $$contrib{$idx.':senderdomain'});
       my $sender=&Apache::loncommon::aboutmewrapper(
                                    $$plainname,
                                    $$contrib{$idx.':sendername'},
                                    $$contrib{$idx.':senderdomain'}).' ('.
                                    $$contrib{$idx.':sendername'}.' at '.
                                    $$contrib{$idx.':senderdomain'}.')';
       my $attachmenturls = $$contrib{$idx.':attachmenturl'};
       my @postversions = ();
       if ($type eq 'allversions' || $type eq 'export') {
           $start = 0;
           if ($$contrib{$idx.':history'}) {
               if ($$contrib{$idx.':history'} =~ m/:/) {
                   @postversions = split/:/,$$contrib{$idx.':history'};
               } else {
                   @postversions = ("$$contrib{$idx.':history'}");
               }
           }
           &get_post_versions($messages,$$contrib{$idx.':message'},1);
           &get_post_versions($subjects,$$contrib{$idx.':subject'},1);
           push @postversions,$$contrib{$idx.':timestamp'};
           $end = @postversions;
       } else {
           &get_post_versions($messages,$$contrib{$idx.':message'},1,$numver);
           &get_post_versions($subjects,$$contrib{$idx.':subject'},1,$numver);
       }
   
       if ($$contrib{$idx.':anonymous'}) {
           $sender.=' ['.&mt('anonymous').'] '.$$screenname;
       }
       if ($type eq 'allversions') {
           $discussion=('<b>'.$sender.'</b><br /><ul>');
       }
       for (my $i=$start; $i<$end; $i++) {
           my ($timesent,$attachmsg);
           my %currattach = ();
           $timesent = &Apache::lonlocal::locallocaltime($postversions[$i]);
           $$messages{$i}=~s/\n/\<br \/\>/g;
           $$messages{$i}=&Apache::lontexconvert::msgtexconverted($$messages{$i});
           $$subjects{$i}=~s/\n/\<br \/\>/g;
           $$subjects{$i}=&Apache::lontexconvert::msgtexconverted($$subjects{$i});
           if ($attachmenturls) {
               &extract_attachments($attachmenturls,$idx,$i,\$attachmsg,$allattachments,\%currattach);
           }
           if ($type eq 'export') {
               $$imsfiles{$idx}{$i} = '';
               if ($attachmsg) {
                   $$attachtxt{$i} = '<br />Attachments:<br />';
                   foreach (sort keys %currattach) {
                       if ($$allattachments{$_}{'filename'} =~ m-^/uploaded/([^/]+/[^/]+)(/feedback)?(/?\d*)/([^/]+)$-) {
                           my $fname = $1.$3.'/'.$4;
                           $$imsfiles{$idx}{$i} .= '<file href="'.$fname.'">'."\n";
                           $$attachtxt{$i}.= '<a href="'.$fname.'">'.$4.'</a><br />';
                       }
                   }
               }
           } else {
               if ($attachmsg) {
                   $$attachtxt{$i} = '<br />Attachments:'.$attachmsg.'<br />';
               } else {
                   $$attachtxt{$i} = '';
               }
           }
           if ($type eq 'allversions') {
               $discussion.= <<"END";
   <li><b>$$subjects{$i}</b>, $timesent<br />
   $$messages{$i}<br />
   $$attachtxt{$i}</li>
   END
           }
       }
       if ($type eq 'allversions') {
           $discussion.=('</ul></body></html>');
           return $discussion;
       } else {
           return;
       }
   }
   
   sub replicate_attachments {
       my ($attachrefs,$tempexport) = @_;
       my $response;
       foreach my $id (keys %{$attachrefs}) {
           if ($$attachrefs{$id}{'filename'} =~ m-^/uploaded/([^/]+)/([^/]+)(/feedback)?(/?\d*)/([^/]+)$-) {
               my $path = $tempexport;
               my $tail = $1.'/'.$2.$4;
               my @extras = split/\//,$tail;
               my $destination = $tempexport.'/'.$1.'/'.$2.$4.'/'.$5;
               if (!-e $destination) {
                   my $i= 0;
                   while ($i<@extras) {
                       $path .= '/'.$extras[$i];
                       if (!-e $path) {
                           mkdir($path,0700);
                       }
                       $i ++;
                   }
                   my ($content,$rtncode);
                   print STDERR "File to replicate is $$attachrefs{$id}{'filename'} in $1,$2\n";
                   my $uploadreply = &Apache::lonnet::getuploaded('GET',$$attachrefs{$id}{'filename'},$1,$2,$content,$rtncode);
                   if ($uploadreply eq 'ok') {
                        my $attachcopy;
                        if ($attachcopy = Apache::File->new('>'.$destination)) {
                            print $attachcopy $content;
                            close($attachcopy);
                        } else {
                            $response .= 'Error copying a file attachment to IMS package: '.$!.'<br />'."\n";
                        }
                   } else {
                       print STDERR "return code from lonnet was $rtncode\n";
                   }
               }
           }
       }
 }  }
   
 sub mail_screen {  sub mail_screen {
Line 920  sub mail_screen { Line 1190  sub mail_screen {
   my @currnewattach = ();    my @currnewattach = ();
   my @currdelold = ();    my @currdelold = ();
   my @keepold = ();    my @keepold = ();
   my @attachments = ();    my %attachments = ();
   my %currattach = ();    my %currattach = ();
   my $attachnum = 0;    my $attachnum = 0;
   my $anonchk = (<<END);    my $anonchk = (<<END);
Line 966  END Line 1236  END
    $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},     $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
    $ENV{'course.'.$ENV{'request.course.id'}.'.num'});     $ENV{'course.'.$ENV{'request.course.id'}.'.num'});
       unless (($contrib{'hidden'}=~/\.$idx\./) || ($contrib{'deleted'}=~/\.$idx\./)) {        unless (($contrib{'hidden'}=~/\.$idx\./) || ($contrib{'deleted'}=~/\.$idx\./)) {
             if ($contrib{$idx.':history'}) {
                 if ($contrib{$idx.':history'} =~ /:/) {
                     my @oldversions = split/:/,$contrib{$idx.':history'};
                     $numoldver = @oldversions;
                 } else {
                     $numoldver = 1;
                 }
             }
           if ($ENV{'form.replydisc'}) {            if ($ENV{'form.replydisc'}) {
               if ($contrib{$idx.':history'}) {                if ($contrib{$idx.':history'}) {
                   if ($contrib{$idx.':history'} =~ /:/) {                    if ($contrib{$idx.':history'} =~ /:/) {
Line 977  END Line 1255  END
               }                }
               my $message;                my $message;
               if ($idx > 0) {                if ($idx > 0) {
                   if ($contrib{$idx.':message'} =~ /::::\Q$numoldver\E::::(.+?)$/si) {                    my %msgversions = ();
                       $message = $1;                    &get_post_versions(\%msgversions,$contrib{$idx.':message'},0,$numoldver);
                   } else {                    $message = $msgversions{$numoldver};
                       $message = $contrib{$idx.':message'};  
                   }  
               } else {  
           $message=$contrib{$idx.':message'};  
               }                }
       $message=~s/\n/\<br \/\>/g;        $message=~s/\n/\<br \/\>/g;
       $quote='<blockquote>'.&Apache::lontexconvert::msgtexconverted($message).'</blockquote>';        $quote='<blockquote>'.&Apache::lontexconvert::msgtexconverted($message).'</blockquote>';
               if ($idx > 0) {                if ($idx > 0) {
                   if ($contrib{$idx.':subject'} =~ /::::\Q$numoldver\E::::(.+?)$/si) {                    my %subversions = ();
                       $subject = $1;                    &get_post_versions(\%subversions,$contrib{$idx.':subject'},1,$numoldver);
                   } else {                    $subject = 'Re: '.$subversions{$numoldver};
                       $subject = $contrib{$idx.':subject'};  
                   }  
                   $subject = 'Re: '.$subject;  
               }                }
               $subject = &HTML::Entities::encode($subject,'<>&"');                $subject = &HTML::Entities::encode($subject,'<>&"');
           } else {            } else {
               $attachmenturls = $contrib{$idx.':attachmenturl'};                $attachmenturls = $contrib{$idx.':attachmenturl'};
               if ($contrib{$idx.':message'} =~ /.*::::(\d+)::::(.*?)$/si) {                if ($idx > 0) {
                   $numoldver = $1;                    my %msgversions = ();
                   $comment = $2;                    &get_post_versions(\%msgversions,$contrib{$idx.':message'},0,$numoldver);
               } else {                    $comment = $msgversions{$numoldver};
                   $comment = $contrib{$idx.':message'};                    my %subversions = ();
               }                    &get_post_versions(\%subversions,$contrib{$idx.':subject'},0,$numoldver);
               $comment = &HTML::Entities::encode($comment,'<>&"');                    $subject = $subversions{$numoldver}; 
               if ($contrib{$idx.':subject'} =~ /.*::::\d+::::(.+?)$/si) {  
                   $subject = $1;  
               } else {  
                   $subject = $contrib{$idx.':subject'};  
               }                }
               $subject = &HTML::Entities::encode($subject,'<>&"');  
               if (defined($contrib{$idx.':replyto'})) {                if (defined($contrib{$idx.':replyto'})) {
                   $parentmsg = $contrib{$idx.':replyto'};                    $parentmsg = $contrib{$idx.':replyto'};
               }                }
Line 1091  $htmlheader Line 1357  $htmlheader
         }          }
   
         if (rec) {          if (rec) {
             if (typeof(document.mailform.onsubmit)!='undefined') {              if (typeof(document.mailform.onsubmit)=='function') {
  document.mailform.onsubmit();   document.mailform.onsubmit();
     }      }
     document.mailform.submit();      document.mailform.submit();
Line 1145  END Line 1411  END
         }          }
         if ($ENV{'form.editdisc'}) {          if ($ENV{'form.editdisc'}) {
             if ($attachmenturls) {              if ($attachmenturls) {
                 &extract_attachments($attachmenturls,$idx,$numoldver,\$attachmsg,\@attachments,\%currattach,\@currdelold);                  &extract_attachments($attachmenturls,$idx,$numoldver,\$attachmsg,\%attachments,\%currattach,\@currdelold);
                 $attachnum = scalar(keys %currattach);                  $attachnum = scalar(keys %currattach);
                 foreach (keys %currattach) {                  foreach (keys %currattach) {
                     $r->print('<input type="hidden" name="keepold" value="'.$_.'" />'."\n");                      $r->print('<input type="hidden" name="keepold" value="'.$_.'" />'."\n");
Line 1652  END Line 1918  END
 END  END
 }  }
   
 sub fail_redirect {  sub get_post_versions {
       my ($versions,$incoming,$htmldecode,$numver) = @_;
       if ($incoming =~ /^<version num="0">/) {
           my $p = HTML::LCParser->new(\$incoming);
           my $done = 0;                                                                       
           while ( (my $token = $p->get_tag("version")) && (!$done)) {
               my $num = $token->[1]{num};
               my $text = $p->get_text("/version");
               if (defined($numver)) {
                   if ($num == $numver) {
                       if ($htmldecode) {
                           $text = &HTML::Entities::decode($text);
                       }
                       $$versions{$numver}=$text;
                       $done = 1;
                   }
               } else {
                   if ($htmldecode) {
                       $text = &HTML::Entities::decode($text);
                   }
                   $$versions{$num}=$text;
               }
           }
       } else {
           if (!defined($numver)) {
               $numver = 0;
           }
           if ($htmldecode) {
               $$versions{$numver} = $incoming;
           } else {
               $$versions{$numver} = &HTML::Entities::encode($incoming,'<>&"');
           }
       }
       return;
   }
   
   sub get_post_attachments {
       my ($attachments,$attachmenturls) = @_;
       my $num;
       if ($attachmenturls =~ m/^<attachment id="0">/) {
           my $p = HTML::LCParser->new(\$attachmenturls);
           while (my $token = $p->get_tag("attachment","filename","post"))  {
               if ($token->[0] eq "attachment") {
                   $num = $token->[1]{id};
                   %{$$attachments{$num}} =();
               } elsif ($token->[0] eq "filename") {
                   $$attachments{$num}{'filename'} = $p->get_text("/filename");
               } elsif ($token->[0] eq "post") {
                   my $id = $token->[1]{id};
                   $$attachments{$num}{$id} = $p->get_text("/post");
               }
           }
       } else {
           %{$$attachments{'0'}} = ();
           $$attachments{'0'}{'filename'} = $attachmenturls;
           $$attachments{'0'}{'0'} = 'n';
       }
   
       return;
   }
   
   sub fail_redirect {;
   my ($r,$feedurl) = @_;    my ($r,$feedurl) = @_;
   if ($feedurl=~/^\/adm\//) { $feedurl.='?register=1' };    if ($feedurl=~/^\/adm\//) { $feedurl.='?register=1' };
   $r->print (<<ENDFAILREDIR);    $r->print (<<ENDFAILREDIR);
Line 1664  sub fail_redirect { Line 1991  sub fail_redirect {
 <body bgcolor="#FFFFFF">  <body bgcolor="#FFFFFF">
 <img align="right" src="/adm/lonIcons/lonlogos.gif" />  <img align="right" src="/adm/lonIcons/lonlogos.gif" />
 <b>Sorry, no recipients  ...</b>  <b>Sorry, no recipients  ...</b>
   <br /><a href="$feedurl">Continue</a>
 </body>  </body>
 </html>  </html>
 ENDFAILREDIR  ENDFAILREDIR
Line 1734  $roletag Line 2062  $roletag
 $sectag  $sectag
 $userpicktag  $userpicktag
 </form>  </form>
   <br /><a href="$feedurl">Continue</a>
 </body>  </body>
 </html>  </html>
 ENDREDIR  ENDREDIR
Line 1742  ENDREDIR Line 2071  ENDREDIR
 sub no_redirect_back {  sub no_redirect_back {
   my ($r,$feedurl) = @_;    my ($r,$feedurl) = @_;
   my $nofeed=&mt('Sorry, no feedback possible on this resource  ...');    my $nofeed=&mt('Sorry, no feedback possible on this resource  ...');
     my $continue=&mt('Continue');
   $r->print (<<ENDNOREDIR);    $r->print (<<ENDNOREDIR);
 <html>  <html>
 <head><title>Feedback not sent</title>  <head><title>Feedback not sent</title>
Line 1757  ENDNOREDIR Line 2087  ENDNOREDIR
 <body bgcolor="#FFFFFF" onLoad='if (window.name!="loncapaclient") { self.close(); }'>  <body bgcolor="#FFFFFF" onLoad='if (window.name!="loncapaclient") { self.close(); }'>
 <img align="right" src="/adm/lonIcons/lonlogos.gif" />  <img align="right" src="/adm/lonIcons/lonlogos.gif" />
 <b>$nofeed</b>  <b>$nofeed</b>
   <br /><a href="$feedurl">$continue</a>
 </body>  </body>
 </html>  </html>
 ENDNOREDIRTWO  ENDNOREDIRTWO
Line 1966  sub send_msg { Line 2297  sub send_msg {
 sub adddiscuss {  sub adddiscuss {
     my ($symb,$email,$anon,$attachmenturl,$subject)=@_;      my ($symb,$email,$anon,$attachmenturl,$subject)=@_;
     my $status='';      my $status='';
     if (&discussion_open() &&      my $realsymb;
       if ($symb=~/^bulletin___/) {
    my $filename=(&Apache::lonnet::decode_symb($symb))[2];
    $filename=~s|^adm/wrapper/||;
    $realsymb=&Apache::lonnet::symbread($filename);
       }
       if (&discussion_open(undef,$realsymb) &&
  &Apache::lonnet::allowed('pch',$ENV{'request.course.id'}.   &Apache::lonnet::allowed('pch',$ENV{'request.course.id'}.
         ($ENV{'request.course.sec'}?'/'.$ENV{'request.course.sec'}:''))) {          ($ENV{'request.course.sec'}?'/'.$ENV{'request.course.sec'}:''))) {
   
Line 2014  sub adddiscuss { Line 2351  sub adddiscuss {
             }              }
             my $numnewver = $numoldver + 1;              my $numnewver = $numoldver + 1;
             if (defined($oldcontrib{$oldidx.':subject'})) {              if (defined($oldcontrib{$oldidx.':subject'})) {
                 if ($oldcontrib{$oldidx.':subject'} =~ /::::\d+::::/) {                  if ($oldcontrib{$oldidx.':subject'} =~ /^<version num="0">/) {
                     $contrib{'subject'} = $oldcontrib{$oldidx.':subject'}.'::::'.$numnewver.'::::'.$contrib{'subject'};                      $contrib{'subject'} = '<version num="'.$numnewver.'">'.&HTML::Entities::encode($contrib{'subject'},'<>&"').'</version>';
                       $contrib{'subject'} = $oldcontrib{$oldidx.':subject'}.$contrib{'subject'};
                 } else {                  } else {
                      $contrib{'subject'} = '::::0::::'.$oldcontrib{$oldidx.':subject'}.'::::1::::'.$contrib{'subject'};                      $contrib{'subject'} = '<version num="0">'.&HTML::Entities::encode($oldcontrib{$oldidx.':subject'},'<>&"').'</version><version num="1">'.&HTML::Entities::encode($contrib{'subject'},'<>&"').'</version>';
                 }                  }
             }               } 
             if (defined($oldcontrib{$oldidx.':message'})) {              if (defined($oldcontrib{$oldidx.':message'})) {
                 if ($oldcontrib{$oldidx.':message'} =~ /::::\d+::::/) {                  if ($oldcontrib{$oldidx.':message'} =~ /^<version num="0">/) {
                     $contrib{'message'} = $oldcontrib{$oldidx.':message'}.'::::'.$numnewver.'::::'.$contrib{'message'};                      $contrib{'message'} = '<version num="'.$numnewver.'">'.&HTML::Entities::encode($contrib{'message'},'<>&"').'</version>';
                       $contrib{'message'} = $oldcontrib{$oldidx.':message'}.$contrib{'message'};
                 } else {                  } else {
                     $contrib{'message'} = '::::0::::'.$oldcontrib{$oldidx.':message'}.'::::1::::'.$contrib{'message'};                      $contrib{'message'} = '<version num="0">'.&HTML::Entities::encode($oldcontrib{$oldidx.':message'},'<>&"').'</version><version num="1">'.&HTML::Entities::encode($contrib{'message'},'<>&"').'</version>';
                 }                  }
             }              }
             $contrib{'history'} .= $oldcontrib{$oldidx.':timestamp'};              $contrib{'history'} .= $oldcontrib{$oldidx.':timestamp'};
Line 2087  sub generate_preview_button { Line 2426  sub generate_preview_button {
 <input type="hidden" name="subject">  <input type="hidden" name="subject">
 <input type="hidden" name="comment" />  <input type="hidden" name="comment" />
 <input type="button" value="$pre"  <input type="button" value="$pre"
 onClick="if (typeof(document.mailform.onsubmit)!='undefined') {document.mailform.onsubmit();};this.form.comment.value=document.mailform.comment.value;this.form.subject.value=document.mailform.subject.value;this.form.submit();" />  onClick="if (typeof(document.mailform.onsubmit)=='function') {document.mailform.onsubmit();};this.form.comment.value=document.mailform.comment.value;this.form.subject.value=document.mailform.subject.value;this.form.submit();" />
 </form>  </form>
 ENDPREVIEW  ENDPREVIEW
 }  }
Line 2102  sub modify_attachments { Line 2441  sub modify_attachments {
     my $bodytag=&Apache::loncommon::bodytag('Discussion Post Attachments',      my $bodytag=&Apache::loncommon::bodytag('Discussion Post Attachments',
                                           '','');                                            '','');
     my $msg = '';      my $msg = '';
     my @attachments = ();      my %attachments = ();
     my %currattach = ();      my %currattach = ();
     if ($idx) {      if ($idx) {
         &extract_attachments($attachmenturls,$idx,$numoldver,\$msg,\@attachments,\%currattach,$currdelold);          &extract_attachments($attachmenturls,$idx,$numoldver,\$msg,\%attachments,\%currattach,$currdelold);
     }      }
     $r->print(<<END);      $r->print(<<END);
 <html>  <html>
Line 2130  END Line 2469  END
             my @currold = keys %currattach;              my @currold = keys %currattach;
             if (@currold > 0) {              if (@currold > 0) {
                 $r->print("The following attachments were part of the most recent saved version of this posting.<br />Check the checkboxes for any you wish to remove<br />\n");                    $r->print("The following attachments were part of the most recent saved version of this posting.<br />Check the checkboxes for any you wish to remove<br />\n");  
                 foreach (@currold) {                  foreach my $id (@currold) {
                     my $id = $_;                      my $attachurl = &HTML::Entities::decode($attachments{$id}{'filename'}); 
                     $attachments[$id] =~ m#/([^/]+)$#;                      $attachurl =~ m#/([^/]+)$#;
                     $r->print('<input type="checkbox" name="deloldattach" value="'.$_.'" />&nbsp;'.$1.'<br />'."\n");                      $r->print('<input type="checkbox" name="deloldattach" value="'.$id.'" />&nbsp;'.$1.'<br />'."\n");
                 }                  }
                 $r->print("<br />");                  $r->print("<br />");
             }              }
Line 2230  sub generate_attachments_button { Line 2569  sub generate_attachments_button {
     my $response = (<<END);      my $response = (<<END);
 <form name="attachment" action="/adm/feedback?attach=$ressymb" method="post">  <form name="attachment" action="/adm/feedback?attach=$ressymb" method="post">
 Click to add/remove attachments:&nbsp;<input type="button" value="$att"  Click to add/remove attachments:&nbsp;<input type="button" value="$att"
 onClick="this.form.subject.value=document.mailform.subject.value;this.form.comment.value=document.mailform.comment.value;  onClick="if (typeof(document.mailform.onsubmit)=='function') {document.mailform.onsubmit();};this.form.comment.value=document.mailform.comment.value;this.form.subject.value=document.mailform.subject.value;
 END  END
     unless ($mode eq 'board') {      unless ($mode eq 'board') {
         $response .= 'javascript:anonchk();';          $response .= 'javascript:anonchk();';
Line 2266  ENDATTACH Line 2605  ENDATTACH
   
 sub extract_attachments {  sub extract_attachments {
     my ($attachmenturls,$idx,$numoldver,$message,$attachments,$currattach,$currdelold) = @_;      my ($attachmenturls,$idx,$numoldver,$message,$attachments,$currattach,$currdelold) = @_;
     if ($attachmenturls =~ m/::::\d+:[\.yn\d]+::::/) {      %{$attachments}=();
         @{$attachments} = split/::::\d+:[\.yn\d]+::::/,$attachmenturls;      &get_post_attachments($attachments,$attachmenturls);
         shift @{$attachments};      foreach my $id (sort keys %{$attachments}) {
         my $searchstr = '::::';          if (exists($$attachments{$id}{$numoldver})) {
         for (my $i=0; $i<@{$attachments}; $i++) {              if (defined($currdelold)) {
             if ($attachmenturls =~ m#^\Q$searchstr\E(\d+)(:[\.yn\d]+)::::#) {                  if (@{$currdelold} > 0) {
                 my $info = $1.$2;                      unless (grep/^$id$/,@{$currdelold}) {
                 my $attachid = $1-1;                          $$currattach{$id} = $$attachments{$id}{$numoldver}; 
                 $searchstr .= $info.'::::'.$$attachments[$i].'::::';  
                 if ($info =~ /\.$numoldver([yn])\./) {  
                     if (defined($currdelold)) {  
                         if (@{$currdelold} > 0) {  
                             unless (grep/^$attachid$/,@{$currdelold}) {  
                                 my $id = $i;  
                                 $$currattach{$id} = $1;  
                             }  
                         } else {  
                             my $id = $i;  
                             $$currattach{$id} = $1;  
                         }  
                     } else {  
                         my $id = $i;  
                         $$currattach{$id} = $1;  
                     }                      }
                   } else {
                       $$currattach{$id} = $$attachments{$id}{$numoldver};
                 }                  }
               } else {
                   $$currattach{$id} = $$attachments{$id}{$numoldver};
             }              }
         }          }
         my @attached = (sort { $a <=> $b } keys %{$currattach});      }
         if (@attached == 1) {      my @attached = (sort { $a <=> $b } keys %{$currattach});
             my $id = $attached[0];      if (@attached == 1) {
             $$attachments[$attached[0]]=~m|/([^/]+)$|;          my $id = $attached[0];
             $$message.='<br /><a href="'.$$attachments[$id].'"><tt>'.          my $attachurl;
             $1.'</tt></a><br />';          if ($attachmenturls =~ m/^<attachment id="0">/) {
               $attachurl = &HTML::Entities::decode($$attachments{$id}{'filename'});
           } else {
               $attachurl = $$attachments{$id}{'filename'};
           }
           $attachurl=~m|/([^/]+)$|;
           $$message.='<br /><a href="'.$attachurl.'"><tt>'.
           $1.'</tt></a><br />';
           &Apache::lonnet::allowuploaded('/adm/feedback',
                                  $attachurl);
       } elsif (@attached > 1) {
           $$message.='<ol>';
           foreach (@attached) {
               my $id = $_;
               my $attachurl = &HTML::Entities::decode($$attachments{$id}{'filename'});
               my ($fname)
                 =($attachurl=~m|/([^/]+)$|);
               $$message .= '<li><a href="'.$attachurl.
                 '"><tt>'.
                 $fname.'</tt></a></li>';
             &Apache::lonnet::allowuploaded('/adm/feedback',              &Apache::lonnet::allowuploaded('/adm/feedback',
                                    $$attachments[$id]);                               $attachurl);
         } elsif (@attached > 1) {  
             $$message.='<ol>';  
             foreach (@attached) {  
                 my $id = $_;  
                 my ($fname)  
                   =($$attachments[$id]=~m|/([^/]+)$|);  
                 $$message .= '<li><a href="'.$$attachments[$id].  
                   '"><tt>'.  
                   $fname.'</tt></a></li>';  
                 &Apache::lonnet::allowuploaded('/adm/feedback',  
                                  $$attachments[$id]);  
             }  
             $$message .= '</ol><br />';  
         }          }
     } else {          $$message .= '</ol>';
         my ($fname)  
            =($attachmenturls=~m|/([^/]+)$|);  
         $$message .='<p>'.&mt('Attachment').  
            ': <a href="'.$attachmenturls.  
            '"><tt>'.  
            $fname.'</tt></a></p>';  
            $$attachments[0] = $attachmenturls;  
            $$currattach{'0'} = 'n';  
            &Apache::lonnet::allowuploaded('/adm/feedback',  
                              $attachmenturls);  
     }      }
 }  }
   
Line 2333  sub construct_attachmenturl { Line 2657  sub construct_attachmenturl {
     my ($currnewattach,$keepold,$symb,$idx)=@_;      my ($currnewattach,$keepold,$symb,$idx)=@_;
     my $oldattachmenturl;      my $oldattachmenturl;
     my $newattachmenturl;      my $newattachmenturl;
     my $startnum = 1;      my $startnum = 0;
     my $currver = 0;      my $currver = 0;
     if (($ENV{'form.editdisc'}) && ($idx)) {      if (($ENV{'form.editdisc'}) && ($idx)) {
         my %contrib=&Apache::lonnet::restore($symb,$ENV{'request.course.id'},          my %contrib=&Apache::lonnet::restore($symb,$ENV{'request.course.id'},
Line 2351  sub construct_attachmenturl { Line 2675  sub construct_attachmenturl {
             $currver = 1;              $currver = 1;
         }          }
         if ($oldattachmenturl) {          if ($oldattachmenturl) {
             if ($oldattachmenturl =~ m/::::\d+:[\.yn\d]+::::/) {              if ($oldattachmenturl =~ m/^<attachment id="0">/) {
                 my @attachments = split/::::\d+:[\.yn\d]+::::/,$oldattachmenturl;                  my %attachments = ();
                 shift @attachments;                  my $prevver = $currver-1;
                 $startnum += @attachments;                  &get_post_attachments(\%attachments,$oldattachmenturl);
                 my $searchstr = '::::';                  my $numattach = scalar(keys %attachments);
                 $newattachmenturl = '::::';                  $startnum += $numattach;
                 for (my $i=0; $i<@attachments; $i++) {                  foreach my $num (sort {$a <=> $b} keys %attachments) {
                     if ($oldattachmenturl =~ m#^\Q$searchstr\E(\d+)(:[\.yn\d]+)::::#) {                      $newattachmenturl .= '<attachment id="'.$num.'"><filename>'.$attachments{$num}{'filename'}.'</filename>';
                         my $attachid = $1 - 1;                      foreach $_ (sort {$a <=> $b} keys %{$attachments{$num}}) {
                         $searchstr .= $1.$2.'::::'.$attachments[$i].'::::';                          unless ($_ eq 'filename') {
                         $newattachmenturl .= $1.$2;                              $newattachmenturl .= '<post id="'.$_.'">'.$attachments{$num}{$_}.'</post>';
                         if (grep/^$attachid$/,@{$keepold}) {  
                             $newattachmenturl .= '.'.$currver.'n.';  
                         }                          }
                         $newattachmenturl .= '::::'.$attachments[$i].'::::';  
                     }                      }
                       if (grep/^$num$/,@{$keepold}) {
                           $newattachmenturl .= '<post id="'.$currver.'">'.$attachments{$num}{$prevver}.'</post>';
                       }
                       $newattachmenturl .= '</attachment>';
                 }                  }
                 $newattachmenturl =~ s/::::$//;  
             } else {              } else {
                 $newattachmenturl = '::::1:.0n.';                  $newattachmenturl = '<attachment id="0"><filename>'.&HTML::Entities::encode($oldattachmenturl).'</filename><post id="0">n</post>';
                 unless (grep/^0$/,@{$keepold}) {                  unless (grep/^0$/,@{$keepold}) {
                     $newattachmenturl .= '.1n.';                      $newattachmenturl .= '<post id="1">n</post>';
                 }                  }
                 $newattachmenturl .= '::::'.$oldattachmenturl;                  $newattachmenturl .= '</attachment>';
                 $startnum ++;                  $startnum ++;
             }              }
         }          }
     }      }
     for (my $i=0; $i<@{$currnewattach}; $i++) {      for (my $i=0; $i<@{$currnewattach}; $i++) {
         my $attachnum = $startnum + $i;          my $attachnum = $startnum + $i;
         $newattachmenturl .= '::::'.$attachnum.':.'.$currver.'n.::::'.$$currnewattach[$i];          $newattachmenturl .= '<attachment id="'.$attachnum.'"><filename>'.&HTML::Entities::encode($$currnewattach[$i]).'</filename><post id="'.$currver.'">n</post></attachment>';
     }      }
     return $newattachmenturl;       return $newattachmenturl; 
 }  }
Line 2397  sub handler { Line 2721  sub handler {
 # --------------------------- Get query string for limited number of parameters  # --------------------------- Get query string for limited number of parameters
   
   &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},    &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},
          ['hide','unhide','deldisc','postdata','preview','replydisc','editdisc','threadedon','threadedoff','onlyunread','allposts','onlyunmark','previous','markread','markonread','markondisp','toggoff','toggon','modifydisp','changes','navmaps','navurl','sortfilter','sortposts','applysort','rolefilter','statusfilter','sectionpick','posterlist','userpick','attach','origpage','currnewattach','deloldattach','keepold','allversions']);           ['hide','unhide','deldisc','postdata','preview','replydisc','editdisc','threadedon','threadedoff','onlyunread','allposts','onlyunmark','previous','markread','markonread','markondisp','toggoff','toggon','modifydisp','changes','navmaps','navurl','sortfilter','sortposts','applysort','rolefilter','statusfilter','sectionpick','posterlist','userpick','attach','origpage','currnewattach','deloldattach','keepold','allversions','export']);
   if ($ENV{'form.discsymb'}) {    if ($ENV{'form.discsymb'}) {
       my $symb = $ENV{'form.discsymb'};        my $symb = $ENV{'form.discsymb'};
       my $readkey = $symb.'_read';        my $readkey = $symb.'_read';
Line 2453  END Line 2777  END
           $ressymb=~s|(bulletin___\d+___)|$1adm/wrapper|;            $ressymb=~s|(bulletin___\d+___)|$1adm/wrapper|;
       }        }
       if ($idx > 0) {        if ($idx > 0) {
             my %messages = ();
             my %subjects = ();
             my %attachmsgs = ();
             my %allattachments = ();
             my %imsfiles = ();
             my ($screenname,$plainname);
           my %contrib=&Apache::lonnet::restore($ressymb,$ENV{'request.course.id'},            my %contrib=&Apache::lonnet::restore($ressymb,$ENV{'request.course.id'},
                $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},                 $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
                $ENV{'course.'.$ENV{'request.course.id'}.'.num'});                 $ENV{'course.'.$ENV{'request.course.id'}.'.num'});
           if ($contrib{$idx.':history'}) {            my $discussion = &get_post_contents(\%contrib,$idx,$seeid,'allversions',\%messages,\%subjects,\%allattachments,\%attachmsgs,\%imsfiles,\$screenname,\$plainname);
               my $attachmenturls = $contrib{$idx.':attachmenturl'};            $r->print($discussion);
               my @postversions = ();  
               if ($contrib{$idx.':history'} =~ m/:/) {  
                   @postversions = split/:/,$contrib{$idx.':history'};  
               } else {  
                   @postversions = ("$contrib{$idx.':history'}");  
               }  
               if (@postversions > 0) {  
                   push @postversions,$contrib{$idx.':timestamp'};  
                   my $screenname=&Apache::loncommon::screenname(  
                                             $contrib{$idx.':sendername'},  
                                             $contrib{$idx.':senderdomain'});  
                   my $plainname=&Apache::loncommon::nickname(  
                                             $contrib{$idx.':sendername'},  
                                             $contrib{$idx.':senderdomain'});  
                   my $sender=&Apache::loncommon::aboutmewrapper(  
                                      $plainname,  
                                      $contrib{$idx.':sendername'},  
                                      $contrib{$idx.':senderdomain'}).' ('.  
                                      $contrib{$idx.':sendername'}.' at '.  
                                      $contrib{$idx.':senderdomain'}.')';  
                   if ($contrib{$idx.':anonymous'}) {  
                       $sender.=' ['.&mt('anonymous').'] '.$screenname;  
                   }  
                   $r->print('<b>'.$sender.'</b><br /><ul>');  
                   for (my $i=0; $i<@postversions; $i++) {  
                       my ($timesent,$message,$subject,$attachmsg);  
                       $timesent = &Apache::lonlocal::locallocaltime($postversions[$i]);  
                       if ($i == @postversions-1) {  
                           ($message)=($contrib{$idx.':message'} =~ /.*::::\Q$i\E::::(.+?)$/si);  
                           ($subject)=($contrib{$idx.':subject'} =~ /.*::::\Q$i\E::::(.+?)$/si);  
                       } else {   
                           ($message)=($contrib{$idx.':message'} =~ /::::\Q$i\E::::(.+?)::::/si);  
                           ($subject)=($contrib{$idx.':subject'} =~ /::::\Q$i\E::::(.+?)::::/si);  
                       }  
                       $message=~s/\n/\<br \/\>/g;  
                       $message=&Apache::lontexconvert::msgtexconverted($message);  
                       $subject=~s/\n/\<br \/\>/g;  
                       $subject=&Apache::lontexconvert::msgtexconverted($subject);  
                       if ($attachmenturls) {  
                           my @attachments = ();  
                           my %currattach = ();  
                           &extract_attachments($attachmenturls,$idx,$i,\$attachmsg,\@attachments,\%currattach);  
                       }  
                       if ($attachmsg) {  
                           $attachmsg = '<br />Attachments:'.$attachmsg.'<br />';  
                       } else {  
                           $attachmsg = '<br />';  
                       }  
                       $r->print (<<END);  
 <li><b>$subject</b>, $timesent<br />  
 $message<br />  
 $attachmsg</li>  
 END  
                   }  
                   $r->print('</ul></body></html>');  
               }  
           }  
       }        }
       return OK;        return OK;
   }    }
Line 2638  END Line 2911  END
 <b>$lt{'mnpa'} $numitems $lt{'robb'}</b>  <b>$lt{'mnpa'} $numitems $lt{'robb'}</b>
 <form name="reldt" action="$feedurl" target="loncapaclient">  <form name="reldt" action="$feedurl" target="loncapaclient">
 </form>  </form>
   <br /><a href="$feedurl">Continue</a>
 </body>  </body>
 </html>  </html>
 ENDREDIR  ENDREDIR
Line 2870  ENDREDIR Line 3144  ENDREDIR
       my ($map,$ind,$url)=&Apache::lonnet::decode_symb($ENV{'form.chgreads'});        my ($map,$ind,$url)=&Apache::lonnet::decode_symb($ENV{'form.chgreads'});
       &redirect_back($r,&Apache::lonnet::clutter($url),        &redirect_back($r,&Apache::lonnet::clutter($url),
        &mt('Changed read status').'<br />','0','0');         &mt('Changed read status').'<br />','0','0');
     } elsif ($ENV{'form.export'}) {
         &Apache::loncommon::content_type($r,'text/html');
         $r->send_http_header;
         my $symb=$ENV{'form.export'};
         my $mode;
         my $status='OPEN';
         my ($map,$ind,$url)=&Apache::lonnet::decode_symb($symb);
         my $previous=$ENV{'form.previous'};
         my $feedurl = &Apache::lonnet::clutter($url);
    # backward compatibility (bulletin boards used to be 'wrapped')
         if ($feedurl=~m|^/adm/wrapper/adm/.*/bulletinboard$|) {
             $mode = 'board';
             $feedurl=~s|^/adm/wrapper||;
         }
         if ($feedurl =~ /\.(problem|exam|quiz|assess|survey|form|library)$/) {
             $mode='problem';
             $status=$Apache::inputtags::status[-1];
         }
         my $discussion = &list_discussion($mode,$status,$symb); 
         my $bodytag=&Apache::loncommon::bodytag('Resource Feedback and Discussion');
         $r->print($bodytag.$discussion);                                     
         return OK;
   } else {    } else {
 # ------------------------------------------------------------- Normal feedback  # ------------------------------------------------------------- Normal feedback
   my $feedurl=$ENV{'form.postdata'};    my $feedurl=$ENV{'form.postdata'};
Removed from v.1.111  
changed lines
  Added in v.1.122

FreeBSD-CVSweb <freebsd-cvsweb@FreeBSD.org>