--- loncom/homework/essayresponse.pm 2008/12/21 19:09:15 1.95 +++ loncom/homework/essayresponse.pm 2015/03/10 17:32:53 1.124 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # essay (ungraded) style responses # -# $Id: essayresponse.pm,v 1.95 2008/12/21 19:09:15 raeburn Exp $ +# $Id: essayresponse.pm,v 1.124 2015/03/10 17:32:53 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -29,7 +29,10 @@ package Apache::essayresponse; use strict; use Apache::lonxml(); +use Apache::lonhtmlcommon; +use Apache::loncommon; use Apache::lonnet; +use Apache::lonnavmaps; use Apache::lonlocal; use LONCAPA qw(:DEFAULT :match); @@ -44,83 +47,119 @@ 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' && - $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 $maxfilesize=&Apache::lonnet::EXT("resource.$part".'_'."$id.maxfilesize"); - if (!defined($maxfilesize)) { - $maxfilesize = 10.0; #FIXME This should become a domain configuration + } elsif ($target eq 'web') { + my $part= $Apache::inputtags::part; + my $coll; + if ($Apache::lonhomework::history{"resource.$part.$id.collaborators"} =~ /\S/) { + $coll = &HTML::Entities::encode($Apache::lonhomework::history{"resource.$part.$id.collaborators"},'<>&"'); } - if ( $Apache::lonhomework::type eq 'survey' ) { - $result.= ' '; - } - $result.='
'; - if ( $Apache::lonhomework::type ne 'survey' ) { - $result.= ''; - } - - if ($ncol > 0) { - $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.=&Apache::inputtags::file_selector($part,$id,$uploadedfiletypes, - $filesfrom,undef,$maxfilesize); - $result.='
'. - '
'. - ''. - '
'.'
'; - $result .= &check_collaborators($ncol,$coll) if ($coll =~ /\w+/); - $result .='
'; - } 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).''; - } + if ($Apache::inputtags::status[-1] eq 'CAN_ANSWER') { + my $ncol= &Apache::lonnet::EXT("resource.$part".'_'."$id.maxcollaborators"); + my $uploadedfiletypes= &Apache::lonnet::EXT("resource.$part".'_'."$id.uploadedfiletypes"); + $uploadedfiletypes=~s/[^\w\,]//g; + my $maxfilesize=&Apache::lonnet::EXT("resource.$part".'_'."$id.maxfilesize"); + if (!defined($maxfilesize)) { + $maxfilesize = 10.0; #FIXME This should become a domain configuration + } + my $hiddendraft; + if (($Apache::lonhomework::type eq 'survey') || + ($Apache::lonhomework::type eq 'surveycred') || + ($Apache::lonhomework::type eq 'anonsurvey') || + ($Apache::lonhomework::type eq 'anonsurveycred')) { + $hiddendraft = ''; + } else { + $result = &draft_row($part,$id,$ncol,$uploadedfiletypes,'essayresponse'); + } + if ($ncol > 0) { + $result .= &get_collab_row($part,$id,$coll,$ncol); + } + 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.=&Apache::inputtags::file_selector($part,$id,$uploadedfiletypes, + $filesfrom,undef,$maxfilesize); + if ($result) { + $result = + '
'.$hiddendraft. + &Apache::lonhtmlcommon::start_pick_box(). + $result. + &Apache::lonhtmlcommon::end_pick_box().'
'; + } else { + $result = $hiddendraft; + } + } else { + $result = &show_status_table($part,$id,$coll); + } + } + return $result; +} - my $file_submission = - &Apache::inputtags::show_past_file_submission($part,$id); - if ($file_submission) { - $result .= ''.$file_submission.''; - } +sub draft_row { + my ($part,$id,$ncol,$uploadedfiletypes,$resptype) = @_; + my $status_text = &mt('Submission type'); + if ($Apache::lonhomework::history{"resource.$part.award"} eq 'DRAFT') { + $status_text .= '
'.&mt('(Currently -- draft)'); + } + my $closure; + unless ($ncol || $uploadedfiletypes) { + $closure = 1; + } + my %label = ( + draft => { + essayresponse => &mt('Save entries below (not submitted for credit yet)'), + externalresponse => &mt('Save entries below (not submitted for grading yet)'), + }, + submit => { + essayresponse => &mt('Submit entries below as answer to receive credit'), + externalresponse => &mt('Submit entries below for grading'), + } + ); + return &Apache::lonhtmlcommon::row_title($status_text). + '
'. + ''. + &Apache::lonhtmlcommon::row_closure($closure); +} - my $port_submission = - &Apache::inputtags::show_past_portfile_submission($part,$id); - if ($port_submission) { - $result .= ''.$port_submission.''; - } +sub get_collab_row { + my ($part,$id,$coll,$ncol,$closure) = @_; + my $output = &Apache::lonhtmlcommon::row_title(&mt('Collaborators')). + '
'; + $output .= &check_collaborators($ncol,$coll) if ($coll =~ /\w+/); + $output .= &Apache::lonhtmlcommon::row_closure($closure); + return $output; +} - if ($result ne '') { - $result = - ''.$result. - '
'; - } +sub show_status_table { + my ($part,$id,$coll) = @_; + my $output; + if ($coll) { + $output = ''.&mt('Collaborated with [_1]',$coll).''; } - return $result; + my $current_files_display = &Apache::inputtags::current_file_submissions($part,$id); + if ($current_files_display) { + $output .= ''.&mt('Submitted files:').'
'. + $current_files_display.''; + } + if ($output ne '') { + return ''.$output. + '
'; + } + return; } sub end_essayresponse { @@ -130,32 +169,52 @@ sub end_essayresponse { 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); - } + &get_collaborators($part,$id); +# Scantron if ( &Apache::response::submitted('scantron') ) { $increment=&Apache::response::scored_response($part,$id); } elsif ( &Apache::response::submitted() ) { my $response = $env{'form.HWVAL_'.$id}; - my $filename = $env{'form.HWFILE'.$part.'_'.$id.'.filename'} || + my $jspart=$part; + $jspart=~s/\./_/g; + my $filename = $env{'form.HWFILE'.$jspart.'_'.$id.'.filename'} || $env{'form.HWFILETOOBIG'.$part.'_'.$id}; - my $portfiles = $env{'form.HWPORT'.$part.'_'.$id}; - if (( $response =~ /[^\s]/) || ($filename =~ /[^\s]/) || ($portfiles =~ /[^\s]/)) { + my $portfiles = $env{'form.HWPORT'.$jspart.'_'.$id}; + my @deletions = &Apache::loncommon::get_env_multiple('form.HWFILE'.$jspart.'_'.$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') { - $award='SUBMITTED'; + 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; my $totalsize=0; - &file_submission($part,$id,'filename',\$award,\$uploadedflag,\$totalsize); - &file_submission($part,$id,'portfiles',\$award,\$uploadedflag,\$totalsize); + &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 # @@ -175,7 +234,7 @@ sub end_essayresponse { { $akey => $response },$adom,$aname); } } - } + } } } elsif ($target eq 'edit') { $result.=&Apache::edit::end_table(); @@ -203,6 +262,7 @@ sub end_essayresponse { if ($target eq 'analyze') { $Apache::lonhomework::analyze{"$part.$id.type"} = 'essayresponse'; + push (@{ $Apache::lonhomework::analyze{"parts"} },"$part.$id"); &Apache::lonhomework::set_bubble_lines(); } } @@ -211,6 +271,16 @@ sub end_essayresponse { return $result; } +sub get_collaborators { + my ($part,$id) = @_; + 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::lonhomework::results{"resource.$part.$id.collaborators"}=$collaborators; + } +} + sub format_prior_response { my ($mode,$answer,$other_data) = @_; my $output; @@ -235,71 +305,141 @@ sub format_prior_response { } if ($answer =~ /\S/) { $output.='

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

'.$answer.'

'; + '
'.&HTML::Entities::encode($answer, '"<>&').'

'; } return '
'.$output.'
'; } sub file_submission { - my ($part,$id,$which,$award,$uploadedflag,$totalsize)=@_; + my ($part,$id,$award,$uploadedflag,$totalsize,$deletions,$context,$info)=@_; my $files; my $jspart=$part; $jspart=~s/\./_/g; - if ($which eq 'portfiles') { - $files= $env{'form.HWPORT'.$jspart.'_'.$id}; - } elsif ($which eq 'filename') { - if ($env{'form.HWFILETOOBIG'.$jspart.'_'.$id} ne '') { - $$award = 'EXCESS_FILESIZE'; - return; - } else { - $files = $env{'form.HWFILE'.$jspart.'_'.$id.'.filename'}; + 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"}) { + my @possfiles = split(/,/,$Apache::lonhomework::history{"resource.$part.$id.portfiles"}); + foreach my $file (@possfiles) { + my ($path,$name) = ($file =~ m{^(.*/)([^/]+)$}); + my ($origname,$version,$ext) = &Apache::lonnet::file_name_version_ext($name); + unless ($version) { + push(@portfiles,$file); + } + } + } + $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; + } elsif ($file =~ m{^/uploaded/\Q$udom\E/\Q$uname\E/essayresponse/[^/]+$}) { + $uploaded_delete = $file; + } + } + } + } + foreach my $current (@portfiles) { + unless ($port_delete{$current}) { + push(@savedportfiles,$current); + } + } + if ($uploaded_delete) { + if ($uploaded_delete eq $uploadedurl) { + $Apache::lonhomework::results{"resource.$part.$id.uploadedfile"} = ""; + $Apache::lonhomework::results{"resource.$part.$id.uploadedurl"} = ""; + } else { + undef($uploaded_delete); + } } } - if ($files =~ /[^\s]/) { - $files =~s/,$//; - my (@submitted_files,@acceptable_files,@accepted_files); - if ($which eq 'portfiles') { - @submitted_files = split(/\s*,\s*/,$files); - } else { - @submitted_files = ($files); + 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'.$part.'_'.$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; } - my $uploadedfiletypes= + } + 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) { + if ($uploadedfiletypes ne '') { $uploadedfiletypes=~s/[^\w\,]//g; $uploadedfiletypes=','.$uploadedfiletypes.','; - foreach my $file (@submitted_files) { - my ($extension)=($file=~/\.(\w+)$/); - if ($uploadedfiletypes=~/\,\s*\Q$extension\E\s*\,/i) { - push(@acceptable_files,$file); + 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'; - if ($which eq 'filename') { - &delete_form_items($jspart,$id); - } + &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; - foreach my $file (@acceptable_files) { - if ($which eq 'filename') { - if (ref($totalsize)) { - $$totalsize += $env{'form.HWFILESIZE'.$jspart.'_'.$id}; - } - } else { - my ($symb,$crsid,$udom,$uname) = &Apache::lonnet::whichuser(); - my ($path,$filename) = ($file =~ m{^(.+)/([^/]+)$}); + 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); - foreach my $dir_line (@list) { + my ($listref,$listerror) = + &Apache::lonnet::dirlist($fullpath,$udom,$uname,1); + if (ref($listref) eq 'ARRAY') { + $dirlist{$fullpath} = $listref; + } + } + 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) = + $size,undef,$mtime,undef,undef,undef,$obs,undef) = split(/\&/,$dir_line,16); if ($filename eq $fname) { my $mbsize = $size/(1024.0*1024.0); @@ -309,89 +449,130 @@ sub file_submission { last; } } - $dirlist{$fullpath} = \@list; + } + 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'; - if ($which eq 'filename') { - &delete_form_items($jspart,$id); - } + delete($env{'form.HWFILE'.$jspart.'_'.$id}); } else { - push(@accepted_files,$file); + $accepted_upload = $acceptable_upload; } } else { - push(@accepted_files,$file); + $accepted_upload = $acceptable_upload; } } - $Apache::lonhomework::results{"resource.$part.$id.$which"}=join(',',@accepted_files); - if (($$award eq 'INVALID_FILETYPE') || ($award eq 'EXCESS_FILESIZE')) { - return; - } - if (ref($uploadedflag)) { - $$uploadedflag=1; - } - my ($symb,$crsid,$domain,$name)=&Apache::lonnet::whichuser(); - if ($which eq 'portfiles') { - &Apache::lonnet::unmark_as_readonly($domain,$name,[$symb,$crsid]); - &Apache::lonnet::mark_as_readonly($domain,$name,\@submitted_files,[$symb,$crsid]); - &Apache::lonnet::clear_selected_files($name); - } - if ($which eq 'filename') { - $Apache::lonhomework::results{"resource.$part.$id.uploadedfile"}= - $files; - my $cleanpart = $part; - $cleanpart =~ s/\W/_/g; - my $cleanid = $id; - $cleanid =~ s/\W/_/g; - my ($map,$resid,$res)=&Apache::lonnet::decode_symb($symb); - my $container; - if ($map =~ /^uploaded/) { - (my $prefix,$container) = ($map =~ m{^uploaded/[^/]+/[^/]+/(default|supplemental)_?([^.]*)\.(?:sequence|page)$}); - if (length($container) > 10) { - $container = substr($container,-10,10); + } + if ($accepted_upload ne '') { + my ($path,$multiresp) = + &Apache::loncommon::get_turnedin_filepath($symb,$uname,$udom, + 'submission'); + if ($path eq '') { + $$award = 'INTERNAL_ERROR'; + } else { + if ($multiresp) { + $path .= '/'.$jspart.'_'.$id; + } + my $prefix = 'portfolio'; + my $formelement = 'HWFILE'.$jspart.'_'.$id; + my $fname = &Apache::lonnet::clean_filename($env{'form.'.$formelement.'.filename'}); + my $url = '/uploaded/'.$udom.'/'.$uname.'/'.$prefix.$path.'/'.$fname; + my @stat = &Apache::lonnet::stat_file($url); + my $conflicts = 0; + if (@stat && $stat[0] ne 'no_such_dir') { + my $current_permissions = + &Apache::lonnet::get_portfile_permissions($udom,$uname); + if (ref($current_permissions) eq 'HASH') { + if (ref($current_permissions->{$path.'/'.$fname}) eq 'ARRAY') { + foreach my $record (@{$current_permissions->{$path.'/'.$fname}}) { + if (ref($record) eq 'ARRAY') { + next if (($record->[0] eq $symb) && + ($record->[1] eq $crsid)); + $conflicts ++; + } + } + } } - if ($container ne '') { - $container = $prefix.'_'.$container; + if ($conflicts) { + $$award = 'FILENAME_INUSE'; + } + } + unless ($conflicts) { + my ($mode,%allfiles,%codebase); + my $result = &Apache::lonnet::userfileupload($formelement,'', + $prefix.$path,$mode,\%allfiles,\%codebase); + if ($result =~ m{^/uploaded/}) { + $stored_upload = $path.'/'.$fname; + unless (grep(/^\Q$stored_upload\E$/,@accepted_portfiles)) { + $Apache::lonhomework::results{"resource.$part.$id.portfiles"} = $stored_upload; + push(@tolock,$stored_upload); + } } else { - $container = $prefix; - } - } else { - ($container) = ($map =~ m{(.+)\.(?:sequence|page)$}); - $container =~ s/\W/_/g; - if (length($container) > 255) { - $container = substr($container,0,254); - } - } - my $subdir = 'essayresponse'; - my %crsdesc = &Apache::lonnet::coursedescription($crsid); - foreach my $item ($crsdesc{'domain'},$crsdesc{'num'},$container,$resid,$cleanpart,$cleanid) { - if ($item ne '') { - $subdir .= '/'.$item; - } - } - $Apache::lonhomework::results{"resource.$part.$id.uploadedurl"}= - &Apache::lonnet::userfileupload('HWFILE'.$jspart.'_'.$id,undef, - $subdir); - &Apache::lonnet::delenv($env{'form.HWFILE'.$jspart.'_'.$id}); - } - } elsif ($which eq 'portfiles' && - $Apache::lonhomework::history{"resource.$part.$id.$which"}) { - my ($symb,$crsid,$domain,$name)=&Apache::lonnet::whichuser(); - &Apache::lonnet::unmark_as_readonly($domain,$name,[$symb,$crsid]); - $Apache::lonhomework::results{"resource.$part.$id.$which"}=""; + $$award = 'INTERNAL_ERROR'; + } + } + } + delete($env{'form.HWFILE'.$jspart.'_'.$id}); + } + if (@accepted_portfiles) { + if ($Apache::lonhomework::results{"resource.$part.$id.portfiles"}) { + $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"}) || + ($uploaded_delete)) { + if (ref($uploadedflag)) { + $$uploadedflag=1; + } + } + if ($context eq 'externalresponse') { + my @todelete = map { "/uploaded/$udom/$uname/portfolio".$_; } keys(%port_delete); + if (@tolock || @todelete) { + if (ref($info) eq 'HASH') { + if (($info->{'ip'}) && ($info->{'is_submit'})) { + my @adds; + if (@tolock) { + @adds = map { "/uploaded/$udom/$uname/portfolio".$_; } @tolock; + } + &Apache::lonnet::automated_portfile_access('ip',\@adds,\@todelete,$info); + } + } + } + } + &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) = @_; - &Apache::lonnet::delenv($env{'form.HWFILE'.$jspart.'_'.$id.'.filename'}); - &Apache::lonnet::delenv($env{'form.HWFILE'.$jspart.'_'.$id.'.mimetype'}); - &Apache::lonnet::delenv($env{'form.HWFILE'.$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', @@ -399,7 +580,7 @@ sub check_collaborators { $env{'course.'.$env{'request.course.id'}.'.num'}); my (@badcollaborators,$result); - my (@collaborators) = split(/,?\s+/,$coll); + my (@collaborators) = split(/[,;\s]+/,$coll); foreach my $entry (@collaborators) { my $collaborator; if ($entry =~ /:/) { @@ -447,7 +628,7 @@ __END__ =head1 NAME -Apache::easyresponse +Apache::essayresponse =head1 SYNOPSIS @@ -462,8 +643,16 @@ described at http://www.lon-capa.org. =item start_essayresponse() +=item draft_row() + +=item get_collab_row() + +=item show_status_table() + =item end_essayresponse() +=item get_collaborators() + =item format_prior_response() =item file_submission()