'."\n";
@@ -357,6 +354,9 @@ sub CreateInterface {
$Str .= '';
$Str .= ' 'x5;
+ $Str .=
+ &mt('Status [_1]',
+ '');
$Str .= ' ';
return $Str;
}
@@ -432,14 +432,9 @@ sub CreateAndParseOutputSelector {
[$elementname]);
#
# Format for output options is 'mode, restrictions';
- my $selected = 'html, without links';
- if (exists($ENV{'form.'.$elementname})) {
- if (ref($ENV{'form.'.$elementname} eq 'ARRAY')) {
- $selected = $ENV{'form.'.$elementname}->[0];
- } else {
- $selected = $ENV{'form.'.$elementname};
- }
- }
+ my $selected = (&Apache::loncommon::get_env_multiple('form.'.$elementname))[0];
+ $selected = 'html, without links' if (!$selected);
+
#
# Set package variables describing output mode
$show_links = 'no';
@@ -479,6 +474,7 @@ my @OutputDataOptions =
grand_maximum => 1,
summary_table => 1,
maximum_row => 1,
+ ignore_weight => 0,
shortdesc => 'Total Score and Maximum Possible for each '.
'Sequence or Folder',
longdesc => 'The score of each student as well as the '.
@@ -497,6 +493,7 @@ my @OutputDataOptions =
grand_maximum => 1,
summary_table => 1,
maximum_row => 1,
+ ignore_weight => 0,
shortdesc => 'Score on each Problem Part',
longdesc =>'The students score on each problem part, computed as'.
'the part weight * part awarded',
@@ -514,8 +511,10 @@ my @OutputDataOptions =
grand_maximum => 0,
summary_table => 0,
maximum_row => 0,
+ ignore_weight => 0,
shortdesc => 'Number of Tries before success on each Problem Part',
longdesc =>'The number of tries before success on each problem part.',
+ non_html_notes => 'negative values indicate an incorrect problem',
},
{ name =>'Parts Correct',
base =>'tries',
@@ -530,6 +529,7 @@ my @OutputDataOptions =
grand_maximum => 1,
summary_table => 1,
maximum_row => 0,
+ ignore_weight => 1,
shortdesc => 'Number of Problem Parts completed successfully.',
longdesc => 'The Number of Problem Parts completed successfully and '.
'the maximum possible for each student',
@@ -552,14 +552,9 @@ sub CreateAndParseOutputDataSelector {
my $Str = '';
my $elementname = 'chartoutputdata';
#
- my $selected = 'scores';
- if (exists($ENV{'form.'.$elementname})) {
- if (ref($ENV{'form.'.$elementname} eq 'ARRAY')) {
- $selected = $ENV{'form.'.$elementname}->[0];
- } else {
- $selected = $ENV{'form.'.$elementname};
- }
- }
+ my $selected = (&Apache::loncommon::get_env_multiple('form.'.$elementname))[0];
+ $selected = 'scores' if (!$selected);
+
#
$chosen_output = $OutputDataOptions[0];
foreach my $option (@OutputDataOptions) {
@@ -582,6 +577,26 @@ sub CreateAndParseOutputDataSelector {
#######################################################
#######################################################
+sub count_parts {
+ my ($navmap,$sequence) = @_;
+ my @resources = &get_resources($navmap,$sequence);
+ my $count = 0;
+ foreach my $res (@resources) {
+ $count += scalar(@{$res->parts});
+ }
+ return $count;
+}
+
+sub get_resources {
+ my ($navmap,$sequence) = @_;
+ my @resources = $navmap->retrieveResources($sequence,
+ sub { shift->is_problem(); },
+ 0,0,0);
+ return @resources;
+}
+
+#######################################################
+#######################################################
=pod
@@ -609,17 +624,49 @@ Return a line of the chart for a student
my %prog_state; # progress state used by loncommon PrgWin routines
my $total_sum_width;
+ my %width; # Holds sequence width information
+ my @sequences;
+ my $navmap; # Have to keep this around since weakref is a bit zealous
+
+sub html_cleanup {
+ undef(%prog_state);
+ undef(%width);
+ #
+ undef($navmap);
+ undef(@sequences);
+}
+
sub html_initialize {
my ($r) = @_;
#
$padding = ' 'x3;
$count = 0;
$nodata_count = 0;
- undef(%prog_state);
+ &html_cleanup();
+ ($navmap,@sequences) =
+ &Apache::lonstatistics::selected_sequences_with_assessments();
+ if (! ref($navmap)) {
+ # Unable to get data, so bail out
+ $r->print("
".
+ &mt('Unable to retrieve course information.').
+ '
');
+ }
+
+ # If we're showing links, show a checkbox to open in new
+ # windows.
+ if ($show_links ne 'no') {
+ $r->print(<new_window = true;
+
+NEW_WINDOW_CHECKBOX
+ }
+
#
- $r->print("
'.$/;
- foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
+ foreach my $seq (@sequences) {
+ my $symb = $seq->symb;
my $ave;
- my $num_students = $Statistics->{$seq->{'symb'}}->{'num_students'};
+ my $num_students = $Statistics->{$symb}->{'num_students'};
if ($num_students > 0) {
$ave = int(100*
- ($Statistics->{$seq->{'symb'}}->{'score'}/$num_students)
+ ($Statistics->{$symb}->{'score'}/$num_students)
)/100;
} else {
$ave = 0;
}
- my $max = $Statistics->{$seq->{'symb'}}->{'max'};
+ my $max = $Statistics->{$symb}->{'max'};
$ave = sprintf("%.2f",$ave);
- $Str .= '
'.$seq->{'title'}.'
'.
+ $Str .= '
'.$seq->compTitle.'
'.
'
'.$ave.'
'.
'
'.$max.' '.'
'."\n";
}
@@ -833,6 +968,7 @@ sub StudentAverageTotal {
}
sub SingleStudentTotal {
+ return if (! defined($navmap));
my $student = &Apache::lonstatistics::current_student();
my $Str = '
'.&mt('Summary table for [_1] ([_2]@[_3])',
$student->{'fullname'},
@@ -850,10 +986,10 @@ sub SingleStudentTotal {
$Str .= '
'.&mt('Maximum').'
'."\n";
my $total = 0;
my $total_max = 0;
- foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
- my $value = $Statistics->{$seq->{'symb'}}->{'score'};
- my $max = $Statistics->{$seq->{'symb'}}->{'max'};
- $Str .= '
'.$seq->{'title'}.'
'.
+ foreach my $seq (@sequences) {
+ my $value = $Statistics->{$seq->symb}->{'score'};
+ my $max = $Statistics->{$seq->symb}->{'max'};
+ $Str .= '
'.&HTML::Entities::encode($seq->compTitle).'
'.
'
'.$value.'
'.
'
'.$max.'
'."\n";
$total += $value;
@@ -889,6 +1025,7 @@ sub SingleStudentTotal {
my $excel_sheet;
my $excel_workbook;
+my $format;
my $filename;
my $rows_output;
@@ -899,9 +1036,12 @@ my $request_aborted;
my $total_formula;
my $maximum_formula;
+my %formula_data;
-sub excel_initialize {
- my ($r) = @_;
+my $navmap;
+my @sequences;
+
+sub excel_cleanup {
#
undef ($excel_sheet);
undef ($excel_workbook);
@@ -913,12 +1053,31 @@ sub excel_initialize {
undef ($total_formula);
undef ($maximum_formula);
#
+ undef(%formula_data);
+ #
+ undef($navmap);
+ undef(@sequences);
+}
+
+sub excel_initialize {
+ my ($r) = @_;
+
+ &excel_cleanup();
+ ($navmap,@sequences) =
+ &Apache::lonstatistics::selected_sequences_with_assessments();
+ if (! ref($navmap)) {
+ # Unable to get data, so bail out
+ $r->print("
".
+ &mt('Unable to retrieve course information.').
+ '
');
+ }
+ #
my $total_columns = scalar(&get_student_fields_to_show());
my $num_students = scalar(@Apache::lonstatistics::Students);
#
- foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
+ foreach my $seq (@sequences) {
if ($chosen_output->{'every_problem'}) {
- $total_columns += $seq->{'num_assess_parts'};
+ $total_columns+=&count_parts($navmap,$seq);
}
# Add 2 because we need a 'sequence_sum' and 'total' column for each
$total_columns += 2;
@@ -938,9 +1097,6 @@ sub excel_initialize {
}
return if ($request_aborted);
#
- $filename = '/prtspool/'.
- $ENV{'user.name'}.'_'.$ENV{'user.domain'}.'_'.
- time.'_'.rand(1000000000).'.xls';
#
$excel_workbook = undef;
$excel_sheet = undef;
@@ -951,11 +1107,12 @@ sub excel_initialize {
# Determine rows
my $header_row = $rows_output++;
my $description_row = $rows_output++;
+ my $notes_row = $rows_output++;
$rows_output++; # blank row
my $summary_header_row;
if ($chosen_output->{'summary_table'}) {
$summary_header_row = $rows_output++;
- $rows_output+= scalar(&Apache::lonstatistics::Sequences_with_Assess());
+ $rows_output+= scalar(@sequences);
$rows_output++;
}
my $sequence_name_row = $rows_output++;
@@ -967,56 +1124,27 @@ sub excel_initialize {
my $first_data_row = $rows_output++;
#
# Create sheet
- $excel_workbook = Spreadsheet::WriteExcel->new('/home/httpd'.$filename);
- #
- # Check for errors
- if (! defined($excel_workbook)) {
- $r->log_error("Error creating excel spreadsheet $filename: $!");
- $r->print("Problems creating new Excel file. ".
- "This error has been logged. ".
- "Please alert your LON-CAPA administrator");
- return ;
- }
- #
- # The excel spreadsheet stores temporary data in files, then put them
- # together. If needed we should be able to disable this (memory only).
- # The temporary directory must be specified before calling 'addworksheet'.
- # File::Temp is used to determine the temporary directory.
- $excel_workbook->set_tempdir($Apache::lonnet::tmpdir);
- #
- my $format = &Apache::loncommon::define_excel_formats($excel_workbook);
+ ($excel_workbook,$filename,$format)=
+ &Apache::loncommon::create_workbook($r);
+ return if (! defined($excel_workbook));
#
# Add a worksheet
- my $sheetname = $ENV{'course.'.$ENV{'request.course.id'}.'.description'};
+ my $sheetname = $env{'course.'.$env{'request.course.id'}.'.description'};
$sheetname = &Apache::loncommon::clean_excel_name($sheetname);
$excel_sheet = $excel_workbook->addworksheet($sheetname);
#
# Put the course description in the header
$excel_sheet->write($header_row,$cols_output++,
- $ENV{'course.'.$ENV{'request.course.id'}.'.description'},
+ $env{'course.'.$env{'request.course.id'}.'.description'},
$format->{'h1'});
$cols_output += 3;
#
# Put a description of the sections listed
my $sectionstring = '';
- my @Sections = @Apache::lonstatistics::SelectedSections;
- if (scalar(@Sections) > 1) {
- if (scalar(@Sections) > 2) {
- my $last = pop(@Sections);
- $sectionstring = "Sections ".join(', ',@Sections).', and '.$last;
- } else {
- $sectionstring = "Sections ".join(' and ',@Sections);
- }
- } else {
- if ($Sections[0] eq 'all') {
- $sectionstring = "All sections";
- } else {
- $sectionstring = "Section ".$Sections[0];
- }
- }
- $excel_sheet->write($header_row,$cols_output++,$sectionstring,
+ my @Sections = &Apache::lonstatistics::get_selected_sections();
+ $excel_sheet->write($header_row,$cols_output++,
+ &Apache::lonstatistics::section_and_enrollment_description('plaintext'),
$format->{'h3'});
- $cols_output += scalar(@Sections);
#
# Put the date in there too
$excel_sheet->write($header_row,$cols_output++,
@@ -1025,7 +1153,13 @@ sub excel_initialize {
$cols_output = 0;
$excel_sheet->write($description_row,$cols_output++,
$chosen_output->{'shortdesc'},
- $format->{'h3'});
+ $format->{'b'});
+ #
+ $cols_output = 0;
+ $excel_sheet->write($notes_row,$cols_output++,
+ $chosen_output->{'non_html_notes'},
+ $format->{'i'});
+
##############################################
# Output headings for the raw data
##############################################
@@ -1040,63 +1174,60 @@ sub excel_initialize {
# Add the remaining column headers
my $total_formula_string = '=0';
my $maximum_formula_string = '=0';
- foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
+ foreach my $seq (@sequences) {
+ my $symb = $seq->symb;
$excel_sheet->write($sequence_name_row,,
- $cols_output,$seq->{'title'},$format->{'bold'});
+ $cols_output,$seq->compTitle,$format->{'bold'});
# Determine starting cell
- $seq->{'Excel:startcell'}=
+ $formula_data{$symb}->{'Excel:startcell'}=
&Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
($first_data_row,$cols_output);
- $seq->{'Excel:startcol'}=$cols_output;
+ $formula_data{$symb}->{'Excel:startcol'}=$cols_output;
my $count = 0;
if ($chosen_output->{'every_problem'}) {
# Put the names of the problems and parts into the sheet
- foreach my $res (@{$seq->{'contents'}}) {
- if ($res->{'type'} ne 'assessment' ||
- ! exists($res->{'parts'}) ||
- ref($res->{'parts'}) ne 'ARRAY' ||
- scalar(@{$res->{'parts'}}) < 1) {
- next;
- }
- if (scalar(@{$res->{'parts'}}) > 1) {
- foreach my $part (@{$res->{'parts'}}) {
+ foreach my $res (&get_resources($navmap,$seq)) {
+ if (scalar(@{$res->parts}) > 1) {
+ foreach my $part (@{$res->parts}) {
$excel_sheet->write($resource_name_row,
$cols_output++,
- $res->{'title'}.' part '.$part,
+ $res->compTitle.' part '.$res->part_display($part),
$format->{'bold'});
$count++;
}
} else {
$excel_sheet->write($resource_name_row,
$cols_output++,
- $res->{'title'},$format->{'bold'});
+ $res->compTitle,$format->{'bold'});
$count++;
}
}
}
# Determine ending cell
if ($count <= 1) {
- $seq->{'Excel:endcell'} = $seq->{'Excel:startcell'};
- $seq->{'Excel:endcol'} = $seq->{'Excel:startcol'};
+ $formula_data{$symb}->{'Excel:endcell'} = $formula_data{$symb}->{'Excel:startcell'};
+ $formula_data{$symb}->{'Excel:endcol'} = $formula_data{$symb}->{'Excel:startcol'};
} else {
- $seq->{'Excel:endcell'} =
+ $formula_data{$symb}->{'Excel:endcell'} =
&Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
($first_data_row,$cols_output-1);
- $seq->{'Excel:endcol'} = $cols_output-1;
+ $formula_data{$symb}->{'Excel:endcol'} = $cols_output-1;
}
# Create the formula for summing up this sequence
- if (! exists($seq->{'Excel:endcell'}) ||
- ! defined($seq->{'Excel:endcell'})) {
- $seq->{'Excel:endcell'} = $seq->{'Excel:startcell'};
- }
- $seq->{'Excel:sum'}= $excel_sheet->store_formula
- ('=SUM('.$seq->{'Excel:startcell'}.
- ':'.$seq->{'Excel:endcell'}.')');
+ if (! exists($formula_data{$symb}->{'Excel:endcell'}) ||
+ ! defined($formula_data{$symb}->{'Excel:endcell'})) {
+ $formula_data{$symb}->{'Excel:endcell'} = $formula_data{$symb}->{'Excel:startcell'};
+ }
+
+ my $start = $formula_data{$symb}->{'Excel:startcell'};
+ my $end = $formula_data{$symb}->{'Excel:endcell'};
+ $formula_data{$symb}->{'Excel:sum'}= $excel_sheet->store_formula
+ ("=IF(COUNT($start\:$end),SUM($start\:$end),\"\")");
# Determine cell the score is held in
- $seq->{'Excel:scorecell'} =
+ $formula_data{$symb}->{'Excel:scorecell'} =
&Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
($first_data_row,$cols_output);
- $seq->{'Excel:scorecol'}=$cols_output;
+ $formula_data{$symb}->{'Excel:scorecol'}=$cols_output;
if ($chosen_output->{'base'} eq 'parts correct total') {
$excel_sheet->write($resource_name_row,$cols_output++,
'parts correct',
@@ -1121,10 +1252,10 @@ sub excel_initialize {
$excel_sheet->write($resource_name_row,$cols_output,
'maximum',
$format->{'bold'});
- $seq->{'Excel:maxcell'} =
+ $formula_data{$symb}->{'Excel:maxcell'} =
&Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
($first_data_row,$cols_output);
- $seq->{'Excel:maxcol'}=$cols_output;
+ $formula_data{$symb}->{'Excel:maxcol'}=$cols_output;
$maximum_formula_string.='+'.
&Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
($first_data_row,$cols_output);
@@ -1160,23 +1291,23 @@ sub excel_initialize {
# Add the maximums for each sequence or assessment
my %total_cell_translation;
my %maximum_cell_translation;
- foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
- $cols_output=$seq->{'Excel:startcol'};
- $total_cell_translation{$seq->{'Excel:scorecell'}} =
+ foreach my $seq (@sequences) {
+ my $symb = $seq->symb;
+ $cols_output=$formula_data{$symb}->{'Excel:startcol'};
+ $total_cell_translation{$formula_data{$symb}->{'Excel:scorecell'}}=
&Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
- ($maximum_data_row,$seq->{'Excel:scorecol'});
- $maximum_cell_translation{$seq->{'Excel:maxcell'}} =
+ ($maximum_data_row,$formula_data{$symb}->{'Excel:scorecol'});
+ $maximum_cell_translation{$formula_data{$symb}->{'Excel:maxcell'}}=
&Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
- ($maximum_data_row,$seq->{'Excel:maxcol'});
+ ($maximum_data_row,$formula_data{$symb}->{'Excel:maxcol'});
my $weight;
my $max = 0;
- foreach my $resource (@{$seq->{'contents'}}) {
- next if ($resource->{'type'} ne 'assessment');
- foreach my $part (@{$resource->{'parts'}}) {
+ foreach my $resource (&get_resources($navmap,$seq)) {
+ foreach my $part (@{$resource->parts}){
$weight = 1;
if ($chosen_output->{'scores'}) {
$weight = &Apache::lonnet::EXT
- ('resource.'.$part.'.weight',$resource->{'symb'},
+ ('resource.'.$part.'.weight',$resource->symb,
undef,undef,undef);
if (!defined($weight) || ($weight eq '')) {
$weight=1;
@@ -1193,16 +1324,15 @@ sub excel_initialize {
#
if ($chosen_output->{'sequence_sum'} &&
$chosen_output->{'every_problem'}) {
- my %replaceCells;
- $replaceCells{$seq->{'Excel:startcell'}} =
- &Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
- ($maximum_data_row,$seq->{'Excel:startcol'});
- $replaceCells{$seq->{'Excel:endcell'}} =
- &Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
- ($maximum_data_row,$seq->{'Excel:endcol'});
+ my %replaceCells=
+ ('^'.$formula_data{$symb}->{'Excel:startcell'}.':'.
+ $formula_data{$symb}->{'Excel:endcell'}.'$' =>
+ &Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell($maximum_data_row,$formula_data{$symb}->{'Excel:startcol'}).':'.
+ &Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell($maximum_data_row,$formula_data{$symb}->{'Excel:endcol'}));
$excel_sheet->repeat_formula($maximum_data_row,$cols_output++,
- $seq->{'Excel:sum'},undef,
- %replaceCells);
+ $formula_data{$symb}->{'Excel:sum'},undef,
+ %replaceCells, %replaceCells);
+
} elsif ($chosen_output->{'sequence_sum'}) {
$excel_sheet->write($maximum_data_row,$cols_output++,$max);
}
@@ -1241,25 +1371,26 @@ sub excel_initialize {
$excel_sheet->write($summary_header_row,$cols_output++,
'Std Dev',$format->{'bold'});
my $row = $summary_header_row+1;
- foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
+ foreach my $seq (@sequences) {
+ my $symb = $seq->symb;
$cols_output = 0;
$excel_sheet->write($row,$cols_output++,
- $seq->{'title'},
+ $seq->compTitle,
$format->{'bold'});
if ($chosen_output->{'maximum_row'}) {
$excel_sheet->write
($row,$cols_output++,
'='.
&Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
- ($maximum_data_row,$seq->{'Excel:scorecol'})
+ ($maximum_data_row,$formula_data{$symb}->{'Excel:scorecol'})
);
}
my $range =
&Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
- ($first_data_row,$seq->{'Excel:scorecol'}).
+ ($first_data_row,$formula_data{$symb}->{'Excel:scorecol'}).
':'.
&Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
- ($first_data_row+$num_students-1,$seq->{'Excel:scorecol'});
+ ($first_data_row+$num_students-1,$formula_data{$symb}->{'Excel:scorecol'});
$excel_sheet->write($row,$cols_output++,
'=AVERAGE('.$range.')');
$excel_sheet->write($row,$cols_output++,
@@ -1275,7 +1406,7 @@ sub excel_initialize {
#
# Let the user know what we are doing
my $studentcount = scalar(@Apache::lonstatistics::Students);
- if ($ENV{'form.SelectedStudent'}) {
+ if ($env{'form.SelectedStudent'}) {
$studentcount = '1';
}
if ($studentcount > 1) {
@@ -1291,7 +1422,8 @@ sub excel_initialize {
# Initialize progress window
%prog_state=&Apache::lonhtmlcommon::Create_PrgWin
($r,'Excel File Compilation Status',
- 'Excel File Compilation Progress', $studentcount);
+ 'Excel File Compilation Progress', $studentcount,
+ 'inline',undef,'Statistics','stats_status');
#
&Apache::lonhtmlcommon::Update_PrgWin($r,\%prog_state,
'Processing first student');
@@ -1300,8 +1432,9 @@ sub excel_initialize {
sub excel_outputstudent {
my ($r,$student) = @_;
- return if ($request_aborted);
- return if (! defined($excel_sheet));
+ if ($request_aborted || ! defined($navmap) || ! defined($excel_sheet)) {
+ return;
+ }
$cols_output=0;
#
# Write out student data
@@ -1309,7 +1442,7 @@ sub excel_outputstudent {
foreach my $field (@to_show) {
my $value = $student->{$field};
if ($field eq 'comments') {
- $value = &Apache::lonmsg::retrieve_instructor_comments
+ $value = &Apache::lonmsgdisplay::retrieve_instructor_comments
($student->{'username'},$student->{'domain'});
}
$excel_sheet->write($rows_output,$cols_output++,$value);
@@ -1320,7 +1453,7 @@ sub excel_outputstudent {
my @tmp = &Apache::loncoursedata::get_current_state($student->{'username'},
$student->{'domain'},
undef,
- $ENV{'request.course.id'});
+ $env{'request.course.id'});
if ((scalar @tmp > 0) && ($tmp[0] !~ /^error:/)) {
%StudentsData = @tmp;
}
@@ -1328,33 +1461,36 @@ sub excel_outputstudent {
# Write out sequence scores and totals data
my %total_cell_translation;
my %maximum_cell_translation;
- foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
- $cols_output = $seq->{'Excel:startcol'};
+ foreach my $seq (@sequences) {
+ my $symb = $seq->symb;
+ $cols_output = $formula_data{$symb}->{'Excel:startcol'};
# Keep track of cells to translate in total cell
- $total_cell_translation{$seq->{'Excel:scorecell'}} =
+ $total_cell_translation{$formula_data{$symb}->{'Excel:scorecell'}} =
&Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
- ($rows_output,$seq->{'Excel:scorecol'});
+ ($rows_output,$formula_data{$symb}->{'Excel:scorecol'});
# and maximum cell
- $maximum_cell_translation{$seq->{'Excel:maxcell'}} =
+ $maximum_cell_translation{$formula_data{$symb}->{'Excel:maxcell'}} =
&Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
- ($rows_output,$seq->{'Excel:maxcol'});
+ ($rows_output,$formula_data{$symb}->{'Excel:maxcol'});
#
my ($performance,$performance_length,$score,$seq_max,$rawdata);
if ($chosen_output->{'tries'} || $chosen_output->{'correct'}){
($performance,$performance_length,$score,$seq_max,$rawdata) =
- &StudentTriesOnSequence($student,\%StudentsData,
- $seq,'no');
+ &student_tries_on_sequence($student,\%StudentsData,
+ $navmap,$seq,'no');
} else {
($performance,$performance_length,$score,$seq_max,$rawdata) =
- &StudentPerformanceOnSequence($student,\%StudentsData,
- $seq,'no');
+ &student_performance_on_sequence($student,\%StudentsData,
+ $navmap,$seq,'no',
+ $chosen_output->{ignore_weight});
}
if ($chosen_output->{'every_problem'}) {
if ($chosen_output->{'correct'}) {
# only indiciate if each item is correct or not
foreach my $value (@$rawdata) {
- # nonzero means correct
- $value = 1 if ($value > 0);
+ # positive means correct, 0 or negative means
+ # incorrect
+ $value = $value > 0 ? 1 : 0;
$excel_sheet->write($rows_output,$cols_output++,$value);
}
} else {
@@ -1371,23 +1507,16 @@ sub excel_outputstudent {
if ($chosen_output->{'sequence_sum'} &&
$chosen_output->{'every_problem'}) {
# Write a formula for the sum of this sequence
- my %replaceCells;
- $replaceCells{$seq->{'Excel:startcell'}} =
- &Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
- ($rows_output,$seq->{'Excel:startcol'});
- $replaceCells{$seq->{'Excel:endcell'}} =
- &Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell
- ($rows_output,$seq->{'Excel:endcol'});
- # The undef is for the format
- if (scalar(keys(%replaceCells)) == 1) {
- $excel_sheet->repeat_formula($rows_output,$cols_output++,
- $seq->{'Excel:sum'},undef,
- %replaceCells,%replaceCells);
- } else {
- $excel_sheet->repeat_formula($rows_output,$cols_output++,
- $seq->{'Excel:sum'},undef,
- %replaceCells);
- }
+ my %replaceCells=
+ ('^'.$formula_data{$symb}->{'Excel:startcell'}.':'.$formula_data{$symb}->{'Excel:endcell'}.'$'
+ =>
+ &Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell($rows_output,$formula_data{$symb}->{'Excel:startcol'}).':'.
+ &Spreadsheet::WriteExcel::Utility::xl_rowcol_to_cell($rows_output,$formula_data{$symb}->{'Excel:endcol'})
+ );
+ # The undef is for the format
+ $excel_sheet->repeat_formula($rows_output,$cols_output++,
+ $formula_data{$symb}->{'Excel:sum'},undef,
+ %replaceCells, %replaceCells);
} elsif ($chosen_output->{'sequence_sum'}) {
if ($score eq ' ') {
$cols_output++;
@@ -1422,14 +1551,13 @@ sub excel_outputstudent {
sub excel_finish {
my ($r) = @_;
- return if ($request_aborted);
- return if (! defined($excel_sheet));
+ if ($request_aborted || ! defined($navmap) || ! defined($excel_sheet)) {
+ &excel_cleanup();
+ return;
+ }
#
# Write the excel file
$excel_workbook->close();
- my $c = $r->connection();
- #
- return if($c->aborted());
#
# Close the progress window
&Apache::lonhtmlcommon::Close_PrgWin($r,\%prog_state);
@@ -1438,6 +1566,7 @@ sub excel_finish {
$r->print(' '.
'Your Excel spreadsheet.'."\n");
$r->rflush();
+ &excel_cleanup();
return;
}
@@ -1465,16 +1594,32 @@ my $outputfile;
my $filename;
my $request_aborted;
my %prog_state; # progress window state
+my $navmap;
+my @sequences;
-sub csv_initialize{
- my ($r) = @_;
- #
- # Clean up
+sub csv_cleanup {
undef($outputfile);
undef($filename);
undef($request_aborted);
undef(%prog_state);
#
+ undef($navmap);
+ undef(@sequences);
+}
+
+sub csv_initialize{
+ my ($r) = @_;
+
+ &csv_cleanup();
+ ($navmap,@sequences) =
+ &Apache::lonstatistics::selected_sequences_with_assessments();
+ if (! ref($navmap)) {
+ # Unable to get data, so bail out
+ $r->print("
".
+ &mt('Unable to retrieve course information.').
+ '
');
+ }
+ #
# Deal with unimplemented requests
$request_aborted = undef;
if ($chosen_output->{'base'} =~ /final table/) {
@@ -1492,25 +1637,28 @@ END
my $studentcount = scalar(@Apache::lonstatistics::Students);
%prog_state=&Apache::lonhtmlcommon::Create_PrgWin
($r,'CSV File Compilation Status',
- 'CSV File Compilation Progress', $studentcount);
+ 'CSV File Compilation Progress', $studentcount,
+ 'inline',undef,'Statistics','stats_status');
#
# Open a file
- $filename = '/prtspool/'.
- $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.");
- $outputfile = undef;
- }
+ ($outputfile,$filename) = &Apache::loncommon::create_text_file($r,'csv');
+ if (! defined($outputfile)) { return ''; }
#
# Datestamp
- my $description = $ENV{'course.'.$ENV{'request.course.id'}.'.description'};
+ my $description = $env{'course.'.$env{'request.course.id'}.'.description'};
print $outputfile '"'.&Apache::loncommon::csv_translate($description).'",'.
'"'.&Apache::loncommon::csv_translate(scalar(localtime(time))).'"'.
"\n";
+ print $outputfile '"'.
+ &Apache::loncommon::csv_translate
+ (&Apache::lonstatistics::section_and_enrollment_description()).
+ '"'."\n";
+ foreach my $item ('shortdesc','non_html_notes') {
+ next if (! exists($chosen_output->{$item}));
+ print $outputfile
+ '"'.&Apache::loncommon::csv_translate($chosen_output->{$item}).'"'.
+ "\n";
+ }
#
# Print out the headings
my $sequence_row = '';
@@ -1519,23 +1667,19 @@ END
$sequence_row .='"",';
$resource_row .= '"'.&Apache::loncommon::csv_translate($field).'",';
}
- foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
+ foreach my $seq (@sequences) {
$sequence_row .= '"'.
- &Apache::loncommon::csv_translate($seq->{'title'}).'",';
+ &Apache::loncommon::csv_translate($seq->compTitle).'",';
my $count = 0;
if ($chosen_output->{'every_problem'}) {
- foreach my $res (@{$seq->{'contents'}}) {
- if ($res->{'type'} ne 'assessment' ||
- ! exists($res->{'parts'}) ||
- ref($res->{'parts'}) ne 'ARRAY' ||
- scalar(@{$res->{'parts'}}) < 1) {
+ foreach my $res (&get_resources($navmap,$seq)) {
+ if (scalar(@{$res->parts}) < 1) {
next;
}
- foreach my $part (@{$res->{'parts'}}) {
+ foreach my $part (@{$res->parts}) {
$resource_row .= '"'.
- &Apache::loncommon::csv_translate($res->{'title'}.
- ', Part '.$part
- ).'",';
+ &Apache::loncommon::csv_translate
+ ($res->compTitle.', Part '.$res->part_display($part)).'",';
$count++;
}
}
@@ -1570,8 +1714,9 @@ END
sub csv_outputstudent {
my ($r,$student) = @_;
- return if ($request_aborted);
- return if (! defined($outputfile));
+ if ($request_aborted || ! defined($navmap) || ! defined($outputfile)) {
+ return;
+ }
my $Str = '';
#
# Output student fields
@@ -1579,7 +1724,7 @@ sub csv_outputstudent {
foreach my $field (@to_show) {
my $value = $student->{$field};
if ($field eq 'comments') {
- $value = &Apache::lonmsg::retrieve_instructor_comments
+ $value = &Apache::lonmsgdisplay::retrieve_instructor_comments
($student->{'username'},$student->{'domain'});
}
$Str .= '"'.&Apache::loncommon::csv_translate($value).'",';
@@ -1590,7 +1735,7 @@ sub csv_outputstudent {
my @tmp = &Apache::loncoursedata::get_current_state($student->{'username'},
$student->{'domain'},
undef,
- $ENV{'request.course.id'});
+ $env{'request.course.id'});
if ((scalar @tmp > 0) && ($tmp[0] !~ /^error:/)) {
%StudentsData = @tmp;
}
@@ -1598,16 +1743,17 @@ sub csv_outputstudent {
# Output performance data
my $total = 0;
my $maximum = 0;
- foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
+ foreach my $seq (@sequences) {
my ($performance,$performance_length,$score,$seq_max,$rawdata);
if ($chosen_output->{'tries'}){
($performance,$performance_length,$score,$seq_max,$rawdata) =
- &StudentTriesOnSequence($student,\%StudentsData,
- $seq,'no');
+ &student_tries_on_sequence($student,\%StudentsData,
+ $navmap,$seq,'no');
} else {
($performance,$performance_length,$score,$seq_max,$rawdata) =
- &StudentPerformanceOnSequence($student,\%StudentsData,
- $seq,'no');
+ &student_performance_on_sequence($student,\%StudentsData,
+ $navmap,$seq,'no',
+ $chosen_output->{ignore_weight});
}
if ($chosen_output->{'every_problem'}) {
if ($chosen_output->{'correct'}) {
@@ -1650,13 +1796,12 @@ sub csv_outputstudent {
sub csv_finish {
my ($r) = @_;
- return if ($request_aborted);
- return if (! defined($outputfile));
+ if ($request_aborted || ! defined($navmap) || ! defined($outputfile)) {
+ &csv_cleanup();
+ return;
+ }
close($outputfile);
#
- my $c = $r->connection();
- return if ($c->aborted());
- #
# Close the progress window
&Apache::lonhtmlcommon::Close_PrgWin($r,\%prog_state);
#
@@ -1664,12 +1809,25 @@ sub csv_finish {
$r->print(' '.
''.&mt('Your csv file.').''."\n");
$r->rflush();
+ &csv_cleanup();
return;
}
}
+# This function will return an HTML string including a star, with
+# a mouseover popup showing the "real" value. An optional second
+# argument lets you show something other than a star.
+sub show_star {
+ my $popup = shift;
+ my $symbol = shift || '*';
+ # Escape the popup for JS.
+ $popup =~ s/([^-a-zA-Z0-9:;,._ ()|!\/?=&*])/'\\' . sprintf("%lo", ord($1))/ge;
+
+ return "$symbol";
+}
+
#######################################################
#######################################################
@@ -1695,8 +1853,8 @@ Inputs:
#######################################################
#######################################################
-sub StudentTriesOnSequence {
- my ($student,$studentdata,$seq,$links) = @_;
+sub student_tries_on_sequence {
+ my ($student,$studentdata,$navmap,$seq,$links) = @_;
$links = 'no' if (! defined($links));
my $Str = '';
my ($sum,$max) = (0,0);
@@ -1704,11 +1862,10 @@ sub StudentTriesOnSequence {
my @TriesData = ();
my $tries;
my $hasdata = 0; # flag - true if the student has any data on the sequence
- foreach my $resource (@{$seq->{'contents'}}) {
- next if ($resource->{'type'} ne 'assessment');
- my $resource_data = $studentdata->{$resource->{'symb'}};
+ foreach my $resource (&get_resources($navmap,$seq)) {
+ my $resource_data = $studentdata->{$resource->symb};
my $value = '';
- foreach my $partnum (@{$resource->{'parts'}}) {
+ foreach my $partnum (@{$resource->parts()}) {
$tries = undef;
$max++;
$performance_length++;
@@ -1736,12 +1893,12 @@ sub StudentTriesOnSequence {
if ($status eq 'excused') {
$symbol = 'x';
$max--;
- } elsif ($status eq 'correct_by_override') {
+ } elsif ($status eq 'correct_by_override' && !$resource->is_task()) {
$symbol = '+';
$sum++;
} elsif ($tries > 0) {
if ($tries > 9) {
- $symbol = '*';
+ $symbol = show_star($tries);
} else {
$symbol = $tries;
}
@@ -1758,7 +1915,7 @@ sub StudentTriesOnSequence {
} elsif ($status eq 'incorrect_by_override') {
$symbol = '-';
} elsif ($status eq 'ungraded_attempted') {
- $symbol = '#';
+ $symbol = 'u';
} elsif ($status eq 'incorrect_attempted' ||
$tries > 0) {
$symbol = '.';
@@ -1768,28 +1925,35 @@ sub StudentTriesOnSequence {
}
}
#
- if (! defined($tries) || $symbol eq '.') {
- $tries = $symbol;
+ if (! defined($tries)) {
+ $tries = 0;
+ }
+ if ($status =~ /^(incorrect|ungraded)/) {
+ # Bug 3390: show '-' for tries on incorrect problems
+ # (csv & excel only)
+ push(@TriesData,-$tries);
+ } else {
+ push (@TriesData,$tries);
}
- push (@TriesData,$tries);
#
if ( ($links eq 'yes' && $symbol ne ' ') ||
($links eq 'all')) {
if (length($symbol) > 1) {
&Apache::lonnet::logthis('length of symbol "'.$symbol.'" > 1');
}
- $symbol = ''.$symbol.'';
+ '&command=submission';
+ $symbol = &link($symbol, $link);
}
$value .= $symbol;
}
$Str .= $value;
}
- if ($seq->{'randompick'}) {
- $max = $seq->{'randompick'};
+ if ($seq->randompick()) {
+ $max = $seq->randompick();
}
if (! $hasdata && $sum == 0) {
$sum = ' ';
@@ -1797,12 +1961,43 @@ sub StudentTriesOnSequence {
return ($Str,$performance_length,$sum,$max,\@TriesData);
}
+=pod
+
+=item &link
+
+Inputs:
+
+=over 4
+
+=item $text
+
+=item $target
+
+=back
+
+Takes the text and creates a link to the $text that honors
+the value of 'new window' if clicked on, but uses a real
+'href' so middle and right clicks still work.
+
+$target and $text are assumed to be already correctly escaped; i.e., it
+can be dumped out directly into the output stream as-is.
+
+=cut
+
+sub link {
+ my ($text,$target) = @_;
+ return
+ "$text";
+}
+
#######################################################
#######################################################
=pod
-=item &StudentPerformanceOnSequence()
+=item &student_performance_on_sequence
Inputs:
@@ -1822,8 +2017,8 @@ Inputs:
#######################################################
#######################################################
-sub StudentPerformanceOnSequence {
- my ($student,$studentdata,$seq,$links) = @_;
+sub student_performance_on_sequence {
+ my ($student,$studentdata,$navmap,$seq,$links,$awarded_only) = @_;
$links = 'no' if (! defined($links));
my $Str = ''; # final result string
my ($score,$max) = (0,0);
@@ -1832,16 +2027,19 @@ sub StudentPerformanceOnSequence {
my @ScoreData = ();
my $partscore;
my $hasdata = 0; # flag, 0 if there were no submissions on the sequence
- foreach my $resource (@{$seq->{'contents'}}) {
- next if ($resource->{'type'} ne 'assessment');
- my $resource_data = $studentdata->{$resource->{'symb'}};
- foreach my $part (@{$resource->{'parts'}}) {
+ foreach my $resource (&get_resources($navmap,$seq)) {
+ my $symb = $resource->symb;
+ my $resource_data = $studentdata->{$symb};
+ foreach my $part (@{$resource->parts()}) {
$partscore = undef;
- my $weight = &Apache::lonnet::EXT('resource.'.$part.'.weight',
- $resource->{'symb'},
- $student->{'domain'},
- $student->{'username'},
- $student->{'section'});
+ my $weight;
+ if (!$awarded_only){
+ $weight = &Apache::lonnet::EXT('resource.'.$part.'.weight',
+ $symb,
+ $student->{'domain'},
+ $student->{'username'},
+ $student->{'section'});
+ }
if (!defined($weight) || ($weight eq '')) {
$weight=1;
}
@@ -1857,7 +2055,7 @@ sub StudentPerformanceOnSequence {
$hasdata = 1;
}
#
- $partscore = $weight*$awarded;
+ $partscore = &Apache::grades::compute_points($weight,$awarded);
if (! defined($awarded)) {
$partscore = undef;
}
@@ -1867,16 +2065,22 @@ sub StudentPerformanceOnSequence {
$symbol = sprintf("%.0f",$symbol);
}
if (length($symbol) > 1) {
- $symbol = '*';
+ $symbol = show_star($symbol);
}
- if (exists($resource_data->{'resource.'.$part.'.solved'})) {
+ if (exists($resource_data->{'resource.'.$part.'.solved'}) &&
+ $resource_data->{'resource.'.$part.'.solved'} ne '') {
my $status = $resource_data->{'resource.'.$part.'.solved'};
if ($status eq 'excused') {
$symbol = 'x';
$max -= $weight; # Do not count 'excused' problems.
+ } elsif ($status eq 'ungraded_attempted') {
+ $symbol = 'u';
}
$hasdata = 1;
- } else {
+ } elsif ($resource_data->{'resource.'.$part.'.award'} eq 'DRAFT') {
+ $symbol = 'd';
+ $hasdata = 1;
+ } elsif (!exists($resource_data->{'resource.'.$part.'.awarded'})){
# Unsolved. Did they try?
if (exists($resource_data->{'resource.'.$part.'.tries'})){
$symbol = '.';
@@ -1892,11 +2096,12 @@ sub StudentPerformanceOnSequence {
push (@ScoreData,$partscore);
#
if ( ($links eq 'yes' && $symbol ne ' ') || ($links eq 'all')) {
- $symbol = ''.$symbol.'';
+ '&command=submission';
+ $symbol = &link($symbol, $link);
}
$Str .= $symbol;
}
@@ -1924,13 +2129,13 @@ problems.
#######################################################
sub CreateLegend {
my $Str = "
".
- " 1 correct by student in 1 try\n".
- " 7 correct by student in 7 tries\n".
+ " digit score or number of tries to get correct ".
" * correct by student in more than 9 tries\n".
" + correct by hand grading or override\n".
" - incorrect by override\n".
" . incorrect attempted\n".
- " # ungraded attempted\n".
+ " u ungraded attempted\n".
+ " d draft answer saved but not submitted\n".
" not attempted (blank field)\n".
" x excused".
"