WARNING: ';
- $Str .= 'No Students enrolled OR Please select a student
';
- $r->print($Str);
- } else {
- if(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
- $r->print(&StudentAverageTotal(\%cache, $students, $sequenceKeys));
- untie(%cache);
- }
- }
+ my $Str;
return;
}
+
#######################################################
#######################################################
@@ -245,23 +135,7 @@ sub BuildStudentAssessmentPage {
Called by &BuildStudentAssessmentPage to create the top part of the
page which displays the chart.
-Inputs:
-
-=over 4
-
-=item $cache The ubiquitous cache
-
-=item $selectedName The name of the currently selected student, or
-'All Students' or 'No Student Selected'.
-
-=item $students Array ref containing the name(s) of the students selected
-for display.
-
-=item $formName The name of the HTML form to use, 'Statistics'
-
-=item $doNotShow Array ref containing the names of columns to not show
-
-=back
+Inputs: None
Returns: A string containing the HTML for the headers and top table for
the chart page.
@@ -271,52 +145,53 @@ the chart page.
#######################################################
#######################################################
sub CreateInterface {
- my($cache,$selectedName,$students,$formName,$doNotShow)=@_;
-
my $Str = '';
- $Str .= &CreateLegend();
- $Str .= '
'."\n";
+Return a line of the chart for a student.
+=cut
+
+#######################################################
+#######################################################
+sub ChartOutputStudent {
+ my $student = shift;
+ my $Str = '';
+ # First, the @StudentData fields need to be listed
+ my @to_show = @Apache::lonstatistics::SelectedStudentData;
+ foreach (@to_show) {
+ if ($_ eq 'all') {
+ @to_show = @Apache::lonstatistics::StudentDataOrder;
+ last;
+ }
+ }
+ foreach my $field (@to_show) {
+ my $title=$student->{$field};
+ my $base =scalar(my @Tmp = split(//,$title));
+ my $width=$Apache::lonstatistics::StudentData{$field}->{'width'};
+ $Str .= $title.' 'x($width-$base).$padding;
+ }
+ # Get ALL the students data
+ my %StudentsData;
+ my @tmp = &Apache::loncoursedata::get_current_state
+ ($student->{'username'},$student->{'domain'},undef,
+ $ENV{'request.course.id'});
+ if ((scalar @tmp > 0) && ($tmp[0] !~ /^error:/)) {
+ %StudentsData = @tmp;
+ }
+ if (scalar(@tmp) < 1) {
+ $Str .= 'No Course Data'."\n";
+ return $Str;
+ }
+ #
+ # By sequence build up the data
+ my $studentstats;
+ foreach my $map_symb (@Apache::lonstatistics::SelectedMaps) {
+ foreach my $seq (@Apache::lonstatistics::Sequences) {
+ next if ($map_symb ne $seq->{'symb'} && $map_symb ne 'all');
+ next if ($seq->{'num_assess'} < 1);
+ my ($performance,$score,$seq_max) =
+ &StudentPerformanceOnSequence($student,\%StudentsData,
+ $seq,'linkify');
+ $Str .= $performance.$padding;
+ $studentstats->{$seq->{'symb'}}->{'score'}= $score;
+ $studentstats->{$seq->{'symb'}}->{'max'} = $seq_max;
+ }
+ }
+ #
+ # Total it up and store the statistics info.
+ my ($score,$max) = (0,0);
+ while (my ($symb,$seq_stats) = each (%{$studentstats})) {
+ $Statistics->{$symb}->{'score'} += $seq_stats->{'score'};
+ $Statistics->{$symb}->{'max'} += $seq_stats->{'max'};
+ $score += $seq_stats->{'score'};
+ $max += $seq_stats->{'max'};
+ }
+ my $scorelength = scalar(my @tmp1 = split(//,$score));
+ my $maxlength = scalar(my @tmp2 = split(//,$max));
+ $Str .= ' '.' 'x($maxlength-$scorelength).$score.'/'.$max;
+ $Str .= " \n";
return $Str;
-}
+}
#######################################################
#######################################################
=pod
-=item &StudentReport()
+=back
-This is the workhorse subroutine - it handles formatting and display of a
-students performance data. It processes one row of the chart.
+=cut
-Input:
+#######################################################
+#######################################################
-=over 4
+}
-=item $cache The ubiquitous cache
+#######################################################
+#######################################################
+
+=pod
-=item $name The name and domain of the current student in name:domain format
+=item &StudentPerformanceOnSequence()
-=item $spacing A string containing spaces.
+Inputs:
+
+=over 4
-=item $showSequences Array ref containing the sequences to display
+=item $student
-=back
+=item $studentdata Hash ref to all student data
-Output: $Str
+=item $seq Hash ref, the sequence we are working on
-$Str: Formatted string that is an entire row of the chart. It is a
-concatenation of student information and student course information.
+=item $links if defined we will output links to each resource.
+
+=back
=cut
#######################################################
#######################################################
-sub StudentReport {
- my ($cache,$name,$spacing,$showSequences)=@_;
- my ($username,$domain)=split(':',$name);
-
+sub StudentPerformanceOnSequence {
+ my ($student,$studentdata,$seq,$links) = @_;
my $Str = '';
- if(defined($cache->{$name.':error'})) {
- return $Str;
- }
- if($cache->{$name.':error'} =~ /course/) {
- $Str .= 'No course data for student ';
- $Str .= ''.$username.'. ';
- return $Str;
- }
-
- my $hasVersion = 'false';
- my $hasFinalData = 'false';
- foreach my $sequence (@$showSequences) {
- my $hasData = 'false';
- my $characterCount=0;
- foreach my $problemID (split(':', $cache->{$sequence.':problems'})) {
- my $problem = $cache->{$problemID.':problem'};
- # All grades (except for versionless parts) are displayed as links
- # to their submission record. Loop through all the parts for the
- # current problem in the correct order and prepare the output links
- foreach(split(/\:/,$cache->{$sequence.':'.$problemID.
- ':parts'})) {
- if($cache->{$name.':'.$problemID.':NoVersion'} eq 'true' ||
- $cache->{$name.':'.$problemID.':'.$_.':code'} eq ' ' ||
- $cache->{$name.':'.$problemID.':'.$_.':code'} eq '') {
- $Str .= ' ';
- $characterCount++;
- next;
- }
- $hasVersion = 'true';
- $hasData = 'true';
- if (lc($ENV{'form.displaymode'}) ne 'display without links') {
- $Str .= '';
- }
- my $code = $cache->{$name.':'.$problemID.':'.$_.':code'};
- my $tries = $cache->{$name.':'.$problemID.':'.$_.':tries'};
- if($code eq '*' && $tries < 10 && $tries ne '') {
- $code = $tries;
+ my $output_width = 0;
+ my ($sum,$max) = (0,0);
+ foreach my $resource (@{$seq->{'contents'}}) {
+ next if ($resource->{'type'} ne 'assessment');
+ my $resource_data = $studentdata->{$resource->{'symb'}};
+ my $value = '';
+ foreach my $partnum (@{$resource->{'parts'}}) {
+ $max++;
+ my $symbol = ' '; # default to space
+ #
+ if (exists($resource_data->{'resource.'.$partnum.'.solved'})) {
+ my $status = $resource_data->{'resource.'.$partnum.'.solved'};
+ if ($status eq 'correct_by_override') {
+ $symbol = '+';
+ $sum++;
+ } elsif ($status eq 'incorrect_by_override') {
+ $symbol = '-';
+ } elsif ($status eq 'ungraded_attempted') {
+ $symbol = '#';
+ } elsif ($status eq 'incorrect_attempted') {
+ $symbol = '.';
+ } elsif ($status eq 'excused') {
+ $symbol = 'x';
+ $max--;
+ } elsif ($status eq 'correct_by_student' &&
+ exists($resource_data->{'resource.'.$partnum.'.tries'})){
+ my $num = $resource_data->{'resource.'.$partnum.'.tries'};
+ if ($num > 9) {
+ $symbol = '*';
+ } elsif ($num > 0) {
+ $symbol = $num;
+ } else {
+ $symbol = ' ';
+ }
+ $sum++;
+ } else {
+ $symbol = ' ';
}
- $Str .= $code;
- if (lc($ENV{'form.displaymode'}) ne 'display without links') {
- $Str .= '';
+ } else {
+ # Unsolved. Did they try?
+ if (exists($resource_data->{'resource.'.$partnum.'.tries'})){
+ $symbol = '.';
+ } else {
+ $symbol = ' ';
}
- $characterCount++;
}
+ #
+ $output_width++;
+ if (defined($links) && $symbol ne ' ') {
+ $symbol = ''.$symbol.'';
+ }
+ $value .= $symbol;
}
-
- # Output the number of correct answers for the current sequence.
- # This part takes up 6 character slots, but is formated right
- # justified.
- my $spacesNeeded=$cache->{$sequence.':columnWidth'}-$characterCount;
- $spacesNeeded -= 3;
- $Str .= (' 'x$spacesNeeded);
-
-# my $outputProblemsCorrect = sprintf("%3d", $cache->{$name.':'.$sequence.
-# ':problemsCorrect'});
-
- my $outputProblemsCorrect = sprintf("%2d/%2d", $cache->{$name.':'.$sequence.
- ':problemsCorrect'},
- $characterCount);
- if($hasData eq 'true') {
- $Str .= ''.$outputProblemsCorrect.'';
- $hasFinalData = 'true';
- } else {
- $Str .= ' ';
- }
- $Str .= $spacing;
- }
-
- # Output the total correct problems over the total number of problems.
- # I don't like this type of formatting, but it is a solution. Need
- # a way to dynamically determine the space requirements.
- my $outputProblemsSolved = sprintf("%4d", $cache->{$name.':problemsSolved'});
- my $outputTotalProblems = sprintf("%4d", $cache->{$name.':totalProblems'});
- if($hasFinalData eq 'true') {
- $Str .= ''.$outputProblemsSolved.
- ' / '.$outputTotalProblems.'';
- } else {
- $Str .= ' ';
+ $Str .= $value;
}
-
- if($hasVersion eq 'false') {
- $Str = 'No course data.';
- }
-
- return $Str;
+ # Put on the totals
+ my $ratio = $sum.'/'.$max;
+ my $ratio_length = scalar(my @tmp1 = split(//,$ratio));
+ # Pad with extra spaces
+ my $width = $seq->{'width'};
+ $Str .= ' 'x($width-$output_width-$ratio_length).$ratio;
+ #
+ return ($Str,$sum,$max);
}
#######################################################
#######################################################
-
-#######################################################
-#######################################################
sub StudentAverageTotal {
my ($cache, $students, $sequenceKeys)=@_;
my $Str = "\nSummary Tables:\n";
@@ -590,29 +514,6 @@ sub StudentAverageTotal {
#######################################################
#######################################################
-#######################################################
-#######################################################
-sub MaxSeqPr {
- my ($cache, $name, $sequence)=@_;
- my $prCount=0;
- foreach my $problemID (split(':', $cache->{$sequence.':problems'})) {
- my $problem = $cache->{$problemID.':problem'};
- foreach(split(/\:/,$cache->{$sequence.':'.$problemID.':parts'})) {
- if($cache->{$name.':'.$problemID.':NoVersion'} eq 'true' ||
- $cache->{$name.':'.$problemID.':'.$_.':code'} eq ' ' ||
- $cache->{$name.':'.$problemID.':'.$_.':code'} eq '') {
- $prCount++;
- next;
- }
- $prCount++;
- }
- }
- return $prCount;
-}
-
-#######################################################
-#######################################################
-
=pod
=item &CreateLegend()
@@ -643,176 +544,7 @@ sub CreateLegend {
#######################################################
#######################################################
-=pod
-
-=item &CreateColumnSelectionBox()
-
-If there are columns not being displayed then this selection box is created
-with a list of those columns. When selections are made and the page
-refreshed, the columns will be removed from this box and the column is
-put back in the chart. If there is no columns to select, no row is added
-to the interface table.
-
-=over 4
-Input: $CacheData, $headings
-
-$CacheData: A pointer to a hash tied to the cached data
-
-$headings: An array of the names of the columns for the student information.
-They are used for displaying which columns are missing.
-
-Output: $notThere
-
-$notThere: The string contains one row of a table. The first column has the
-name of the selection box. The second contains the selection box
-which has a size of four.
-
-=back
-
-=cut
-
-#######################################################
-#######################################################
-sub CreateColumnSelectionBox {
- my ($doNotShow)=@_;
-
- my $notThere = '';
- $notThere .= '';
-
- return $notThere;
-}
-
-#######################################################
-#######################################################
-
-=pod
-
-=item &CreateColumnSelectors()
-
-This function generates the checkboxes above the column headings. The
-column will be removed if the checkbox is unchecked.
-
-=over 4
-
-Input: $CacheData, $headings
-
-$CacheData: A pointer to a hash tied to the cached data
-
-$headings: An array of the names of the columns for the student
-information. They are used to know what are the student information columns
-
-Output: $present
-
-$present: The string contains the first row of a table. Each column contains
-a checkbox which is left justified. Currently left justification is used
-for consistency of location over the column in which it presides.
-
-=back
-
-=cut
-
-#######################################################
-#######################################################
-sub CreateColumnSelectors {
- my ($infoHeadings, $sequenceHeadings, $sequenceKeys)=@_;
-
- my $present = '';
- for(my $index=0; $index<(scalar @$infoHeadings); $index++) {
- $present .= '