--- loncom/homework/structuretags.pm	2001/09/14 20:17:35	1.67
+++ loncom/homework/structuretags.pm	2001/10/30 19:18:15	1.71
@@ -152,7 +152,7 @@ sub start_problem {
     &page_start($target,$token,$tagstack,$parstack,$parser,$safeeval);
 
 
-  if ($target eq 'web' || $target eq 'grade') {
+  if ($target eq 'web' || $target eq 'grade' || $target eq 'answer') {
     #handle exam checkout
     if ($Apache::lonhomework::type eq 'exam') {
       my $token=$Apache::lonhomework::history{"resource.0.outtoken"};
@@ -252,17 +252,20 @@ sub end_problem {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
   my $result='';
   my $status=$Apache::inputtags::status['-1'];
-  if ($target eq 'grade' || $target eq 'web' ) {
+  if ($target eq 'grade' || $target eq 'web' || $target eq 'answer' ) {
     if ( $target eq 'grade' && $Apache::inputtags::part eq '0' &&
        $status eq 'CAN_ANSWER') {
       # if part is zero, no <part>s existed, so we need to the grading
       &Apache::inputtags::grade;
-    } elsif ($Apache::inputtags::part eq '0') {
+    } elsif ( $target eq 'web' && $Apache::inputtags::part eq '0') {
       # if part is zero, no <part>s existed, so we need show the current 
       # grading status
       $result.= &Apache::inputtags::gradestatus($Apache::inputtags::part);
     }
-    if ($target eq 'web') { 
+    if (
+	($target eq 'web' && ($ENV{'request.state'} ne 'construct')) ||
+	($target eq 'answer')
+       ) {
       if ($status eq 'CAN_ANSWER') {
 	$result.="</form></body>\n"; 
       } elsif ($status eq 'SHOW_ANSWER' || $status eq 'CANNOT_ANSWER') {
@@ -321,7 +324,7 @@ sub end_library {
 sub start_block {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
 
-  if ($target eq 'web' || $target eq 'grade') {
+  if ($target eq 'web' || $target eq 'grade' || $target eq 'answer') {
     my $code = @$parstack[$#$parstack];
     $code =~ s/\"//g;
     $code .=';return $condition;';
@@ -376,49 +379,52 @@ sub end_while {
 # </randomlist>
 sub start_randomlist {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
-  my $body= &Apache::lonxml::get_all_text("/randomlist",$$parser[$#$parser]);
-  my $b_parser= HTML::TokeParser->new(\$body);
-  my $b_tok;
-  my @randomlist;
-  my $list_item;
-
-  while($b_tok = $b_parser->get_token() ) {
-    if($b_tok->[0] eq 'S') { # start tag
-    # get content of the tag until matching end tag
-    # get all text upto the matching tag
-    # and push the content into @randomlist
-      $list_item = &Apache::lonxml::get_all_text('/'.$b_tok->[1],$b_parser);
-      $list_item = "$b_tok->[4]"."$list_item"."</$b_tok->[1]>";
-      push(@randomlist,$list_item);
-   #  print "<br /><b>START-TAG $b_tok->[1], $b_tok->[4], $list_item</b>";
-    }
-    if($b_tok->[0] eq 'T') { # text
-    # what to do with text in between tags?
-      #  print "<b>TEXT $b_tok->[1]</b><br />";
-    }
-    # if($b_tok->[0] eq 'E') { # end tag, should not happen
+  if ($target eq 'answer' || $target eq 'grade' || $target eq 'web') {
+    my $body= &Apache::lonxml::get_all_text("/randomlist",$$parser[$#$parser]);
+    my $b_parser= HTML::TokeParser->new(\$body);
+    my $b_tok;
+    my @randomlist;
+    my $list_item;
+    while($b_tok = $b_parser->get_token() ) {
+      if($b_tok->[0] eq 'S') { # start tag
+	# get content of the tag until matching end tag
+	# get all text upto the matching tag
+	# and push the content into @randomlist
+	$list_item = &Apache::lonxml::get_all_text('/'.$b_tok->[1],$b_parser);
+	$list_item = "$b_tok->[4]"."$list_item"."</$b_tok->[1]>";
+	push(@randomlist,$list_item);
+	#  print "<br /><b>START-TAG $b_tok->[1], $b_tok->[4], $list_item</b>";
+      }
+      if($b_tok->[0] eq 'T') { # text
+	# what to do with text in between tags?
+	#  print "<b>TEXT $b_tok->[1]</b><br />";
+      }
+      # if($b_tok->[0] eq 'E') { # end tag, should not happen
       #  print "<b>END-TAG $b_tok->[1]</b><br />";
-    # }
-  }
-  my @idx_arr = (0 .. $#randomlist);
-  &Apache::structuretags::shuffle(\@idx_arr);
-  my $bodytext = '';
-  for(0 .. $#randomlist) {
-    $bodytext .= "$randomlist[ $idx_arr[$_] ]";
-  }
+      # }
+    }
 
-  &Apache::lonxml::newparser($parser,\$bodytext);
+    my @idx_arr = (0 .. $#randomlist);
+    &Apache::structuretags::shuffle(\@idx_arr);
+    my $bodytext = '';
+    for(0 .. $#randomlist) {
+      $bodytext .= "$randomlist[ $idx_arr[$_] ]";
+    }
+    &Apache::lonxml::newparser($parser,\$bodytext);
+  }
   return "";
 }
 
 sub shuffle {
     my $a=shift;
     my $i;
-    &Apache::response::setrandomnumber();
-    for($i=@$a;--$i;) {
-      my $j=int rand($i+1);
-      next if $i == $j;
-      @$a[$i,$j] = @$a[$j,$i];
+    if (defined(@$a)) {
+      &Apache::response::setrandomnumber();
+      for($i=@$a;--$i;) {
+	my $j=int rand($i+1);
+	next if $i == $j;
+	@$a[$i,$j] = @$a[$j,$i];
+      }
     }
 }
 
@@ -435,7 +441,7 @@ sub start_part {
   @Apache::inputtags::previous=();
   if ($target eq 'meta') {
     return &Apache::response::mandatory_part_meta;
-  } elsif ($target eq 'web' || $target eq 'grade') {
+  } elsif ($target eq 'web' || $target eq 'grade' || $target eq 'answer') {
     my ($status,$accessmsg) = &Apache::lonhomework::check_access($id);
     push (@Apache::inputtags::status,$status);
     my $expression='$external::datestatus="'.$status.'";';
@@ -468,7 +474,7 @@ sub end_part {
 
 sub start_preduedate {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
-  if ($target eq 'web' || $target eq 'grade') {
+  if ($target eq 'web' || $target eq 'grade' || $target eq 'answer') {
     if ($Apache::inputtags::status['-1'] ne 'CAN_ANSWER' &&
 	$Apache::inputtags::status['-1'] ne 'CANNOT_ANSWER' ) {
       &Apache::lonxml::get_all_text("/preduedate",$$parser[$#$parser]);
@@ -497,7 +503,7 @@ sub end_postanswerdate {
 
 sub start_notsolved {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
-  if ($target eq 'web' || $target eq 'grade') {
+  if ($target eq 'web' || $target eq 'grade' || $target eq 'answer') {
     my $gradestatus=$Apache::lonhomework::history{"resource.$Apache::inputtags::part.solved"};
     &Apache::lonxml::debug("not solved has :$gradestatus:");
     if ($gradestatus =~ /^correct/) {
@@ -514,7 +520,7 @@ sub end_notsolved {
 
 sub start_solved {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
-  if ($target eq 'web' || $target eq 'grade') {
+  if ($target eq 'web' || $target eq 'grade' || $target eq 'answer') {
     my $gradestatus=$Apache::lonhomework::history{"resource.$Apache::inputtags::part.solved"};
     if ($gradestatus !~ /^correct/) {
       &Apache::lonxml::get_all_text("/solved",$$parser[$#$parser]);
@@ -539,7 +545,7 @@ sub end_startouttext {
   my $text='';
 
   if ($target eq 'edit') {
-    $text=&Apache::lonxml::get_all_text("endouttext",$$parser[$#$parser]);
+    $text=&Apache::lonxml::get_all_text("endouttext",$$parser[-1]);
     $result.=&Apache::edit::start_table($token)."<tr><td>Text Block</td>
 <td>Delete:".
   &Apache::edit::deletelist($target,$token)
@@ -573,13 +579,13 @@ sub delete_startouttext {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
 #  my $text=&Apache::lonxml::get_all_text("endouttext",$$parser['-1']);
   my $text=$$parser['-1']->get_text("/endouttext");
-  my $token=$$parser['-1']->get_token();
-  &Apache::lonxml::debug("Deleting :$text: and :$token->[0]:$token->[1]:$token->[2]: for startouttext");
-  &Apache::lonxml::end_tag($tagstack,$parstack,$token);
+  my $ntoken=$$parser['-1']->get_token();
+  &Apache::lonxml::debug("Deleting :$text: and :$ntoken->[0]:$ntoken->[1]:$ntoken->[2]: for startouttext");
+  &Apache::lonxml::end_tag($tagstack,$parstack,$ntoken);
   # Deleting 2 parallel tag pairs, but we need the numbers later to look like 
   # they did the last time round
-  &Apache::lonxml::increasedepth($token);
-  &Apache::lonxml::decreasedepth($token);
+  &Apache::lonxml::increasedepth($ntoken);
+  &Apache::lonxml::decreasedepth($ntoken);
   return 1;
 }