--- loncom/interface/lonsupportreq.pm 2022/01/01 19:55:31 1.79.2.9 +++ loncom/interface/lonsupportreq.pm 2015/06/23 02:42:34 1.81 @@ -1,7 +1,5 @@ -# The LearningOnline Network with CAPA -# Helpdesk request form # -# $Id: lonsupportreq.pm,v 1.79.2.9 2022/01/01 19:55:31 raeburn Exp $ +# $Id: lonsupportreq.pm,v 1.81 2015/06/23 02:42:34 musolffc Exp $ # # Copyright Michigan State University Board of Trustees # @@ -29,8 +27,6 @@ package Apache::lonsupportreq; use strict; -use MIME::Types; -use MIME::Lite; use CGI::Cookie(); use Apache::Constants qw(:common); use Apache::loncommon(); @@ -69,9 +65,7 @@ sub handler { } } my $origurl = $env{'form.origurl'}; - if ($origurl =~ m{^https?://[^/]+(.*)$}) { - $origurl = $1; - } + $origurl =~ s{^https?://}{}; $origurl =~ s/(`)//g; $origurl =~ s/\$/\(\$\)/g; my $command = $env{'form.command'}; @@ -83,18 +77,18 @@ sub handler { } return OK; } - + sub print_request_form { my ($r,$origurl,$function) = @_; - my ($os,$browser,$bversion,$uname,$udom,$uhome,$urole,$usec,$email,$cid, + my ($os,$browser,$bversion,$uhost,$uname,$udom,$uhome,$urole,$usec,$email,$cid, $cdom,$cnum,$ctitle,$ccode,$sectionlist,$lastname,$firstname,$server, - $formname,$public,$homeserver,$knownuser,$captcha_form,$captcha_error, - $captcha,$recaptcha_version,$extra_validations,%groupid); + $formname,$public,$homeserver); $function = &Apache::loncommon::get_users_function() if (!$function); $ccode = ''; $os = $env{'browser.os'}; $browser = $env{'browser.type'}; $bversion = $env{'browser.version'}; + $uhost = $env{'request.host'}; if (($env{'user.name'} eq 'public') && ($env{'user.domain'} eq 'public')) { $public = 1; } else { @@ -109,118 +103,13 @@ sub print_request_form { } } } - if (($env{'user.name'} =~ /^$match_username$/) && - ($env{'user.domain'} =~ /^$match_domain$/) && (!$public)) { - $knownuser = 1; - } else { - my $lonhost = $r->dir_config('lonHostID'); - ($captcha_form,$captcha_error,$captcha,$recaptcha_version) = - &Apache::loncommon::captcha_display('login',$lonhost); - } - &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['codedom', - 'useremail','useraccount']); - if ($homeserver) { $uhome = $env{'user.home'}; $urole = $env{'request.role'}; $usec = $env{'request.course.sec'}; $cid = $env{'request.course.id'}; } - if ($cid) { - $cdom = $env{'course.'.$cid.'.domain'}; - $cnum = $env{'course.'.$cid.'.num'}; - } - if ($cdom && $cnum) { - my %csettings = &Apache::lonnet::get('environment',['description','internal.coursecode','internal.sectionnums'],$cdom,$cnum); - $ctitle = $csettings{'description'}; - $ccode = $csettings{'internal.coursecode'}; - $sectionlist = $csettings{'internal.sectionnums'}; - my @sections = split(/,/,$sectionlist); - foreach my $section (@sections) { - my ($sec,$grp) = split(/:/,$section); - $groupid{$sec} = $grp; - } - } - $formname = 'logproblem'; - my $codedom = &get_domain(); - my %helpform; - my %domconfig = - &Apache::lonnet::get_dom('configuration',['contacts'],$codedom); - if (ref($domconfig{'contacts'}) eq 'HASH') { - if (ref($domconfig{'contacts'}{'helpform'}) eq 'HASH') { - %helpform = %{$domconfig{'contacts'}{'helpform'}}; - } - } - - my %codes; - my @codetitles; - my %cat_titles; - my %cat_order; - my %idlist; - my %idnums; - my %idlist_titles; - my $totcodes = 0; - my $jscript = ''; - my $loaditems = qq| -function initialize_codes() { - return; -} - |; - unless ($helpform{'course'} eq 'no') { - my $instcats = &Apache::lonnet::get_dom_instcats($codedom); - if (ref($instcats) eq 'HASH') { - if ((ref($instcats->{'codetitles'}) eq 'ARRAY') && (ref($instcats->{'codes'}) eq 'HASH') && - (ref($instcats->{'cat_titles'}) eq 'HASH') && (ref($instcats->{'cat_order'}) eq 'HASH')) { - %codes = %{$instcats->{'codes'}}; - @codetitles = @{$instcats->{'codetitles'}}; - %cat_titles = %{$instcats->{'cat_titles'}}; - %cat_order = %{$instcats->{'cat_order'}}; - $totcodes = scalar(keys(%codes)); - } - if ($totcodes > 0) { - my $numtypes = @codetitles; - &Apache::courseclassifier::build_code_selections(\%codes,\@codetitles,\%cat_titles,\%cat_order,\%idlist,\%idnums,\%idlist_titles); - my ($scripttext,$longtitles) = &Apache::courseclassifier::javascript_definitions(\@codetitles,\%idlist,\%idlist_titles,\%idnums,\%cat_titles); - my $longtitles_str = join('","',@{$longtitles}); - my $allidlist = $idlist{$codetitles[0]}; - my $knowncodes; - if (($ccode ne '') && ($cnum ne '')) { - my (%singlecodes,@singlecodetitles,%singlecat_titles,%singlecat_order); - my $crscode_reply = &Apache::lonnet::auto_instcode_format($cnum,$codedom,{ $cnum => $ccode },\%singlecodes, - \@singlecodetitles,\%singlecat_titles,\%singlecat_order); - if (ref($singlecodes{$cnum}) eq 'HASH') { - $knowncodes = $singlecodes{$cnum}; - } - } - $jscript .= &Apache::courseclassifier::courseset_js_start($formname,$longtitles_str,$allidlist); - $jscript .= $scripttext; - $jscript .= &Apache::courseclassifier::javascript_code_selections($formname,\@codetitles,$knowncodes,$sectionlist); - $loaditems = ''; - } - } - } - my %missing = &Apache::lonlocal::texthash( - username => 'You must include a name.', - phone => 'You must include a phone.', - ); - &js_escape(\%missing); - if (keys(%helpform)) { - foreach my $key (keys(%helpform)) { - if (($helpform{$key} eq 'req') && ($missing{$key} ne '')) { - next if (($key eq 'username') && (($knownuser) || - (($env{'user.name'} =~ /^$match_username$/) && (!$public)))); - $extra_validations .= <<"END"; - - if (document.$formname.$key.value == '') { - alert("$missing{$key}"); - return; - } - -END - } - } - } my $machine = &Apache::lonnet::absolute_url(); my $sourceurl = $machine.$origurl; $server = $machine.&Apache::loncommon::cleanup_html($origurl); @@ -235,7 +124,7 @@ END name => 'Name', subm => 'Submit Request', emad => 'Your e-mail address', - emac => 'Cc', + emac => 'Cc', unme => 'username', doma => 'domain', entu => 'Enter the username you use to log-in to LON-CAPA', @@ -254,7 +143,7 @@ END subj => 'Subject', detd => 'Detailed Description', opfi => 'Optional file upload', - uplf => 'Upload a file (e.g., a screenshot) relevant to your help request', + uplf => 'Upload a file (e.g., a screenshot) relevant to your help request (1 MB max.)', fini => 'Finish', clfm => 'Clear Form', ); @@ -274,12 +163,21 @@ function validate() { alert("$js_lt{'rdes'}."); return; } - $extra_validations document.logproblem.submit(); } END $scripttag .= &Apache::lonhtmlcommon::javascript_valid_email(); + if ($cid) { + $cdom = $env{'course.'.$cid.'.domain'}; + $cnum = $env{'course.'.$cid.'.num'}; + } + if ($cdom && $cnum) { + my %csettings = &Apache::lonnet::get('environment',['description','internal.coursecode','internal.sectionnums'],$cdom,$cnum); + $ctitle = $csettings{'description'}; + $ccode = $csettings{'internal.coursecode'}; + $sectionlist = $csettings{'internal.sectionnums'}; + } if ($homeserver) { if ($env{'environment.permanentemail'}) { @@ -296,6 +194,14 @@ END $firstname = $env{'environment.firstname'}; } } + my @sections = split(/,/,$sectionlist); + my %groupid; + foreach my $section (@sections) { + my ($sec,$grp) = split(/:/,$section); + $groupid{$sec} = $grp; + } + &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['codedom', + 'useremail','useraccount']); if ($env{'form.origurl'} eq '/adm/createaccount') { if ($email eq '') { if ($env{'form.useremail'} =~ /^[^\@]+\@[^\@]+$/) { @@ -308,11 +214,57 @@ END } } } + my $codedom = &get_domain(); my $details_title; if ($codedom) { $details_title = '
('.$codedom.')'; } - + my %coursecodes; + my %codes; + my @codetitles; + my %cat_titles; + my %cat_order; + my %idlist; + my %idnums; + my %idlist_titles; + my $caller = 'global'; + my $totcodes = 0; + my $format_reply; + my $jscript = ''; + my $loaditems = qq| +function initialize_codes() { + return; +} + |; + if ($cnum) { + $coursecodes{$cnum} = $ccode; + if ($ccode eq '') { + $totcodes = &Apache::courseclassifier::retrieve_instcodes(\%coursecodes,$codedom,$totcodes); + } else { + $coursecodes{$cnum} = $ccode; + $caller = $cnum; + $totcodes ++; + } + } else { + $totcodes = &Apache::courseclassifier::retrieve_instcodes(\%coursecodes,$codedom,$totcodes); + } + if ($totcodes > 0) { + if ($ccode eq '') { + $format_reply = &Apache::lonnet::auto_instcode_format($caller,$codedom,\%coursecodes,\%codes,\@codetitles,\%cat_titles,\%cat_order); + if ($format_reply eq 'ok') { + my $numtypes = @codetitles; + &Apache::courseclassifier::build_code_selections(\%codes,\@codetitles,\%cat_titles,\%cat_order,\%idlist,\%idnums,\%idlist_titles); + my ($scripttext,$longtitles) = &Apache::courseclassifier::javascript_definitions(\@codetitles,\%idlist,\%idlist_titles,\%idnums,\%cat_titles); + my $longtitles_str = join('","',@{$longtitles}); + my $allidlist = $idlist{$codetitles[0]}; + $jscript .= &Apache::courseclassifier::courseset_js_start($formname,$longtitles_str,$allidlist); + $jscript .= $scripttext; + $jscript .= &Apache::courseclassifier::javascript_code_selections($formname,@codetitles); + $loaditems = ''; + } + } + } + my $js = <<"ENDJS"; + ENDJS - if ($knownuser) { - $js .="\n".''; - } elsif ($recaptcha_version >=2) { - $js.= "\n".''."\n"; - } my %add_entries = ( style => "margin-top:0px;margin-bottom:0px;", onload => "initialize_codes();", @@ -347,117 +295,107 @@ ENDJS if ($homeserver) { $formtype = ' enctype="multipart/form-data"'; } - my $topsubmit = '      '; - my $shownsubmit; $r->print('
'."\n"); my $output = &Apache::lonhtmlcommon::start_pick_box(). &Apache::lonhtmlcommon::row_headline(). ''. &mt('(All fields marked with * are required.)'). ''. - &Apache::lonhtmlcommon::row_closure(); - unless ($helpform{'username'} eq 'no') { - my ($reqd,$namefield,$fullname); - if ((defined($lastname) && $lastname ne '') && (defined($firstname) && $firstname ne '')) { - $fullname = "$firstname $lastname"; - $namefield = $fullname.'&').'" />'."\n"; - } else { - if (defined($firstname) && $firstname ne '') { - $fullname = $firstname; - } elsif (defined($lastname) && $lastname ne '') { - $fullname = " $lastname"; - } - $namefield = '&').'" />'."\n"; - if ($helpform{'username'} eq 'req') { - $reqd = '*'; - } + &Apache::lonhtmlcommon::row_closure(). + &Apache::lonhtmlcommon::row_title($html_lt{'name'},undef,$css[$num])."\n"; + my $fullname = ''; + if ((defined($lastname) && $lastname ne '') && (defined($firstname) && $firstname ne '')) { + $fullname = "$firstname $lastname"; + $output .= $fullname.'&').'" />'."\n"; + } else { + if (defined($firstname) && $firstname ne '') { + $fullname = $firstname; + } elsif (defined($lastname) && $lastname ne '') { + $fullname = " $lastname"; } - $output .= &Apache::lonhtmlcommon::row_title($html_lt{'name'}.$reqd,undef,$css[$num])."\n".$namefield. - $topsubmit. - &Apache::lonhtmlcommon::row_closure()."\n"; - $shownsubmit = 1; - $num ++; - $i = $num%2; + $output .= '&').'" />'."\n"; } + $output .= '      '. + &Apache::lonhtmlcommon::row_closure()."\n"; + $num ++; + $i = $num%2; $output .= &Apache::lonhtmlcommon::row_title( ''. $html_lt{'emad'}.' *' ,undef,$css[$i]). '&').'" />'."\n"; - unless ($shownsubmit) { - $output .= $topsubmit; - } - $output .= &Apache::lonhtmlcommon::row_closure(); + &HTML::Entities::encode($email,'"<>&').'" />
'."\n". + &Apache::lonhtmlcommon::row_closure(); $num ++; $i = $num%2; - if ($knownuser) { - if ($homeserver) { - unless ($helpform{'cc'} eq 'no') { - $output .= &Apache::lonhtmlcommon::row_title($html_lt{'emac'},undef,$css[$i]). - '
'."\n". - &Apache::lonhtmlcommon::row_closure(); - $num ++; - $i = $num%2; - } + if (($env{'user.name'} =~ /^$match_username$/) && (!$public)) { + if ($homeserver) { + $output .= &Apache::lonhtmlcommon::row_title($html_lt{'emac'},undef,$css[$i]). + '
'."\n". + &Apache::lonhtmlcommon::row_closure(); + $num ++; + $i = $num%2; } } - unless ($helpform{'user'} eq 'no') { - $output .= &Apache::lonhtmlcommon::row_title("$html_lt{'unme'}/$html_lt{'doma'}",undef,$css[$i]); - my $udom_input = '&').'" />'."\n"; - my $uname_input = '&').'" />'."\n"; - if ($knownuser) { - $output .= ''.$html_lt{'unme'}.': '.$uname.'  '.$html_lt{'doma'}.': '.$udom.$udom_input.$uname_input; + $output .= &Apache::lonhtmlcommon::row_title("$html_lt{'unme'}/$html_lt{'doma'}",undef,$css[$i]); + my $udom_input = '&').'" />'."\n"; + my $uname_input = '&').'" />'."\n"; + if (($env{'user.name'} =~ /^$match_username$/) && + ($env{'user.domain'} =~ /^$match_domain$/) && (!$public)) { + $output .= ''.$html_lt{'unme'}.': '.$uname.'  '.$html_lt{'doma'}.': '.$udom.$udom_input.$uname_input; + } else { + my $udomform = ''; + my $unameform = ''; + if (($env{'user.domain'} =~ /^$match_domain$/) && (!$public)) { + $output .= $html_lt{'entu'}; + } elsif (($env{'user.name'} =~ /^$match_username$/) && (!$public)) { + $output .= $html_lt{'chdo'}; } else { - my $udomform = ''; - my $unameform = ''; - if (($env{'user.domain'} =~ /^$match_domain$/) && (!$public)) { - $output .= $html_lt{'entu'}; - } elsif (($env{'user.name'} =~ /^$match_username$/) && (!$public)) { - $output .= $html_lt{'chdo'}; - } else { - $output .= $html_lt{'entr'}; - } - $output .= '
'."\n"; - if (!$public) { - if ($env{'user.domain'} =~ /^$match_domain$/) { - $udomform = ''.$html_lt{'doma'}.': '.$udom.$udom_input; - } elsif ($env{'user.name'} =~ /^$match_username$/) { - $unameform = ''.$html_lt{'unme'}.': '.$uname.'  '.$uname_input; - } - } - if ($udomform eq '') { - $udomform = ''.$html_lt{'doma'}.': '; - $udomform .= &Apache::loncommon::select_dom_form($codedom,'udom')."\n"; - } - if ($unameform eq '') { - $unameform= ''.$html_lt{'unme'}.':   '; + $output .= $html_lt{'entr'}; + } + $output .= '
'."\n"; + if (!$public) { + if ($env{'user.domain'} =~ /^$match_domain$/) { + $udomform = ''.$html_lt{'doma'}.': '.$udom.$udom_input; + } elsif ($env{'user.name'} =~ /^$match_username$/) { + $unameform = ''.$html_lt{'unme'}.': '.$uname.'  '.$uname_input; } - $output .= $unameform.$udomform; } - $output .= &Apache::lonhtmlcommon::row_closure(); - $num ++; + if ($udomform eq '') { + $udomform = ''.$html_lt{'doma'}.': '; + $udomform .= &Apache::loncommon::select_dom_form($codedom,'udom')."\n"; + } + if ($unameform eq '') { + $unameform= ''.$html_lt{'unme'}.':   '; + } + $output .= $unameform.$udomform; } + $output .= &Apache::lonhtmlcommon::row_closure(); + $num ++; $i = $num%2; $output .= &Apache::lonhtmlcommon::row_title("$html_lt{'urlp'}",undef,$css[$i]). $server."\n".'&').'" />'."\n". + &Apache::lonhtmlcommon::row_closure(). + &Apache::lonhtmlcommon::row_title("$html_lt{'phon'}",undef,'LC_evenrow_value'). + '
'."\n". &Apache::lonhtmlcommon::row_closure(); - unless ($helpform{'phone'} eq 'no') { - my $reqd; - if ($helpform{'phone'} eq 'req') { - $reqd = '*'; - } - $output .= &Apache::lonhtmlcommon::row_title($html_lt{'phon'}.$reqd,undef,'LC_evenrow_value'). - '
'."\n". - &Apache::lonhtmlcommon::row_closure(); - $num ++; - $i = $num%2; - } - unless ($helpform{'course'} eq 'no') { - $output .= &Apache::lonhtmlcommon::row_title("$html_lt{'crsd'}$details_title",undef,$css[$i]); + $num ++; + $i = $num%2; + $output .= &Apache::lonhtmlcommon::row_title("$html_lt{'crsd'}$details_title",undef,$css[$i]); + if ($cnum) { + if ($coursecodes{$cnum}) { + foreach my $item (@codetitles) { + $output .= ''.$item.': '.$codes{$cnum}{$item}.'; '; + } + $output .= ' &').'" />'."\n"; + } else { + $output .= $html_lt{'enin'}.':  + '."\n"; + } + } else { if ($totcodes > 0) { my $numtitles = @codetitles; if ($numtitles == 0) { @@ -468,13 +406,9 @@ ENDJS my $lasttitle = $numtitles; if ($numtitles > 4) { $lasttitle = 4; - } - my $onchange; - if ($sectionlist) { - $onchange = 'toggleSecVis()'; - } + } $output .= ''; for (my $i=1; $i<$numtitles; $i++) { $output .= ''."\n"; @@ -511,7 +445,7 @@ ENDJS $output .= '
'.$codetitles[0].'
'."\n". - ''."\n". ' \n"; my @items = (); my @longitems = (); @@ -503,7 +437,7 @@ ENDJS $output .= '		'.$codetitles[$i].'
'."\n". - ''."\n". ''."\n". ''."\n". '
'; if ($numtitles > 4) { $output .= '

