Diff for /loncom/interface/lonfeedback.pm between versions 1.110 and 1.130

version 1.110, 2004/07/29 22:42:25 version 1.130, 2004/11/11 22:47:55
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 Apache::lonnavmaps;
   use Apache::lonenc();
   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 61  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 65  sub discussion_visible { Line 69  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 92  sub list_discussion { Line 100  sub list_discussion {
             $ressymb=~s|(bulletin___\d+___)|$1adm/wrapper|;              $ressymb=~s|(bulletin___\d+___)|$1adm/wrapper|;
         }          }
     }      }
       $ressymb=&Apache::lonenc::check_encrypt($ressymb);
 # Get discussion display settings for this discussion  # Get discussion display settings for this discussion
     my $lastkey = $ressymb.'_lastread';      my $lastkey = $ressymb.'_lastread';
     my $showkey = $ressymb.'_showonlyunread';      my $showkey = $ressymb.'_showonlyunread';
       my $markkey = $ressymb.'_showonlyunmark',
     my $visitkey = $ressymb.'_visit';      my $visitkey = $ressymb.'_visit';
     my $ondispkey = $ressymb.'_markondisp';      my $ondispkey = $ressymb.'_markondisp';
     my $userpickkey = $ressymb.'_userpick';      my $userpickkey = $ressymb.'_userpick';
     my %dischash = &Apache::lonnet::get('nohist_'.$ENV{'request.course.id'}.'_discuss',[$lastkey,$showkey,$visitkey,$ondispkey,$userpickkey],$ENV{'user.domain'},$ENV{'user.name'});      my $toggkey = $ressymb.'_readtoggle';
       my $readkey = $ressymb.'_read';
   
       my %dischash = &Apache::lonnet::get('nohist_'.$ENV{'request.course.id'}.'_discuss',[$lastkey,$showkey,$markkey,$visitkey,$ondispkey,$userpickkey,$toggkey,$readkey],$ENV{'user.domain'},$ENV{'user.name'});
     my %discinfo = ();      my %discinfo = ();
     my $showonlyunread = 0;      my $showonlyunread = 0;
       my $showunmark = 0; 
     my $markondisp = 0;      my $markondisp = 0;
     my $prevread = 0;      my $prevread = 0;
     my $previous = 0;      my $previous = 0;
Line 154  sub list_discussion { Line 167  sub list_discussion {
     }      }
   
 # Get discussion display default settings for user  # Get discussion display default settings for user
     my %userenv = &Apache::lonnet::get('environment',['discdisplay','discmarkread'],$ENV{'user.domain'},$ENV{'user.name'});      if ($ENV{'environment.discdisplay'} eq 'unread') {
     my $discdisplay=$userenv{'discdisplay'};  
     if ($discdisplay eq 'unread') {  
         $showonlyunread = 1;          $showonlyunread = 1;
     }      }
     my $discmarkread=$userenv{'discmarkread'};      if ($ENV{'environment.discmarkread'} eq 'ondisp') {
     if ($discmarkread eq 'ondisp') {  
         $markondisp = 1;          $markondisp = 1;
     }      }
   
 # Override user's default if user specified display setting for this discussion  # Override user's default if user specified display setting for this discussion
     if (defined($dischash{$ondispkey})) {      if (defined($dischash{$ondispkey})) {
         $markondisp = $dischash{$ondispkey};          unless ($dischash{$ondispkey} eq '') {
               $markondisp = $dischash{$ondispkey};
           }
     }      }
     if ($markondisp) {      if ($markondisp) {
         $discinfo{$lastkey} = time;          $discinfo{$lastkey} = time;
     }      }
   
     if (defined($dischash{$showkey})) {      if (defined($dischash{$showkey})) {
         $showonlyunread = $dischash{$showkey};          unless ($dischash{$showkey} eq '') {
               $showonlyunread = $dischash{$showkey};
           }
       }
   
       if (defined($dischash{$markkey})) {
           unless ($dischash{$markkey} eq '') {
               $showunmark = $dischash{$markkey};
           }
     }      }
   
     if (defined($dischash{$visitkey})) {      if (defined($dischash{$visitkey})) {
         $visit = $dischash{$visitkey};          unless ($dischash{$visitkey} eq '') {
               $visit = $dischash{$visitkey};
           }
     }      }
     $visit ++;      $visit ++;
   
