--- loncom/interface/lonfeedback.pm	2021/12/31 20:39:00	1.370.2.5.2.1
+++ loncom/interface/lonfeedback.pm	2017/11/03 21:36:27	1.374
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Feedback
 #
-# $Id: lonfeedback.pm,v 1.370.2.5.2.1 2021/12/31 20:39:00 raeburn Exp $
+# $Id: lonfeedback.pm,v 1.374 2017/11/03 21:36:27 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -44,7 +44,7 @@ use HTML::LCParser();
 #use HTML::Tidy::libXML;
 use Apache::lonspeller();
 use Apache::longroup;
-use Archive::Zip qw( :ERROR_CODES );
+use Cwd;
 use LONCAPA qw(:DEFAULT :match);
 
 sub discussion_open {
@@ -118,16 +118,10 @@ sub list_discussion {
             $outputtarget = 'export';
         }
     }
-    my ($nofooter,$nodisclink,$nofdbklink);
     if (not &discussion_visible($status)) {
         if ($mode ne 'board') {
-            ($nofooter,$nodisclink,$nofdbklink) = &check_menucoll();
-            if ($nofooter || $nofdbklink) {
-                return '<br />';
-            } else {
-                &Apache::lonenc::check_encrypt(\$ressymb);
-                return '<br /><div class="LC_feedback_link">'.&send_message_link($ressymb)."</div>";
-            }
+            &Apache::lonenc::check_encrypt(\$ressymb); 
+            return '<br /><div class="LC_feedback_link">'.&send_message_link($ressymb)."</div>";
         }
     }
     if ($group ne '' && $mode eq 'board') {
@@ -136,27 +130,17 @@ sub list_discussion {
         }
     }
 
