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

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

version 1.99.2.3, 2004/08/27 22:13:24 version 1.111, 2004/08/01 16:05:14
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::lonspeller();
   
 sub discussion_open {  sub discussion_open {
     my ($status)=@_;      my ($status)=@_;
Line 64  sub discussion_visible { Line 65  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 (not &discussion_visible($status)) { return ''; }      if (not &discussion_visible($status)) { return ''; }
     my @bgcols = ("#cccccc","#eeeeee");      my @bgcols = ("#cccccc","#eeeeee");
Line 72  sub list_discussion { Line 72  sub list_discussion {
     if ($mode eq 'board') { $discussiononly=1; }      if ($mode eq 'board') { $discussiononly=1; }
     unless ($ENV{'request.course.id'}) { return ''; }      unless ($ENV{'request.course.id'}) { return ''; }
     my $crs='/'.$ENV{'request.course.id'};      my $crs='/'.$ENV{'request.course.id'};
       my $cid=$ENV{'request.course.id'};
     if ($ENV{'request.course.sec'}) {      if ($ENV{'request.course.sec'}) {
  $crs.='_'.$ENV{'request.course.sec'};   $crs.='_'.$ENV{'request.course.sec'};
     }                       }                 
Line 80  sub list_discussion { Line 81  sub list_discussion {
  $symb=&Apache::lonnet::symbread();   $symb=&Apache::lonnet::symbread();
     }      }
     unless ($symb) { return ''; }      unless ($symb) { return ''; }
       my %usernamesort = ();
       my %namesort =();
       my %subjectsort = ();
 # backward compatibility (bulletin boards used to be 'wrapped')  # backward compatibility (bulletin boards used to be 'wrapped')
     my $ressymb=$symb;      my $ressymb=$symb;
     if ($mode eq 'board') {      if ($mode eq 'board') {
Line 92  sub list_discussion { Line 95  sub list_discussion {
 # 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 %dischash = &Apache::lonnet::get('nohist_'.$ENV{'request.course.id'}.'_discuss',[$lastkey,$showkey,$visitkey,$ondispkey],$ENV{'user.domain'},$ENV{'user.name'});      my $userpickkey = $ressymb.'_userpick';
       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;
     my $visit = 0;      my $visit = 0;
     my $newpostsflag = 0;      my $newpostsflag = 0;
       my @posters = split/\&/,$dischash{$userpickkey};
   
 # Retain identification of "NEW" posts identified in last display, if continuing 'previous' browsing of posts.  # Retain identification of "NEW" posts identified in last display, if continuing 'previous' browsing of posts.
     &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['previous']);      &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['previous','sortposts','rolefilter','statusfilter','sectionpick','totposters']);
       my $sortposts = $ENV{'form.sortposts'};
       my $rolefilter = $ENV{'form.rolefilter'};
       my $statusfilter = $ENV{'form.statusfilter'};
       my $sectionpick = $ENV{'form.sectionpick'};
       my $totposters = $ENV{'form.totposters'};
     $previous = $ENV{'form.previous'};      $previous = $ENV{'form.previous'};
     if ($previous > 0) {      if ($previous > 0) {
         $prevread = $previous;          $prevread = $previous;
Line 114  sub list_discussion { Line 129  sub list_discussion {
         }          }
     }      }
   
   # Get information about students and non-students in course for filtering display of posts
       my %roleshash = ();
       my %roleinfo = ();
       if ($rolefilter) {
           %roleshash = &Apache::lonnet::dump('nohist_userroles',$ENV{'course.'.$ENV{'request.course.id'}.'.domain'},$ENV{'course.'.$ENV{'request.course.id'}.'.num'});
           foreach (keys %roleshash) {
               my ($role,$uname,$udom,$sec) = split/:/,$_;
               my ($end,$start) = split/:/,$roleshash{$_};
               my $now = time;
               my $status = 'Active';
               if (($now < $start) || ($end > 0 && $now > $end)) {
                   $status = 'Expired';
               }
               push @{$roleinfo{$uname.':'.$udom}}, $role.':'.$sec.':'.$status;
           }
           my ($classlist) = &Apache::loncoursedata::get_classlist(
                                 $ENV{'request.course.id'},
                                 $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
                                 $ENV{'course.'.$ENV{'request.course.id'}.'.num'});
           my $sec_index = &Apache::loncoursedata::CL_SECTION();
           my $status_index = &Apache::loncoursedata::CL_STATUS();
           while (my ($student,$data) = each %$classlist) {
               my ($section,$status) = ($data->[$sec_index],
                                    $data->[$status_index]);
               push @{$roleinfo{$student}}, 'st:'.$section.':'.$status;
           }
       }
   
 # 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'});      my %userenv = &Apache::lonnet::get('environment',['discdisplay','discmarkread'],$ENV{'user.domain'},$ENV{'user.name'});
     my $discdisplay=$userenv{'discdisplay'};      my $discdisplay=$userenv{'discdisplay'};
Line 137  sub list_discussion { Line 180  sub list_discussion {
         $showonlyunread = $dischash{$showkey};          $showonlyunread = $dischash{$showkey};
     }      }
   
       if (defined($dischash{$markkey})) {
           $showunmark = $dischash{$markkey};
       }
   
     if (defined($dischash{$visitkey})) {      if (defined($dischash{$visitkey})) {
         $visit = $dischash{$visitkey};          $visit = $dischash{$visitkey};
     }      }
Line 146  sub list_discussion { Line 193  sub list_discussion {
     my $viewgrades=(&Apache::lonnet::allowed('vgr',$crs)      my $viewgrades=(&Apache::lonnet::allowed('vgr',$crs)
  && ($symb=~/\.(problem|exam|quiz|assess|survey|form)$/));   && ($symb=~/\.(problem|exam|quiz|assess|survey|form)$/));
     my @discussionitems=();      my @discussionitems=();
       my %shown = ();
       my @posteridentity=();
     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'});
Line 164  sub list_discussion { Line 213  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;
   
Line 182  sub list_discussion { Line 231  sub list_discussion {
                 $newpostsflag = 1;                  $newpostsflag = 1;
             }              }
     my $hidden=($contrib{'hidden'}=~/\.$idx\./);      my $hidden=($contrib{'hidden'}=~/\.$idx\./);
               my $studenthidden=($contrib{'studenthidden'}=~/\.$idx\./);
     my $deleted=($contrib{'deleted'}=~/\.$idx\./);      my $deleted=($contrib{'deleted'}=~/\.$idx\./);
     my $origindex='0.';      my $origindex='0.';
     if (($contrib{$idx.':replyto'}) && ($ENV{'environment.threadeddiscussion'})) {              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  # this is a follow-up message
  $original[$idx]=$original[$contrib{$idx.':replyto'}];      $original[$idx]=$original[$contrib{$idx.':replyto'}];
  $depth[$idx]=$depth[$contrib{$idx.':replyto'}]+1;      $depth[$idx]=$depth[$contrib{$idx.':replyto'}]+1;
  $origindex=$index[$contrib{$idx.':replyto'}];      $origindex=$index[$contrib{$idx.':replyto'}];
  if ($depth[$idx]>$maxdepth) { $maxdepth=$depth[$idx]; }      if ($depth[$idx]>$maxdepth) { $maxdepth=$depth[$idx]; }
                   } else {
                       $original[$idx]=0;
                       $depth[$idx]=0;
                   }
     } else {      } else {
 # this is an original message  # this is an original message
  $original[$idx]=0;   $original[$idx]=0;
Line 202  sub list_discussion { Line 258  sub list_discussion {
     }      }
     unless ((($hidden) && (!$seeid)) || ($deleted)) {      unless ((($hidden) && (!$seeid)) || ($deleted)) {
  $visible++;   $visible++;
  my $message=$contrib{$idx.':message'};                  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=~s/\n/\<br \/\>/g;
  $message=&Apache::lontexconvert::msgtexconverted($message);   $message=&Apache::lontexconvert::msgtexconverted($message);
                 my $subject=$contrib{$idx.':subject'};            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)) {                  if (defined($subject)) {
                     $subject=~s/\n/\<br \/\>/g;                      $subject=~s/\n/\<br \/\>/g;
                     $subject=&Apache::lontexconvert::msgtexconverted($subject);                      $subject=&Apache::lontexconvert::msgtexconverted($subject);
                 }                  }
  if ($contrib{$idx.':attachmenturl'}) {   if ($attachmenturls) {
     my ($fname)                      my @attachments = ();
                         =($contrib{$idx.':attachmenturl'}=~m|/([^/]+)$|);                      my %currattach = ();
     &Apache::lonnet::allowuploaded('/adm/feedback',                      &extract_attachments($attachmenturls,$idx,$numoldver,\$message,\@attachments,\%currattach);
    $contrib{$idx.':attachmenturl'});  
     $message.='<p>'.&mt('Attachment').  
  ': <a href="'.$contrib{$idx.':attachmenturl'}.'"><tt>'.  
  $fname.'</tt></a></p>';  
  }   }
  if ($message) {   if ($message) {
     if ($hidden) {      if ($hidden) {
  $message='<font color="#888888">'.$message.'</font>';   $message='<font color="#888888">'.$message.'</font>';
                           if ($studenthidden) {
                               $message .='<br /><br />Deleted by poster (student).';
                           }
     }      }
     my $screenname=&Apache::loncommon::screenname(      my $screenname=&Apache::loncommon::screenname(
     $contrib{$idx.':sendername'},      $contrib{$idx.':sendername'},
Line 231  sub list_discussion { Line 312  sub list_discussion {
     $contrib{$idx.':senderdomain'});      $contrib{$idx.':senderdomain'});
           
     my $sender=&mt('Anonymous');      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)) {      if ((!$contrib{$idx.':anonymous'}) || ($seeid)) {
  $sender=&Apache::loncommon::aboutmewrapper(   $sender=&Apache::loncommon::aboutmewrapper(
  $plainname,   $plainname,
Line 242  sub list_discussion { Line 337  sub list_discussion {
     $sender.=' ['.&mt('anonymous').'] '.      $sender.=' ['.&mt('anonymous').'] '.
  $screenname;   $screenname;
  }   }
  if ($seeid) {  # Set up for sorting by domain, then username
     if ($hidden) {                          unless (defined($usernamesort{$contrib{$idx.':senderdomain'}})) {
  $sender.=' <a href="/adm/feedback?unhide='.                              %{$usernamesort{$contrib{$idx.':senderdomain'}}} = ();
     $ressymb.':::'.$idx;                          }
                           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) {                                  if ($newpostsflag) {
                                     $sender .= '&previous='.$prevread;                                      $sender .= '&previous='.$prevread;
                                 }                                  }
                                 $sender .= '">'.&mt('Make Visible').'</a>';                                  $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 {      } else {
  $sender.=' <a href="/adm/feedback?hide='.   $sender.=' <a href="/adm/feedback?hide='.
     $ressymb.':::'.$idx;      $ressymb.':::'.$idx;
Line 259  sub list_discussion { Line 397  sub list_discussion {
                                 $sender .= '">'.&mt('Hide').'</a>';                                  $sender .= '">'.&mt('Hide').'</a>';
     }                           }                     
     $sender.=' <a href="/adm/feedback?deldisc='.      $sender.=' <a href="/adm/feedback?deldisc='.
  $ressymb.':::'.$idx;      $ressymb.':::'.$idx;
                                 if ($newpostsflag) {                              if ($newpostsflag) {
                                     $sender .= '&previous='.$prevread;                                  $sender .= '&previous='.$prevread;
                                 }                              }
                                 $sender .= '">'.&mt('Delete').'</a>';                              $sender .= '">'.&mt('Delete').'</a>';
  }   }
     } else {      } else {
  if ($screenname) {   if ($screenname) {
     $sender='<i>'.$screenname.'</i>';      $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) &&      if (&discussion_open($status) &&
  &Apache::lonnet::allowed('pch',   &Apache::lonnet::allowed('pch',
Line 286  sub list_discussion { Line 442  sub list_discussion {
  $vgrlink=&Apache::loncommon::submlink('Submissions',   $vgrlink=&Apache::loncommon::submlink('Submissions',
             $contrib{$idx.':sendername'},$contrib{$idx.':senderdomain'},$symb);              $contrib{$idx.':sendername'},$contrib{$idx.':senderdomain'},$symb);
     }      }
                       my $ctlink;
                       if ($dischash{$readkey}=~/\.$idx\./) { 
                           $ctlink = '<b>'.&mt('Mark unread').'?</b>&nbsp;<input type="checkbox" name="postunread_'.$idx.'" />';
                       } else {
                           $ctlink = '<b>'.&mt('Mark read').'?</b>&nbsp;<input type="checkbox" name="postread_'.$idx.'" />';
                       }
 #figure out at what position this needs to print  #figure out at what position this needs to print
     my $thisindex=$idx;      my $thisindex=$idx;
     if ($ENV{'environment.threadeddiscussion'}) {      if ( (($ENV{'environment.threadeddiscussion'}) && (($sortposts eq '') || ($sortposts eq 'ascdate'))) || ($sortposts eq 'thread')) {
  $thisindex=$origindex.substr('00'.$replies[$depth[$idx]],-2,2);   $thisindex=$origindex.substr('00'.$replies[$depth[$idx]],-2,2);
     }      }
     $alldiscussion{$thisindex}=$idx;      $alldiscussion{$thisindex}=$idx;
     $index[$idx]=$thisindex;                      $shown{$idx} = 0;
                       $index[$idx]=$thisindex;
                     my $spansize = 2;                      my $spansize = 2;
                     if ($showonlyunread && $prevread > $posttime) {                      if ($showonlyunread && $prevread > $posttime) {
                         $notshown{$idx} = 1;                          $notshown{$idx} = 1;
                       } elsif ($showunmark && $dischash{$readkey}=~/\.$idx\./) {
                           $notshown{$idx} = 1;
                     } else {                      } 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) {                          if ($prevread > 0 && $prevread <= $posttime) {
                             $newitem{$idx} = 1;                              $newitem{$idx} = 1;
                             $discussionitems[$idx] .= '                              $discussionitems[$idx] .= '
Line 311  sub list_discussion { Line 524  sub list_discussion {
                         $discussionitems[$idx] .= '<td align ="left">&nbsp;&nbsp;'.                          $discussionitems[$idx] .= '<td align ="left">&nbsp;&nbsp;'.
                             '<b>'.$subject.'</b>&nbsp;&nbsp;'.                              '<b>'.$subject.'</b>&nbsp;&nbsp;'.
                             $sender.'</b> '.$vgrlink.' ('.                              $sender.'</b> '.$vgrlink.' ('.
                             localtime($posttime).')</td></tr>'.                              &Apache::lonlocal::locallocaltime($posttime).')</td>';
                             '</table><blockquote>'.$message.'</blockquote></p>';                          if ($dischash{$toggkey}) {
                               $discussionitems[$idx].='<td align="right">&nbsp;&nbsp;'.
                                 $ctlink.'</td>';
                           }
                           $discussionitems[$idx].= '</tr></table><blockquote>'.$message.'</blockquote></p>';
                           if ($contrib{$idx.':history'}) {
                               my @postversions = ();
                               $discussionitems[$idx] .= '<br />'.&mt('This post has been edited by the author.');
                               if ($seeid) {
                                   $discussionitems[$idx] .= '&nbsp;&nbsp;<a href="/adm/feedback?allversions='.$ressymb.':::'.$idx.'">'.&mt('Display all versions').'</a>';
                               }
                               $discussionitems[$idx].='<br/>'.&mt('Earlier version(s) were posted on: ');
                               if ($contrib{$idx.':history'} =~ m/:/) {
                                   @postversions = split/:/,$contrib{$idx.':history'};
                               } else {
                                   @postversions = ("$contrib{$idx.':history'}");
                               }
                               for (my $i=0; $i<@postversions; $i++) {
                                   my $version = $i+1;
                                   $discussionitems[$idx] .= '<b>'.$version.'.</b> - '.&Apache::lonlocal::locallocaltime($postversions[$i]).'  ';
                               }
                           }
                     }                      }
                 }                  }
             }              }
Line 328  sub list_discussion { Line 562  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 354  sub list_discussion { Line 597  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 367  sub list_discussion { Line 624  sub list_discussion {
 # Print the discusssion  # Print the discusssion
  if ($outputtarget ne 'tex') {   if ($outputtarget ne 'tex') {
             my $colspan=$maxdepth+1;              my $colspan=$maxdepth+1;
     $discussion.='<table bgcolor="#AAAAAA" cellpadding="2" cellspacing="2" border="0">';              $discussion.= qq|
   <script>
      function studentdelete (symb,idx,newflag,previous) {
          var symbparm = symb+':::'+idx
          var prevparm = ""
          if (newflag == 1) {
              prevparm = "&previous="+previous
          }
          if (confirm("Are you sure you want to delete this post?\\nDeleted posts will no longer be visible to you and other students,\\nbut will continue to be visible to your instructor")) {
              document.location.href = "/adm/feedback?hide="+symbparm+prevparm
          }  
      }
   </script>
               |;
       $discussion.='<form name="readchoices" method="post" action="/adm/feedback?chgreads='.$symb.'"><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 381  sub list_discussion { Line 652  sub list_discussion {
  if ($newpostsflag) {   if ($newpostsflag) {
     $discussion .= '&previous='.$prevread;      $discussion .= '&previous='.$prevread;
  }   }
  $discussion .='">'.&mt('Chronological View').'</a>&nbsp;&nbsp;</td>';   $discussion .='">'.&mt('Chronological View').'</a>&nbsp;&nbsp;
     }                                <a href= "/adm/feedback?sortfilter='.$ressymb;
                   if ($newpostsflag) {
                       $discussion .= '&previous='.$prevread;
                   }
                   $discussion .='">'.&mt('Sorting/Filtering options').'</a>&nbsp;&nbsp';
               } else {
                   $discussion .= '<td align="left">';
               }
               $discussion .='<a href= "/adm/feedback?export='.$ressymb;
               if ($newpostsflag) {
                   $discussion .= '&previous='.$prevread;
               }
               $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 409  sub list_discussion { Line 692  sub list_discussion {
                 $discussion .= '&previous='.$prevread;                  $discussion .= '&previous='.$prevread;
             }              }
             $discussion .= '">'.&mt('Show all posts').'</a> '.&mt('to display').' '.              $discussion .= '">'.&mt('Show all posts').'</a> '.&mt('to display').' '.
                          $numhidden.' '.&mt('previously viewed posts').                           $numhidden.' ';
                          '<br/></td></tr>';              if ($showunmark) {
                   $discussion .= &mt('posts previously marked read');
               } else {
                   $discussion .= &mt('previously viewed posts');
               }
               $discussion .= '<br/></td></tr>';
           }
   
   # Choose sort mechanism
           my @showposts = ();
           if ($sortposts eq 'descdate') {
               @showposts = (sort { $b <=> $a } keys %alldiscussion);
           } elsif ($sortposts eq 'thread') {
               @showposts = (sort { $a <=> $b } keys %alldiscussion);
           } elsif ($sortposts eq 'subject') {
               foreach (sort keys %subjectsort) {
                   push @showposts, @{$subjectsort{$_}};
               }
           } elsif ($sortposts eq 'username') {
               foreach my $domain (sort keys %usernamesort) {
                   foreach (sort keys %{$usernamesort{$domain}}) {
                       push @showposts, @{$usernamesort{$domain}{$_}};
                   }
               }
           } elsif ($sortposts eq 'lastfirst') {
               foreach my $last (sort keys %namesort) {
                    foreach (sort keys %{$namesort{$last}}) {
                        push @showposts, @{$namesort{$last}{$_}};
                    }
               }
           } else {
               $sortposts = 'ascdate';
               @showposts =  (sort { $a <=> $b } keys %alldiscussion);
         }          }
  foreach (sort { $a <=> $b } keys %alldiscussion) {          foreach (@showposts) {
             unless ($notshown{$alldiscussion{$_}} eq '1') {              unless (($sortposts eq 'thread') || ($sortposts eq 'ascdate' && $ENV{'environment.threadeddiscussion'})) {
                   $alldiscussion{$_} = $_;
               }
               unless ( ($notshown{$alldiscussion{$_}} eq '1') || ($shown{$alldiscussion{$_}} == 0) ) {
                 if ($outputtarget ne 'tex') {                  if ($outputtarget ne 'tex') {
     $discussion.="\n<tr>";      $discussion.="\n<tr>";
  } else {   } else {
Line 439  sub list_discussion { Line 757  sub list_discussion {
  $threadinsert='<br /><strong>Reply: '.$thisdepth.'</strong>';   $threadinsert='<br /><strong>Reply: '.$thisdepth.'</strong>';
     }      }
     $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([^>]+)>(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                      #FIXME xmlparse can't be safely called from inside xmlparse
Line 453  sub list_discussion { Line 771  sub list_discussion {
         }          }
  if ($outputtarget ne 'tex') {   if ($outputtarget ne 'tex') {
             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 464  sub list_discussion { Line 782  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 ($discussiononly) {      if ($discussiononly) {
           my $now = time;
           my $attachnum = 0;
           my $newattachmsg = '';
           my @currnewattach = ();
           my @currdelold = ();
           my $comment = '';
           my $subject = '';
           if ($ENV{'form.origpage'}) {
               &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['addnewattach','deloldattach','delnewattach','timestamp','idx','subject','comment']);
               $subject = &HTML::Entities::encode($ENV{'form.subject'},'<>&"');
               $comment = &HTML::Entities::encode($ENV{'form.comment'},'<>&"');
               my @keepold = ();
               &process_attachments(\@currnewattach,\@currdelold,\@keepold);
               if (@currnewattach > 0) {
                   $attachnum += @currnewattach;
               }
           }
  $discussion.=(<<ENDDISCUSS);   $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" />
 <input type="hidden" name="symb" value="$ressymb" />  <input type="hidden" name="symb" value="$ressymb" />
 <input type="hidden" name="sendit" value="true" />  <input type="hidden" name="sendit" value="true" />
 <br />  <input type="hidden" name="timestamp" value="$now" />
 <font size="1">Note: in anonymous discussion, your name is visible only to  <br /><a name="newpost"></a>
 course faculty</font><br />  <font size="1">Note: in anonymous discussion, your name is visible only 
 <b>Title:</b>&nbsp;<input type="text" name="subject" value="" size="30" /><br /><br />  to course faculty</font><br />
 <textarea name="comment" cols="80" rows="14" wrap="hard"></textarea>  <b>Title:</b>&nbsp;<input type="text" name="subject" value="$subject" size="30" /><br /><br />
 <p>  <textarea name="comment" cols="80" rows="14" wrap="hard">$comment</textarea>
 Attachment (128 KB max size): <input type="file" name="attachment" />  
 </p>  
 </form>  
 ENDDISCUSS  ENDDISCUSS
           if ($ENV{'form.origpage'}) {
               $discussion.='<input type="hidden" name="origpage" value="'.$ENV{'form.origpage'}.'" />'."\n";
               foreach (@currnewattach) {
                   $discussion.='<input type="hidden" name="currnewattach" value="'.$_.'" />'."\n";
               }
           }
           $discussion.="</form>\n";
         if ($outputtarget ne 'tex') {          if ($outputtarget ne 'tex') {
               $discussion.=&generate_attachments_button('',$attachnum,$ressymb,$now,\@currnewattach,\@currdelold,'',$mode);
               if (@currnewattach > 0) {
                   $newattachmsg .= '<b>New attachments</b><br />';
                   if (@currnewattach > 1) {
                       $newattachmsg .= '<ol>';
                       foreach my $item (@currnewattach) {
                           $item =~ m#.*/([^/]+)$#;
                           $newattachmsg .= '<li><a href="'.$item.'">'.$1.'</a></li>'."\n";
                       }
                       $newattachmsg .= '</ol>'."\n";
                   } else {
                       $currnewattach[0] =~ m#.*/([^/]+)$#;
                       $newattachmsg .= '<a href="'.$currnewattach[0].'">'.$1.'</a><br />'."\n";
                   }
               }
               $discussion.=$newattachmsg;
     $discussion.=&generate_preview_button();      $discussion.=&generate_preview_button();
  }   }
     } else {      } else {
Line 519  ENDDISCUSS Line 894  ENDDISCUSS
     '<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 mail_screen {  sub mail_screen {
   my ($r,$feedurl,$options) = @_;    my ($r,$feedurl,$options) = @_;
     if (exists($ENV{'form.origpage'})) {
         &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['subject','comment','currnewattach','addnewattach','deloldattach','delnewattach','timestamp','idx','anondiscuss','discuss']);
     }
   my $bodytag=&Apache::loncommon::bodytag('Resource Feedback and Discussion',    my $bodytag=&Apache::loncommon::bodytag('Resource Feedback and Discussion',
                                           '','onLoad="window.focus();"');                                            '','onLoad="window.focus();setposttype();"');
   my $title=&Apache::lonnet::gettitle($feedurl);    my $title=&Apache::lonnet::gettitle($feedurl);
   if (!$title) { $title = $feedurl; }    if (!$title) { $title = $feedurl; }
   my $quote='';    my $quote='';
   my $subject = '';    my $subject = '';
     my $comment = '';
   my $prevtag = '';    my $prevtag = '';
   if ($ENV{'form.replydisc'}) {    my $parentmsg = '';
       my ($symb,$idx)=split(/\:\:\:/,$ENV{'form.replydisc'});    my ($symb,$idx,$attachmenturls);
     my $numoldver = 0;
     my $attachmsg = '';
     my $newattachmsg = '';
     my @currnewattach = ();
     my @currdelold = ();
     my @keepold = ();
     my @attachments = ();
     my %currattach = ();
     my $attachnum = 0;
     my $anonchk = (<<END);
     function anonchk() {
        if (document.mailform.anondiscuss.checked == true) {
             document.attachment.anondiscuss.value = '1'
        }
        if (document.mailform.discuss.checked == true) {
             document.attachment.discuss.value = '1'
        }
        return
      }
   END
     my $anonscript;
     if (exists($ENV{'form.origpage'})) {
         $anonscript = (<<END);
     function setposttype() {
         var anondisc = $ENV{'form.anondiscuss'};
         var disc = $ENV{'form.discuss'};
         if (anondisc == 1) {
             document.mailform.anondiscuss.checked = true
         }
         if (disc == 1) {
             document.mailform.discuss.checked = true
         }
         return
     }
   END
     } else {
         $anonscript = (<<END);
     function setposttype() {
         return
     }
   END
     }
     if (($ENV{'form.replydisc'}) || ($ENV{'form.editdisc'})) {
         if ($ENV{'form.replydisc'}) {
             ($symb,$idx)=split(/\:\:\:/,$ENV{'form.replydisc'});
         } else {
             ($symb,$idx)=split(/\:\:\:/,$ENV{'form.editdisc'});
         }
       my %contrib=&Apache::lonnet::restore($symb,$ENV{'request.course.id'},        my %contrib=&Apache::lonnet::restore($symb,$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'});
       unless (($contrib{'hidden'}=~/\.$idx\./) || ($contrib{'deleted'}=~/\.$idx\./)) {        unless (($contrib{'hidden'}=~/\.$idx\./) || ($contrib{'deleted'}=~/\.$idx\./)) {
   my $message=$contrib{$idx.':message'};            if ($ENV{'form.replydisc'}) {
   $message=~s/\n/\<br \/\>/g;                if ($contrib{$idx.':history'}) {
   $quote='<blockquote>'.&Apache::lontexconvert::msgtexconverted($message).'</blockquote>';                    if ($contrib{$idx.':history'} =~ /:/) {
           if ($idx > 0) {                        my @oldversions = split/:/,$contrib{$idx.':history'};
               $subject = 'Re: '.$contrib{$idx.':subject'};                        $numoldver = @oldversions;
                     } else {
                         $numoldver = 1;
                     }
                 }
                 my $message;
                 if ($idx > 0) {
                     if ($contrib{$idx.':message'} =~ /::::\Q$numoldver\E::::(.+?)$/si) {
                         $message = $1;
                     } else {
                         $message = $contrib{$idx.':message'};
                     }
                 } else {
             $message=$contrib{$idx.':message'};
                 }
         $message=~s/\n/\<br \/\>/g;
         $quote='<blockquote>'.&Apache::lontexconvert::msgtexconverted($message).'</blockquote>';
                 if ($idx > 0) {
                     if ($contrib{$idx.':subject'} =~ /::::\Q$numoldver\E::::(.+?)$/si) {
                         $subject = $1;
                     } else {
                         $subject = $contrib{$idx.':subject'};
                     }
                     $subject = 'Re: '.$subject;
                 }
                 $subject = &HTML::Entities::encode($subject,'<>&"');
             } else {
                 $attachmenturls = $contrib{$idx.':attachmenturl'};
                 if ($contrib{$idx.':message'} =~ /.*::::(\d+)::::(.*?)$/si) {
                     $numoldver = $1;
                     $comment = $2;
                 } else {
                     $comment = $contrib{$idx.':message'};
                 }
                 $comment = &HTML::Entities::encode($comment,'<>&"');
                 if ($contrib{$idx.':subject'} =~ /.*::::\d+::::(.+?)$/si) {
                     $subject = $1;
                 } else {
                     $subject = $contrib{$idx.':subject'};
                 }
                 $subject = &HTML::Entities::encode($subject,'<>&"');
                 if (defined($contrib{$idx.':replyto'})) {
                     $parentmsg = $contrib{$idx.':replyto'};
                 }
                 unless (exists($ENV{'form.origpage'})) {
                     my $anonflag = 0;
                     if ($contrib{$idx.':anonymous'}) {
                         $anonflag = 1;
                     }
                     $anonscript = (<<END);
     function setposttype () {
         var currtype = $anonflag
         if (currtype == 1) {
             document.mailform.elements.discuss.checked = false
             document.mailform.elements.anondiscuss.checked = true
         }
         if (currtype == 0) {
             document.mailform.elements.anondiscuss.checked = false
             document.mailform.elements.discuss.checked = true
         }
         return
     }
   END
                 }
           }            }
       }        }
       if ($ENV{'form.previous'}) {        if ($ENV{'form.previous'}) {
           $prevtag = '<input type="hidden" name="previous" value="'.$ENV{'form.previous'}.'" />';            $prevtag = '<input type="hidden" name="previous" value="'.$ENV{'form.previous'}.'" />';
       }        }
   }    }
   
     if ($ENV{'form.origpage'}) {
         $subject = $ENV{'form.subject'};
         $comment = $ENV{'form.comment'};
         &process_attachments(\@currnewattach,\@currdelold,\@keepold);
     }
   my $latexHelp=&Apache::loncommon::helpLatexCheatsheet();    my $latexHelp=&Apache::loncommon::helpLatexCheatsheet();
   my $htmlheader=&Apache::lonhtmlcommon::htmlareaheaders();    my $htmlheader=&Apache::lonhtmlcommon::htmlareaheaders();
   my $send=&mt('Send');    my $send=&mt('Send');
   $r->print(<<ENDDOCUMENT);    $r->print(<<END);
 <html>  <html>
 <head>  <head>
 <title>The LearningOnline Network with CAPA</title>  <title>The LearningOnline Network with CAPA</title>
Line 595  $htmlheader Line 1091  $htmlheader
         }          }
   
         if (rec) {          if (rec) {
             if (typeof(document.mailform.onsubmit)=='function') {              if (typeof(document.mailform.onsubmit)!='undefined') {
  document.mailform.onsubmit();   document.mailform.onsubmit();
     }      }
     document.mailform.submit();      document.mailform.submit();
Line 603  $htmlheader Line 1099  $htmlheader
             alert('Please check a feedback type.');              alert('Please check a feedback type.');
  }   }
     }      }
       $anonchk
       $anonscript
 //-->  //-->
 </script>  </script>
 </head>  </head>
Line 612  $bodytag Line 1110  $bodytag
 enctype="multipart/form-data">  enctype="multipart/form-data">
 $prevtag  $prevtag
 <input type="hidden" name="postdata" value="$feedurl" />  <input type="hidden" name="postdata" value="$feedurl" />
   END
     if ($ENV{'form.replydisc'}) {
         $r->print(<<END);
 <input type="hidden" name="replydisc" value="$ENV{'form.replydisc'}" />  <input type="hidden" name="replydisc" value="$ENV{'form.replydisc'}" />
   END
     } elsif ($ENV{'form.editdisc'}) {
        $r->print(<<END);
   <input type="hidden" name="editdisc" value ="$ENV{'form.editdisc'}" />
   <input type="hidden" name="parentmsg" value ="$parentmsg" />
   END
     }
     $r->print(<<END);
 Please check at least one of the following feedback types:  Please check at least one of the following feedback types:
 $options<hr />  $options<hr />
 $quote  $quote
Line 621  $quote Line 1130  $quote
 $latexHelp  $latexHelp
 Title: <input type="text" name="subject" size="30" value="$subject" /></p>  Title: <input type="text" name="subject" size="30" value="$subject" /></p>
 <p>  <p>
 <textarea name="comment" id="comment" cols="60" rows="10" wrap="hard">  <textarea name="comment" id="comment" cols="60" rows="10" wrap="hard">$comment
 </textarea></p>  </textarea></p>
 <p>  <p>
   END
       if ( ($ENV{'form.editdisc'}) || ($ENV{'form.replydisc'}) ) {
           if ($ENV{'form.origpage'}) {
               foreach (@currnewattach) {
                   $r->print('<input type="hidden" name="currnewattach" value="'.$_.'" />'."\n");
               }
               foreach (@currdelold) {
                   $r->print('<input type="hidden" name="deloldattach" value="'.$_.'" />'."\n");
               }
           }
           if ($ENV{'form.editdisc'}) {
               if ($attachmenturls) {
                   &extract_attachments($attachmenturls,$idx,$numoldver,\$attachmsg,\@attachments,\%currattach,\@currdelold);
                   $attachnum = scalar(keys %currattach);
                   foreach (keys %currattach) {
                       $r->print('<input type="hidden" name="keepold" value="'.$_.'" />'."\n");
                   }
               }
           }
       } else {
           $r->print(<<END);
 Attachment (128 KB max size): <input type="file" name="attachment" />  Attachment (128 KB max size): <input type="file" name="attachment" />
 </p>  </p>
   END
       }
       $r->print(<<END);
 <p>  <p>
 <input type="hidden" name="sendit" value="1" />  <input type="hidden" name="sendit" value="1" />
 <input type="button" value="$send" onClick='gosubmit();' />  <input type="button" value="$send" onClick='gosubmit();' />
 </p>  </p>
 </form>  </form>
 ENDDOCUMENT  END
 $r->print(&generate_preview_button().      if ($ENV{'form.editdisc'} || $ENV{'form.replydisc'}) {
 &Apache::lonhtmlcommon::htmlareaselectactive('comment').          my $now = time;
 '</body></html>');          my $ressymb = $symb;
           my $postidx = '';
           if ($ENV{'form.editdisc'}) {
               $postidx = $idx;
           }
           if (@currnewattach > 0) {
               $attachnum += @currnewattach;
           }
           $r->print(&generate_attachments_button($postidx,$attachnum,$ressymb,$now,\@currnewattach,\@currdelold,$numoldver));
           if ($attachnum > 0) {
               if (@currnewattach > 0) {
                   $newattachmsg .= '<b>New attachments</b><br />';
                   if (@currnewattach > 1) {
                       $newattachmsg .= '<ol>';
                       foreach my $item (@currnewattach) {
                           $item =~ m#.*/([^/]+)$#;
                           $newattachmsg .= '<li><a href="'.$item.'">'.$1.'</a></li>'."\n";
                       }
                       $newattachmsg .= '</ol>'."\n";
                   } else {
                       $currnewattach[0] =~ m#.*/([^/]+)$#;
                       $newattachmsg .= '<a href="'.$currnewattach[0].'">'.$1.'</a><br />'."\n";
                   }
               }
               if ($attachmsg) {
                   $r->print("<b>Retained attachments</b>:$attachmsg<br />\n");
               }
               if ($newattachmsg) {
                   $r->print("$newattachmsg<br />");
               }
           }
       }
       $r->print(&generate_preview_button().
                 &Apache::lonhtmlcommon::htmlareaselectactive('comment').
                 '</body></html>');
 }  }
   
 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 658  sub print_display_options { Line 1225  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 722  function setDisp() { Line 1351  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 739  $lt{'sdpf'}<br/> $lt{'prca'}  <ol><li>$l Line 1368  $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 755  $lt{'sdpf'}<br/> $lt{'prca'}  <ol><li>$l Line 1391  $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 767  END Line 1405  END
     return;      return;
 }  }
   
   sub print_sortfilter_options {
       my ($r,$symb,$previous,$feedurl) = @_;
    # backward compatibility (bulletin boards used to be 'wrapped')
       if ($feedurl=~m|^/adm/wrapper/adm/.*/bulletinboard$|) {
           $feedurl=~s|^/adm/wrapper||;
       }
       my @sections = ();
       my $section_sel = '';
       my $numsections = 0;
       my $numvisible = 5;
       my ($classlist) = &Apache::loncoursedata::get_classlist(
                                 $ENV{'request.course.id'},
                                 $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
                                 $ENV{'course.'.$ENV{'request.course.id'}.'.num'});
                                                                                      
       my $sec_index = &Apache::loncoursedata::CL_SECTION();
       my $status_index = &Apache::loncoursedata::CL_STATUS();
       my %sectioncount = ();
       while (my ($student,$data) = each %$classlist) {
           my ($section,$status) = ($data->[$sec_index],
                                    $data->[$status_index]);
           unless ($section eq '' || $section =~ /^\s*$/) {
               if (!defined($sectioncount{$section})) {
                   $sectioncount{$section} = 1;
                   $numsections ++;
               } else {
                   $sectioncount{$section} ++;
               }
           }
       }
                                                                                      
       if ($ENV{'request.course.sec'} !~ /^\s*$/) {
           @sections = ($ENV{'request.course.sec'});
           $numvisible = 1;
       } else {
           @sections = sort {$a cmp $b} keys(%sectioncount);
           unshift(@sections,'all'); # Put 'all' at the front of the list
           if ($numsections < 4) {
               $numvisible = $numsections + 1;
           }
       }
       foreach (@sections) {
           $section_sel .= "  <option value=\"$_\" />$_\n";
       }
                                                                                      
       my $function = &Apache::loncommon::get_users_function();
       my $tabcolor = &Apache::loncommon::designparm($function.'.tabbg',
                                                       $ENV{'user.domain'});
       my $bodytag=&Apache::loncommon::bodytag('Discussion options',
                                             '','');
       my %lt = &Apache::lonlocal::texthash(
           'diso' => 'Discussion sorting and filtering options',
           'diop' => 'Display Options',
           'curr' => 'Current setting ',
           'actn' => 'Action',
           'prca' => 'Options can be set that control the sort order of the posts, in addition to which posts are displayed.',
           'soor' => 'Sort order',
           'disp' => 'Specific user roles',
           'actv' => 'Specific role status',
           'spse' => 'Specific sections',
           'psub' => 'Pick specific users (by name)',
           'shal' => 'Show a list of current posters'
       );
       $r->print(<<END);
   <html>
   <head>
   <title>$lt{'diso'}</title>
   <meta http-equiv="pragma" content="no-cache" />
   </head>
   $bodytag
   <form name="modifyshown" method="post" action="/adm/feedback">
   <b>$lt{'diso'}</b><br/> $lt{'prca'}
   <br /><br />
   <table border="0">
    <tr>
     <td><b>$lt{'soor'}</b></td>
     <td>&nbsp;</td>
     <td><b>$lt{'disp'}</b></td>
     <td>&nbsp;</td>
     <td><b>$lt{'actv'}</b></td>
     <td>&nbsp;</td>
     <td><b>$lt{'spse'}</b></td>
     <td>&nbsp;</td>
     <td><b>$lt{'psub'}</b></td>
    </tr>
    <tr>
     <td>
      <select name="sortposts">
       <option value="ascdate" />Date order - oldest first
       <option value="descdate" />Date order - newest first
       <option value="thread" />Threaded
       <option value="subject" />By subject
       <option value="username" />By domain and username
       <option value="lastfirst" />By last name, first name
      </select>
     </td>
     <td>&nbsp;</td>
     <td>
      <select name="rolefilter" multiple="true" size="5">
       <option value="all" />All users
       <option value="st" />Students
       <option value="cc" />Course Coordinators
       <option value="in" />Instructors
       <option value="ta" />TAs
       <option value="pr" />Exam proctors
       <option value="cr" />Custom roles
      </select>
     </td>
     <td>&nbsp;</td>
     <td>
      <select name="statusfilter">
       <option value="all" />Roles of any status
       <option value="Active" />Only active roles
       <option value="Expired" />Only inactive roles
      </select>
     </td>
     <td>&nbsp;</td>
     <td>
      <select name="sectionpick" multiple="true" size="$numvisible">
       $section_sel
      </select>
     </td>
     <td>&nbsp;</td>
     <td><input type="checkbox" name="posterlist" value="$symb" />$lt{'shal'}</td>
    </tr>
   </table>
   <br />
   <br />
   <input type="hidden" name="previous" value="$previous" />
   <input type="hidden" name="applysort" value="$symb" />
   <input type="button" name="sub" value="Store Changes" onClick="javascript:document.modifyshown.submit()" />
   <br />
   <br />
   </form>
   </body>
   </html>
   END
   }
   
   sub print_showposters {
       my ($r,$symb,$previous,$feedurl,$sortposts) = @_;
    # backward compatibility (bulletin boards used to be 'wrapped')
       if ($feedurl=~m|^/adm/wrapper/adm/.*/bulletinboard$|) {
           $feedurl=~s|^/adm/wrapper||;
       }
   # backward compatibility (bulletin boards used to be 'wrapped')
       my $ressymb=$symb;
       if ($ressymb =~ /bulletin___\d+___/) {
           unless ($ressymb =~ m|bulletin___\d+___adm/wrapper|) {
               $ressymb=~s|(bulletin___\d+___)|$1adm/wrapper|;
           }
       }
       my $crs='/'.$ENV{'request.course.id'};
       if ($ENV{'request.course.sec'}) {
           $crs.='_'.$ENV{'request.course.sec'};
       }
       $crs=~s/\_/\//g;
       my $seeid=&Apache::lonnet::allowed('rin',$crs);
       my %contrib=&Apache::lonnet::restore($ressymb,$ENV{'request.course.id'},
                             $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
                             $ENV{'course.'.$ENV{'request.course.id'}.'.num'});
       my %namesort = ();
       my %postcounts = ();
       my %lt=&Apache::lonlocal::texthash(
                        'diso' => 'Discussion filtering options',
       );
       my $bodytag=&Apache::loncommon::bodytag('Discussion options',
                                             '','');
       if ($contrib{'version'}) {
           for (my $idx=1;$idx<=$contrib{'version'};$idx++) {
               my $hidden=($contrib{'hidden'}=~/\.$idx\./);
               my $deleted=($contrib{'deleted'}=~/\.$idx\./);
               unless ((($hidden) && (!$seeid)) || ($deleted)) {
                   if ((!$contrib{$idx.':anonymous'}) || ($seeid)) {
                       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}} = ();
                       }
                       my $poster =  $contrib{$idx.':sendername'}.':'.$contrib{$idx.':senderdomain'};
                       $postcounts{$poster} ++;
                       if (defined($namesort{$lastname}{$firstname})) {
                           if (!grep/^$poster$/,@{$namesort{$lastname}{$firstname}}) {
                               push @{$namesort{$lastname}{$firstname}}, $poster;
                           }
                       } else {
                           @{$namesort{$lastname}{$firstname}} = ("$poster");
                       }
                   }
               }
           }
       }
       $r->print(<<END);
   <html>
   <head>
   <title>$lt{'diso'}</title>
   <meta http-equiv="pragma" content="no-cache" />
   </head>
   $bodytag
    <form name="pickpostersform" method="post">
     <table border="0">
      <tr>
       <td bgcolor="#777777">
        <table border="0" cellpadding="3">
         <tr bgcolor="#e6ffff">
          <td><b>No.</b></td>
          <td><b>Select</b></td>
          <td><b>Fullname</b><font color="#999999">(Username/domain)</font></td>
          <td><b>Posts</td>
         </tr>
   END
       my $count = 0;
       foreach my $last (sort keys %namesort) {
           foreach my $first (sort keys %{$namesort{$last}}) {
               foreach (sort @{$namesort{$last}{$first}}) {
                   my ($uname,$udom) = split/:/,$_;
                   if (!$uname || !$udom) { 
                       next;
                   } else {
                       $count ++;
                       $r->print('<tr bgcolor="#ffffe6"><td align="right">'.$count.'</td><td align="center"><input name="stuinfo" type="checkbox" value="'.$_.'" /></td><td>'.$last.', '.$first.' ('.$uname.','.$udom.')</td><td>'.$postcounts{$_}.'</td></tr>');
                   }
               }
           }
       }
       $r->print(<<END);
        </table>
       </td>
      </tr>
     </table>
   <br />
   <input type="hidden" name="sortposts" value="$sortposts" />
   <input type="hidden" name="userpick" value="$symb" />
   <input type="button" name="store" value="Display posts" onClick="javascript:document.pickpostersform.submit()" />
   </form>
   </body>
   </html>
   END
   }
   
 sub fail_redirect {  sub fail_redirect {
   my ($r,$feedurl) = @_;    my ($r,$feedurl) = @_;
   if ($feedurl=~/^\/adm\//) { $feedurl.='?register=1' };    if ($feedurl=~/^\/adm\//) { $feedurl.='?register=1' };
Line 785  ENDFAILREDIR Line 1670  ENDFAILREDIR
 }  }
   
 sub redirect_back {  sub redirect_back {
   my ($r,$feedurl,$typestyle,$sendsomething,$sendposts,$status,$previous) = @_;    my ($r,$feedurl,$typestyle,$sendsomething,$sendposts,$status,$previous,$sort,$rolefilter,$statusfilter,$secpick,$numpicks) = @_;
   my $prevtag = '';    my $sorttag = '';
     my $roletag = '';
     my $statustag = '';
     my $sectag = '';
     my $userpicktag = '';
   my $qrystr = '';    my $qrystr = '';
     my $prevtag = '';
  # 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||;
   }    }
   
   if ($feedurl=~/^\/adm\//) { $feedurl.='?register=1' };    if ($feedurl=~/^\/adm\//) { $feedurl.='?register=1' };
   if ($previous > 0) {    if ($previous > 0) {
       $qrystr = 'previous='.$previous;        $qrystr = 'previous='.$previous;
Line 803  sub redirect_back { Line 1692  sub redirect_back {
       }        }
       $prevtag = '<input type="hidden" name="previous" value="'.$previous.'" />';        $prevtag = '<input type="hidden" name="previous" value="'.$previous.'" />';
   }    }
     if (defined($sort)) {
         my $sortqry = 'sortposts='.$sort;
         if (($feedurl =~ /\?register=1/) || ($feedurl =~ /\?previous=/)) {
             $feedurl .= '&'.$sortqry;
         } else {
             $feedurl .= '?'.$sortqry;
         }
         $sorttag = '<input type="hidden" name="sortposts" value="'.$sort.'" />';
         if ( (defined($numpicks)) && ($numpicks > 0) ) {
             my $userpickqry = 'totposters='.$numpicks;
             $feedurl .= '&'.$userpickqry;
             $userpicktag = '<input type="hidden" name="totposters" value="'.$numpicks.'" />';
         } else {
             my $roleqry = 'rolefilter='.$rolefilter;
             $feedurl .= '&'.$roleqry;
             $roletag = '<input type="hidden" name="rolefilter" value="'.$rolefilter.'" />';
             $feedurl .= '&statusfilter='.$statusfilter;
             $statustag ='<input type="hidden" name="statusfilter" value="'.$statusfilter.'" />';
             $feedurl .= '&sectionpick='.$secpick;
             $sectag = '<input type="hidden" name="sectionpick" value="'.$secpick.'" />';
         }
     }
   $r->print (<<ENDREDIR);    $r->print (<<ENDREDIR);
 <html>  <html>
 <head>  <head>
Line 817  $typestyle Line 1728  $typestyle
 <font color="red">$status</font>  <font color="red">$status</font>
 <form name="reldt" action="$feedurl" target="loncapaclient">  <form name="reldt" action="$feedurl" target="loncapaclient">
 $prevtag  $prevtag
   $sorttag
   $statustag
   $roletag
   $sectag
   $userpicktag
 </form>  </form>
 </body>  </body>
 </html>  </html>
Line 825  ENDREDIR Line 1741  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  ...');
   $r->print (<<ENDNOREDIR);    $r->print (<<ENDNOREDIR);
 <html>  <html>
 <head><title>Feedback not sent</title>  <head><title>Feedback not sent</title>
Line 839  ENDNOREDIR Line 1756  ENDNOREDIR
 </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>Sorry, no feedback possible on this resource  ...</b>  <b>$nofeed</b>
 </body>  </body>
 </html>  </html>
 ENDNOREDIRTWO  ENDNOREDIRTWO
Line 849  sub screen_header { Line 1766  sub screen_header {
     my ($feedurl) = @_;      my ($feedurl) = @_;
     my $msgoptions='';      my $msgoptions='';
     my $discussoptions='';      my $discussoptions='';
     unless ($ENV{'form.replydisc'}) {      unless (($ENV{'form.replydisc'}) || ($ENV{'form.editdisc'})) {
  if (($feedurl=~/^\/res\//) && ($feedurl!~/^\/res\/adm/)) {   if (($feedurl=~/^\/res\//) && ($feedurl!~/^\/res\/adm/)) {
     $msgoptions=       $msgoptions= 
  '<p><input type="checkbox" name="author" /> '.   '<p><input type="checkbox" name="author" /> '.
Line 906  sub resource_output { Line 1823  sub resource_output {
 sub clear_out_html {  sub clear_out_html {
   my ($message,$override)=@_;    my ($message,$override)=@_;
   unless (&Apache::lonhtmlcommon::htmlareablocked()) { return $message; }    unless (&Apache::lonhtmlcommon::htmlareablocked()) { return $message; }
   # Always allow the <m>-tag
     my %html=(M=>1);
   # Check if more is allowed
   my $cid=$ENV{'request.course.id'};    my $cid=$ENV{'request.course.id'};
   if (($ENV{"course.$cid.allow_limited_html_in_feedback"} =~ m/yes/i) ||    if (($ENV{"course.$cid.allow_limited_html_in_feedback"} =~ m/yes/i) ||
       ($override)) {        ($override)) {
       # allows <B> <I> <P> <A> <LI> <OL> <UL> <EM> <BR> <TT> <STRONG>         # allows <B> <I> <P> <A> <LI> <OL> <UL> <EM> <BR> <TT> <STRONG> 
       # <BLOCKQUOTE> <DIV .*> <DIV> <IMG> <M> <SPAN> <H1> <H2> <H3> <H4> <SUB>        # <BLOCKQUOTE> <DIV .*> <DIV> <IMG> <M> <SPAN> <H1> <H2> <H3> <H4> <SUB>
       # <SUP>        # <SUP>
       my %html=(B=>1, I=>1, P=>1, A=>1, LI=>1, OL=>1, UL=>1, EM=>1,        %html=(B=>1, I=>1, P=>1, A=>1, LI=>1, OL=>1, UL=>1, EM=>1,
  BR=>1, TT=>1, STRONG=>1, BLOCKQUOTE=>1, DIV=>1, IMG=>1,       BR=>1, TT=>1, STRONG=>1, BLOCKQUOTE=>1, DIV=>1, IMG=>1,
                 M=>1, SUB=>1, SUP=>1, SPAN=>1,        M=>1, SUB=>1, SUP=>1, SPAN=>1, 
  H1=>1, H2=>1, H3=>1, H4=>1, H5=>1);       H1=>1, H2=>1, H3=>1, H4=>1, H5=>1);
     }
       $message =~ s/\<(\/?\s*(\w+)[^\>\<]*)/  # Do the substitution of everything that is not explicitly allowed
     $message =~ s/\<(\/?\s*(\w+)[^\>\<]*)/
   {($html{uc($2)}&&(length($1)<1000))?"\<$1":"\&lt;$1"}/ge;    {($html{uc($2)}&&(length($1)<1000))?"\<$1":"\&lt;$1"}/ge;
       $message =~ s/(\<?\s*(\w+)[^\<\>]*)\>/    $message =~ s/(\<?\s*(\w+)[^\<\>]*)\>/
   {($html{uc($2)}&&(length($1)<1000))?"$1\>":"$1\&gt;"}/ge;    {($html{uc($2)}&&(length($1)<1000))?"$1\>":"$1\&gt;"}/ge;
   } else {  
       $message=~s/\</\&lt\;/g;  
       $message=~s/\>/\&gt\;/g;  
   }  
   return $message;    return $message;
 }  }
   
Line 1070  sub adddiscuss { Line 1987  sub adddiscuss {
  $contrib{'anonymous'}='true';   $contrib{'anonymous'}='true';
     }      }
     if (($symb) && ($email)) {      if (($symb) && ($email)) {
        $status='Adding to class discussion'.($anon?' (anonymous)':'').': '.          if ($ENV{'form.editdisc'}) {
         &Apache::lonnet::store(\%contrib,$symb,$ENV{'request.course.id'},              my %newcontrib = ();
               $contrib{'ip'}=$ENV{'REMOTE_ADDR'};
               $contrib{'host'}=$Apache::lonnet::perlvar{'lonHostID'};
               $contrib{'timestamp'} = time;
               $contrib{'history'} = '';
               my $numoldver = 0;
               my ($oldsymb,$oldidx)=split(/\:\:\:/,$ENV{'form.editdisc'});
               $oldsymb=~s|(bulletin___\d+___)adm/wrapper/|$1|;
   # get timestamp for last post and history
               my %oldcontrib=&Apache::lonnet::restore($oldsymb,$ENV{'request.course.id'},
                        $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
                        $ENV{'course.'.$ENV{'request.course.id'}.'.num'});
               if (defined($oldcontrib{$oldidx.':replyto'})) {
                   $contrib{'replyto'} = $oldcontrib{$oldidx.':replyto'};
               }
               if (defined($oldcontrib{$oldidx.':history'})) {
                   if ($oldcontrib{$oldidx.':history'} =~ /:/) {
                       my @oldversions = split/:/,$oldcontrib{$oldidx.':history'};
                       $numoldver = @oldversions;
                   } else {
                       $numoldver = 1;
                   }
                   $contrib{'history'} = $oldcontrib{$oldidx.':history'}.':';
               }
               my $numnewver = $numoldver + 1;
               if (defined($oldcontrib{$oldidx.':subject'})) {
                   if ($oldcontrib{$oldidx.':subject'} =~ /::::\d+::::/) {
                       $contrib{'subject'} = $oldcontrib{$oldidx.':subject'}.'::::'.$numnewver.'::::'.$contrib{'subject'};
                   } else {
                        $contrib{'subject'} = '::::0::::'.$oldcontrib{$oldidx.':subject'}.'::::1::::'.$contrib{'subject'};
                   }
               } 
               if (defined($oldcontrib{$oldidx.':message'})) {
                   if ($oldcontrib{$oldidx.':message'} =~ /::::\d+::::/) {
                       $contrib{'message'} = $oldcontrib{$oldidx.':message'}.'::::'.$numnewver.'::::'.$contrib{'message'};
                   } else {
                       $contrib{'message'} = '::::0::::'.$oldcontrib{$oldidx.':message'}.'::::1::::'.$contrib{'message'};
                   }
               }
               $contrib{'history'} .= $oldcontrib{$oldidx.':timestamp'};
               foreach (keys %contrib) {
                   my $key = $oldidx.':'.&Apache::lonnet::escape($oldsymb).':'.$_;                                                                               
                   $newcontrib{$key} = $contrib{$_};
               }
               my $put_reply = &Apache::lonnet::putstore($ENV{'request.course.id'},
                     \%newcontrib,
                     $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
                     $ENV{'course.'.$ENV{'request.course.id'}.'.num'});
               $status='Editing class discussion'.($anon?' (anonymous)':'');
           } else {
              $status='Adding to class discussion'.($anon?' (anonymous)':'').': '.
              &Apache::lonnet::store(\%contrib,$symb,$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'});
           }
         my %storenewentry=($symb => time);          my %storenewentry=($symb => time);
         $status.='<br />'.&mt('Updating discussion time').': '.          $status.='<br />'.&mt('Updating discussion time').': '.
         &Apache::lonnet::put('discussiontimes',\%storenewentry,          &Apache::lonnet::put('discussiontimes',\%storenewentry,
Line 1101  sub show_preview { Line 2070  sub show_preview {
     my $r=shift;      my $r=shift;
     my $message=&clear_out_html($ENV{'form.comment'});      my $message=&clear_out_html($ENV{'form.comment'});
     $message=~s/\n/\<br \/\>/g;      $message=~s/\n/\<br \/\>/g;
       $message=&Apache::lonspeller::markeduptext($message);
     $message=&Apache::lontexconvert::msgtexconverted($message);      $message=&Apache::lontexconvert::msgtexconverted($message);
     my $subject=&clear_out_html($ENV{'form.subject'});      my $subject=&clear_out_html($ENV{'form.subject'});
     $subject=~s/\n/\<br \/\>/g;      $subject=~s/\n/\<br \/\>/g;
Line 1111  sub show_preview { Line 2081  sub show_preview {
 }  }
   
 sub generate_preview_button {  sub generate_preview_button {
     my $pre=&mt("Show Preview");      my $pre=&mt("Show Preview and Check Spelling");
     return(<<ENDPREVIEW);      return(<<ENDPREVIEW);
 <form name="preview" action="/adm/feedback?preview=1" method="post" target="preview">  <form name="preview" action="/adm/feedback?preview=1" method="post" target="preview">
 <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)=='fuction') {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)!='undefined') {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 {
       my ($r,$currnewattach,$currdelold,$symb,$idx,$attachmenturls)=@_;
       my $subject=&clear_out_html($ENV{'form.subject'});
       $subject=~s/\n/\<br \/\>/g;
       $subject=&Apache::lontexconvert::msgtexconverted($subject);
       my $timestamp=$ENV{'form.timestamp'};
       my $numoldver=$ENV{'form.numoldver'};
       my $bodytag=&Apache::loncommon::bodytag('Discussion Post Attachments',
                                             '','');
       my $msg = '';
       my @attachments = ();
       my %currattach = ();
       if ($idx) {
           &extract_attachments($attachmenturls,$idx,$numoldver,\$msg,\@attachments,\%currattach,$currdelold);
       }
       $r->print(<<END);
   <html>
   <head>
   <title>Managing Attachments</title>
   <script>
    function setAction () {
      document.modattachments.action = document.modattachments.origpage.value;
      document.modattachments.submit();
    }
   </script> 
   </head>
   $bodytag
   <form name="modattachments" method="post" enctype="multipart/form-data" action="/adm/feedback?attach=$symb">
    <table border="2">
     <tr>
      <td>
       <b>Subject:</b>$subject</b><br /><br />
   END
       if ($idx) {
           if ($attachmenturls) {
               my @currold = keys %currattach;
               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");  
                   foreach (@currold) {
                       my $id = $_;
                       $attachments[$id] =~ m#/([^/]+)$#;
                       $r->print('<input type="checkbox" name="deloldattach" value="'.$_.'" />&nbsp;'.$1.'<br />'."\n");
                   }
                   $r->print("<br />");
               }
           }
       }
       if (@{$currnewattach} > 0) {
           $r->print("The following attachments have been uploaded for inclusion with this posting.<br />Check the checkboxes for any you wish to remove<br />\n");
           foreach (@{$currnewattach}) {
               $_ =~ m#/([^/]+)$#;
               $r->print('<input type="checkbox" name="delnewattach" value="'.$_.'" />&nbsp;'.$1.'<br />'."\n");
           }
           $r->print("<br />"); 
       }
       $r->print(<<END);
      Add a new attachment to this post.&nbsp;<input type="file" name="addnewattach" /><input type="button" name="upload" value="Upload" onClick="this.form.submit()" />    
      </td>
     </tr>
    </table>
   <input type="hidden" name="subject" value="$ENV{'form.subject'}" />
   <input type="hidden" name="comment" value="$ENV{'form.comment'}" />
   <input type="hidden" name="timestamp" value="$ENV{'form.timestamp'}" />
   <input type="hidden" name="idx" value="$ENV{'form.idx'}" />
   <input type="hidden" name="numoldver" value="$ENV{'form.numoldver'}" />
   <input type="hidden" name="origpage" value="$ENV{'form.origpage'}" />
   <input type="hidden" name="anondiscuss" value="$ENV{'form.anondiscuss'}" />
   <input type="hidden" name="discuss" value="$ENV{'form.discuss'}" />
   END
       foreach (@{$currnewattach}) {
           $r->print('<input type="hidden" name="currnewattach" value="'.$_.'" />'."\n");
       }
       foreach (@{$currdelold}) {
           $r->print('<input type="hidden" name="deloldattach" value="'.$_.'" />'."\n");
       }
       $r->print(<<END);
    <input type="button" name="rtntoedit" value="Store Changes" onClick="setAction()"/>
   </form>
   </body>
   </html>
   END
       return;
   }
   
   sub process_attachments {
       my ($currnewattach,$currdelold,$keepold) = @_;
       if (exists($ENV{'form.currnewattach'})) {
           if (ref($ENV{'form.currnewattach'}) eq 'ARRAY') {
               @{$currnewattach} = @{$ENV{'form.currnewattach'}};
           } else {
               $$currnewattach[0] = $ENV{'form.currnewattach'};
           }
       }
       if (exists($ENV{'form.deloldattach'})) {
           if (ref($ENV{'form.deloldattach'}) eq 'ARRAY') {
               @{$currdelold} = @{$ENV{'form.deloldattach'}};
           } else {
               $$currdelold[0] = $ENV{'form.deloldattach'};
           }
       }
       if (exists($ENV{'form.delnewattach'})) {
           my @currdelnew = ();
           my @currnew = ();
           if (ref($ENV{'form.delnewattach'}) eq 'ARRAY') {
               @currdelnew = @{$ENV{'form.delnewattach'}};
           } else {
               $currdelnew[0] = $ENV{'form.delnewattach'};
           }
           foreach my $newone (@{$currnewattach}) {
               my $delflag = 0;
               foreach (@currdelnew) {
                   if ($newone eq $_) {
                       $delflag = 1;
                       last;
                   }
               }
               unless ($delflag) {
                   push @currnew, $newone;
               }
           }
           @{$currnewattach} = @currnew;
       }
       if (exists($ENV{'form.keepold'})) {
           if (ref($ENV{'form.keepold'}) eq 'ARRAY') {
               @{$keepold} = @{$ENV{'form.keepold'}};
           } else {
               $$keepold[0] = $ENV{'form.keepold'};
           }
       }
   }
   
   sub generate_attachments_button {
       my ($idx,$attachnum,$ressymb,$now,$currnewattach,$deloldattach,$numoldver,$mode) = @_;
       my $origpage = $ENV{'REQUEST_URI'};
       my $att=$attachnum.' '.&mt("attachments");
       my $response = (<<END);
   <form name="attachment" action="/adm/feedback?attach=$ressymb" method="post">
   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;
   END
       unless ($mode eq 'board') {
           $response .= 'javascript:anonchk();';
       }
       $response .= (<<ENDATTACH);
   this.form.submit();" />
   <input type="hidden" name="origpage" value="$origpage" />
   <input type="hidden" name="idx" value="$idx" />
   <input type="hidden" name="timestamp" value="$now" />
   <input type="hidden" name="subject" />
   <input type="hidden" name="comment" />
   <input type="hidden" name="anondiscuss" value = "0";
   <input type="hidden" name="discuss" value = "0";
   <input type="hidden" name="numoldver" value="$numoldver" />
   ENDATTACH
       if (defined($deloldattach)) {
           if (@{$deloldattach} > 0) {
               foreach (@{$deloldattach}) {
                   $response .= '<input type="hidden" name="deloldattach" value="'.$_.'" />'."\n";
               }
           }
       }
       if (defined($currnewattach)) {
           if (@{$currnewattach} > 0) {
               foreach (@{$currnewattach}) {
                   $response .= '<input type="hidden" name="currnewattach" value="'.$_.'" />'."\n";
               }
           }
       }
       $response .= '</form>';
       return $response;
   }
   
   sub extract_attachments {
       my ($attachmenturls,$idx,$numoldver,$message,$attachments,$currattach,$currdelold) = @_;
       if ($attachmenturls =~ m/::::\d+:[\.yn\d]+::::/) {
           @{$attachments} = split/::::\d+:[\.yn\d]+::::/,$attachmenturls;
           shift @{$attachments};
           my $searchstr = '::::';
           for (my $i=0; $i<@{$attachments}; $i++) {
               if ($attachmenturls =~ m#^\Q$searchstr\E(\d+)(:[\.yn\d]+)::::#) {
                   my $info = $1.$2;
                   my $attachid = $1-1;
                   $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;
                       }
                   }
               }
           }
           my @attached = (sort { $a <=> $b } keys %{$currattach});
           if (@attached == 1) {
               my $id = $attached[0];
               $$attachments[$attached[0]]=~m|/([^/]+)$|;
               $$message.='<br /><a href="'.$$attachments[$id].'"><tt>'.
               $1.'</tt></a><br />';
               &Apache::lonnet::allowuploaded('/adm/feedback',
                                      $$attachments[$id]);
           } 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 {
           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);
       }
   }
   
   sub construct_attachmenturl {
       my ($currnewattach,$keepold,$symb,$idx)=@_;
       my $oldattachmenturl;
       my $newattachmenturl;
       my $startnum = 1;
       my $currver = 0;
       if (($ENV{'form.editdisc'}) && ($idx)) {
           my %contrib=&Apache::lonnet::restore($symb,$ENV{'request.course.id'},
                          $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
                          $ENV{'course.'.$ENV{'request.course.id'}.'.num'});
           $oldattachmenturl = $contrib{$idx.':attachmenturl'};
           if ($contrib{$idx.':history'}) {
               if ($contrib{$idx.':history'} =~ /:/) {
                   my @oldversions = split/:/,$contrib{$idx.':history'};
                   $currver = 1 + scalar(@oldversions);
               } else {
                   $currver = 2;
               }
           } else {
               $currver = 1;
           }
           if ($oldattachmenturl) {
               if ($oldattachmenturl =~ m/::::\d+:[\.yn\d]+::::/) {
                   my @attachments = split/::::\d+:[\.yn\d]+::::/,$oldattachmenturl;
                   shift @attachments;
                   $startnum += @attachments;
                   my $searchstr = '::::';
                   $newattachmenturl = '::::';
                   for (my $i=0; $i<@attachments; $i++) {
                       if ($oldattachmenturl =~ m#^\Q$searchstr\E(\d+)(:[\.yn\d]+)::::#) {
                           my $attachid = $1 - 1;
                           $searchstr .= $1.$2.'::::'.$attachments[$i].'::::';
                           $newattachmenturl .= $1.$2;
                           if (grep/^$attachid$/,@{$keepold}) {
                               $newattachmenturl .= '.'.$currver.'n.';
                           }
                           $newattachmenturl .= '::::'.$attachments[$i].'::::';
                       }
                   }
                   $newattachmenturl =~ s/::::$//;
               } else {
                   $newattachmenturl = '::::1:.0n.';
                   unless (grep/^0$/,@{$keepold}) {
                       $newattachmenturl .= '.1n.';
                   }
                   $newattachmenturl .= '::::'.$oldattachmenturl;
                   $startnum ++;
               }
           }
       }
       for (my $i=0; $i<@{$currnewattach}; $i++) {
           my $attachnum = $startnum + $i;
           $newattachmenturl .= '::::'.$attachnum.':.'.$currver.'n.::::'.$$currnewattach[$i];
       }
       return $newattachmenturl; 
   }
     
 sub handler {  sub handler {
   my $r = shift;    my $r = shift;
   if ($r->header_only) {    if ($r->header_only) {
Line 1133  sub handler { Line 2397  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','threadedon','threadedoff','onlyunread','allposts','previous','markread','markonread','markondisp','modifydisp','changes','navmaps','navurl']);           ['hide','unhide','deldisc','postdata','preview','replydisc','editdisc','threadedon','threadedoff','onlyunread','allposts','onlyunmark','previous','markread','markonread','markondisp','toggoff','toggon','modifydisp','changes','navmaps','navurl','sortfilter','sortposts','applysort','rolefilter','statusfilter','sectionpick','posterlist','userpick','attach','origpage','currnewattach','deloldattach','keepold','allversions']);
   if ($ENV{'form.navmaps'}) {    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'}) {
         &Apache::loncommon::content_type($r,'text/html');
         $r->send_http_header;
         my $bodytag=&Apache::loncommon::bodytag('Discussion Post Versions',
                                             '','');
         $r->print (<<END);
   <html>
   <head>
   <title>Post Versions</title>
   <meta http-equiv="pragma" content="no-cache" />
   </head>
   $bodytag
   END
         my $crs='/'.$ENV{'request.course.id'};
         if ($ENV{'request.course.sec'}) {
             $crs.='_'.$ENV{'request.course.sec'};
         }
         $crs=~s/\_/\//g;
         my $seeid=&Apache::lonnet::allowed('rin',$crs);
         my ($symb,$idx)=split(/\:\:\:/,$ENV{'form.allversions'});
         my $ressymb=$symb;
         unless ($ressymb =~ m|bulletin___\d+___adm/wrapper|) {
             $ressymb=~s|(bulletin___\d+___)|$1adm/wrapper|;
         }
         if ($idx > 0) {
             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{$idx.':history'}) {
                 my $attachmenturls = $contrib{$idx.':attachmenturl'};
                 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;
     }
     if ($ENV{'form.posterlist'}) {
         &Apache::loncommon::content_type($r,'text/html');
         $r->send_http_header;
         my $symb=$ENV{'form.posterlist'};
         my $sortposts = $ENV{'form.sortposts'};
         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$|) {
             $feedurl=~s|^/adm/wrapper||;
         }
         &print_showposters($r,$symb,$previous,$feedurl,$sortposts);
         return OK;
     }
     if ($ENV{'form.userpick'}) {
         &Apache::loncommon::content_type($r,'text/html');
         $r->send_http_header;
         my $symb=$ENV{'form.userpick'};
         my ($map,$ind,$url)=&Apache::lonnet::decode_symb($symb);
         my $previous=$ENV{'form.previous'};
   # backward compatibility (bulletin boards used to be 'wrapped')
         my $ressymb=$symb;
         unless ($ressymb =~ m|bulletin___\d+___adm/wrapper|) {
             $ressymb=~s|(bulletin___\d+___)|$1adm/wrapper|;
         }
         my $sort=$ENV{'form.sortposts'};
         my @posters = ();
         if (ref($ENV{'form.stuinfo'}) eq 'ARRAY') {
             @posters = $ENV{'form.stuinfo'};
         } else {
             $posters[0] = $ENV{'form.stuinfo'};
         }
         my $numpicks = @posters;
         if (defined($ENV{'form.userpick'})) {
             my %discinfo = ();
             $discinfo{$ressymb.'_userpick'} = join('&',@posters);
             &Apache::lonnet::put('nohist_'.$ENV{'request.course.id'}.'_discuss',\%discinfo,$ENV{'user.domain'},$ENV{'user.name'});
         }
         my $feedurl = &Apache::lonnet::clutter($url);
    # backward compatibility (bulletin boards used to be 'wrapped')
         if ($feedurl=~m|^/adm/wrapper/adm/.*/bulletinboard$|) {
             $feedurl=~s|^/adm/wrapper||;
         }
         &redirect_back($r,$feedurl,&mt('Changed sort/filter').'<br />','0','0','',$previous,$sort,'','','',$numpicks);
         return OK;
     }
     if ($ENV{'form.applysort'}) {
         &Apache::loncommon::content_type($r,'text/html');
         $r->send_http_header;
         my $symb=$ENV{'form.applysort'};
         my ($map,$ind,$url)=&Apache::lonnet::decode_symb($symb);
         my $previous=$ENV{'form.previous'};
         my $sort = $ENV{'form.sortposts'};
         my $rolefilter = $ENV{'form.rolefilter'};
         my $statusfilter = $ENV{'form.statusfilter'};
         my $secpick = $ENV{'form.sectionpick'};
         my $feedurl = &Apache::lonnet::clutter($url);
    # backward compatibility (bulletin boards used to be 'wrapped')
         if ($feedurl=~m|^/adm/wrapper/adm/.*/bulletinboard$|) {
             $feedurl=~s|^/adm/wrapper||;
         }
         &redirect_back($r,$feedurl,&mt('Changed sort/filter').'<br />','0','0','',$previous,$sort,$rolefilter,$statusfilter,$secpick);
         return OK;
     } elsif ($ENV{'form.sortfilter'}) {
         &Apache::loncommon::content_type($r,'text/html');
         $r->send_http_header;
         my $symb=$ENV{'form.sortfilter'};
         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$|) {
             $feedurl=~s|^/adm/wrapper||;
         }
         &print_sortfilter_options($r,$symb,$previous,$feedurl);
         return OK;
     } elsif ($ENV{'form.navmaps'}) {
       my %discinfo = ();        my %discinfo = ();
       my @resources = ();        my @resources = ();
       if ($ENV{'form.navmaps'} =~ /:/) {        if ($ENV{'form.navmaps'} =~ /:/) {
Line 1187  ENDREDIR Line 2648  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 1222  ENDREDIR Line 2700  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 1233  ENDREDIR Line 2718  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 1276  ENDREDIR Line 2764  ENDREDIR
   
                   
     my $currenthidden=$contrib{'hidden'};      my $currenthidden=$contrib{'hidden'};
       my $currentstudenthidden=$contrib{'studenthidden'};
   
       my $crs='/'.$ENV{'request.course.id'};
       if ($ENV{'request.course.sec'}) {
           $crs.='_'.$ENV{'request.course.sec'};
       }
       $crs=~s/\_/\//g;
       my $seeid=&Apache::lonnet::allowed('rin',$crs);
           
     if ($ENV{'form.hide'}) {      if ($ENV{'form.hide'}) {
  $currenthidden.='.'.$idx.'.';   $currenthidden.='.'.$idx.'.';
           unless ($seeid) {
               $currentstudenthidden.='.'.$idx.'.';
           }
     } else {      } else {
         $currenthidden=~s/\.$idx\.//g;          $currenthidden=~s/\.$idx\.//g;
     }      }
     my %newhash=('hidden' => $currenthidden);      my %newhash=('hidden' => $currenthidden);
       if ( ($ENV{'form.hide'}) && (!$seeid) ) {
           $newhash{'studenthidden'} = $currentstudenthidden;
       }
   
     &Apache::lonnet::store(\%newhash,$symb,$ENV{'request.course.id'},      &Apache::lonnet::store(\%newhash,$symb,$ENV{'request.course.id'},
                      $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},                       $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
Line 1336  ENDREDIR Line 2838  ENDREDIR
       $r->content_type('text/html');        $r->content_type('text/html');
       $r->send_http_header;        $r->send_http_header;
       &show_preview($r);        &show_preview($r);
     } elsif ($ENV{'form.attach'}) {
   # -------------------------------------------------------- Work on attachments
         &Apache::loncommon::content_type($r,'text/html');
         $r->send_http_header;
         &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['subject','comment','addnewattach','delnewattach','timestamp','numoldver','idx','anondiscuss','discuss']);
         my @currnewattach = ();
         my @currdelold = ();
         my @keepold = ();
         &process_attachments(\@currnewattach,\@currdelold,\@keepold);
         if (exists($ENV{'form.addnewattach.filename'})) {
             unless (length($ENV{'form.addnewattach'})>131072) {
                 my $subdir = 'feedback/'.$ENV{'form.timestamp'};
                 my $newattachment=&Apache::lonnet::userfileupload('addnewattach',undef,$subdir);
                 push @currnewattach, $newattachment;
             }
         }
         my $attachmenturls = '';
         my $idx = $ENV{'form.idx'};
         my $symb = $ENV{'form.attach'};
         if ($idx) {
             my %contrib=&Apache::lonnet::restore($symb,$ENV{'request.course.id'},
                            $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
                            $ENV{'course.'.$ENV{'request.course.id'}.'.num'});
             $attachmenturls = $contrib{$idx.':attachmenturl'};
         }
         &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');
   } else {    } else {
 # ------------------------------------------------------------- Normal feedback  # ------------------------------------------------------------- Normal feedback
   my $feedurl=$ENV{'form.postdata'};    my $feedurl=$ENV{'form.postdata'};
Line 1349  ENDREDIR Line 2883  ENDREDIR
       $symb=(split(/\:\:\:/,$ENV{'form.replydisc'}))[0];        $symb=(split(/\:\:\:/,$ENV{'form.replydisc'}))[0];
       my ($map,$id,$url)=&Apache::lonnet::decode_symb($symb);        my ($map,$id,$url)=&Apache::lonnet::decode_symb($symb);
       $feedurl=&Apache::lonnet::clutter($url);        $feedurl=&Apache::lonnet::clutter($url);
     } elsif ($ENV{'form.editdisc'}) {
         $symb=(split(/\:\:\:/,$ENV{'form.editdisc'}))[0];
         my ($map,$id,$url)=&Apache::lonnet::decode_symb($symb);
         $feedurl=&Apache::lonnet::clutter($url);
     } elsif ($ENV{'form.origpage'}) {
         $symb=""; 
   } else {    } else {
       $symb=&Apache::lonnet::symbread($feedurl);        $symb=&Apache::lonnet::symbread($feedurl);
   }    }
Line 1385  ENDREDIR Line 2925  ENDREDIR
     unless ($ENV{'form.sendit'}) {      unless ($ENV{'form.sendit'}) {
       my $options=&screen_header($feedurl);        my $options=&screen_header($feedurl);
       if ($options) {        if ($options) {
  &mail_screen($r,$feedurl,$options);          &mail_screen($r,$feedurl,$options);
       } else {        } else {
  &fail_redirect($r,$feedurl);   &fail_redirect($r,$feedurl);
       }        }
Line 1407  ENDREDIR Line 2947  ENDREDIR
       &Apache::lonnet::delenv('allowed.vgr');        &Apache::lonnet::delenv('allowed.vgr');
 # Get attachments, if any, and not too large  # Get attachments, if any, and not too large
       my $attachmenturl='';        my $attachmenturl='';
       if ($ENV{'form.attachment.filename'}) {        if (($ENV{'form.origpage'}) || ($ENV{'form.editdisc'}) || ($ENV{'form.replydisc'})) {
             my ($symb,$idx);
             if ($ENV{'form.replydisc'}) {
                 ($symb,$idx)=split(/\:\:\:/,$ENV{'form.replydisc'});
             } elsif ($ENV{'form.editdisc'}) {
                 ($symb,$idx)=split(/\:\:\:/,$ENV{'form.editdisc'});
             } elsif ($ENV{'form.origpage'}) {
                 $symb = $ENV{'form.symb'};
             }
             my @currnewattach = ();
             my @deloldattach = ();
             my @keepold = ();
             &process_attachments(\@currnewattach,\@deloldattach,\@keepold);
             $symb=~s|(bulletin___\d+___)adm/wrapper/|$1|;
             $attachmenturl=&construct_attachmenturl(\@currnewattach,\@keepold,$symb,$idx);
         } elsif ($ENV{'form.attachment.filename'}) {
   unless (length($ENV{'form.attachment'})>131072) {    unless (length($ENV{'form.attachment'})>131072) {
       $attachmenturl=&Apache::lonnet::userfileupload('attachment',undef,'feedback');        $attachmenturl=&Apache::lonnet::userfileupload('attachment',undef,'feedback');
   }    }
Removed from v.1.99.2.3  
changed lines
  Added in v.1.111

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