Line 187  sub list_discussion { Line 209  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 205  sub list_discussion { Line 226  sub list_discussion {
     $ENV{'environment.remote'} eq 'off' ) {      $ENV{'environment.remote'} eq 'off' ) {
  $target='target="LONcom"';   $target='target="LONcom"';
     }      }
       
     my $now = time;      my $now = time;
     $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);  
     }  
 #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;  
                     } 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></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 542  sub list_discussion { Line 247  sub list_discussion {
         'cuse' => 'Current discussion settings',          'cuse' => 'Current discussion settings',
         'allposts' => 'All posts',          'allposts' => 'All posts',
         'unread' => 'New posts only',          'unread' => 'New posts only',
           'unmark' => 'Unread only',
         'ondisp' => 'Once displayed',          'ondisp' => 'Once displayed',
         'onmark' => 'Once marked read',          'onmark' => 'Once marked not NEW',
           'toggoff' => 'Off',
           'toggon' => 'On',
         'disa' => 'Posts to be displayed',          'disa' => 'Posts to be displayed',
         'npce' => 'Posts cease to be marked "NEW"',          'npce' => 'Posts cease to be marked "NEW"',
           'epcb' => 'Each post can be toggled read/unread', 
         'chgt' => 'Change',          'chgt' => 'Change',
         'disp' => 'Display',          'disp' => 'Display',
         'nolo' => 'Not new',          'nolo' => 'Not new',
           'togg' => 'Toggle read/unread',
     );      );
   
     my $currdisp = $lt{'allposts'};      my $currdisp = $lt{'allposts'};
     my $currmark = $lt{'onmark'};      my $currmark = $lt{'onmark'};
       my $currtogg = $lt{'toggoff'};
     my $dispchange = $lt{'unread'};      my $dispchange = $lt{'unread'};
     my $markchange = $lt{'ondisp'};      my $markchange = $lt{'ondisp'};
       my $toggchange = $lt{'toggon'};
     my $chglink = '/adm/feedback?modifydisp='.$ressymb;      my $chglink = '/adm/feedback?modifydisp='.$ressymb;
     my $displink = 'onlyunread';      my $displinkA = 'onlyunread';
       my $displinkB = 'onlyunmark';
     my $marklink = 'markondisp';      my $marklink = 'markondisp';
       my $togglink = 'toggon';
   
     if ($markondisp) {      if ($markondisp) {
         $currmark = $lt{'ondisp'};          $currmark = $lt{'ondisp'};
Line 568  sub list_discussion { Line 282  sub list_discussion {
     if ($showonlyunread) {      if ($showonlyunread) {
         $currdisp = $lt{'unread'};          $currdisp = $lt{'unread'};
         $dispchange = $lt{'allposts'};          $dispchange = $lt{'allposts'};
         $displink = 'allposts';          $displinkA = 'allposts';
       }
   
       if ($showunmark) {
           $currdisp = $lt{'unmark'};
           $dispchange = $lt{'unmark'};
           $displinkA='allposts';
           $displinkB='onlyunread';
           $showonlyunread = 0;
     }      }
   
       if ($dischash{$toggkey}) {
           $currtogg = $lt{'toggon'};
           $toggchange = $lt{'toggoff'};
           $togglink = 'toggoff';
       } 
         
     $chglink .= '&changes='.$displink.'_'.$marklink;      $chglink .= '&changes='.$displinkA.'_'.$displinkB.'_'.$marklink.'_'.$togglink;
   
     if ($newpostsflag) {      if ($newpostsflag) {
         $chglink .= '&previous='.$prevread;          $chglink .= '&previous='.$prevread;
Line 579  sub list_discussion { Line 307  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;
               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 595  sub list_discussion { Line 361  sub list_discussion {
    }     }
 </script>  </script>
             |;              |;
     $discussion.='<table bgcolor="#AAAAAA" cellpadding="2" cellspacing="2" border="0">';      $discussion.='<form name="readchoices" method="post" action="/adm/feedback?chgreads='.$ressymb.'"><table bgcolor="#AAAAAA" cellpadding="2" cellspacing="2" border="0">';
     $discussion .='<tr><td bgcolor="#DDDDBB" colspan="'.$colspan.'">'.      $discussion .='<tr><td bgcolor="#DDDDBB" colspan="'.$colspan.'">'.
  '<table border="0" width="100%" bgcolor="#DDDDBB"><tr>';   '<table border="0" width="100%" bgcolor="#DDDDBB"><tr>';
     if ($visible>2) {      if ($visible>2) {
Line 625  sub list_discussion { Line 391  sub list_discussion {
             $discussion .= '">'.&mt('Export').'?</a>&nbsp;&nbsp;</td>';              $discussion .= '">'.&mt('Export').'?</a>&nbsp;&nbsp;</td>';
     if ($newpostsflag) {      if ($newpostsflag) {
  if (!$markondisp) {   if (!$markondisp) {
     $discussion .='<td align="right"><a href="/adm/feedback?markread='.$ressymb.'">'.&mt('Mark new posts as read').'</a>&nbsp;&nbsp;';      $discussion .='<td align="right"><a href="/adm/feedback?markread='.$ressymb.'">'.&mt('Mark NEW posts no longer new').'</a>&nbsp;&nbsp;';
  } else {   } else {
     $discussion .= '<td>&nbsp;</td>';      $discussion .= '<td>&nbsp;</td>';
  }   }
Line 633  sub list_discussion { Line 399  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').' '.
                            $numhidden.' ';
                   if ($showunmark) {
                       $discussion .= &mt('posts previously marked read');
                   } else {
                       $discussion .= &mt('previously viewed posts');
                   }
                   $discussion .= '<br/></td></tr>';
             }              }
             $discussion .= '">'.&mt('Show all posts').'</a> '.&mt('to display').' '.  
                          $numhidden.' '.&mt('previously viewed posts').  
                          '<br/></td></tr>';  
         }          }
   
 # Choose sort mechanism  # Choose sort mechanism
Line 679  sub list_discussion { Line 445  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 711  sub list_discussion { Line 473  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 $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">
              <td colspan="$colspan" valign="top">               <td colspan="$colspan" valign="top">
               <table border="0" bgcolor="#FFFFFF" width="100%" cellspacing="2" cellpadding="2">                <table border="0" bgcolor="#FFFFFF" width="100%" cellspacing="2" cellpadding="2">
Line 734  sub list_discussion { Line 527  sub list_discussion {
                    <td>                     <td>
                     <font size="-1"><b>$lt{'cuse'}</b>:</td>                      <font size="-1"><b>$lt{'cuse'}</b>:</td>
                    <td>&nbsp;</td>                     <td>&nbsp;</td>
                      <td><font size="-1">
 END  END
             if ($newpostsflag) {              if ($newpostsflag) {
                 $discussion .=                   $discussion .= 
                    '<td><font size="-1">1.&nbsp;'.$lt{'disp'}.'&nbsp;-&nbsp;<i>'.$currdisp.'</i>&nbsp;&nbsp;2.&nbsp;'.$lt{'nolo'}.'&nbsp;-&nbsp;<i>'.$currmark.'</i></font></td>';                     '1.&nbsp;'.$lt{'disp'}.'&nbsp;-&nbsp;<i>'.$currdisp.'</i>&nbsp;&nbsp;2.&nbsp;'.$lt{'nolo'}.'&nbsp;-&nbsp;<i>'.$currmark.'</i>';
                   if ($dischash{$toggkey}) {
                      $discussion .= '&nbsp;&nbsp;3.&nbsp;'.$lt{'togg'}.'&nbsp;-&nbsp;<i>'.$currtogg.'</i>';
                   }
             } else {              } else {
                 $discussion .=                  if ($dischash{$toggkey}) {
                    '<td><font size="-1">'.$lt{'disp'}.'&nbsp;-&nbsp;<i>'.$currdisp.'</i></font></td>';                     $discussion .= '1.&nbsp;'.$lt{'disp'}.'&nbsp;-&nbsp;<i>'.$currdisp.'</i>&nbsp;2.&nbsp;'.$lt{'togg'}.'&nbsp;-&nbsp;<i>'.$currtogg.'</i>';
                   } else {
                       $discussion .=
                            $lt{'disp'}.'&nbsp;-&nbsp;<i>'.$currdisp.'</i>';
                   }
             }              }
             $discussion .= <<END;              $discussion .= <<END;
                      </font></td>
                    <td>&nbsp;</td>                     <td>&nbsp;</td>
                    <td>                     <td>
                     <font size="-1"><b><a href="$chglink">$lt{'chgt'}</a>?</font></b></td>                      <font size="-1"><b><a href="$chglink">$lt{'chgt'}</a>?</font></b>
                      </td>
                   </tr>                    </tr>
                  </table>                   </table>
                 </td>                  </td>
   END
               if ($dischash{$toggkey}) {
                   my $storebutton = &mt('Store read/unread changes');
                   $discussion.='<td align="right">'.
                 '<input type="hidden" name="discsymb" value="'.$ressymb.'">'."\n".
                 '<input type="button" name="readoptions" value="'.$storebutton.'"'.
                 ' onClick="this.form.submit();">'."\n".
                 '</td>';
               }
               $discussion .= (<<END);
                </tr>                 </tr>
               </table>                </table>
              </td>               </td>
             </tr>              </tr>
            </table>             </table>
            <br /><br />             <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 - <br />'.$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 768  END Line 623  END
         my $subject = '';          my $subject = '';
         if ($ENV{'form.origpage'}) {          if ($ENV{'form.origpage'}) {
             &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['addnewattach','deloldattach','delnewattach','timestamp','idx','subject','comment']);              &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['addnewattach','deloldattach','delnewattach','timestamp','idx','subject','comment']);
             $subject = &HTML::Entities::encode($ENV{'form.subject'},'<>&"');              $subject = &Apache::lonnet::unescape($ENV{'form.subject'});
             $comment = &HTML::Entities::encode($ENV{'form.comment'},'<>&"');              $comment = &Apache::lonnet::unescape($ENV{'form.comment'});
             my @keepold = ();              my @keepold = ();
             &process_attachments(\@currnewattach,\@currdelold,\@keepold);              &process_attachments(\@currnewattach,\@currdelold,\@keepold);
             if (@currnewattach > 0) {              if (@currnewattach > 0) {
                 $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 815  ENDDISCUSS Line 671  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'}.
  ($ENV{'request.course.sec'}?'/'.$ENV{'request.course.sec'}:''))) {   ($ENV{'request.course.sec'}?'/'.$ENV{'request.course.sec'}:''))) {
     if ($outputtarget ne 'tex') {      if ($outputtarget ne 'tex') {
  $discussion.='<table bgcolor="#BBBBBB"><tr><td><a href="/adm/feedback?replydisc='.   $discussion.='<table bgcolor="#BBBBBB"><tr><td><a href="/adm/feedback?replydisc='.
     $symb.':::" '.$target.'>'.      $ressymb.':::" '.$target.'>'.
     '<img src="/adm/lonMisc/chat.gif" border="0" />'.      '<img src="/adm/lonMisc/chat.gif" border="0" />'.
     &mt('Post Discussion').'</a></td></tr></table>';      &mt('Post Discussion').'</a></td></tr></table>';
     }      }
  }   }
     }      }
    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'});
       ($$screenname,$$plainname)=($$contrib{$idx.':screenname'},
    $$contrib{$idx.':plainname'});
       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);
                   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 file attachment - '.$5.' to IMS package: '.$!.'<br />'."\n";
                       }
                   } else {
                       &Apache::lonnet::logthis("Replication of attachment failed when building IMS export of discussion posts - domain: $1, course: $2, file: $$attachrefs{$id}{'filename'} -error: $rtncode");
                       $response .= 'Error copying file attachment - '.$5.' to IMS package: '.$rtncode.'<br />'."\n";
                   }
               }
           }
       }
       return $response;
 }  }
   
 sub mail_screen {  sub mail_screen {
Line 852  sub mail_screen { Line 1198  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 898  END Line 1244  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 909  END Line 1263  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 974  END Line 1316  END
   }    }
   
   if ($ENV{'form.origpage'}) {    if ($ENV{'form.origpage'}) {
       $subject = $ENV{'form.subject'};        $subject = &Apache::lonnet::unescape($ENV{'form.subject'});
       $comment = $ENV{'form.comment'};        $comment = &Apache::lonnet::unescape($ENV{'form.comment'});
       &process_attachments(\@currnewattach,\@currdelold,\@keepold);        &process_attachments(\@currnewattach,\@currdelold,\@keepold);
   }    }
   my $latexHelp=&Apache::loncommon::helpLatexCheatsheet();    my $latexHelp=&Apache::loncommon::helpLatexCheatsheet();
Line 1023  $htmlheader Line 1365  $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 1077  END Line 1419  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 1137  END Line 1479  END
 }  }
   
 sub print_display_options {  sub print_display_options {
     my ($r,$symb,$previous,$dispchg,$markchg,$feedurl) = @_;      my ($r,$symb,$previous,$dispchgA,$dispchgB,$markchg,$toggchg,$feedurl) = @_;
  # backward compatibility (bulletin boards used to be 'wrapped')   # backward compatibility (bulletin boards used to be 'wrapped')
     if ($feedurl=~m|^/adm/wrapper/adm/.*/bulletinboard$|) {      if ($feedurl=~m|^/adm/wrapper/adm/.*/bulletinboard$|) {
         $feedurl=~s|^/adm/wrapper||;          $feedurl=~s|^/adm/wrapper||;
Line 1157  sub print_display_options { Line 1499  sub print_display_options {
         'deff' => 'Default for all discussions',          'deff' => 'Default for all discussions',
         'prca' => 'Preferences can be set for this discussion that determine ....',          'prca' => 'Preferences can be set for this discussion that determine ....',
         'whpo' => 'Which posts are displayed when you display this bulletin board or resource, and',          'whpo' => 'Which posts are displayed when you display this bulletin board or resource, and',
         'unwh' => 'Under what circumstances posts are identfied as "New."',          'unwh' => 'Under what circumstances posts are identified as "NEW", and',
           'wipa' => 'Whether individual posts can be marked as read/unread',
         'allposts' => 'All posts',          'allposts' => 'All posts',
         'unread' => 'New posts only',          'unread' => 'New posts only',
           'unmark' => 'Posts not marked read',
         'ondisp' => 'Once displayed',          'ondisp' => 'Once displayed',
         'onmark' => 'Once marked as read',          'onmark' => 'Once marked not NEW ',
           'toggon' => 'Shown',
           'toggoff' => 'Not shown',
         'disa' => 'Posts displayed?',          'disa' => 'Posts displayed?',
         'npmr' => 'New posts cease to be identified as "New"?',          'npmr' => 'New posts cease to be identified as "NEW"?',
           'dotm' => 'Option to mark each post as read/unread?',  
         'chgt' => 'Change to ',          'chgt' => 'Change to ',
         'mkdf' => 'Set to ',          'mkdf' => 'Set to ',
         'yhni' => 'You have not indicated that you wish to change either of the discussion settings',          'yhni' => 'You have not indicated that you wish to change any of the discussion settings',
         'ywbr' => 'You will be returned to the previous page if you click OK.'          'ywbr' => 'You will be returned to the previous page if you click OK.'
     );      );
   
     my $dispchange = $lt{'unread'};      my $dispchangeA = $lt{'unread'};
       my $dispchangeB = $lt{'unmark'};
     my $markchange = $lt{'ondisp'};      my $markchange = $lt{'ondisp'};
       my $toggchange = $lt{'toggon'};
     my $currdisp = $lt{'allposts'};      my $currdisp = $lt{'allposts'};
     my $currmark = $lt{'onmark'};      my $currmark = $lt{'onmark'};
     my $discdisp = 'allposts';      my $discdisp = 'allposts';
     my $discmark = 'onmark';      my $discmark = 'onmark';
       my $currtogg = $lt{'toggoff'};
       my $disctogg = 'toggoff';
                                                                                                                                                                               
     if ($dispchg eq 'allposts') {      if ($dispchgA eq 'allposts') {
         $dispchange = $lt{'allposts'};          $dispchangeA = $lt{'allposts'};
         $currdisp = $lt{'unread'};          $currdisp = $lt{'unread'};
         $discdisp = 'unread';          $discdisp = 'unread';
     }      }
                                                                                         
     if ($markchg eq 'markonread') {      if ($markchg eq 'markonread') {
         $markchange = $lt{'onmark'};          $markchange = $lt{'onmark'};
         $currmark = $lt{'ondisp'};          $currmark = $lt{'ondisp'};
         $discmark = 'ondisp';          $discmark = 'ondisp';
     }      }
   
       if ($dispchgB eq 'onlyunread') {
           $dispchangeB = $lt{'unread'};
           $currdisp = $lt{'unmark'};
           $discdisp = 'unmark';
       }
       if ($toggchg eq 'toggoff') {
           $toggchange = $lt{'toggoff'};
           $currtogg = $lt{'toggon'};
           $disctogg = 'toggon';
       }
     $r->print(<<END);      $r->print(<<END);
 <html>  <html>
 <head>  <head>
 <title>$lt{'dido'}</title>  <title>$lt{'dido'}</title>
 <meta http-equiv="pragma" content="no-cache" />  <meta http-equiv="pragma" content="no-cache" />
 <script>  <script>
   function discdispChk(caller) {
       var disctogg = '$toggchg'
       if (caller == 0) {
           if (document.modifydisp.discdisp[0].checked == true) {
               if (document.modifydisp.discdisp[1].checked == true) {
                   document.modifydisp.discdisp[1].checked = false
               }
           }
       }
       if (caller == 1) {
           if (document.modifydisp.discdisp[1].checked == true) {
               if (document.modifydisp.discdisp[0].checked == true) {
                   document.modifydisp.discdisp[0].checked = false
               }
               if (disctogg == 'toggon') {
                   document.modifydisp.disctogg.checked = true
               }
               if (disctogg == 'toggoff') {
                   document.modifydisp.disctogg.checked = false
               }
           }
       }
       if (caller == 2) {
           var dispchgB = '$dispchgB'
           if (disctogg == 'toggoff') {
               if (document.modifydisp.disctogg.checked == true) {
                   if (dispchgB == 'onlyunmark') {
                       document.modifydisp.discdisp[1].checked = false
                   }
               }
           }
       }  
   }
   
 function setDisp() {  function setDisp() {
     var prev = "$previous"      var prev = "$previous"
     var chktotal = 0      var chktotal = 0
     if (document.modifydisp.discdisp.checked == true) {      if (document.modifydisp.discdisp[0].checked == true) {
         document.modifydisp.$dispchg.value = "$symb"          document.modifydisp.$dispchgA.value = "$symb"
           chktotal ++
       }
       if (document.modifydisp.discdisp[1].checked == true) {
           document.modifydisp.$dispchgB.value = "$symb"
         chktotal ++          chktotal ++
     }      }
     if (document.modifydisp.discmark.checked == true) {      if (document.modifydisp.discmark.checked == true) {
         document.modifydisp.$markchg.value = "$symb"          document.modifydisp.$markchg.value = "$symb"
         chktotal ++          chktotal ++
     }      }
       if (document.modifydisp.disctogg.checked == true) {
           document.modifydisp.$toggchg.value = "$symb"
           chktotal ++
       }
     if (chktotal > 0) {       if (chktotal > 0) { 
         document.modifydisp.submit()          document.modifydisp.submit()
     } else {      } else {
Line 1221  function setDisp() { Line 1625  function setDisp() {
 </head>  </head>
 $bodytag  $bodytag
 <form name="modifydisp" method="post" action="/adm/feedback">  <form name="modifydisp" method="post" action="/adm/feedback">
 $lt{'sdpf'}<br/> $lt{'prca'}  <ol><li>$lt{'whpo'}</li><li>$lt{'unwh'}</li></ol>  $lt{'sdpf'}<br/> $lt{'prca'}  <ol><li>$lt{'whpo'}</li><li>$lt{'unwh'}</li><li>$lt{'wipa'}</li></ol>
 <br />  <br />
 <table border="0" cellpadding="0" cellspacing="0">  <table border="0" cellpadding="0" cellspacing="0">
  <tr>   <tr>
Line 1238  $lt{'sdpf'}<br/> $lt{'prca'}  <ol><li>$l Line 1642  $lt{'sdpf'}<br/> $lt{'prca'}  <ol><li>$l
        <tr bgcolor="#dddddd">         <tr bgcolor="#dddddd">
        <td>$lt{'disa'}</td>         <td>$lt{'disa'}</td>
        <td>$lt{$discdisp}</td>         <td>$lt{$discdisp}</td>
        <td><input type="checkbox" name="discdisp" />&nbsp;$lt{'chgt'} "$dispchange"</td>         <td><input type="checkbox" name="discdisp" onClick="discdispChk('0')" />&nbsp;$lt{'chgt'} "$dispchangeA"
              <br />
              <input type="checkbox" name="discdisp" onClick="discdispChk('1')" />&nbsp;$lt{'chgt'} "$dispchangeB"
          </td>
       </tr><tr bgcolor="#eeeeee">        </tr><tr bgcolor="#eeeeee">
        <td>$lt{'npmr'}</td>         <td>$lt{'npmr'}</td>
        <td>$lt{$discmark}</td>         <td>$lt{$discmark}</td>
        <td><input type="checkbox" name="discmark" />$lt{'chgt'} "$markchange"</td>         <td><input type="checkbox" name="discmark" />$lt{'chgt'} "$markchange"</td>
         </tr><tr bgcolor="#dddddd">
          <td>$lt{'dotm'}</td>
          <td>$lt{$disctogg}</td>
          <td><input type="checkbox" name="disctogg" onClick="discdispChk('2')" />$lt{'chgt'} "$toggchange"</td>
       </tr>        </tr>
      </table>       </table>
     </td>      </td>
Line 1254  $lt{'sdpf'}<br/> $lt{'prca'}  <ol><li>$l Line 1665  $lt{'sdpf'}<br/> $lt{'prca'}  <ol><li>$l
 <br />  <br />
 <br />  <br />
 <input type="hidden" name="previous" value="$previous" />  <input type="hidden" name="previous" value="$previous" />
 <input type="hidden" name="$dispchg" value=""/>  <input type="hidden" name="$dispchgA" value=""/>
   <input type="hidden" name="$dispchgB" value=""/>
 <input type="hidden" name="$markchg" value=""/>  <input type="hidden" name="$markchg" value=""/>
   <input type="hidden" name="$toggchg" value="" />
 <input type="button" name="sub" value="Store Changes" onClick="javascript:setDisp()" />  <input type="button" name="sub" value="Store Changes" onClick="javascript:setDisp()" />
 <br />  <br />
 <br />  <br />
Line 1513  END Line 1926  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 1525  sub fail_redirect { Line 1999  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 1575  sub redirect_back { Line 2050  sub redirect_back {
           $sectag = '<input type="hidden" name="sectionpick" value="'.$secpick.'" />';            $sectag = '<input type="hidden" name="sectionpick" value="'.$secpick.'" />';
       }        }
   }    }
     $feedurl=&Apache::lonenc::check_encrypt($feedurl);
   $r->print (<<ENDREDIR);    $r->print (<<ENDREDIR);
 <html>  <html>
 <head>  <head>
Line 1595  $roletag Line 2071  $roletag
 $sectag  $sectag
 $userpicktag  $userpicktag
 </form>  </form>
   <br /><a href="$feedurl">Continue</a>
 </body>  </body>
 </html>  </html>
 ENDREDIR  ENDREDIR
Line 1603  ENDREDIR Line 2080  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 1610  sub no_redirect_back { Line 2088  sub no_redirect_back {
 ENDNOREDIR  ENDNOREDIR
   
   if ($feedurl!~/^\/adm\/feedback/) {     if ($feedurl!~/^\/adm\/feedback/) { 
     $r->print('<meta HTTP-EQUIV="Refresh" CONTENT="2; url='.$feedurl.'">');        $r->print('<meta HTTP-EQUIV="Refresh" CONTENT="2; url='.
    &Apache::lonenc::check_encrypt($feedurl).'">');
   }    }
       $feedurl=&Apache::lonenc::check_encrypt($feedurl);
   $r->print (<<ENDNOREDIRTWO);    $r->print (<<ENDNOREDIRTWO);
 </head>  </head>
 <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 1827  sub send_msg { Line 2307  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 1875  sub adddiscuss { Line 2361  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 1948  sub generate_preview_button { Line 2436  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
 }  }
   
 sub modify_attachments {  sub modify_attachments {
     my ($r,$currnewattach,$currdelold,$symb,$idx,$attachmenturls)=@_;      my ($r,$currnewattach,$currdelold,$symb,$idx,$attachmenturls)=@_;
     my $subject=&clear_out_html($ENV{'form.subject'});      my $orig_subject = &Apache::lonnet::unescape($ENV{'form.subject'});
       my $subject=&clear_out_html($orig_subject);
     $subject=~s/\n/\<br \/\>/g;      $subject=~s/\n/\<br \/\>/g;
     $subject=&Apache::lontexconvert::msgtexconverted($subject);      $subject=&Apache::lontexconvert::msgtexconverted($subject);
     my $timestamp=$ENV{'form.timestamp'};      my $timestamp=$ENV{'form.timestamp'};
Line 1963  sub modify_attachments { Line 2452  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 1984  $bodytag Line 2473  $bodytag
  <table border="2">   <table border="2">
   <tr>    <tr>
    <td>     <td>
     <b>Subject:</b>$subject</b><br /><br />      <b>Subject:</b> $subject</b><br /><br />
 END  END
     if ($idx) {      if ($idx) {
         if ($attachmenturls) {          if ($attachmenturls) {
             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 2091  sub generate_attachments_button { Line 2580  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=escape(document.mailform.comment.value);this.form.subject.value=escape(document.mailform.subject.value);
 END  END
     unless ($mode eq 'board') {      unless ($mode eq 'board') {
         $response .= 'javascript:anonchk();';          $response .= 'javascript:anonchk();';
Line 2127  ENDATTACH Line 2616  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 2194  sub construct_attachmenturl { Line 2668  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 2212  sub construct_attachmenturl { Line 2686  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; 
 }  }
   
   sub has_discussion {
       my $resourcesref = shift;
       my $navmap = Apache::lonnavmaps::navmap->new();
       my @allres=$navmap->retrieveResources();
       foreach my $resource (@allres) {
           if ($resource->hasDiscussion()) {
               my $ressymb;
               if ($resource->symb() =~ m-(___adm/\w+/\w+)/(\d+)/bulletinboard$-) {
                   $ressymb = 'bulletin___'.$2.$1.'/'.$2.'/bulletinboard';
               } else {
                   $ressymb = $resource->symb();
               }
               push @{$resourcesref}, $ressymb;
           }
       }
       return;
   } 
       
 sub handler {  sub handler {
   my $r = shift;    my $r = shift;
Line 2258  sub handler { Line 2750  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','previous','markread','markonread','markondisp','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','navtime','navmaps','navurl','sortfilter','sortposts','applysort','rolefilter','statusfilter','sectionpick','posterlist','userpick','attach','origpage','currnewattach','deloldattach','keepold','allversions','export']);
     if ($ENV{'form.discsymb'}) {
         my $symb = $ENV{'form.discsymb'};
         my $readkey = $symb.'_read';
         my %readinghash = ();
         my $chgcount = 0;
         %readinghash = &Apache::lonnet::get('nohist_'.$ENV{'request.course.id'}.'_discuss',[$readkey],$ENV{'user.domain'},$ENV{'user.name'});
         foreach my $key (keys %ENV) {
             if ($key =~ m/^form\.postunread_(\d+)/) {
                 if ($readinghash{$readkey} =~ /\.$1\./) {
                     $readinghash{$readkey} =~ s/\.$1\.//;
                     $chgcount ++;
                 }
             } elsif ($key =~ m/^form\.postread_(\d+)/) {
                 unless ($readinghash{$readkey} =~ /\.$1\./) {
                     $readinghash{$readkey} .= '.'.$1.'.';
                     $chgcount ++;
                 }
             }
         }
         if ($chgcount > 0) {
             &Apache::lonnet::put('nohist_'.$ENV{'request.course.id'}.'_discuss',\%readinghash,$ENV{'user.domain'},$ENV{'user.name'});
         }
         &Apache::loncommon::content_type($r,'text/html');
         $r->send_http_header;
         my ($map,$ind,$url)=&Apache::lonnet::decode_symb($symb);
         my $previous=$ENV{'form.previous'};
         my $feedurl = &Apache::lonnet::clutter($url);
         &redirect_back($r,$feedurl,&mt('Marked postings read/unread').'<br />','0','0','',$previous,'','','',);
         return OK;
     }
   if ($ENV{'form.allversions'}) {    if ($ENV{'form.allversions'}) {
       &Apache::loncommon::content_type($r,'text/html');        &Apache::loncommon::content_type($r,'text/html');
       $r->send_http_header;        $r->send_http_header;
Line 2284  END Line 2806  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 2426  END Line 2897  END
       }        }
       &print_sortfilter_options($r,$symb,$previous,$feedurl);        &print_sortfilter_options($r,$symb,$previous,$feedurl);
       return OK;        return OK;
   } elsif ($ENV{'form.navmaps'}) {    } elsif ($ENV{'form.navtime'}) {
       my %discinfo = ();        my %discinfo = ();
       my @resources = ();        my @resources = ();
       if ($ENV{'form.navmaps'} =~ /:/) {        if (defined($ENV{'form.navmaps'})) {
           @resources = split/:/,$ENV{'form.navmaps'};            if ($ENV{'form.navmaps'} =~ /:/) {
                 @resources = split/:/,$ENV{'form.navmaps'};
             } else {
                 @resources = ("$ENV{'form.navmaps'}");
             }
       } else {        } else {
           @resources = ("$ENV{'form.navmaps'}");            &has_discussion(\@resources);
       }        }
       my $numitems = @resources;        my $numitems = @resources;
       my $feedurl = '/adm/navmaps';        my $feedurl = '/adm/navmaps';
Line 2441  END Line 2916  END
       }        }
       my %lt = &Apache::lonlocal::texthash(        my %lt = &Apache::lonlocal::texthash(
           'mnpa' => 'Marked "New" posts as read in a total of',            'mnpa' => 'Marked "New" posts as read in a total of',
           'robb' => 'resources/bulletin boards.'            'robb' => 'resources/bulletin boards.',
             'twnp' => 'There are currently no resources or bulletin boards with unread discussion postings.'
       );               );       
       foreach (@resources) {        foreach (@resources) {
 # backward compatibility (bulletin boards used to be 'wrapped')  # backward compatibility (bulletin boards used to be 'wrapped')
Line 2452  END Line 2928  END
               }                }
           }            }
           my $lastkey = $ressymb.'_lastread';            my $lastkey = $ressymb.'_lastread';
           $discinfo{$lastkey} = time;            $discinfo{$lastkey} = $ENV{'form.navtime'};
         }
         my $textline = "<b>$lt{'mnpa'} $numitems $lt{'robb'}</b>";
         if ($numitems > 0) {
             &Apache::lonnet::put('nohist_'.$ENV{'request.course.id'}.'_discuss',\%discinfo,$ENV{'user.domain'},$ENV{'user.name'});
         } else {
             $textline = "<b>$lt{'twnp'}</b>";
       }        }
       &Apache::lonnet::put('nohist_'.$ENV{'request.course.id'}.'_discuss',\%discinfo,$ENV{'user.domain'},$ENV{'user.name'});  
       &Apache::loncommon::content_type($r,'text/html');        &Apache::loncommon::content_type($r,'text/html');
       $r->send_http_header;        $r->send_http_header;
       $r->print (<<ENDREDIR);        $r->print (<<ENDREDIR);
Line 2466  END Line 2947  END
 </head>  </head>
 <body bgcolor="#FFFFFF" onLoad='if (window.name!="loncapaclient") { this.document.reldt.submit(); self.close(); }'>  <body bgcolor="#FFFFFF" onLoad='if (window.name!="loncapaclient") { this.document.reldt.submit(); self.close(); }'>
 <img align="right" src="/adm/lonIcons/lonlogos.gif" />  <img align="right" src="/adm/lonIcons/lonlogos.gif" />
 <b>$lt{'mnpa'} $numitems $lt{'robb'}</b>  $textline
 <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 2479  ENDREDIR Line 2961  ENDREDIR
       my $symb=$ENV{'form.modifydisp'};        my $symb=$ENV{'form.modifydisp'};
       my ($map,$ind,$url)=&Apache::lonnet::decode_symb($symb);        my ($map,$ind,$url)=&Apache::lonnet::decode_symb($symb);
       my $previous=$ENV{'form.previous'};        my $previous=$ENV{'form.previous'};
       my ($dispchg,$markchg) = split/_/,$ENV{'form.changes'};        my ($dispchgA,$dispchgB,$markchg,$toggchg) = split/_/,$ENV{'form.changes'};
       my $feedurl = &Apache::lonnet::clutter($url);        my $feedurl = &Apache::lonnet::clutter($url);
  # backward compatibility (bulletin boards used to be 'wrapped')     # backward compatibility (bulletin boards used to be 'wrapped')  
       if ($feedurl=~m|^/adm/wrapper/adm/.*/bulletinboard$|) {        if ($feedurl=~m|^/adm/wrapper/adm/.*/bulletinboard$|) {
           $feedurl=~s|^/adm/wrapper||;            $feedurl=~s|^/adm/wrapper||;
       }        }
       &print_display_options($r,$symb,$previous,$dispchg,$markchg,$feedurl);        &print_display_options($r,$symb,$previous,$dispchgA,$dispchgB,$markchg,$toggchg,$feedurl);
       return OK;        return OK;
   } elsif (($ENV{'form.markondisp'}) || ($ENV{'form.markonread'}) || ($ENV{'form.allposts'}) || ($ENV{'form.onlyunread'}) ) {    } elsif (($ENV{'form.markondisp'}) || ($ENV{'form.markonread'}) || ($ENV{'form.allposts'}) || ($ENV{'form.onlyunread'}) || $ENV{'form.onlyunmark'} || $ENV{'form.toggoff'} || $ENV{'form.toggon'} ) {
       &Apache::loncommon::content_type($r,'text/html');        &Apache::loncommon::content_type($r,'text/html');
       $r->send_http_header;        $r->send_http_header;
       my $previous=$ENV{'form.previous'};        my $previous=$ENV{'form.previous'};
       my ($map,$ind,$url);        my ($map,$ind,$url);
         if ( ($ENV{'form.toggoff'}) || ($ENV{'form.toggon'}) ) {
   # ------------------------------ Modify setting for read/unread toggle for each post 
             my $symb=$ENV{'form.toggoff'}?$ENV{'form.toggoff'}:$ENV{'form.toggon'};
             my $ressymb = $symb;
             ($map,$ind,$url)=&Apache::lonnet::decode_symb($symb);
             unless ($ressymb =~ m|bulletin___\d+___adm/wrapper|) {
                 $ressymb=~s|(bulletin___\d+___)|$1adm/wrapper|;
             }
             my %discinfo = ();
             my $toggkey = $ressymb.'_readtoggle';
             if ($ENV{'form.toggon'}) {
                 $discinfo{$toggkey} = 1;
             } elsif ($ENV{'form.toggoff'}) {
                 $discinfo{$toggkey} = 0;
             }
             &Apache::lonnet::put('nohist_'.$ENV{'request.course.id'}.'_discuss',\%discinfo,$ENV{'user.domain'},$ENV{'user.name'});
         }
       if (($ENV{'form.markondisp'}) || ($ENV{'form.markonread'})) {        if (($ENV{'form.markondisp'}) || ($ENV{'form.markonread'})) {
 # ---------------------- Modify setting for identification of 'NEW' posts in this discussion  # ---------------------- Modify setting for identification of 'NEW' posts in this discussion
           my $symb=$ENV{'form.markondisp'}?$ENV{'form.markondisp'}:$ENV{'form.markonread'};            my $symb=$ENV{'form.markondisp'}?$ENV{'form.markondisp'}:$ENV{'form.markonread'};
Line 2514  ENDREDIR Line 3013  ENDREDIR
           }            }
           &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'});
       }        }
       if (($ENV{'form.allposts'}) || ($ENV{'form.onlyunread'})) {        if (($ENV{'form.allposts'}) || ($ENV{'form.onlyunread'}) || ($ENV{'form.onlyunmark'}) ) {
 # ----------------------------------------------------------------- Modify display setting for this discussion   # ----------------------------------------------------------------- Modify display setting for this discussion 
           my $symb=$ENV{'form.allposts'}?$ENV{'form.allposts'}:$ENV{'form.onlyunread'};            my $symb;
             if ($ENV{'form.allposts'}) {
                 $symb = $ENV{'form.allposts'};
             } elsif ($ENV{'form.onlyunread'}) {
                 $symb = $ENV{'form.onlyunread'};
             } else {
                 $symb = $ENV{'form.onlyunmark'};
             }
           my $ressymb = $symb;            my $ressymb = $symb;
           ($map,$ind,$url)=&Apache::lonnet::decode_symb($symb);            ($map,$ind,$url)=&Apache::lonnet::decode_symb($symb);
           unless ($ressymb =~ m|bulletin___\d+___adm/wrapper|) {            unless ($ressymb =~ m|bulletin___\d+___adm/wrapper|) {
Line 2525  ENDREDIR Line 3031  ENDREDIR
           my %discinfo = ();            my %discinfo = ();
           if ($ENV{'form.allposts'}) {            if ($ENV{'form.allposts'}) {
               $discinfo{$ressymb.'_showonlyunread'} = 0;                $discinfo{$ressymb.'_showonlyunread'} = 0;
                 $discinfo{$ressymb.'_showonlyunmark'} = 0;
           } elsif ($ENV{'form.onlyunread'}) {            } elsif ($ENV{'form.onlyunread'}) {
               $discinfo{$ressymb.'_showonlyunread'} = 1;                $discinfo{$ressymb.'_showonlyunread'} = 1;
             } else {
                 $discinfo{$ressymb.'_showonlyunmark'} = 1;
           }            }
           &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'});
       }        }
       if (($ENV{'form.markonread'}) || ($ENV{'form.allposts'}) || ($ENV{'form.onlyunread'}) ) {        if (($ENV{'form.markonread'}) || ($ENV{'form.allposts'}) || ($ENV{'form.onlyunread'}) || ($ENV{'form.onlyunmark'}) ||($ENV{'form.toggoff'}) || ($ENV{'form.toggon'}) ) {
           &redirect_back($r,&Apache::lonnet::clutter($url),&mt('Changed display status').'<br />','0','0','',$previous);            &redirect_back($r,&Apache::lonnet::clutter($url),&mt('Changed display status').'<br />','0','0','',$previous);
       } else {        } else {
           &redirect_back($r,&Apache::lonnet::clutter($url),&mt('Changed display status').'<br />','0','0');            &redirect_back($r,&Apache::lonnet::clutter($url),&mt('Changed display status').'<br />','0','0');
       }        }
       return OK;        return OK;
   } elsif ($ENV{'form.markread'}) {    } elsif ($ENV{'form.markread'}) {
 # ----------------------------------------------------------------- Mark new posts as read  # ----------------------------------------------------------------- Mark new posts not NEW 
       &Apache::loncommon::content_type($r,'text/html');        &Apache::loncommon::content_type($r,'text/html');
       $r->send_http_header;        $r->send_http_header;
       my $symb=$ENV{'form.markread'};        my $symb=$ENV{'form.markread'};
Line 2668  ENDREDIR Line 3177  ENDREDIR
           $attachmenturls = $contrib{$idx.':attachmenturl'};            $attachmenturls = $contrib{$idx.':attachmenturl'};
       }        }
       &modify_attachments($r,\@currnewattach,\@currdelold,$symb,$idx,$attachmenturls);        &modify_attachments($r,\@currnewattach,\@currdelold,$symb,$idx,$attachmenturls);
     } elsif ($ENV{'form.chgreads'}) {
         &Apache::loncommon::content_type($r,'text/html');
         $r->send_http_header;
         my ($map,$ind,$url)=&Apache::lonnet::decode_symb($ENV{'form.chgreads'});
         &redirect_back($r,&Apache::lonnet::clutter($url),
          &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.110  
changed lines
  Added in v.1.130


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