--- loncom/interface/statistics/lonsurveyreports.pm 2008/10/30 17:45:25 1.19 +++ loncom/interface/statistics/lonsurveyreports.pm 2010/02/28 23:58:55 1.24 @@ -1,6 +1,6 @@ # The LearningOnline Network with CAPA # -# $Id: lonsurveyreports.pm,v 1.19 2008/10/30 17:45:25 bisitz Exp $ +# $Id: lonsurveyreports.pm,v 1.24 2010/02/28 23:58:55 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -63,7 +63,7 @@ sub BuildSurveyReportsPage { ); &Apache::loncommon::store_course_settings('survey_reports', \%Saveable_Parameters); - &Apache::loncommon::restore_course_settings('survey_resports', + &Apache::loncommon::restore_course_settings('survey_reports', \%Saveable_Parameters); # &Apache::lonstatistics::PrepareClasslist(); @@ -98,7 +98,7 @@ sub BuildSurveyReportsPage { # $r->print('
'); $r->print('

'. - &Apache::lonlocal::locallocaltime(time).','. + &Apache::lonlocal::locallocaltime(time).', '. &Apache::lonstatistics::section_and_enrollment_description(). '

'); $r->rflush(); @@ -177,7 +177,7 @@ sub SurveyProblemSelector { my @resources = &Apache::lonstathelpers::get_resources($navmap,$seq); foreach my $res (@resources) { foreach my $part (@{$res->parts}) { - if ($res->is_survey($part)) { + if (($res->is_survey($part)) || ($res->is_anonsurvey($part))) { push(@SurveyProblems,{res=>$res,seq=>$seq,part=>$part}); last; } @@ -211,10 +211,10 @@ sub SurveyProblemSelector { $Str .= ''. ''.''. - ''. + ''. ''. (' 'x2). - ''.&mt('View survey').''.''.$/; + ''.&mt('View survey').''.$/; } $Str .= ''; return $Str; @@ -339,7 +339,7 @@ sub make_text_report { } else { $r->print('

' .&mt('There is no essay or string response data to output for this survey.') - .''); + .'

'); } $r->rflush(); return; @@ -569,15 +569,20 @@ sub make_HTML_report { sprintf("%.2f",$count/$total*100).'%'. &Apache::loncommon::end_data_table_row().$/; } - $Str .= &Apache::loncommon::start_data_table_row(). - ''.&mt('Foil Name').''. - ''.&mt('Text').''. - ''.&mt('Freq').''. - ''.&mt('Percent').''. - &Apache::loncommon::end_data_table_row().$/. - $tmp; + $Str.='' + .'' + .&Apache::loncommon::start_data_table() + .&Apache::loncommon::start_data_table_header_row() + .''.&mt('Foil Name').'' + .''.&mt('Text').'' + .''.&mt('Frequency').'' + .''.&mt('Percent').'' + .&Apache::loncommon::end_data_table_header_row().$/ + .$tmp + .&Apache::loncommon::end_data_table() + .''; } - $Str.= &Apache::loncommon::end_data_table().'
'; + $Str.= ''; $r->print($Str); $r->rflush(); } @@ -653,11 +658,12 @@ sub CreateInterface { $prob_checkbox .= 'checked="checked" '; } $prob_checkbox .= 'value="true" />'; - $Str .= ''. - '
'. - '