--- loncom/interface/statistics/lonstathelpers.pm 2009/03/25 15:49:53 1.58 +++ loncom/interface/statistics/lonstathelpers.pm 2011/03/12 02:58:49 1.65 @@ -1,6 +1,6 @@ # The LearningOnline Network with CAPA # -# $Id: lonstathelpers.pm,v 1.58 2009/03/25 15:49:53 bisitz Exp $ +# $Id: lonstathelpers.pm,v 1.65 2011/03/12 02:58:49 www Exp $ # # Copyright Michigan State University Board of Trustees # @@ -135,9 +135,20 @@ Skips 'survey' problems. #################################################### #################################################### sub problem_selector { - my ($AcceptedResponseTypes,$sequence_addendum) = @_; + my ($AcceptedResponseTypes,$sequence_addendum,$symbmode,$all,$prefix,$smallbox,$onclick) = @_; +# all: also make sequences selectable +# prefix: prefix for all form names +# smallbox: use smaller box +# onclick: javascript to execute when clicked my $Str; - $Str = "\n\n"; + my $jsadd=''; + if ($onclick) { + $jsadd="onClick='$onclick'"; + } + $Str = &Apache::loncommon::start_scrollbox(($smallbox?'520px':'620px'), + ($smallbox?'500px':'600px'), + ($smallbox?'50px':'300px')). + &Apache::loncommon::start_data_table(); my $rb_count =0; my ($navmap,@sequences) = &Apache::lonstatistics::selected_sequences_with_assessments('all'); @@ -158,41 +169,48 @@ sub problem_selector { resptype=>$resptype}); my $checked = ''; if ($env{'form.problemchoice'} eq $value) { - $checked = 'checked="checked" '; + $checked = ' checked="checked"'; } my $title = $res->compTitle; if (! defined($title) || $title eq '') { ($title) = ($res->src =~ m:/([^/]*)$:); } - $seq_str .= ''. - qq{}. - ''. - '' + :qq{}). + ''. + '\n"; + $seq_str .= "". &Apache::loncommon::end_data_table_row()."\n"; $rb_count++; } } } } if ($seq_str ne '') { - $Str .= ''. - ''. - "\n".$seq_str; + $Str .= &Apache::loncommon::start_data_table_header_row(). + ''. + &Apache::loncommon::end_data_table_header_row()."\n".$seq_str; if (defined($sequence_addendum)) { - $Str .= ''. + $Str .= &Apache::loncommon::start_data_table_header_row(). (''x2). ''. - "\n"; + &Apache::loncommon::end_data_table_header_row()."\n"; } } + } - $Str .= "
'; + $seq_str .= &Apache::loncommon::start_data_table_row(). + ($symbmode? + ''; if (scalar(@response_ids) > 1) { $seq_str .= &mt('response').' '.$respid; } my $link = $res->link.'?symb='.&escape($res->shown_symb); $seq_str .= (' 'x2). ''.&mt('view').''; - $seq_str .= "
 '.$seq->compTitle.'
'. + ($all?'':''). + $seq->compTitle.'
 '.$sequence_addendum.'
\n"; + $Str .= &Apache::loncommon::end_data_table().&Apache::loncommon::end_scrollbox()."\n"; return $Str; } @@ -222,7 +240,7 @@ and their contents. A checkbox is provi #################################################### #################################################### sub MultipleProblemSelector { - my ($navmap,$inputname,$formname)=@_; + my ($navmap,$inputname,$formname,$anoncounter)=@_; my $cid = $env{'request.course.id'}; my $Str; # Massage the input as needed. @@ -257,7 +275,64 @@ sub MultipleProblemSelector { } END - $Str .= + my $checkanonjs = <<"END"; + + + +END + if (ref($anoncounter) eq 'HASH') { + if (keys(%{$anoncounter}) > 0) { + my $anonwarning = &mt('Your selection includes both problems with and without anonymous submissions.').'\n'.&mt('You must select either only anonymous or only named problems.').'\n\n'.&mt('If a selection contains both anonymous and named parts, [_1]use the Anoymous/Named buttons to ensure selections will be either all anonymous [_1]or all named.','\n'); + $checkanonjs = <<"END"; + + + +END + } + } + $Str .= $checkanonjs. ''.&mt('Select All').''. (' 'x4). ''.&mt('Unselect All').''; @@ -288,9 +363,27 @@ END $seq_id++, $inputname)); } elsif ($curRes->is_problem) { + my $anonpart = 0; + my $namedpart = 0; + my @parts = @{$curRes->parts()}; + if (ref($anoncounter) eq 'HASH') { + if (keys(%{$anoncounter}) > 0) { + my @parts = @{$curRes->parts()}; + my $symb = $curRes->symb(); + foreach my $part (@parts) { + if ((exists($anoncounter->{$symb."\0".$part})) || + $curRes->is_anonsurvey($part)) { + $anonpart ++; + } else { + $namedpart ++ + } + } + } + } if (@Accumulator && $Accumulator[-1] ne '') { &{$Accumulator[-1]}($curRes, - exists($selected->{$curRes->symb})); + exists($selected->{$curRes->symb}), + $anonpart,$namedpart); } } } @@ -318,20 +411,45 @@ sub new_accumulator { return sub { if (@_) { - my ($res,$checked) = @_; + my ($res,$checked,$anonpart,$namedpart) = @_; $target.=''. (' 'x2).''.&mt('view').''. - ''.$/; + ''; + my $mixed = ''; + if ($anonpart) { + if ($namedpart) { + my $checknamed = ''; + my $checkedanon = ' checked="checked"'; + if ($env{'form.mixed_'.$seq_id.':'.$anon_id} eq $esc_symb) { + $checknamed = $checkedanon; + $checkedanon = ''; + } + $mixed = ' ('. + &mt('Both anonymous and named submissions -- display: [_1]Anonymous [_2]Named[_3]', + ''.(' 'x2).' '. + '').')'; + } else { + $target .= ' '.&mt('(Anonymous Survey)'); + } + } + $target.= $mixed.''.$/; } else { if (defined($target)) { return { title => $title, @@ -464,7 +582,8 @@ sub get_prev_curr_next { my @resources = &get_resources($navmap,$seq); foreach my $res (@resources) { foreach my $part (@{$res->parts}) { - if ($res->is_survey($part) && ($granularity eq 'part_survey')){ + if (($res->is_survey($part) || ($res->is_anonsurvey($part))) && + ($granularity eq 'part_survey')) { push (@Resource, { symb => $res->symb, part => $part, @@ -609,7 +728,7 @@ sub GetStudentAnswers { my $answer = &Apache::lonstathelpers::get_student_answer ($resource,$sname,$sdom,$partid,$respid); &Apache::lonhtmlcommon::Increment_PrgWin($r,\%prog_state, - &mt('last student')); + 'last student'); $answers{$answer}++; $student->{'answer'} = $answer; }