'.$codetitles[$numtitles].'
'."\n". - ''."\n". ''."\n". ''."\n"; } @@ -520,43 +454,40 @@ ENDJS $output .= $html_lt{'enin'}.':  '."\n"; } - my $reqd; - if ($helpform{'course'} eq 'req') { - $reqd = '*'; - } - $output .= '
'.$html_lt{'enct'}.$reqd.':  - &').'" />'."\n"; - $output .= &Apache::lonhtmlcommon::row_closure(); - $num ++; - $i = $num%2; - } - unless ($helpform{'section'} eq 'no') { - $output .= &Apache::lonhtmlcommon::row_title($html_lt{'secn'},undef,$css[$i]); - if ($sectionlist) { - $output .= "
'."\n"; + } + $output .= &Apache::lonhtmlcommon::row_closure(); + $num ++; + $i = $num%2; + $output .= &Apache::lonhtmlcommon::row_title($html_lt{'secn'},undef,$css[$i]); + if ($sectionlist) { + $output .= "
'."\n". - ''."\n"; - } else { - $output .= ''."\n"; } - $output .= &Apache::lonhtmlcommon::row_closure(); - $num ++; - $i = $num%2; + $output .= ""; + } else { + $output .= ''."\n"; } + $output .= &Apache::lonhtmlcommon::row_closure(); + $num ++; + $i = $num%2; $output .= &Apache::lonhtmlcommon::row_title( ''. $html_lt{'subj'}.' *' @@ -572,33 +503,27 @@ ENDJS &Apache::lonhtmlcommon::row_closure(); $num ++; $i = $num%2; - if ($knownuser) { + if (($env{'user.name'} =~ /^$match_username$/) && (!$public)) { if ($homeserver) { - unless ($helpform{'screenshot'} eq 'no') { - my $max = 1048576; - my $showmax = 1.00; - if ($helpform{'maxsize'} =~ /^\d+\.\d*$/) { - $max *= $helpform{'maxsize'}; - $showmax = $helpform{'maxsize'}; - } - $showmax = ' ('.sprintf("%.2f",$showmax).' '.&mt('MB max.').')'; - $output .= &Apache::lonhtmlcommon::row_title($html_lt{'opfi'},undef,$css[$i]) - .' ' - .'' - .'
'."\n".$html_lt{'uplf'}.$showmax."\n" - .&Apache::lonhtmlcommon::row_closure(); - $num ++; - $i = $num%2; - } + $output .= &Apache::lonhtmlcommon::row_title($html_lt{'opfi'},undef,$css[$i]) + .' ' + .'' + .'
'."\n".$html_lt{'uplf'}."\n" + .&Apache::lonhtmlcommon::row_closure(); + $num ++; + $i = $num%2; } } else { - if ($captcha_form) { + my $lonhost = $r->dir_config('lonHostID'); + my ($captchaform,$error) = + &Apache::loncommon::captcha_display('login',$lonhost); + if ($captchaform) { $output .= &Apache::lonhtmlcommon::row_title( ''. &mt('Validation'). ' *' ,undef,$css[$i]). - $captcha_form."\n". + $captchaform."\n". &Apache::lonhtmlcommon::row_closure(); $num ++; $i = $num%2; @@ -636,8 +561,8 @@ sub print_request_receipt { if (($env{'user.name'} eq 'public') && ($env{'user.domain'} eq 'public')) { $public = 1; } - my $lonhost = $r->dir_config('lonHostID'); unless (($env{'user.name'} =~ /^$match_username$/) && (!$public)) { + my $lonhost = $r->dir_config('lonHostID'); my ($captcha_chk,$captcha_error) = &Apache::loncommon::captcha_response('login',$lonhost); if ($captcha_chk != 1) { @@ -663,80 +588,62 @@ sub print_request_receipt { return; } } - my $ip = &Apache::lonnet::get_requestor_ip(); - my @ENVvars = ('HTTP_HOST','HTTP_USER_AGENT','SERVER_ADDR','SERVER_NAME'); + my @ENVvars = ('HTTP_HOST','HTTP_USER_AGENT','REMOTE_ADDR','SERVER_ADDR','SERVER_NAME'); my @envvars = ('browser.os','browser.type','browser.version','user.home','request.role'); my @loncvars = ('user.name','user.domain','request.course.sec','request.course.id'); - my @cookievars; - if ($ENV{'SERVER_PORT'} == 443) { - @cookievars = ('lonLinkID'); - } else { - @cookievars = ('lonID'); - } + my @cookievars = ('lonID'); my $admin = $Apache::lonnet::perlvar{'lonAdminMail'}; my $origmail = $Apache::lonnet::perlvar{'lonSupportEMail'}; my $defdom = &get_domain(); + my $to = &Apache::loncommon::build_recipient_list(undef,'helpdeskmail', + $defdom,$origmail); my $from = $admin; - my %helpform; + my $bcc; my %domconfig = &Apache::lonnet::get_dom('configuration',['contacts'],$defdom); if (ref($domconfig{'contacts'}) eq 'HASH') { - if (ref($domconfig{'contacts'}{'helpform'}) eq 'HASH') { - %helpform = %{$domconfig{'contacts'}{'helpform'}}; + if (exists($domconfig{'contacts'}{'helpdeskmail'})) { + if (ref($domconfig{'contacts'}{'helpdeskmail'}) eq 'HASH') { + my $bccmail = $domconfig{'contacts'}{'helpdeskmail'}{'bcc'}; + if ($bccmail ne '') { + my @bccs = split(/,/,$bccmail); + my @ok_bccs; + foreach my $bcc (@bccs) { + $bcc =~ s/^\s+//g; + $bcc =~ s/\s+$//g; + if ($bcc =~ m/^[^\@]+\@[^\@]+$/) { + if (!(grep(/^\Q$bcc\E$/,@ok_bccs))) { + push(@ok_bccs,$bcc); + } + } + } + if (@ok_bccs > 0) { + $bcc = join(', ',@ok_bccs); + } + } + } } } my $reporttime = &Apache::lonlocal::locallocaltime(time); - my %formvars = ( - email => 1, - sourceurl => 1, - subject => 1, - description => 1, - ); - unless ($helpform{'username'} eq 'no') { - $formvars{'username'} = 1; - } - unless ($helpform{'user'} eq 'no') { - $formvars{'uname'} = 1; - $formvars{'udom'} = 1; - } - unless ($helpform{'phone'} eq 'no') { - $formvars{'phone'} = 1; - } - unless ($helpform{'section'} eq 'no') { - $formvars{'section'} = 1; - } - unless ($helpform{'course'} eq 'no') { - $formvars{'course'} = 1; - } - unless ($helpform{'cc'} eq 'no') { - $formvars{'cc'} = 1; - } - unless ($helpform{'screenshot'} eq 'no') { - $formvars{'screenshot'} = 1; - } - my $coursecode; - if ($formvars{'course'}) { - $coursecode = $env{'form.coursecode'}; + my @formvars = ('username','email','uname','udom','sourceurl','phone','section','coursecode','title','subject','description','screenshot'); + + &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},\@formvars); + my $coursecode = $env{'form.coursecode'}; + if ($coursecode eq '') { + my $totcodes = 0; + my %coursecodes; + $totcodes = &Apache::courseclassifier::retrieve_instcodes(\%coursecodes,$defdom,$totcodes); + my @standardnames = &Apache::loncommon::get_standard_codeitems(); + if ($totcodes > 0) { + my $noregexps = 1; + $coursecode = + &Apache::courseclassifier::instcode_from_selectors($defdom,$noregexps); + } if ($coursecode eq '') { - my $totcodes = 0; - my $instcats = &Apache::lonnet::get_dom_instcats($defdom); - if (ref($instcats) eq 'HASH') { - if (ref($instcats->{'codes'}) eq 'HASH') { - $totcodes = scalar(keys(%{$instcats->{'codes'}})); - } - } - my @standardnames = &Apache::loncommon::get_standard_codeitems(); - if ($totcodes > 0) { - my $noregexps = 1; - $coursecode = - &Apache::courseclassifier::instcode_from_selectors($defdom,$noregexps); - } - if ($coursecode eq '') { - foreach my $item (@standardnames) { - if ((defined($env{'form.'.$item})) && ($env{'form.'.$item} ne '-1')) { - $coursecode .= $env{'form.'.$item}; - } + foreach my $item (@standardnames) { + if ((defined($env{'form.'.$item})) && ($env{'form.'.$item} ne '-1')) { + $coursecode .= $env{'form.'.$item}; } } } @@ -767,20 +674,8 @@ sub print_request_receipt { $env{'user.domain'}); } } - my ($cid,$cdom,$cnum,$sectionlist,$ccode); - if ($homeserver) { - $cid = $env{'request.course.id'}; - } - if ($cid) { - $cdom = $env{'course.'.$cid.'.domain'}; - $cnum = $env{'course.'.$cid.'.num'}; - } - if ($cdom && $cnum) { - my %csettings = &Apache::lonnet::get('environment',['description','internal.coursecode','internal.sectionnums'],$cdom,$cnum); - $ccode = $csettings{'internal.coursecode'}; - $sectionlist = $csettings{'internal.sectionnums'}; - } - if (($homeserver) && (defined($env{'form.cc'})) && ($formvars{'cc'})) { + + if (($homeserver) && (defined($env{'form.cc'}))) { my @ccs; if ($env{'form.cc'} =~ /,/) { @ccs = split(/,/,$env{'form.cc'}); @@ -814,48 +709,20 @@ sub print_request_receipt { } } $env{'form.user'} = "'".$env{'form.uname'}.':'.$env{'form.udom'}."'"; - if ($formvars{'course'}) { - $env{'form.crsi'} = $env{'form.title'}; - if ($coursecode ne '') { - $env{'form.crsi'} .= ' - '.$coursecode; - } - } - if ($formvars{'section'}) { - my $section; - if ($sectionlist) { - if ($coursecode eq $ccode) { - $section = $env{'form.sectionsel'} - } else { - $section = $env{'form.sectiontxt'}; - } - } else { - $section = $env{'form.section'}; - } - $env{'form.crsi'} .= ' - '.$lt{'sect'}.': '.$section; - } - my $supportmsg; - if ($formvars{'username'}) { - $supportmsg .= "$lt{'username'}: $env{'form.username'}\n"; - } - $supportmsg .= "$lt{'email'}: $env{'form.email'}\n"; - if (($homeserver) && (defined($env{'form.cc'})) && ($formvars{'cc'})) { - $supportmsg .= "$lt{'cc'}: $okcclist\n"; - } - if ($formvars{'user'}) { - $supportmsg .= "$lt{'user'}: $env{'form.user'}\n"; - } - if ($formvars{'phone'}) { - $supportmsg .= "$lt{'phone'}: $env{'form.phone'}\n"; - } - if ($formvars{'course'}) { - $supportmsg .= "$lt{'crsi'}: $env{'form.crsi'}\n"; - } - $supportmsg .= "$lt{'subject'}: $env{'form.subject'} + $env{'form.crsi'} = $env{'form.title'}.' - '.$coursecode.' - '.$lt{'sect'}.': '.$env{'form.section'}; + my $supportmsg = <'. "$lt{$item}: $showurl
\n"; } elsif ($item eq 'cc') { - next if ($helpform{'cc'} eq 'no'); if ($okcclist) { my $showcclist = &Apache::loncommon::cleanup_html($okcclist); $displaymsg .= @@ -882,10 +748,6 @@ $lt{'date'}: $reporttime "$lt{$item}: $showcclist
\n"; } } else { - if (($item eq 'username') || ($item eq 'user') || ($item eq 'phone')) { - next if ($helpform{$item} eq 'no'); - } - next if (($item eq 'crsi') && ($helpform{'course'} eq 'no')); my $showitem = $env{'form.'.$item}; $showitem = &Apache::loncommon::cleanup_html($showitem); $displaymsg .= @@ -894,33 +756,6 @@ $lt{'date'}: $reporttime } } } - - my ($requname,$requdom,$reqemail); - foreach my $field ('uname','udom','email') { - $env{'form.'.$field} =~ s/^\s+//; - $env{'form.'.$field} =~ s/\s+$//; - } - if ($env{'form.uname'} =~ /^$match_username$/) { - $requname = $env{'form.uname'}; - } - if ($env{'form.udom'} =~ /^$match_domain$/) { - $requdom = $env{'form.udom'}; - } - if ($env{'form.email'} =~ m/^[^\@]+\@[^\@]+$/) { - $reqemail = $env{'form.email'}; - } - - my $dom_in_effect; - unless ($env{'user.domain'} eq 'public') { - $dom_in_effect = $env{'user.domain'}; - } - if ($dom_in_effect eq '') { - $dom_in_effect = $requdom; - } - if ($dom_in_effect eq '') { - $dom_in_effect = $defdom; - } - $displaymsg .= ''. $lt{'date'}.': '.$reporttime.'
'."\n"; @@ -943,11 +778,6 @@ END &print_header($r,$url,'process'); } my $bad_email = 0; - my ($to,$bcc,$addtext) = - &Apache::loncommon::build_recipient_list(undef,'helpdeskmail', - $dom_in_effect,$origmail, - $requname,$requdom, - $reqemail); if ($to =~ /,/) { my @ok_email; foreach my $email (split(/,/,$to)) { @@ -985,8 +815,12 @@ END } $r->print(&Apache::loncommon::confirmwrapper($message)); - if ($reqemail ne '') { - $from = $reqemail; + if (defined($env{'form.email'})) { + $env{'form.email'} =~ s/^\s+//; + $env{'form.email'} =~ s/\s+$//; + if ($env{'form.email'} =~ m/^[^\@]+\@[^\@]+$/) { + $from = $env{'form.email'}; + } } if (defined($env{'form.cc'})) { @@ -1005,27 +839,15 @@ END my $fname; my $attachmentpath = ''; - my $showsize = ''; - + my $attachmentsize = ''; if ((defined($env{'user.name'})) && (!$public)) { if ($homeserver && $env{'form.screenshot.filename'}) { - unless ($helpform{'screenshot'} eq 'no') { - my $attachmentsize = length($env{'form.screenshot'}); - my $max = 1048576; - my $showmax = 1.00; - if ($helpform{'maxsize'} =~ /^\d+\.\d*$/) { - $max *= $helpform{'maxsize'}; - $showmax = $helpform{'maxsize'}; - } - $showmax = '('.sprintf("%.2f",$showmax).' MB)'; - $showsize = $attachmentsize/1048576; - $showsize = '('.sprintf("%.2f",$showsize).' MB)'; - if ($attachmentsize > $max) { - $displaymsg .= '
'. - &mt('The uploaded screenshot file [_1] included with your request exceeded the maximum allowed size [_2], and has therefore been discarded.',$showsize,$showmax).''; - } else { - $attachmentpath=&Apache::lonnet::userfileupload('screenshot',undef,'helprequests'); - } + $attachmentsize = length($env{'form.screenshot'}); + if ($attachmentsize > 1048576) { + $displaymsg .= '
'. + &mt('The uploaded screenshot file ([_1] bytes) included with your request exceeded the maximum allowed size - 1 MB, and has therefore been discarded.',$attachmentsize).''; + } else { + $attachmentpath=&Apache::lonnet::userfileupload('screenshot',undef,'helprequests'); } } } @@ -1035,14 +857,13 @@ END if ($$cookie{'lonID'} =~ /lonID=($LONCAPA::handle_re);/) { $cookies{'lonID'} = $1; } - if ($$cookie{'lonLinkID'} =~ /lonLinkID=([a-f0-9]+_linked);/) { - $cookies{'lonLinkID'} = $1; - } + if ($attachmentpath =~ m-/([^/]+)$-) { $fname = $1; $displaymsg .= '
' - .&mt('An uploaded screenshot file [_1] was included in the request sent by [_2].' - ,''.$fname.' '.$showsize, + .&mt('An uploaded screenshot file [_1] ([_2] bytes) was included in the request sent by [_3].' + ,''.$fname.'' + ,$attachmentsize ,$env{'user.name'}.':'.$env{'user.domain'} ); $supportmsg .= "\n"; @@ -1052,12 +873,11 @@ END foreach my $var(@ENVvars) { $supportmsg .= "$var: $ENV{$var}\n"; } - $supportmsg .= "REMOTE_ADDR: $ip\n"; foreach my $var (@envvars) { $supportmsg .= "$var: $env{$var}\n"; } } - + my $cc_string; if ($homeserver) { if (@ok_ccs > 0) { @@ -1065,66 +885,27 @@ END } } - my $attachment_text = ''; + my $attachment_text; unless ($homeserver && $attachmentpath) { + my $envdata = ''; foreach my $var (@cookievars) { - $attachment_text .= "$var: $cookies{$var}\n"; + $envdata .= "$var: $cookies{$var}\n"; } foreach my $var (@ENVvars) { - $attachment_text .= "$var: $ENV{$var}\n"; + $envdata .= "$var: $ENV{$var}\n"; } - $attachment_text .= "REMOTE_ADDR: $ip\n"; foreach my $var (@envvars) { - $attachment_text .= "$var: $env{$var}\n"; + $envdata .= "$var: $env{$var}\n"; } foreach my $var (@loncvars) { - $attachment_text .= "$var: $env{$var}\n"; + $envdata .= "$var: $env{$var}\n"; } + $attachment_text = $envdata; } - - if ($addtext) { - my ($addloc,$addstr) = split(/:/,$addtext,2); - $addstr = &unescape($addstr); - if ($addloc eq 's') { - $subject = $addstr.' '.$subject; - } elsif ($addloc eq 'b') { - $supportmsg = $addstr."\n".$supportmsg; - } - } - - my $msg = MIME::Lite->new( - From => $from, - To => $to, - Subject => $subject, - Type =>'TEXT', - Data => $supportmsg, - ); - if ($homeserver) { - if (@ok_ccs > 0) { - my $cc_string = join(', ',@ok_ccs); - $msg->add("Cc" => $cc_string); - } - } - if ($bcc ne '') { - $msg->add("Bcc" => $bcc); - } - $msg->attr("content-type" => "text/plain"); - $msg->attr("content-type.charset" => "UTF-8"); - - if ($homeserver && $attachmentpath) { - my ($type, $encoding) = MIME::Types::by_suffix($attachmentpath); - $msg->attach(Type => $type, - Path => $attachmentpath, - Filename => $fname - ); - - } elsif ($attachment_text ne '') { - $msg->attach(Type => 'TEXT', - Data => $attachment_text); - } - -### Send it: - $msg->send('sendmail'); + + # Compose and send a MIME email + &Apache::loncommon::mime_email($from, $to, $subject, $supportmsg, $cc_string, $bcc, + $attachmentpath, $fname, $attachment_text); if ($attachmentpath =~ m|$Apache::lonnet::perlvar{'lonDaemons'}/tmp/helprequests/(\d+)/[^/]+|) { unlink($attachmentpath); @@ -1148,10 +929,6 @@ END $var.': '.$ENV{$var}.', '; } } - if ($ip ne '') { - $envmsg .= ''. - 'REMOTE_ADDR: '.$ip.', '; - } foreach my $var (@envvars) { if ($env{$var} ne '') { $envmsg .= ''. @@ -1178,7 +955,7 @@ sub print_header { login => 'Log-in help', ask => 'Ask helpdesk', getst => 'Getting started guide', - back => 'Back to last location', + back => 'Back to last location', headline => 'help/support', stud => 'Students', ifyo => 'If your problem is still unresolved, the form below can be used to send a question to the LON-CAPA helpdesk.',