--- loncom/homework/default_homework.lcpm	2004/03/26 20:10:42	1.77
+++ loncom/homework/default_homework.lcpm	2004/05/27 03:46:02	1.83
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA 
 # used by lonxml::xmlparse() as input variable $safeinit to Apache::run::run()
 #
-# $Id: default_homework.lcpm,v 1.77 2004/03/26 20:10:42 albertel Exp $
+# $Id: default_homework.lcpm,v 1.83 2004/05/27 03:46:02 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -69,6 +69,7 @@ sub caparesponse_check {
 	$answer=~s/ +/ /g;
 	$response=~s/ +/ /g;
     }
+    if ($type eq 'float') { $response=~s/,//g; }
     if (length($response) > 500) { return "TOO_LONG: Answer too long"; }
 
     if ($type eq '' ) {
@@ -142,7 +143,7 @@ sub caparesponse_check {
     elsif ($result =='15') { $result='UNIT_IRRECONCIBLE'; }
     else  {$result = "ERROR: Unknown Result:$result:$@:";}
 
-    return ("$result:\nRetError $reterror:\nError $error:\nAnswer $answer:\nResponse $response:\n type-$type|$tol|$tol_type|$sig:$sig_lbound:$sig_ubound|$unit|\n$message",$reterror);
+    return ("$result:\nRetError $reterror:\nAnswer $answer:\nResponse $response:\n type-$type|$tol|$tol_type|$sig:$sig_lbound:$sig_ubound|$unit|\n$message",$reterror);
 }
 
 
@@ -172,7 +173,7 @@ sub caparesponse_check_list {
 	    $unit=$2;
 	}
     }
-    $result.="Final final response :$responselist['-1']:\n";
+    $result.="Final final response :$responselist['-1']:$unit:\n";
     $result.=":$#list: answers\n";
     $unit=~s/\s//;
     my $i=0;
@@ -480,12 +481,17 @@ sub floor  {return (($_[0]-CORE::int($_[
 
 sub format {
     my ($value,$fmt)=@_;
-    my $dollarmode;
-    if ($fmt =~ /^\$(.*)/) { $fmt=$1; $dollarmode=1; } 
+    my ($dollarmode,$commamode,$alwaysperiod,$options);
+    if ($fmt =~ /^([^\d]*)(.*)/) { $options=$1; $fmt=$2; } 
+    #if ($options =~ /\$/) { $dollamode=1; }
+    #if ($options =~ /,/)  { $commamode=1; }
+    if ($options =~ /\./) { $alwaysperiod=1; }
     $fmt=~s/e/E/g;
     my $result=sprintf('%.'.$fmt,$value);
+    if ($alwaysperiod && $fmt eq '0f') { $result .='.'; }
     $result=~s/(E[+-]*)0/$1/;
-    if ($dollarmode) {$result=&dollarmode($result);}
+    #if ($dollarmode) {$result=&dollarformat($result);}
+    #if ($commamode) {$result=&commaformat($result);}
     return $result;
 }
 
@@ -513,7 +519,7 @@ sub chemparse {
 	
 	my $molecule = $2;
 	# subscripts
-	$molecule =~ s|(?<=[a-zA-Z\[\s])(\d+)|<sub>$1</sub>|g;
+	$molecule =~ s|(?<=[a-zA-Z\)\]\s])(\d+)|<sub>$1</sub>|g;
 	# superscripts
 	$molecule =~ s|\^(\d*[+\-]*)|<sup>$1</sup>|g;
 	# strip whitespace
@@ -531,14 +537,25 @@ sub chemparse {
 sub prettyprint {
     my ($value,$fmt,$target)=@_;
     my $result;
-    my $dollarmode;
     if (!$target) { $target = $external::target; }
     if ($fmt =~ /chem/i) { return(&chemparse($value)); }
-    if ($fmt =~ /^\$(.*)/) { $fmt=$1; $dollarmode=1; } 
+    my ($dollarmode,$commamode,$alwaysperiod,$options);
+    if ($fmt =~ /^([^\d]*)(.*)/) { $options=$1; $fmt=$2; } 
+    if ($options =~ /\$/) { $dollamode=1; }
+    if ($options =~ /,/)  { $commamode=1; }
+    if ($options =~ /\./) { $alwaysperiod=1; }
     if ($fmt) { $value=sprintf('%.'.$fmt,$value); }
+    if ($alwaysperiod && $fmt eq '0f') {
+	if ($target eq 'tex') {
+	    $value .='\\ensuremath{.}';
+	} else {
+	    $value .='.';
+	}
+    }
     if ($value =~ /([0-9\.\-\+]+)E([0-9\-\+]+)/i ) {
 	my $frac=$1;
 	if ($dollarmode) { $frac=&dollarformat($frac); }
+	if ($commamode) { $frac=&commaformat($frac); }
 	my $exponent=$2;
 	$exponent=~s/^\+0*//;
 	$exponent=~s/^-0*/-/;
@@ -558,13 +575,13 @@ sub prettyprint {
     } else {
 	$result=$value;
 	if ($dollarmode) { $result=&dollarformat($result,$target); }
+	if ($commamode) { $result=&commaformat($result,$target); }
     }
     return $result;
 }
 
-sub dollarformat {
+sub commaformat {
     my ($number,$target) = @_;
-    if (!$target) { $target = $external::target; }
     if ($number =~ /\./) {
 	while ($number =~ /([^\.,]+)([^\.,][^\.,][^\.,])([,0-9]*\.[0-9]*)$/) {
 	    $number = $1.','.$2.$3;
@@ -574,6 +591,13 @@ sub dollarformat {
 	    $number = $1.','.$2.$3;
 	}
     }
+    return $number;
+}
+
+sub dollarformat {
+    my ($number,$target) = @_;
+    if (!$target) { $target = $external::target; }
+    $number=&commaformat($number,$target);
     if ($target eq 'tex') {
 	$number='\$'.$number; #' stupid emacs
     } else {