--- loncom/homework/essayresponse.pm 2004/12/14 20:38:51 1.55 +++ loncom/homework/essayresponse.pm 2010/08/22 19:00:15 1.109 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # essay (ungraded) style responses # -# $Id: essayresponse.pm,v 1.55 2004/12/14 20:38:51 albertel Exp $ +# $Id: essayresponse.pm,v 1.109 2010/08/22 19:00:15 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -29,8 +29,13 @@ package Apache::essayresponse; use strict; use Apache::lonxml(); -use Apache::lonnet(); +use Apache::lonhtmlcommon; +use Apache::loncommon; +use Apache::lonnet; +use Apache::lonnavmaps; use Apache::lonlocal; +use LONCAPA qw(:DEFAULT :match); + BEGIN { &Apache::lonxml::register('Apache::essayresponse',('essayresponse')); @@ -42,55 +47,91 @@ sub start_essayresponse { my $id = &Apache::response::start_response($parstack,$safeeval); if ($target eq 'meta') { $result=&Apache::response::meta_package_write('essayresponse'); - } elsif ($target eq 'web') { + } elsif ($target eq 'web' && + $Apache::inputtags::status[-1] eq 'CAN_ANSWER') { my $part= $Apache::inputtags::part; my $ncol= &Apache::lonnet::EXT("resource.$part".'_'."$id.maxcollaborators"); my $coll= &HTML::Entities::encode($Apache::lonhomework::history{"resource.$part.$id.collaborators"},'<>&"'); my $uploadedfiletypes= &Apache::lonnet::EXT("resource.$part".'_'."$id.uploadedfiletypes"); $uploadedfiletypes=~s/[^\w\,]//g; - my $uploadedfile= &HTML::Entities::encode($Apache::lonhomework::history{"resource.$part.$id.uploadedfile"},'<>&"'); - $result='
'; - $result.=''; + my $maxfilesize=&Apache::lonnet::EXT("resource.$part".'_'."$id.maxfilesize"); + if (!defined($maxfilesize)) { + $maxfilesize = 10.0; #FIXME This should become a domain configuration + } + if (($Apache::lonhomework::type eq 'survey') || + ($Apache::lonhomework::type eq 'surveycred') || + ($Apache::lonhomework::type eq 'anonsurvey') || + ($Apache::lonhomework::type eq 'anonsurveycred')) { + $result.= ''; + } + my $status_text = &mt('Submission type'); + if ($Apache::lonhomework::history{"resource.$part.award"} eq 'DRAFT') { + $status_text .= '
'.&mt('(Currently -- draft)'); + } + $result.= '

'.&Apache::lonhtmlcommon::start_pick_box(). + &Apache::lonhtmlcommon::row_title($status_text); + if (($Apache::lonhomework::type ne 'survey') && + ($Apache::lonhomework::type ne 'surveycred') && + ($Apache::lonhomework::type ne 'anonsurvey') && + ($Apache::lonhomework::type ne 'anonsurveycred')) { + my $closure; + unless ($ncol || $uploadedfiletypes) { + $closure = 1; + } + $result.= + '
'. + ''. + &Apache::lonhtmlcommon::row_closure($closure); + } + if ($ncol > 0) { - $result .='

'; + $result .= &Apache::lonhtmlcommon::row_closure(); } - if ($uploadedfiletypes) { - $result.=''; + my $filesfrom = 'both'; + my $stuname = &Apache::lonnet::EXT('user.name'); + my $studom = &Apache::lonnet::EXT('user.domain'); + if (!&Apache::lonnet::usertools_access($stuname,$studom,'portfolio')) { + $filesfrom = 'uploadonly'; } - $result.='
'. - ' '. - &mt('Submit entries below as answer to receive credit').'
'. - ' '. - &mt('Save entries below as a draft answer (not submitting them for credit yet)'). - '
'. - 'Collaborators:
'. - &mt('(Enter maximum [_1] collaborators using username or username@domain, e.g. smithje or smithje@[_2].)',$ncol,$ENV{'user.domain'}).'
'; + $result.= &Apache::lonhtmlcommon::row_title(&mt('Collaborators')). + '
'; $result .= &check_collaborators($ncol,$coll) if ($coll =~ /\w+/); - $result .='
'.&mt('Submit a file: (only one file can be uploaded)'). - '

