--- loncom/interface/lonfeedback.pm	2022/11/14 15:12:30	1.370.2.5.2.3
+++ loncom/interface/lonfeedback.pm	2022/01/18 16:55:30	1.388
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Feedback
 #
-# $Id: lonfeedback.pm,v 1.370.2.5.2.3 2022/11/14 15:12:30 raeburn Exp $
+# $Id: lonfeedback.pm,v 1.388 2022/01/18 16:55:30 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -1003,9 +1003,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>
@@ -1307,8 +1312,9 @@ sub build_posting_display {
                                 @{$$usernamesort{$contrib{$idx.':senderdomain'}}{$contrib{$idx.':sendername'}}} = ("$idx");
                             }
 # Set up for sorting by last name, then first name
-                            my %names = &Apache::loncommon::getnames($contrib{$idx.':sendername'},
-                                                                     $contrib{$idx.':senderdomain'});
+                            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 '') {
@@ -2109,7 +2115,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
@@ -2757,7 +2763,7 @@ sub fail_redirect {
   my $logo=&Apache::loncommon::lonhttpdurl('/adm/lonIcons/lonlogos.gif');
   my %parms=('only_body'   => 1);
   if ($delay !~ /^\d+(|\.\d+)$/) {
-      $delay = 2;
+      $delay = 0;
   }
   if ($env{'form.modal'}) {
       my $onload = 'document.forms.reldt.submit()';
@@ -2773,6 +2779,9 @@ sub fail_redirect {
   }
   $r->print(&Apache::loncommon::start_page('Feedback not sent',undef,\%parms));
   my $windowname = 'loncapaclient';
+  if ($env{'request.lti.login'}) {
+     $windowname .= 'lti';
+  }
   $r->print(<<ENDFAILREDIR);
 <img align="right" src="$logo" />
 <p class="LC_warning">$lt{'sorr'}</p>
@@ -2784,7 +2793,7 @@ ENDFAILREDIR
 
 sub redirect_back {
   my ($r,$feedurl,$typestyle,$sendsomething,$sendposts,$blog,$status,$previous,$sort,
-      $rolefilter,$statusfilter,$sectionpick,$grouppick,$numpicks,$group,$toolarge,$delay) = @_;
+      $rolefilter,$statusfilter,$sectionpick,$grouppick,$numpicks,$group,$toolarge) = @_;
   my $sorttag = '';
   my $roletag = '';
   my $statustag = '';
@@ -2883,24 +2892,19 @@ sub redirect_back {
   &Apache::lonenc::check_encrypt(\$feedurl);
   my $logo=&Apache::loncommon::lonhttpdurl('/adm/lonIcons/lonlogos.gif');
   my %parms=('only_body'   => 1);
-  if ($delay !~ /^\d+(|\.\d+)$/) {
-      $delay = 0;
-  }
   if ($env{'form.modal'}) {
       my $onload = 'document.forms.reldt.submit()';
-      if ($delay) {
-          my $js_delay = int(1000 * $delay);
-          $onload = "setTimeout(function(){
-                        document.forms.reldt.submit();
-                     },$js_delay);";
-      }
       $parms{'add_entries'}={'onload' => $onload};
   } else {
-      $parms{'redirect'}=[$delay,$feedurl];
+      $parms{'redirect'}=[0,$feedurl];
   }
   my $start_page=
       &Apache::loncommon::start_page('Feedback sent',undef,\%parms);
   my $end_page = &Apache::loncommon::end_page();
+  my $windowname = 'loncapaclient';
+  if ($env{'request.lti.login'}) {
+     $windowname .= 'lti';
+  }
   $r->print(<<ENDREDIR);
 $start_page
 <img align="right" src="$logo" />
@@ -2909,7 +2913,7 @@ $typestyle
 $blog
 $toolarge
 <font color="red">$status</font>
-<form name="reldt" action="$feedurl" target="loncapaclient">
+<form name="reldt" action="$feedurl" target="$windowname">
 $prevtag
 $sorttag
 $statustag
@@ -2942,7 +2946,10 @@ sub no_redirect_back {
                          },$js_delay);";
           }
           $parms{'add_entries'}={'onload' => $onload};
