--- loncom/homework/bridgetask.pm	2005/03/31 21:24:29	1.8
+++ loncom/homework/bridgetask.pm	2005/04/04 18:14:58	1.9
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA 
 # definition of tags that give a structure to a document
 #
-# $Id: bridgetask.pm,v 1.8 2005/03/31 21:24:29 albertel Exp $
+# $Id: bridgetask.pm,v 1.9 2005/04/04 18:14:58 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -37,10 +37,23 @@ use Apache::lonmenu;
 use Apache::lonlocal;
 use Apache::lonxml;
 use Time::HiRes qw( gettimeofday tv_interval );
+
 BEGIN {
     &Apache::lonxml::register('Apache::bridgetask',('Task','IntroParagraph','Dimension','Instance','InstanceText','Criteria','ClosingParagraph'));
 }
 
+sub initialize_bridgetask {
+    # id of current Dimension, 0 means that no dimension is current 
+    # (inside <Task> only)
+    $Apache::bridgetask::dimension='';
+    # list of all Dimension ids seen
+    @Apache::bridgetask::dimensionlist=();
+    # list of all current Instance ids
+    @Apache::bridgetask::instance=();
+    # list of all Instance ids seen in this problem
+    @Apache::bridgetask::instancelist=();
+}
+
 sub proctor_check_auth {
     my ($slot)=@_;
     my $user=$ENV{'form.proctorname'};
@@ -63,9 +76,9 @@ sub proctor_check_auth {
 
 sub add_previous_version_button {
     my $result;
-    $result.=' Show a previously done version: <select name="previousversion">
+    $result.=&mt(' Show a previously done version: [_1]','<select name="previousversion">
 <option>Pick one</option>
-</select>';
+</select>');
     return $result;
 }
 
@@ -88,7 +101,7 @@ sub start_Task {
 	$target eq 'tex') {
 	($status,$accessmsg,$slot) = 
 	    &Apache::lonhomework::check_task_access('0');
-	push (@Apache::inputtags::status,$status);
+	push(@Apache::inputtags::status,$status);
 	my $expression='$external::datestatus="'.$status.'";';
 	$expression.='$external::gradestatus="'.$Apache::lonhomework::history{"resource.0.solved"}.'";';
 	&Apache::run::run($expression,$safeeval);
@@ -164,10 +177,13 @@ sub end_Task {
 	    ($target eq 'answer') || ($target eq 'tex')
 	   ) {
 	    if ($target eq 'web') {
-		$result.='<table border="1">'.
-		    &Apache::inputtags::file_selector("bridge","task","*",
-						      'portfolioonly').
-						      "</table>";
+		if ($status eq 'CAN_ANSWER') {
+		    $result.='<table border="1">'.
+			&Apache::inputtags::file_selector('0',"bridgetask","*",
+							  'portfolioonly').
+							  "</table>";
+		    $result.=&Apache::inputtags::gradestatus('0');
+		}
 		$result.=&Apache::lonxml::xmlend().'</html>';
 	    }
 	}
@@ -201,11 +217,13 @@ sub end_ClosingParagraph {
 }
 
 my %dimension;
-my $dim_id;
 sub start_Dimension {
     my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
     undef(%dimension);
-    $dim_id=$Apache::lonxml::curdepth;
+    my $dim_id=$Apache::lonxml::curdepth;
+    $Apache::bridgetask::dimension=$dim_id;
+    push(@Apache::bridgetask::dimensionlist,$dim_id);
+    undef(@Apache::bridgetask::instance);
     return '';
 }
 
@@ -216,9 +234,11 @@ sub end_Dimension {
 	#FIXME just grabbing the first one for now, need
 	#to randomly pick one until all have been seen
 	#then start repicking
-	my $instance=$dimension{'instances'}->[0];
-	$result=$dimension{'intro'}.
-	    $dimension{$instance.'.text'};
+	&Apache::response::pushrandomnumber();
+	my $num_instances=scalar(@{$dimension{'instances'}});
+	my $which=&Math::Random::random_uniform_integer(1,0,$num_instances-1);
+	my $instance=$dimension{'instances'}->[$which];
+	$result=$dimension{'intro'}.$dimension{$instance.'.text'};
 	#FIXME: don't show the criteria yet
 	#foreach my $id (@{$dimension{$instance.'.criterias'}}) {
 	#    $result.=$dimension{$instance.'.criteria.'.$id};
@@ -249,11 +269,11 @@ sub end_IntroParagraph {
     return $result;
 }
 
-my $instance_id;
 sub start_Instance {
     my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
     push(@{$dimension{'instances'}},$Apache::lonxml::curdepth);
-    $instance_id=$Apache::lonxml::curdepth;
+    push(@Apache::bridgetask::instance,$Apache::lonxml::curdepth);
+    push(@Apache::bridgetask::instancelist,$Apache::lonxml::curdepth);
     return '';
 }
 
@@ -272,6 +292,7 @@ sub start_InstanceText {
 
 sub end_InstanceText {
     my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
+    my $instance_id=$Apache::bridgetask::instance[-1];
     if ($target eq 'web') {
 	$dimension{$instance_id.'.text'}=&Apache::lonxml::endredirection();
     }
@@ -288,6 +309,7 @@ sub start_Criteria {
 
 sub end_Criteria {
     my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
+    my $instance_id=$Apache::bridgetask::instance[-1];
     if ($target eq 'web') {
 	my $criteria=&Apache::lonxml::endredirection();
 	my $id=$Apache::lonxml::curdepth;