-    unless ($outputtarget eq 'export') {
-        ($nofooter,$nodisclink,$nofdbklink) = &check_menucoll();
-    }
-
-    unless ($nofooter) {
-        my ($blocked,$blocktext) = 
-            &Apache::loncommon::blocking_status('boards');
-        if ($blocked) {
-            my $footer = '<br /><div class="LC_feedback_link">';
-            unless ($nodisclink) {
-                $footer .= '<span class="LC_feedback_link">'.$blocktext.'</span>';
-            }
-            &Apache::lonenc::check_encrypt(\$ressymb);
-            if ($mode ne 'board') {
-                unless ($nofdbklink) {
-                    $footer.=&send_message_link($ressymb);
-                }
-            }
-            $footer.='</div>';
-            return $footer;
+    my ($blocked,$blocktext) = 
+        &Apache::loncommon::blocking_status('boards');
+    if ($blocked) {
+        $blocktext = '<br /><div class="LC_feedback_link"><span class="LC_feedback_link">'.$blocktext."</span>";
+        &Apache::lonenc::check_encrypt(\$ressymb);
+        if ($mode ne 'board') {
+            $blocktext.=&send_message_link($ressymb).'</div>';
+        }else{
+            $blocktext.="</div>";
         }
+        return $blocktext; 
     }
 
     my @bgcols = ("LC_disc_old_item","LC_disc_new_item");
@@ -690,7 +674,7 @@ END
                                              $newpostsflag,$group,
                                              $prevread,$markondisp,$seehidden);
             $discussion .= "</table></form>\n";
-        }
+        } 
         if ($outputtarget eq 'export') {
             if ($manifestok) {
                 while ($currdepth > 0) {
@@ -714,19 +698,18 @@ END
 
                     if (($env{'user.name'} =~ /^$match_username$/)
                         && ($env{'user.domain'} =~ /^$match_domain$/)) {
-                        my $now = time();
+                        my $now = time(); 
                         my $imszipfile = '/prtspool/'.
                                       join('_',$env{'user.name'},$env{'user.domain'},$now).
                                       '_'.rand(1000000000).'.zip';
-                        my $zip = Archive::Zip->new();
-                        $zip->addTree($tempexport);
+                        my $cwd = &getcwd(); 
                         my $imszip = '/home/httpd/'.$imszipfile;
-                        if ($zip->writeToFileNamed($imszip) == AZ_OK) {
-                            $discussion .= &mt('Download the zip file from [_1]Discussion Posting Archive[_2]',
+                        chdir $tempexport;
+                        open(OUTPUT, "zip -r $imszip *  2> /dev/null |");
+                        close(OUTPUT);
+                        chdir $cwd;
+                        $discussion .= &mt('Download the zip file from [_1]Discussion Posting Archive[_2]',
                                            '<a href="'.$imszipfile.'">','</a>').'<br />';
-                        } else {
-                            $discussion .=  &mt('Failed to create zip file').'<br />';
-                        }
                         if ($copyresult) {
                             $discussion .= '<span class="LC_error">'.
                                            &mt('The following errors occurred during export:').
@@ -790,60 +773,33 @@ END
                           &mt('This discussion is closed.').'</span>';
         }
     } elsif ($outputtarget ne 'tex') {
-        unless ($nofooter) {
-            $discussion.='<div class="LC_feedback_link">';
-            unless ($nodisclink) {
-                if (&discussion_open($status) &&
-                    &Apache::lonnet::allowed('pch',
-    	                $env{'request.course.id'}.
-	                ($env{'request.course.sec'}?'/'.$env{'request.course.sec'}:''))) {
-                    $discussion.= &send_feedback_link($ressymb);
-                    if ($env{'request.role.adv'}) {
-                        my $close = &Apache::lonnet::EXT('resource.0.discussend',$ressymb);
-                        my $canvote = &Apache::lonnet::EXT('resource.0.discussvote',$ressymb);
-                        if (defined($close) && $close ne '' && $close < time) {
-                            if ($canvote eq 'notended') {
-                                $discussion .= '&nbsp;'.&mt('(Posting and voting closed for [_1] roles)',
-                                                            &Apache::lonnet::plaintext('st',$crstype));
-                            } else {
-                                $discussion .= '&nbsp;'.&mt('(Closed for [_1] roles)',
-                                                            &Apache::lonnet::plaintext('st',$crstype));
-                            }
-                        }
+        $discussion.='<div class="LC_feedback_link">';
+        if (&discussion_open($status) &&
+            &Apache::lonnet::allowed('pch',
+    	        $env{'request.course.id'}.
+	        ($env{'request.course.sec'}?'/'.$env{'request.course.sec'}:''))) {
+            $discussion.= &send_feedback_link($ressymb);
+            if ($env{'request.role.adv'}) {
+                my $close = &Apache::lonnet::EXT('resource.0.discussend',$ressymb);
+                my $canvote = &Apache::lonnet::EXT('resource.0.discussvote',$ressymb);
+                if (defined($close) && $close ne '' && $close < time) {
+                    if ($canvote eq 'notended') {
+                        $discussion .= '&nbsp;'.&mt('(Posting and voting closed for [_1] roles)',
+                                                    &Apache::lonnet::plaintext('st',$crstype));
+                    } else {
+                        $discussion .= '&nbsp;'.&mt('(Closed for [_1] roles)',
+                                                    &Apache::lonnet::plaintext('st',$crstype));
                     }
-	        } else {
-                    $discussion.= '<span class="LC_feedback_link">'.&mt('This discussion is closed.').'</span>';
                 }
             }
-            unless ($nofdbklink) {
-	        $discussion.= &send_message_link($ressymb);
-            }
-            $discussion.='</div>';
+	} else {
+            $discussion.= '<span class="LC_feedback_link">'.&mt('This discussion is closed.').'</span>';
         }
+	$discussion.= &send_message_link($ressymb).'</div>';
     }
     return $discussion;
 }
 
-sub check_menucoll {
-    my ($nofooter,$nodisclink,$nofdbklink);
-    my ($menucoll,$deeplinkmenu,$menuref) = &Apache::loncommon::menucoll_in_effect();
-    if ($menucoll) {
-        if (ref($menuref) eq 'HASH') {
-            if ($menuref->{'foot'} eq 'n') {
-                $nofooter = 1;
-            } else {
-                unless ($menuref->{'disc'}) {
-                    $nodisclink = 1;
-                }
-                unless ($menuref->{'fdbk'}) {
-                    $nofdbklink = 1;
-                }
-            }
-        }
-    }
-    return ($nofooter,$nodisclink,$nofdbklink);
-}
-
 sub can_see_hidden {
     my ($mode,$ressymb,$feedurl,$group,$cdom,$cnum,$crs) = @_;
     my $seehidden;
@@ -898,7 +854,7 @@ sub send_feedback_link {
                  &discussion_link($ressymb,
                     '<img alt="" class="LC_noBorder" src="'.
                     &Apache::loncommon::lonhttpdurl('/adm/lonMisc/chat.gif').
-                    '" /><span class="LC_menubuttons_inline_text">'.&mt('Post Discussion').'</span>',
+                    '" border="0" /><span class="LC_menubuttons_inline_text">'.&mt('Post Discussion').'</span>',
                     'replydisc').
            '</span>';
 }
@@ -909,7 +865,7 @@ sub send_message_link {
                  &discussion_link($ressymb,
                     '<img alt="" class="LC_noBorder" src="'.
                     &Apache::loncommon::lonhttpdurl('/res/adm/pages/feedback.png').
-                    '" /><span class="LC_menubuttons_inline_text">'.&mt('Send Feedback').'</span>',
+                    '" border="0" /><span class="LC_menubuttons_inline_text">'.&mt('Send Feedback').'</span>',
                     'sendmessageonly').
                  '</span>';
     return $output;
@@ -1003,9 +959,14 @@ sub postingform_display {
             }
         }
     }
+    my $postanon;
+    if (&Apache::lonnet::allowed('pac',$env{'request.course.id'}.
+       ($env{'request.course.sec'}?'/'.$env{'request.course.sec'}:''))) {
+        $postanon = '<input type="submit" name="anondiscuss" value="'.$lt{'poan'}.'" />';
+    }
     $postingform .= (<<ENDDISCUSS);
 <form action="/adm/feedback" method="post" name="mailform" enctype="multipart/form-data"> <input type="submit" name="discuss" value="$lt{'podi'}" />
-<input type="submit" name="anondiscuss" value="$lt{'poan'}" /> <input type="hidden" name="symb" value="$ressymb" />
+$postanon<input type="hidden" name="symb" value="$ressymb" />
 <input type="hidden" name="sendit" value="true" />
 <input type="hidden" name="timestamp" value="$now" />
 <a name="newpost"></a>
@@ -1819,11 +1780,14 @@ sub mail_screen {
 
   my %lt = &Apache::lonlocal::texthash(
             'myqu' => 'Question/comment/feedback:',
+            'title' => 'Title',
             'reta' => 'Retained attachments',
             'atta' => 'Attachment',
            );
-  if ($env{'form.editdisc'} || $env{'form.replydisc'}){
-      $lt{'myqu'} = &mt('Post Discussion');
+  if($env{'form.editdisc'} || $env{'form.replydisc'}){
+    %lt = &Apache::lonlocal::texthash(
+            'myqu' => 'Post Discussion',
+    );
   }
   my $restitle = &get_resource_title($caller_symb,$feedurl);
   my $quote='';
@@ -2098,8 +2062,8 @@ END
     } else {
         $r->print(<<END);
 <p>
-$lt{'atta'} $attachmaxtext: <input type="file" name="attachment" class="LC_flUpload" />
-<input type="hidden" id="LC_free_space" value="131072" />
+$lt{'atta'} $attachmaxtext: <input type="file" name="attachment" class="flUpload" />
+<input type="hidden" id="free_space" value="131072" />
 </p>
 END
     }
@@ -2110,7 +2074,7 @@ END
         $r->print('<input type="hidden" name="ref" value="'.$env{'form.ref'}.'" />');
     }
     $r->print(<<END);
-<input type="hidden" name="sendit" value="1" />
+<input type="hidden" name="sendit" value="1" /><br /><br />
 <input type="button" value="$send" onclick='gosubmit();' />
 </form>
 END
@@ -2985,12 +2949,16 @@ sub screen_header {
 				      $env{'request.course.id'}.
 				      ($env{'request.course.sec'}?'/'.$env{'request.course.sec'}:'')) || 
             (($group ne '') && ($symb =~ m{^bulletin___\d+___adm/wrapper/adm/\Q$cdom\E/\Q$cnum\E/\d+/bulletinboard$}) && (&check_group_priv($group,'pgd') eq 'ok')))) {
-	    $discussoptions='<label><input type="radio" name="discuss" value="nonanon" checked="checked" /> '.
-		$contribdisc.
-	        '</label><br /><label><input type="radio" name="discuss" value="anon" /> '.
-		$anoncontrib.
-		' <i>('.$namevis.')</i></label> '.
-		'<a href="/adm/preferences?action=changescreenname">'.&mt('Change Screenname').'</a>';
+            $discussoptions='<label><input type="radio" name="discuss" value="nonanon" checked="checked" /> '.
+                            $contribdisc.
+                            '</label>'; 
+            if (&Apache::lonnet::allowed('pac',$env{'request.course.id'}.
+               ($env{'request.course.sec'}?'/'.$env{'request.course.sec'}:''))) {
+	        $discussoptions .= '<br /><label><input type="radio" name="discuss" value="anon" /> '.
+		                   $anoncontrib.
+		                   ' <i>('.$namevis.')</i></label> '.
+		                   '<a href="/adm/preferences?action=changescreenname">'.&mt('Change Screenname').'</a>';
+            }
             my $blockblog = &Apache::loncommon::blocking_status('blogs');
             if (!$blockblog) {
                 $discussoptions.= &add_blog_checkbox($crstype);
@@ -3213,7 +3181,7 @@ sub adddiscuss {
     if (($symb) && ($email)) {
         my $now = time;
         if ($env{'form.editdisc'}) {
-            $contrib{'ip'}=&Apache::lonnet::get_requestor_ip();
+            $contrib{'ip'}=$ENV{'REMOTE_ADDR'};
             $contrib{'host'}=$Apache::lonnet::perlvar{'lonHostID'};
             $contrib{'timestamp'} = $now;
             $contrib{'history'} = '';
@@ -3547,7 +3515,7 @@ sub modify_attachments {
    document.modattachments.action = document.modattachments.origpage.value;
    document.modattachments.submit();
  }
-</script>
+</script> 
 <script type="text/javascript" src="/res/adm/includes/file_upload.js"></script>
 END
 
@@ -3590,8 +3558,8 @@ END
     $r->print('<b>'.$subject.'</b>');
     $r->print(&Apache::lonhtmlcommon::row_closure());
     $r->print(&Apache::lonhtmlcommon::row_title($lt{'adda'}));
-    $r->print('<input type="file" name="addnewattach" class="LC_flUpload" />'
-        .'<input type="hidden" id="LC_free_space" value="131072" />'
+    $r->print('<input type="file" name="addnewattach" class="flUpload" />'
+        .'<input type="hidden" id="free_space" value="131072" />'
         .'<input type="button" name="upload" value="Upload" '
         .'onclick="this.form.submit()" />  '.$attachmaxtext);
     if(($idx)||(ref($currnewattach) eq 'ARRAY') && (@{$currnewattach} > 0)){
@@ -4583,8 +4551,6 @@ ENDREDIR
 	  ($env{'request.course.id'} && ($feedurl!~m:^/adm:))
 	  ||
 	  ($env{'request.course.id'} && ($symb=~/^bulletin\_\_\_/))
-          ||
-          (($env{'request.course.id'}) && ($feedurl =~ /ext\.tool$/))
 	  ) {
 	  &Apache::loncommon::content_type($r,'text/html');
 	  $r->send_http_header;
@@ -4686,7 +4652,11 @@ ENDREDIR
 	     && $env{'form.discuss'} !~ /^(?:author|question|course|policy)/)
 	   || $env{'form.anondiscuss'} ne '') {
 	  my $subject = &clear_out_html($env{'form.subject'});
-	  my $anonmode=($env{'form.discuss'} eq 'anon' || $env{'form.anondiscuss'} );
+          my $anonmode;
+          if (&Apache::lonnet::allowed('pac',$env{'request.course.id'}.
+             ($env{'request.course.sec'}?'/'.$env{'request.course.sec'}:''))) {
+	      $anonmode=($env{'form.discuss'} eq 'anon' || $env{'form.anondiscuss'} );
+          }
 	  $typestyle.=&adddiscuss($symb,$message,$anonmode,$attachmenturl,
 				  $subject,$group);
 	  $numpost++;