'. - &mt('Allowed filetypes: [_1]',$uploadedfiletypes).'
'. - 'OR:
Select Portfolio Files'. - '
'. - ''. - '
'; - if ($Apache::lonhomework::history{"resource.$part.$id.portfiles"}=~/[^\s]/){ - $result.="Portfolio files previously selected: ".&Apache::lonnet::unescape($Apache::lonhomework::history{"resource.$part.$id.portfiles"})."
"; - } - - if ($uploadedfile) { - my $url=$Apache::lonhomework::history{"resource.$part.$id.uploadedurl"}; - push (@Apache::lonxml::extlinks,$url); - &Apache::lonnet::allowuploaded('/adm/essayresponse',$url); - my $curfile=' '. - $uploadedfile.''; - $result.=&mt('Currently submitted: [_1]',$curfile); - } else { - $result.=&mt('(Hand in a file you have prepared on your computer)'); - } - $result.='
'; + $result.=&Apache::inputtags::file_selector($part,$id,$uploadedfiletypes, + $filesfrom,undef,$maxfilesize); + $result.=&Apache::lonhtmlcommon::end_pick_box().'

'; + } elsif ($target eq 'web' && + $Apache::inputtags::status[-1] ne 'CAN_ANSWER') { + my $part= $Apache::inputtags::part; + my @msgs; + if ($Apache::lonhomework::history{"resource.$part.$id.collaborators"} =~ /\S/) { + my $coll= &HTML::Entities::encode($Apache::lonhomework::history{"resource.$part.$id.collaborators"},'<>&"'); + $result .= ''.&mt('Collaborated with [_1]',$coll).''; + } + + my $current_files_display = &Apache::inputtags::current_file_submissions($part,$id); + if ($current_files_display) { + $result .= ''.&mt('Submitted files:').'
'. + $current_files_display.''; + } + + if ($result ne '') { + $result = + ''.$result. + '
'; + } } return $result; } @@ -99,139 +140,479 @@ sub end_essayresponse { my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_; my $part = $Apache::inputtags::part; my $id = $Apache::inputtags::response[-1]; - my $increment = 1; + my $increment = &Apache::response::repetition(); my $result; if ( $target eq 'grade' ) { - my $collaborators = $ENV{'form.HWCOL'.$part.'_'.$id}; - if ($collaborators =~ /[^\s]/) { - my $previous_list= &HTML::Entities::encode($Apache::lonhomework::history{"resource.$part.$id.collaborators"},'<>&"'); - $Apache::lonhomework::results{"resource.$part.$id.collaborators"}=$collaborators - if ($collaborators ne $previous_list); - } - if ( defined($ENV{'form.submitted'}) && - $ENV{'form.submitted'} eq 'scantron' ) { +# Deal with collaborators + my $collaborators = $env{'form.HWCOL'.$part.'_'.$id}; + my $previous_list= &HTML::Entities::encode($Apache::lonhomework::history{"resource.$part.$id.collaborators"},'<>&"'); + if ($collaborators ne $previous_list) { +# &Apache::lonnet::logthis("New collaborators [$collaborators] [$previous_list]"); + $Apache::lonhomework::results{"resource.$part.$id.collaborators"}=$collaborators; + } +# Scantron + if ( &Apache::response::submitted('scantron') ) { $increment=&Apache::response::scored_response($part,$id); - } elsif ( defined($ENV{'form.submitted'}) ) { - my $response = $ENV{'form.HWVAL_'.$id}; - my $filename= $ENV{'form.HWFILE'.$part.'_'.$id.'.filename'}; - my $portfiles = $ENV{'form.HWPORT'.$part.'_'.$id}; - &Apache::lonnet::logthis("portfiles now $portfiles"); - if (( $response =~ /[^\s]/) || ($filename =~ /[^\s]/) || ($portfiles =~ /[^\s]/)) { - my $award; - if ($ENV{'form.HWDRAFT'.$part.'_'.$id} eq 'yes') { - $award='SUBMITTED'; - } else { - $award='DRAFT'; + } elsif ( &Apache::response::submitted() ) { + my $response = $env{'form.HWVAL_'.$id}; + my $filename = $env{'form.HWFILE'.$part.'_'.$id.'.filename'} || + $env{'form.HWFILETOOBIG'.$part.'_'.$id}; + my $portfiles = $env{'form.HWPORT'.$part.'_'.$id}; + my @deletions = &Apache::loncommon::get_env_multiple('form.HWFILE'.$part.'_'.$id.'_delete'); + my ($is_submit,$was_draft); + if ($env{'form.HWDRAFT'.$part.'_'.$id} eq 'yes') { + $is_submit = 1; + } + if ($Apache::lonhomework::history{"resource.$part.award"} eq 'DRAFT') { + $was_draft = 1; + } + if (($response =~ /[^\s]/) || ($filename =~ /[^\s]/) || ($portfiles =~ /[^\s]/) || + (@deletions > 0) || ($was_draft && $is_submit)) { + my $award='DRAFT'; + if ($env{'form.HWDRAFT'.$part.'_'.$id} eq 'yes') { + if ($Apache::lonhomework::type eq 'anonsurvey') { + $award='ANONYMOUS'; + } elsif ($Apache::lonhomework::type eq 'anonsurveycred') { + $award='ANONYMOUS_CREDIT'; + } elsif ($Apache::lonhomework::type eq 'surveycred') { + $award='SUBMITTED_CREDIT'; + } else { + $award='SUBMITTED'; + } } my $uploadedflag=0; - if ($filename =~ /[^\s]/) { - my $uploadedfiletypes= &Apache::lonnet::EXT("resource.$part".'_'."$id.uploadedfiletypes"); - $uploadedfiletypes=~s/[^\w\,]//g; - $uploadedfiletypes=','.$uploadedfiletypes.','; - my ($extension)=($filename=~/\.(\w+)$/); - if ($uploadedfiletypes=~/\,$extension\,/i) { - $Apache::lonhomework::results{"resource.$part.$id.uploadedfile"}=$filename; - $Apache::lonhomework::results{"resource.$part.$id.uploadedurl"}= - &Apache::lonnet::userfileupload('HWFILE'.$part.'_'.$id,undef,'essayresponse'); - $uploadedflag=1; - } else { - $award='INVALID_FILETYPE'; - } - } else { - $Apache::lonhomework::results{"resource.$part.$id.uploadedfile"}=""; - $Apache::lonhomework::results{"resource.$part.$id.uploadedurl"}=""; - } - if ($portfiles =~ /[^\s]/) { - $portfiles =~s/,$//; - $Apache::lonhomework::results{"resource.$part.$id.portfiles"}=$portfiles; - my @submitted_files = split /,/,$portfiles; - my ($dsymb,$crsid,$domain,$name)=&Apache::lonxml::whichuser(); - &Apache::lonnet::unmark_as_readonly($domain,$name,$dsymb.$crsid); - &Apache::lonnet::mark_as_readonly($domain,$name,\@submitted_files,$dsymb.$crsid); - &Apache::lonnet::clear_selected_files($name); - $uploadedflag=1; - } else { - $Apache::lonhomework::results{"resource.$part.$id.portfiles"}=""; - } + my $totalsize=0; + &file_submission($part,$id,\$award,\$uploadedflag,\$totalsize,\@deletions); $Apache::lonhomework::results{"resource.$part.$id.submission"}=$response; $Apache::lonhomework::results{"resource.$part.$id.awarddetail"}=$award; my %previous=&Apache::response::check_for_previous($response,$part,$id); - unless ($uploadedflag) { &Apache::response::handle_previous(\%previous,$award); } + if ($uploadedflag) { + if ($award eq 'FILENAME_INUSE') { + delete($Apache::lonhomework::results{"resource.$id.tries"}); + } + } else { + &Apache::response::handle_previous(\%previous,$award); + } # # Store with resource author for similarity testing # if ($award eq 'SUBMITTED') { my ($symb,$crsid,$domain,$name)= - &Apache::lonxml::whichuser(); + &Apache::lonnet::whichuser(); if ($crsid) { - my $akey=$name.'.'.$domain.'.'.$crsid; + my $akey=join('.',&escape($name),&escape($domain), + &escape($crsid)); my $essayurl= &Apache::lonnet::declutter($ENV{'REQUEST_URI'}); my ($adom,$aname,$apath)= - ($essayurl=~/^(\w+)\/(\w+)\/(.*)$/); - $apath=&Apache::lonnet::escape($apath); + ($essayurl=~/^($LONCAPA::domain_re)\/($LONCAPA::username_re)\/(.*)$/); + $apath=&escape($apath); $apath=~s/\W/\_/gs; &Apache::lonnet::put('nohist_essay_'.$apath, { $akey => $response },$adom,$aname); } } - } + } } } elsif ($target eq 'edit') { $result.=&Apache::edit::end_table(); - } elsif ($target eq 'tex') { - if ($Apache::lonhomework::type eq 'exam') { - my $repetition=&Apache::response::repetition(); - $result.='\begin{enumerate}'; - if ($ENV{'request.state'} eq "construct" ) {$result.='\item[\strut]';} - for (my $i=0;$i<$repetition;$i++) { - $result.='\item[\textbf{'.($Apache::lonxml::counter+$i). - '}.]\textit{'.&mt('Leave blank on scoring form'). - '}\vskip 0 mm'; - } - $result.= '\end{enumerate}'; - $increment=$repetition; - } + + } elsif ($target eq 'tex' + && $Apache::lonhomework::type eq 'exam') { + $result .= &Apache::inputtags::exam_score_line($target); + + } elsif ($target eq 'answer') { + $result.=&Apache::response::answer_header($$tagstack[-1]); + my $answer = &mt('Essay will be hand graded.'); + $result.=&Apache::response::answer_part($$tagstack[-1],$answer, + {'no_verbatim' => 1}); + $result.=&Apache::response::answer_footer($$tagstack[-1]); } + if ($target eq 'web') { + &Apache::response::setup_prior_tries_hash(\&format_prior_response, + ['portfiles', + 'uploadedurl']); + } + if ($target eq 'grade' || $target eq 'web' || $target eq 'answer' || $target eq 'tex' || $target eq 'analyze') { - &Apache::lonxml::increment_counter($increment); + &Apache::lonxml::increment_counter($increment, "$part.$id"); + + if ($target eq 'analyze') { + $Apache::lonhomework::analyze{"$part.$id.type"} = 'essayresponse'; + push (@{ $Apache::lonhomework::analyze{"parts"} },"$part.$id"); + &Apache::lonhomework::set_bubble_lines(); + } } &Apache::response::end_response; return $result; } +sub format_prior_response { + my ($mode,$answer,$other_data) = @_; + my $output; + + my (undef,undef,$udom,$uname) = &Apache::lonnet::whichuser(); + my $port_url = '/uploaded/'.$udom.'/'.$uname.'/portfolio/'; + + my $file_list; + + foreach my $file (split(/\s*,\s*/, + $other_data->[0].','.$other_data->[1])) { + next if ($file!~/\S/); + if ($file !~ m{^/uploaded/}) { $file=$port_url.$file; } + $file=~s|/+|/|g; + &Apache::lonnet::allowuploaded('/adm/essayresponse',$file); + $file_list.='
  • file icon '.$file. + '
  • '."\n"; + } + if ($file_list) { + $output.= &mt('Submitted Files').''; + } + if ($answer =~ /\S/) { + $output.='

    '.&mt('Submitted text'). + '

    '.$answer.'

    '; + } + + return '
    '.$output.'
    '; +} + +sub file_submission { + my ($part,$id,$award,$uploadedflag,$totalsize,$deletions)=@_; + my $files; + my $jspart=$part; + $jspart=~s/\./_/g; + my ($symb,$crsid,$udom,$uname) = &Apache::lonnet::whichuser(); + my %crsinfo = &Apache::lonnet::coursedescription($crsid); + my $cdom = $crsinfo{'domain'}; + my $cnum = $crsinfo{'num'}; + my (@portfiles,$uploadedurl,@submitted_portfiles,$submitted_upload, + @acceptable_portfiles,$acceptable_upload,@accepted_portfiles, + $accepted_upload,@savedportfiles,$stored_upload,@tolock, + %port_delete,$uploaded_delete); + if ($Apache::lonhomework::history{"resource.$part.$id.portfiles"} || + $Apache::lonhomework::history{"resource.$part.$id.uploadedurl"}) { + if ($Apache::lonhomework::history{"resource.$part.$id.portfiles"}) { + @portfiles = split(/,/,$Apache::lonhomework::history{"resource.$part.$id.portfiles"}); + } + $uploadedurl = $Apache::lonhomework::history{"resource.$part.$id.uploadedurl"}; + if (ref($deletions) eq 'ARRAY') { + if (@{$deletions} > 0) { + foreach my $file (@{$deletions}) { + $file = &HTML::Entities::decode($file); + if (grep(/^\Q$file\E$/,@portfiles)) { + $port_delete{$file} = 1; + } elsif ($file =~ m{^/uploaded/\Q$udom\E/\Q$uname\E/essayresponse/\Q$cdom\E/\Q$cnum\E/}) { + $uploaded_delete = $file; + } + } + } + } + foreach my $current (@portfiles) { + unless ($port_delete{$current}) { + push(@savedportfiles,$current); + } + } + if ($uploaded_delete) { + $Apache::lonhomework::results{"resource.$part.$id.uploadedfile"} = ""; + $Apache::lonhomework::results{"resource.$part.$id.uploadedurl"} = ""; + } + } + if ($env{'form.HWPORT'.$jspart.'_'.$id} ne '') { + my $newfiles= $env{'form.HWPORT'.$jspart.'_'.$id}; + $newfiles =~s/,$//; + if ($newfiles =~ /[^\s]/) { + foreach my $file (split(/\s*,\s*/,$newfiles)) { + if ($file =~ /[^\s]/) { + push(@submitted_portfiles,$file); + } + } + } + } + if ($env{'form.HWFILETOOBIG'.$jspart.'_'.$id} ne '') { + $$award = 'EXCESS_FILESIZE'; + } elsif ($env{'form.HWFILE'.$jspart.'_'.$id.'.filename'} ne '') { + my $newfile = $env{'form.HWFILE'.$jspart.'_'.$id.'.filename'}; + if ($newfile =~ /[^\s]/) { + $submitted_upload = $newfile; + } + } + if (@savedportfiles) { + foreach my $file (reverse(@savedportfiles)) { + unless(grep(/^\Q$file\E$/,@submitted_portfiles)) { + unshift(@submitted_portfiles,$file); + } + } + } + if (@submitted_portfiles || $submitted_upload) { + my $uploadedfiletypes= + &Apache::lonnet::EXT("resource.$part".'_'."$id.uploadedfiletypes"); + if ($uploadedfiletypes ne '') { + $uploadedfiletypes=~s/[^\w\,]//g; + $uploadedfiletypes=','.$uploadedfiletypes.','; + if (@submitted_portfiles) { + foreach my $file (@submitted_portfiles) { + my ($extension)=($file=~/\.(\w+)$/); + if ($uploadedfiletypes=~/\,\s*\Q$extension\E\s*\,/i) { + push(@acceptable_portfiles,$file); + } + } + } + if ($submitted_upload) { + my ($upload_ext)=($submitted_upload=~/\.(\w+)$/); + if ($uploadedfiletypes=~/\,\s*\Q$upload_ext\E\s*\,/i) { + $acceptable_upload = $submitted_upload; + } else { + $$award='INVALID_FILETYPE'; + &delete_form_items($jspart,$id); + } + } + } else { + @acceptable_portfiles = @submitted_portfiles; + $acceptable_upload = $submitted_upload; + } + } + if ((@acceptable_portfiles) || ($acceptable_upload ne '')) { + my $maxfilesize=&Apache::lonnet::EXT("resource.$part".'_'."$id.maxfilesize"); + if (!$maxfilesize) { + $maxfilesize = 10.0; #FIXME This should become a domain configuration + } + my %dirlist; + if (@acceptable_portfiles) { + foreach my $file (@acceptable_portfiles) { + my ($path,$filename) = ($file =~ m{^(.*/)([^/]+)$}); + my $fullpath = '/userfiles/portfolio'.$path; + if (!exists($dirlist{$fullpath})) { + my @list = &Apache::lonnet::dirlist($fullpath,$udom,$uname,1); + $dirlist{$fullpath} = \@list; + } + if (ref($dirlist{$fullpath}) eq 'ARRAY') { + foreach my $dir_line (@{$dirlist{$fullpath}}) { + my ($fname,$dom,undef,$testdir,undef,undef,undef,undef, + $size,undef,$mtime,undef,undef,undef,$obs,undef) = + split(/\&/,$dir_line,16); + if ($filename eq $fname) { + my $mbsize = $size/(1024.0*1024.0); + if (ref($totalsize)) { + $$totalsize += $mbsize; + } + last; + } + } + } + if (ref($totalsize)) { + if ($$totalsize > $maxfilesize) { + $$award='EXCESS_FILESIZE'; + &delete_form_items($jspart,$id); + } else { + push(@accepted_portfiles,$file); + } + } else { + push(@accepted_portfiles,$file); + } + } + } + if ($acceptable_upload ne '') { + if (ref($totalsize)) { + $$totalsize += $env{'form.HWFILESIZE'.$jspart.'_'.$id}; + if ($$totalsize > $maxfilesize) { + $$award='EXCESS_FILESIZE'; + delete($env{'form.HWFILE'.$jspart.'_'.$id}); + } else { + $accepted_upload = $acceptable_upload; + } + } else { + $accepted_upload = $acceptable_upload; + } + } + } + if ($accepted_upload ne '') { + my ($map,$resid,$resurl)=&Apache::lonnet::decode_symb($symb); + my $turnindir; + my %userhash = &Apache::lonnet::userenvironment($udom,$uname,'turnindir'); + $turnindir = $userhash{'turnindir'}; + if ($turnindir eq '') { + $turnindir = &mt('turned in'); + $turnindir =~ s/\W+/_/g; + my %newhash = ( + 'turnindir' => $turnindir, + ); + &Apache::lonnet::put('environment',\%newhash,$udom,$uname); + } + my $prefix = 'portfolio'; + my $path = '/'.$turnindir.'/'; + my $restitle=&Apache::lonnet::gettitle($symb); + $restitle =~ s/\W+/_/g; + if ($restitle eq '') { + $restitle = ($resurl =~ m{/[^/]+$}); + if ($restitle eq '') { + $restitle = time; + } + } + my @pathitems; + my $navmap = Apache::lonnavmaps::navmap->new(); + if (defined($navmap)) { + my $mapres = $navmap->getResourceByUrl($map); + if (ref($mapres)) { + my $pcslist = $mapres->map_hierarchy(); + if ($pcslist ne '') { + foreach my $pc (split(/,/,$pcslist)) { + my $res = $navmap->getByMapPc($pc); + if (ref($res)) { + my $title = $res->compTitle(); + $title =~ s/\W+/_/g; + if ($title ne '') { + push(@pathitems,$title); + } + } + } + } + my $maptitle = $mapres->compTitle(); + $maptitle =~ s/\W+/_/g; + if ($maptitle ne '') { + push(@pathitems,$maptitle); + } + } else { + $$award = 'INTERNAL_ERROR'; + } + } else { + $$award = 'INTERNAL_ERROR'; + } + push(@pathitems,$restitle); + $path .= join('/',@pathitems); + my $formelement = 'HWFILE'.$jspart.'_'.$id; + my $fname = $env{'form.'.$formelement.'.filename'}; + &Apache::lonnet::clean_filename($fname); + my $url = '/uploaded/'.$udom.'/'.$uname.'/'.$prefix.$path.'/'.$fname; + my @stat = &Apache::lonnet::stat_file($url); + if (@stat && $stat[0] ne 'no_such_dir') { + $$award = 'FILENAME_INUSE'; + } else { + my ($mode,%allfiles,%codebase); + my $result = &Apache::lonnet::userfileupload($formelement,'', + $prefix.$path,$mode,\%allfiles,\%codebase); + if ($result =~ m{^/uploaded/}) { + $stored_upload = $path.'/'.$fname; + $Apache::lonhomework::results{"resource.$part.$id.portfiles"} = $stored_upload; + push(@tolock,$stored_upload); + } else { + $$award = 'INTERNAL_ERROR'; + } + } + delete($env{'form.HWFILE'.$jspart.'_'.$id}); + } + if (@accepted_portfiles) { + if ($stored_upload) { + $Apache::lonhomework::results{"resource.$part.$id.portfiles"} .= ','; + } + $Apache::lonhomework::results{"resource.$part.$id.portfiles"}.=join(',',@accepted_portfiles); + push(@tolock,@accepted_portfiles); + } + if (!defined($Apache::lonhomework::results{"resource.$part.$id.portfiles"})) { + if (keys(%port_delete) > 0) { + $Apache::lonhomework::results{"resource.$part.$id.portfiles"} = ""; + } + } + if ($Apache::lonhomework::history{"resource.$part.$id.portfiles"} ne $Apache::lonhomework::results{"resource.$part.$id.portfiles"}) { + if (ref($uploadedflag)) { + $$uploadedflag=1; + } + } + &Apache::lonnet::unmark_as_readonly($udom,$uname,[$symb,$crsid]); + &Apache::lonnet::mark_as_readonly($udom,$uname,[@tolock],[$symb,$crsid]); + &Apache::lonnet::clear_selected_files($uname); + return; +} + +sub delete_form_items { + my ($jspart,$id) = @_; + delete($env{'form.HWFILE'.$jspart.'_'.$id.'.filename'}); + delete($env{'form.HWFILE'.$jspart.'_'.$id.'.mimetype'}); + delete($env{'form.HWFILE'.$jspart.'_'.$id}); +} + sub check_collaborators { my ($ncol,$coll) = @_; my %classlist=&Apache::lonnet::dump('classlist', - $ENV{'course.'.$ENV{'request.course.id'}.'.domain'}, - $ENV{'course.'.$ENV{'request.course.id'}.'.num'}); + $env{'course.'.$env{'request.course.id'}.'.domain'}, + $env{'course.'.$env{'request.course.id'}.'.num'}); my (@badcollaborators,$result); - my (@collaborators) = split(/\,?\s+/,$coll); - foreach (@collaborators) { - my $collaborator = $_; - if (/@/) { - $collaborator =~ s/@/:/; + + my (@collaborators) = split(/[,;\s]+/,$coll); + foreach my $entry (@collaborators) { + my $collaborator; + if ($entry =~ /:/) { + $collaborator = $entry; } else { - $collaborator = $_.':'.$ENV{'user.domain'}; + $collaborator = $entry.':'.$env{'user.domain'}; } - push @badcollaborators, $_ if (!grep /^$collaborator/i,keys %classlist); + if ($collaborator !~ /^$match_username:$match_domain$/) { + if (!grep(/^\Q$entry\E$/,@badcollaborators)) { + push(@badcollaborators,$entry); + } + } elsif (!grep(/^\Q$collaborator\E$/i,keys(%classlist))) { + if (!grep(/^\Q$entry\E$/,@badcollaborators)) { + push(@badcollaborators,$entry); + } + } } - if (scalar(@badcollaborators)) { - $result = '
    The following user'. - (scalar(@badcollaborators) > 1 ? 's are' : ' is').' invalid: '. - join(', ',@badcollaborators).'. Please correct.
    '; + my $numbad = scalar(@badcollaborators); + if ($numbad) { + $result = '
    '; + if ($numbad == 1) { + $result .= &mt('The following user is invalid:'); + } else { + $result .= &mt('The following [_1] users are invalid:',$numbad); + } + $result .= ' '.join(', ',@badcollaborators).'. '.&mt('Please correct.'). + '
    '; } my $toomany = scalar(@collaborators) - $ncol; if ($toomany > 0) { $result .= '
    '. - 'You have too many collaborators. Please remove '.$toomany.' collaborator'. - ($toomany > 1 ? 's' :'').'.
    '; + &mt('You have too many collaborators.').' '. + &mt('Please remove [quant,_1,collaborator].',$toomany). + ''; } return $result; } 1; __END__ + + +=pod + +=head1 NAME + +Apache::essayresponse + +=head1 SYNOPSIS + +Handler to evaluate essay (ungraded) style responses. + +This is part of the LearningOnline Network with CAPA project +described at http://www.lon-capa.org. + +=head1 SUBROUTINES + +=over + +=item start_essayresponse() + +=item end_essayresponse() + +=item format_prior_response() + +=item file_submission() + +=item delete_form_items() + +=item check_collaborators() + +=back + +=cut 500 Internal Server Error

    Internal Server Error

    The server encountered an internal error or misconfiguration and was unable to complete your request.

    Please contact the server administrator at root@localhost to inform them of the time this error occurred, and the actions you performed just before this error.

    More information about this error may be available in the server error log.