');
if (! scalar(@Problems) || ! defined($Problems[0])) {
$r->print('resource is undefined');
} else {
@@ -117,16 +125,14 @@ sub BuildStudentSubmissionsPage {
my $resource = $Problems[0];
$r->print('
'.$resource->title.'
');
$r->print('
'.$resource->src.'
');
- if ($ENV{'form.renderprob'} eq 'true') {
- $r->print(
- &Apache::lonstathelpers::render_resource({src => $resource->src})
- );
+ if ($env{'form.renderprob'} eq 'true') {
+ $r->print(&Apache::lonstathelpers::render_resource($resource));
$r->rflush();
}
}
- if ($ENV{'form.output'} eq 'excel') {
+ if ($env{'form.output'} eq 'excel') {
&prepare_excel_output($r,\@Problems,\@Students);
- } elsif ($ENV{'form.output'} eq 'csv') {
+ } elsif ($env{'form.output'} eq 'csv') {
&prepare_csv_output($r,\@Problems,\@Students);
} else {
&prepare_html_output($r,\@Problems,\@Students);
@@ -141,7 +147,7 @@ sub BuildStudentSubmissionsPage {
&mt('Computing correct answers greatly increasese the amount of time required to prepare a report.').
'');
$r->print('
'.
- &mt('please select problems and use the Prepare Report button to continue.').
+ &mt('Please select problems and use the [_1]Prepare Report[_2] button to continue.','','').
'
');
$r->print(&Apache::lonstathelpers::MultipleProblemSelector
(undef,'problemchoice','Statistics'));
@@ -153,10 +159,10 @@ sub BuildStudentSubmissionsPage {
##
sub get_extra_response_headers {
my @extra_resp_headers;
- if ($ENV{'form.correctans'} eq 'true') {
+ if ($env{'form.correctans'} eq 'true') {
push(@extra_resp_headers,'Correct');
}
- if ($ENV{'form.prob_status'} eq 'true') {
+ if ($env{'form.prob_status'} eq 'true') {
push(@extra_resp_headers,'Award Detail');
push(@extra_resp_headers,'Time');
push(@extra_resp_headers,'Attempt');
@@ -189,6 +195,8 @@ sub get_headers {
shift(@basic_headers); # Get rid of 'Correct'
}
@headers = (@foils,@basic_headers);
+ } elsif (lc($resptype) eq 'task') {
+ @headers = ('Grader','Status',@basic_headers,'Submission');
} else {
@headers = ('Submission',@basic_headers);
}
@@ -217,7 +225,12 @@ sub prepare_html_output {
my @extra_resp_headers = &get_extra_response_headers();
#
# Create the table header
- my @student_columns = ('username','domain','id');
+ my @student_columns = @Apache::lonstatistics::SelectedStudentData;
+ foreach (@student_columns) {
+ if($_ eq 'all') {
+ @student_columns = ('fullname','username','domain','id','section','status','groups','comments');
+ }
+ }
#
my %headers;
my $student_column_count = scalar(@student_columns);
@@ -257,20 +270,27 @@ sub prepare_html_output {
$headers{'response'} .=
'
'.
&mt('Response [_1]',$responses->[$i]).'
';
- $headers{'student'}.= '
'.join('
',
+ $headers{'student'}.= '
'.
+ join('
',
@headers).
- '
';
+ '
';
}
}
+ if ($part_span == 0) {
+ next;
+ }
if (! $single_part) {
my $tmpname = $partid;
if ($partid =~/^\d+$/) {
$tmpname = $prob->part_display($partid);
}
+ if ($tmpname !~ /^part/) {
+ $tmpname = 'Part '.$tmpname;
+ }
$headers{'part'} .= qq{
';
}
#
# Figure out what it is we need to output for this student
@@ -336,8 +364,9 @@ sub prepare_html_output {
'html','normal',
@extra_resp_headers);
my $width = scalar(@headers);
+ next if ($width < 1);
my $resp_data;
- $resp_data->{'fake'} = qq{
};
+ $resp_data->{'fake'} = qq{
};
if (! defined($results)) {
$results = [];
}
@@ -346,7 +375,7 @@ sub prepare_html_output {
$maxrow = scalar(@$results);
}
for (my $j=scalar(@$results)-1;$j>=0;$j--) {
- if ($ENV{'form.all_sub'} ne 'true') {
+ if ($env{'form.all_sub'} ne 'true') {
next if ($j ne scalar(@$results)-1);
}
my $response = &hashify_response($results->[$j],
@@ -361,6 +390,14 @@ sub prepare_html_output {
$response,
$single_response).
'');
+ } elsif (lc($resptype) eq 'task') {
+ my $results =
+ &html_task_results(\@headers,
+ $prob,$partid,$respid,
+ $response,$resptype);
+ if ($results) {
+ push(@{$resp_data->{'real'}},$results);
+ }
} else {
push(@{$resp_data->{'real'}},
&html_non_essay_results(\@headers,
@@ -420,13 +457,14 @@ sub prepare_html_output {
undef(@essays);
}
} # end of student loop
+ $r->print(''.$/);
return;
}
sub hashify_response {
my ($response,$prob,$student,$partid,$respid) =@_;
my $resp_hash = {};
- if ($ENV{'form.correctans'} eq 'true') {
+ if ($env{'form.correctans'} eq 'true') {
$resp_hash->{'Correct'} =
&Apache::lonstathelpers::get_student_answer
($prob,$student->{'username'},$student->{'domain'},
@@ -434,14 +472,27 @@ sub hashify_response {
}
$resp_hash->{'Submission'} =
$response->[&Apache::loncoursedata::RDs_submission()];
- $resp_hash->{'Award Detail'} =
- $response->[&Apache::loncoursedata::RDs_awarddetail()];
$resp_hash->{'Time'} =
$response->[&Apache::loncoursedata::RDs_timestamp()];
$resp_hash->{'Attempt'} =
$response->[&Apache::loncoursedata::RDs_tries()];
$resp_hash->{'Awarded'} =
$response->[&Apache::loncoursedata::RDs_awarded()];
+ if ($prob->is_task()) {
+ $resp_hash->{'Grader'} =
+ $response->[&Apache::loncoursedata::RDs_response_eval_2()];
+ if ($resp_hash->{'Attempt'} eq '0') {
+ $resp_hash->{'Attempt'} = '';
+ }
+ $resp_hash->{'Award Detail'} =
+ $response->[&Apache::loncoursedata::RDs_part_award()];
+ $resp_hash->{'Status'} =
+ $response->[&Apache::loncoursedata::RDs_response_eval()];
+ } else {
+ $resp_hash->{'Award Detail'} =
+ $response->[&Apache::loncoursedata::RDs_awarddetail()];
+ }
+
return $resp_hash;
}
@@ -465,7 +516,7 @@ sub html_essay_results {
if (defined($respid)) {
$id .= ' '.$respid;
}
- $Str .= ''.$id.''.(' 'x4);
+ $Str .= ''.$id.''.(' 'x4);
}
#
shift(@$headers); # Get rid of the Submission header
@@ -474,11 +525,11 @@ sub html_essay_results {
$correct = &html_format_essay_sub($response->{'Correct'});
shift(@$headers);
}
- $Str .= ''.
+ $Str .= ''.
join('',
map {
(' 'x4).&mt($_.': [_1]',$response->{$_});
- } @$headers).'';
+ } @$headers).'';
if (@$headers || ! $single_response) {
$Str .= ' ';
}
@@ -504,20 +555,34 @@ sub html_format_essay_sub {
return $submission;
}
+sub html_task_results {
+ my ($headers,$prob,$partid,$respid,$response,$resptype) = @_;
+ if (! ref($headers) || ref($headers) ne 'ARRAY' || ! scalar(@$headers)) {
+ return '';
+ }
+
+ my @values;
+ @values = map { $response->{$_}; } @$headers;
+
+ my $td = '
';
+ my $str = $td.join('
'.$td,@values).'';
+ return $str;
+}
+
sub html_non_essay_results {
my ($headers,$prob,$partid,$respid,$response,$resptype) = @_;
if (! ref($headers) || ref($headers) ne 'ARRAY' || ! scalar(@$headers)) {
return '';
}
#
- my $submission = &HTML::Entities::decode($response->{'Submission'});
+ my $submission = &HTML::Entities::decode(&unescape($response->{'Submission'}));
return '' if (! defined($submission) || $submission eq '');
$submission =~ s/\\\"/\"/g;
$submission =~ s/\\\'/\'/g;
if ($resptype eq 'radiobutton') {
$submission = &HTML::Entities::encode($submission,'<>&"');
$submission =~ s/=([^=])$//;
- $submission = ''.$submission.'';
+ $submission = ''.$submission.'';
}
$response->{'Submission'} = $submission;
#
@@ -525,14 +590,14 @@ sub html_non_essay_results {
if ($resptype =~ /^(option|match|rank)$/) {
my %submission =
map {
- my ($foil,$value) = split('=',&Apache::lonnet::unescape($_));
+ my ($foil,$value) = split('=',&unescape($_));
($foil,$value);
} split('&',$response->{'Submission'});
my %correct;
if (exists($response->{'Correct'})) {
%correct =
map {
- my ($foil,$value)=split('=',&Apache::lonnet::unescape($_));
+ my ($foil,$value)=split('=',&unescape($_));
($foil,$value);
} split('&',$response->{'Correct'});
}
@@ -550,6 +615,8 @@ sub html_non_essay_results {
$option = $submission{$header};
}
push(@values,&HTML::Entities::encode($option));
+ } elsif ($original_header eq 'Time') {
+ push(@values,&Apache::lonlocal::locallocaltime($response->{$original_header}));
} else {
# A normal column
push(@values,$response->{$original_header});
@@ -622,28 +689,15 @@ sub prepare_excel_output {
$r->rflush();
#
# Create the excel spreadsheet
- my $filename = '/prtspool/'.
- $ENV{'user.name'}.'_'.$ENV{'user.domain'}.'_'.
- time.'_'.rand(1000000000).'.xls';
- my $workbook = Spreadsheet::WriteExcel->new('/home/httpd'.$filename);
- if (! defined($workbook)) {
- $r->log_error("Error creating excel spreadsheet $filename: $!");
- $r->print('
'.&mt("Unable to create new Excel file. ".
- "This error has been logged. ".
- "Please alert your LON-CAPA administrator").
- '
');
- return undef;
- }
- #
- $workbook->set_tempdir('/home/httpd/perl/tmp');
- #
- my $format = &Apache::loncommon::define_excel_formats($workbook);
+ my ($workbook,$filename,$format) =
+ &Apache::loncommon::create_workbook($r);
+ return if (! defined($workbook));
my $worksheet = $workbook->addworksheet('Student Submission Data');
#
# Add headers to the worksheet
my $rows_output = 0;
$worksheet->write($rows_output++,0,
- $ENV{'course.'.$ENV{'request.course.id'}.'.description'},
+ $env{'course.'.$env{'request.course.id'}.'.description'},
$format->{'h1'});
$rows_output++;
my $cols_output = 0;
@@ -655,7 +709,7 @@ sub prepare_excel_output {
$worksheet->write($partid_row,0,'Part ID',$format->{'bold'});
$worksheet->write($respid_row,0,'Response ID',$format->{'bold'});
# Student headers
- my @StudentColumns = ('username','domain','id');
+ my @StudentColumns = ('username','domain','id','section');
foreach (@StudentColumns) {
$worksheet->write($header_row,$cols_output++,ucfirst($_),
$format->{'bold'});
@@ -732,7 +786,7 @@ sub prepare_excel_output {
my $response_start_col = $start_col{$prob->symb}->{$partid}->{$respid};
for (my $j=scalar(@$results)-1;$j>=0;$j--) {
$cols_output = $response_start_col;
- if ($ENV{'form.all_sub'} ne 'true') {
+ if ($env{'form.all_sub'} ne 'true') {
next if ($j ne scalar(@$results)-1);
}
my $response = &hashify_response($results->[$j],
@@ -741,8 +795,10 @@ sub prepare_excel_output {
$partid,
$respid);
my @response_data =
- &excel_response_data(\@headers,$prob,$partid,
- $respid,$response,$resptype);
+ &compile_response_data(\@headers,$response,
+ $prob,$partid,$respid,
+ $resptype,
+ \&excel_format_item);
$worksheet->write_row($rows_output++,$cols_output,
\@response_data);
$cols_output+=scalar(@response_data);
@@ -754,14 +810,13 @@ sub prepare_excel_output {
}
}
# Fill in the remaining rows with the students data
- for (my $row = $student_row+1;$row<=$max_row;$row++) {
+ for (my $row = $student_row+1;$row<$max_row;$row++) {
my $cols = 0;
foreach my $field (@StudentColumns) {
$worksheet->write($row,$cols++,
$student->{$field});
}
}
- $rows_output++;
&Apache::lonhtmlcommon::Increment_PrgWin($r,\%prog_state,
'last student');
}
@@ -782,14 +837,21 @@ sub prepare_excel_output {
return;
}
-sub excel_response_data {
- my ($headers,$prob,$partid,$respid,$response,$resptype) = @_;
+sub compile_response_data {
+ my ($headers,$response,$prob,$partid,$respid,$resptype,$format) = @_;
if (! ref($headers) || ref($headers) ne 'ARRAY' || ! scalar(@$headers)) {
return ();
}
+ if (ref($format) ne 'CODE') {
+ $format = sub { return $_[0]; };
+ }
#
- my $submission = &HTML::Entities::decode($response->{'Submission'});
- return '' if (! defined($submission) || $submission eq '');
+ my $submission =
+ &HTML::Entities::decode
+ (&unescape($response->{'Submission'}));
+ if (!$prob->is_task()) {
+ return () if (! defined($submission) || $submission eq '');
+ }
$submission =~ s/\\\"/\"/g;
$submission =~ s/\\\'/\'/g;
if ($resptype eq 'radiobutton') {
@@ -801,14 +863,14 @@ sub excel_response_data {
if ($resptype =~ /^(option|match|rank)$/) {
my %submission =
map {
- my ($foil,$value) = split('=',&Apache::lonnet::unescape($_));
+ my ($foil,$value) = split('=',&unescape($_));
($foil,$value);
} split('&',$response->{'Submission'});
my %correct;
if (exists($response->{'Correct'})) {
%correct =
map {
- my ($foil,$value)=split('=',&Apache::lonnet::unescape($_));
+ my ($foil,$value)=split('=',&unescape($_));
($foil,$value);
} split('&',$response->{'Correct'});
}
@@ -825,16 +887,15 @@ sub excel_response_data {
} elsif (exists($submission{$header})) {
$option = $submission{$header};
}
- push(@values,&excel_format_item($option,$header));
+ push(@values,&{$format}($option,$header));
} else {
# A normal column
- push(@values,
- &excel_format_item($response->{$original_header},
+ push(@values,&{$format}($response->{$original_header},
$original_header));
}
}
} else {
- @values = map { &excel_format_item($response->{$_},$_); } @$headers;
+ @values = map { &{$format}($response->{$_},$_); } @$headers;
}
return @values;
}
@@ -842,7 +903,7 @@ sub excel_response_data {
sub excel_format_item {
my ($item,$type) = @_;
if ($type eq 'Time') {
- &Apache::lonstathelpers::calc_serial($item);
+ $item = &Apache::lonstathelpers::calc_serial($item);
} else {
if ($item =~ m/^=/) {
$item = ' '.$item;
@@ -880,63 +941,64 @@ sub prepare_csv_output {
# Open a file
my $outputfile;
my $filename = '/prtspool/'.
- $ENV{'user.name'}.'_'.$ENV{'user.domain'}.'_'.
+ $env{'user.name'}.'_'.$env{'user.domain'}.'_'.
time.'_'.rand(1000000000).'.csv';
unless ($outputfile = Apache::File->new('>/home/httpd'.$filename)) {
$r->log_error("Couldn't open $filename for output $!");
- $r->print("Problems occured in writing the csv file. ".
- "This error has been logged. ".
- "Please alert your LON-CAPA administrator.");
+ $r->print(
+ '
'
+ .&mt('Problems occurred in writing the CSV file.')
+ .' '.&mt('This error has been logged.')
+ .' '.&mt('Please alert your LON-CAPA administrator.')
+ .'
'
+ );
$outputfile = undef;
}
#
# Compute the number of columns per response
- my @extra_resp_headers;
- if ($ENV{'form.correctans'} eq 'true') {
- push(@extra_resp_headers,'Correct');
- }
- if ($ENV{'form.prob_status'} eq 'true') {
- push(@extra_resp_headers,'Award Detail');
- push(@extra_resp_headers,'Time');
- push(@extra_resp_headers,'Attempt');
- push(@extra_resp_headers,'Awarded');
- }
- my $response_multiplier = scalar(@extra_resp_headers);
+ my @extra_resp_headers = &get_extra_response_headers();
#
# Create the table header
- my @student_columns = ('username','domain','id');
+ my @student_columns = ('username','domain','id','section');
#
my %headers;
push(@{$headers{'student'}},@student_columns);
# Pad for the student data
foreach my $row ('problem','part','response') {
- foreach (@student_columns) {
- push(@{$headers{$row}},'');
- }
+ $headers{$row}=[map {''} @student_columns];
}
#
# we put the headers into the %headers hash
- my $prob_start_idx = 0;
+ my %problem_analysis;
+ my %start_col;
+ my $max_column = scalar(@student_columns);
foreach my $prob (@$problems) {
- $headers{'problem'}->[$prob_start_idx] = $prob->compTitle;
- my $part_start_idx = $prob_start_idx;
+ my %analysis = &Apache::lonstathelpers::get_problem_data($prob->src);
+ $problem_analysis{$prob->src}=\%analysis;
+ $headers{'problem'}->[$max_column] = $prob->compTitle;
foreach my $partid (@{$prob->parts}) {
- $headers{'part'}->[$part_start_idx] = $prob->part_display($partid);
+ $headers{'part'}->[$max_column] = $prob->part_display($partid);
my $responses = [$prob->responseIds($partid)];
+ my $resptypes = [$prob->responseType($partid)];
for (my $i=0;$i[$resp_idx]=
+ my @headers = &get_headers($prob,$partid,$responses->[$i],
+ $resptypes->[$i],
+ $problem_analysis{$prob->src},
+ 'csv','display',
+ @extra_resp_headers);
+ $start_col{$prob->symb}->{$partid}->{$responses->[$i]}=
+ $max_column;
+ $headers{'response'}->[$max_column]=
&mt('Response [_1]',$responses->[$i]);
- for (my $j=0;$j<=$#extra_resp_headers;$j++) {
- $headers{'student'}->[$resp_idx+$j]=$extra_resp_headers[$j];
+ for (my $j=0;$j<=$#headers;$j++) {
+ $headers{'student'}->[$max_column+$j]=$headers[$j];
}
+ $max_column += scalar(@headers);
}
- $part_start_idx += scalar(@$responses)*$response_multiplier;
}
- $prob_start_idx += $prob->countResponses * $response_multiplier;
}
foreach my $row ('problem','part','response','student') {
- print $outputfile '"'.join('","',map { ''; } @student_columns).'","'.
+ print $outputfile '"'.
join('","',
map {
&Apache::loncommon::csv_translate($_);
@@ -947,15 +1009,17 @@ sub prepare_csv_output {
foreach my $student (@$students) {
last if ($c->aborted());
my @rows;
- my $prob_start_idx = 0;
foreach my $prob (@$problems) {
- my $part_start_idx = 0;
foreach my $partid (@{$prob->parts}) {
my @responses = $prob->responseIds($partid);
my @response_type = $prob->responseType($partid);
for (my $i=0;$i<=$#responses;$i++) {
- my $resp_start_idx = $response_multiplier * $i;
my $respid = $responses[$i];
+ my $resptype = $response_type[$i];
+ my @headers = &get_headers($prob,$partid,$respid,$resptype,
+ $problem_analysis{$prob->src},
+ 'csv','normal',
+ @extra_resp_headers);
my $results =
&Apache::loncoursedata::get_response_data_by_student
($student,$prob->symb(),$respid);
@@ -963,30 +1027,31 @@ sub prepare_csv_output {
$results = [];
}
for (my $j=0; $j[$idx];
- my @data = &compile_response_data($response,$student,
+ my $response = &hashify_response($results->[$idx],
+ $prob,$student,
+ $partid,$respid);
+ my @data = &compile_response_data(\@headers,$response,
$prob,$partid,
- $respid);
+ $respid,$resptype,
+ \&csv_format_item);
+ my $resp_start_idx =
+ $start_col{$prob->symb}->{$partid}->{$respid};
for (my $k=0;$k<=$#data;$k++) {
- $rows[$j]->[$prob_start_idx + $part_start_idx +
- $resp_start_idx + $k] = $data[$k];
+ $rows[$j]->[$resp_start_idx + $k] = $data[$k];
}
}
}
- $part_start_idx += scalar(@responses)*$response_multiplier;
}
- $prob_start_idx += $prob->countResponses * $response_multiplier;
}
foreach my $row (@rows) {
print $outputfile '"'.join('","',
map { $student->{$_}; }
@student_columns).'"';
-
- for (my $i=0;$i<$prob_start_idx;$i++) {
+ for (my $i=scalar(@student_columns);$i<$max_column;$i++) {
my $value = &Apache::loncommon::csv_translate($row->[$i]);
$value ||='';
print $outputfile ',"'.$value.'"';
@@ -1002,31 +1067,20 @@ sub prepare_csv_output {
# Close the progress window
&Apache::lonhtmlcommon::Close_PrgWin($r,\%prog_state);
#
- # Tell the user where to get their csv file
+ # Tell the user where to get their CSV file
$r->print(' '.
- ''.&mt('Your csv file.').''."\n");
+ ''.&mt('Your CSV file.').''."\n");
$r->rflush();
return;
}
-sub compile_response_data {
- my ($response,$student,$prob,$partid,$respid) = @_;
- my @rowdata;
- push(@rowdata,&Apache::lonnet::unescape($response->[&Apache::loncoursedata::RDs_submission()]));
- if ($ENV{'form.correctans'} eq 'true') {
- my $correct = &Apache::lonstathelpers::get_student_answer
- ($prob,$student->{'username'},$student->{'domain'},
- $partid,$respid);
- push(@rowdata,$correct);
- }
- if ($ENV{'form.prob_status'}) {
- push(@rowdata,$response->[&Apache::loncoursedata::RDs_awarddetail()]);
- push(@rowdata,&Apache::lonlocal::locallocaltime
- ($response->[&Apache::loncoursedata::RDs_timestamp()]));
- push(@rowdata,$response->[&Apache::loncoursedata::RDs_tries()]);
- push(@rowdata,$response->[&Apache::loncoursedata::RDs_awarded()]);
+sub csv_format_item {
+ my ($item,$type) = @_;
+ if ($type eq 'Time') {
+ $item = localtime($item);
}
- return @rowdata;
+ $item =&Apache::loncommon::csv_translate($item);
+ return $item;
}
#########################################################
@@ -1042,7 +1096,7 @@ sub CreateInterface {
my $output_selector = $/.'