-          my $windowname = 'loncapaclient'; 
+          my $windowname = 'loncapaclient';
+          if ($env{'request.lti.login'}) {
+              $windowname .= 'lti';
+          }
           $form_for_modal = <<ENDFORM;
 <form name="reldt" action="$feedurl" target="$windowname">
 </form>
@@ -2953,6 +2960,7 @@ ENDFORM
           $parms{'redirect'}=[$delay,$feedurl];
       }
   }
+ 
   my $start_page=
       &Apache::loncommon::start_page('Feedback not sent',undef,
 				     \%parms);
@@ -2976,7 +2984,7 @@ sub screen_header {
                     question => 'Question about resource content',
                     comment => 'Question/Comment/Feedback about course content',
                     policy => 'Question/Comment/Feedback about course policy',
-                  );
+                  );  
     my $contribdisc = &mt('Contribution to course discussion of resource');
     my $anoncontrib = &mt('Anonymous contribution to course discussion of resource');
     my $namevis = &mt('name only visible to course faculty');
@@ -2985,7 +2993,7 @@ sub screen_header {
         $crstype = &Apache::loncommon::course_type();
         if ($crstype eq 'Community') {
             $default{'comment'} = &mt('Question/Comment/Feedback about community content');
-            $default{'policy'} = &mt('Question/Comment/Feedback about community policy');
+            $default{'policy'} = &mt('Question/Comment/Feedback about community policy'); 
             $contribdisc = &mt('Contribution to community discussion of resource');
             $anoncontrib = &mt('Anonymous contribution to community discussion of resource');
             $namevis = &mt('name only visible to community facilitators');
@@ -3018,10 +3026,13 @@ sub screen_header {
         if (!$blocked && &discussion_open(undef,$realsymb) &&
             (&Apache::lonnet::allowed('pch',
                                       $env{'request.course.id'}.
-                                      ($env{'request.course.sec'}?'/'.$env{'request.course.sec'}:'')) || 
+                                      ($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')))) {
             $discoptions{'nonanon'} = 1;
-            $discoptions{'anon'} = 1;
+            if (&Apache::lonnet::allowed('pac',$env{'request.course.id'}.
+               ($env{'request.course.sec'}?'/'.$env{'request.course.sec'}:''))) {
+                $discoptions{'anon'} = 1;
+            }
             $blockblog = &Apache::loncommon::blocking_status('blogs');
         }
     }
@@ -3032,10 +3043,10 @@ sub screen_header {
     }
     if (keys(%fdbkoptions)) {
         if ($fdbkoptions{'author'}) {
-            $msgoptions =
-                '<label><input type="radio" name="discuss" value="author"'.$checkradio.' /> '.
-                &mt('Feedback to resource author').'</label><br />';
-        }
+	    $msgoptions =
+		'<label><input type="radio" name="discuss" value="author"'.$checkradio.' /> '.
+		&mt('Feedback to resource author').'</label><br />';
+	}
         foreach my $item ('question','comment','policy') {
             my $type = $item;
             if ($item eq 'comment') {
@@ -3045,8 +3056,8 @@ sub screen_header {
             if ($fdbkoptions{$type}) {
                 $msgoptions .=
                     '<label><input type="radio" name="discuss" value="'.$type.'"'.$checkradio.' />'.
-                    ($optionhash?$optionhash:$default{$item}).'</label><br />';
-            }
+                    ($optionhash?$optionhash:$default{$item}).'</label><br />'; 
+	    }
         }
     }
     if (keys(%discoptions)) {
@@ -3150,17 +3161,9 @@ sub feedback_available {
 }
 
 sub send_msg {
-    my ($title,$feedurl,$email,$citations,$attachmenturl,$symb,$clientip,%to)=@_;
+    my ($title,$feedurl,$email,$citations,$attachmenturl,$symb,%to)=@_;
     my $status='';
     my $sendsomething=0;
-    my $delay;
-    my $senthide;
-    my %setters;
-    my ($startblock,$endblock,$triggerblock,$by_ip,$blockdom) =
-        &Apache::loncommon::blockcheck(\%setters,'com',$clientip);
-    if ($by_ip) {
-        $senthide = 1;
-    }
     my $restitle = &get_resource_title($symb,$feedurl);
     if ($title=~/^Error/) { $title=&mt('Feedback').': '.$title; }
     unless ($title=~/\w/) { $title=&mt('Feedback'); }
@@ -3174,8 +3177,7 @@ sub send_msg {
 	    } else {
 		unless (&Apache::lonmsg::user_normal_msg($user,$domain,
 							 $title.' ['.$restitle.']',$email,$citations,$feedurl,
-							 $attachmenturl,undef,undef,$symb,$restitle,undef,
-                                                         undef,undef,undef,$senthide)=~/ok/) {
+							 $attachmenturl,undef,undef,$symb,$restitle)=~/ok/) {
 		    $status.='<br />'.&mt('Error sending message to').' '.$key.'<br />';
 		} else {
 		    $sendsomething++;
@@ -3183,24 +3185,6 @@ sub send_msg {
 	    }
 	}
     }
-    if ($sendsomething && $senthide) {
-        if ($by_ip) {
-            my $showdom = &Apache::lonnet::domain($blockdom);
-            if ($showdom eq '') {
-                $showdom = $blockdom;
-            }
-            $delay = 4;
-            $status.='<br />'.&mt("Message content of feedback you send to instructor(s) from your current IP address: [_1] will be unavailable in your 'Sent' folder.",$clientip).
-                     '<ul><li>'.
-                     &mt('This does not affect delivery of feedback to your instructor(s).').
-                     '</li><li>'.
-                     &mt('Note: some types of communication functionality are blocked for certain IP address(es).').
-                     '</li><li>'.
-                     &mt('This restriction was set by an administrator in the [_1] LON-CAPA domain.',
-                         $showdom).
-                     '</li></ul><br />';
-        }
-    }
 
 # Records of number of feedback messages are kept under the "symb" called "_feedback"
 # There are two entries within the framework of a course:
@@ -3218,7 +3202,7 @@ sub send_msg {
 	    }
         }
     }
-    return ($status,$sendsomething,$delay);
+    return ($status,$sendsomething);
 }
 
 # Routine to get the complete feedback records
@@ -3642,7 +3626,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
 
@@ -4174,11 +4158,15 @@ sub handler {
 					  'only_body'   => 1,
 					  'add_entries' => \%onload});
       my $end_page = &Apache::loncommon::end_page();
+      my $windowname = 'loncapaclient';
+      if ($env{'request.lti.login'}) {
+          $windowname .= 'lti';
+      }
       $r->print (<<ENDREDIR);
 $start_page
 <img align="right" src="$logo" />
 $textline
-<form name="reldt" action="$feedurl" target="loncapaclient">
+<form name="reldt" action="$feedurl" target="$windowname">
 </form>
 <br />
 $end_page
@@ -4712,7 +4700,7 @@ ENDREDIR
 	  if ($options) {
 	      &mail_screen($r,$feedurl,$options,$symb,$attachmax{'text'});
 	  } else {
-	      &fail_redirect($r,$feedurl,2);
+	      &fail_redirect($r,$feedurl,1);
 	  }
 	  return OK;
       }
@@ -4778,10 +4766,9 @@ ENDREDIR
       my ($typestyle,%to) = &Apache::lonmsg::decide_receiver($feedurl);
 
 # Actually send mail
-      my $clientip = &Apache::lonnet::get_requestor_ip($r);
-      my ($status,$numsent,$delay)=&send_msg(&clear_out_html($env{'form.subject'}),
-				             $feedurl,$email,$citations,
-				             $attachmenturl,$usersymb,$clientip,%to);
+      my ($status,$numsent)=&send_msg(&clear_out_html($env{'form.subject'}),
+				      $feedurl,$email,$citations,
+				      $attachmenturl,$usersymb,%to);
 
 # Discussion? Store that.
       my $numpost=0;
@@ -4789,7 +4776,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++;
@@ -4813,7 +4804,7 @@ ENDREDIR
 	  
 # Receipt screen and redirect back to where came from
       &redirect_back($r,$feedurl,$typestyle,$numsent,$numpost,$blog,$status,$env{'form.previous'},
-                     undef,undef,undef,undef,undef,undef,$group,$toolarge,$delay);
+                     undef,undef,undef,undef,undef,undef,$group,$toolarge);
   }
   return OK;
 }