--- loncom/interface/Attic/lonspreadsheet.pm	2002/10/21 17:59:36	1.119
+++ loncom/interface/Attic/lonspreadsheet.pm	2002/10/24 15:34:10	1.127
@@ -1,5 +1,5 @@
 #
-# $Id: lonspreadsheet.pm,v 1.119 2002/10/21 17:59:36 matthew Exp $
+# $Id: lonspreadsheet.pm,v 1.127 2002/10/24 15:34:10 matthew Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -122,20 +122,17 @@ my $tmpdir;
 ##
 sub mask {
     my ($lower,$upper)=@_;
-
-    $lower=~/([A-Za-z]|\*)(\d+|\*)/;
-    my $la=$1;
-    my $ld=$2;
-
-    $upper=~/([A-Za-z]|\*)(\d+|\*)/;
-    my $ua=$1;
-    my $ud=$2;
+    #
+    my ($la,$ld) = ($lower=~/([A-Za-z]|\*)(\d+|\*)/);
+    my ($ua,$ud) = ($upper=~/([A-Za-z]|\*)(\d+|\*)/);
+    #
     my $alpha='';
     my $num='';
-
+    #
     if (($la eq '*') || ($ua eq '*')) {
        $alpha='[A-Za-z]';
     } else {
+        
        if (($la=~/[A-Z]/) && ($ua=~/[A-Z]/) ||
            ($la=~/[a-z]/) && ($ua=~/[a-z]/)) {
           $alpha='['.$la.'-'.$ua.']';
@@ -188,8 +185,8 @@ sub mask {
                }
                $num.=')';
            } else {
-               if ($lda[$#lda]!=$uda[$#uda]) {
-                  $num.='['.$lda[$#lda].'-'.$uda[$#uda].']';
+               if ($lda[-1]!=$uda[-1]) {
+                  $num.='['.$lda[-1].'-'.$uda[-1].']';
 	       }
            }
        }
@@ -208,7 +205,6 @@ sub initsheet {
     $safeeval->deny(":base_io");
     $safehole->wrap(\&Apache::lonnet::EXT,$safeeval,'&EXT');
     $safehole->wrap(\&Apache::lonspreadsheet::mask,$safeeval,'&mask');
-    $safehole->wrap(\&Apache::lonspreadsheet::templaterow,$safeeval,'&templaterow');
     $safeeval->share('$@');
     my $code=<<'ENDDEFS';
 # ---------------------------------------------------- Inside of the safe space
@@ -238,8 +234,6 @@ undef %c; # Holds the constants for a sh
     # sheets, this is the A column.  Used in &MINPARM, &MAXPARM, &expandnamed,
     # &sett, and &setconstants.  There is no &getconstants.
     # &setconstants is called by &loadstudent, &loadcourse, &load assessment,
-undef %rowlabel;  # Holds the 'prefix' for each row.  Set by &setrowlabels.
-    # &setrowlabels is called by &updateclasssheet, &updatestudentassesssheet,
 undef @os;  # Holds the names of other spreadsheets - this is used to specify
     # the spreadsheets that are available for the assessment sheet.
     # Set by &setothersheets.  &setothersheets is called by &handler.  A
@@ -945,31 +939,60 @@ sub calc {
     return '';
 }
 
+# ------------------------------------------- End of "Inside of the safe space"
+ENDDEFS
+    $safeeval->reval($code);
+    return $safeeval;
+}
+
 #
 # This is actually used for the student spreadsheet, not the assessment sheet
 # Do not be fooled by the name!
 #
+sub templaterow {
+    my $sheet = shift;
+    my @cols=();
+    $cols[0]='<b><font size=+1>Template</font></b>';
+    foreach ('A','B','C','D','E','F','G','H','I','J','K','L','M',
+	     'N','O','P','Q','R','S','T','U','V','W','X','Y','Z',
+	     'a','b','c','d','e','f','g','h','i','j','k','l','m',
+	     'n','o','p','q','r','s','t','u','v','w','x','y','z') {
+        my $fm=$sheet->{'f'}->{'template_'.$_};
+        $fm=~s/[\'\"]/\&\#34;/g;
+        push(@cols,"'template_$_','$fm'".'___eq___'.$fm);
+    }
+    return @cols;
+}
+
+
 sub outrowassess {
     # $n is the current row number
-    my $n=shift;
+    my $sheet = shift;
+    my $n=shift; 
+    my $csv = $ENV{'form.showcsv'};
     my @cols=();
     if ($n) {
-        my ($usy,$ufn)=split(/__&&&\__/,$f{'A'.$n});
-        if ($rowlabel{$usy}) {
-            $cols[0]=$rowlabel{$usy}.'<br>'.
+        my ($usy,$ufn)=split(/__&&&\__/,$sheet->{'f'}->{'A'.$n});
+        if ($sheet->{'rowlabel'}->{$usy}) {
+            $cols[0]=&format_rowlabel($sheet->{'rowlabel'}->{$usy});
+            if (! $csv) {
+                $cols[0].='<br>'.
                 '<select name="sel_'.$n.'" onChange="changesheet('.$n.')">'.
                     '<option name="default">Default</option>';
+            }
         } else { 
             $cols[0]=''; 
         }
-        foreach (@os) {
-            $cols[0].='<option name="'.$_.'"';
-            if ($ufn eq $_) {
-                $cols[0].=' selected';
+        if (! $csv) {
+            foreach (@{$sheet->{'othersheets'}}) {
+                $cols[0].='<option name="'.$_.'"';
+                if ($ufn eq $_) {
+                    $cols[0].=' selected';
+                }
+                $cols[0].='>'.$_.'</option>';
             }
-            $cols[0].='>'.$_.'</option>';
+            $cols[0].='</select>';
         }
-        $cols[0].='</select>';
     } else {
         $cols[0]='<b><font size=+1>Export</font></b>';
     }
@@ -977,18 +1000,19 @@ sub outrowassess {
 	     'N','O','P','Q','R','S','T','U','V','W','X','Y','Z',
 	     'a','b','c','d','e','f','g','h','i','j','k','l','m',
 	     'n','o','p','q','r','s','t','u','v','w','x','y','z') {
-        my $fm=$f{$_.$n};
+        my $fm=$sheet->{'f'}->{$_.$n};
         $fm=~s/[\'\"]/\&\#34;/g;
-        push(@cols,"'$_$n','$fm'".'___eq___'.$sheet_values{$_.$n});
+        push(@cols,"'$_$n','$fm'".'___eq___'.$sheet->{'values'}->{$_.$n});
     }
     return @cols;
 }
 
 sub outrow {
+    my $sheet=shift;
     my $n=shift;
     my @cols=();
     if ($n) {
-       $cols[0]=$rowlabel{$f{'A'.$n}};
+        $cols[0]=&format_rowlabel($sheet->{'rowlabel'}->{$sheet->{'f'}->{'A'.$n}});
     } else {
        $cols[0]='<b><font size=+1>Export</font></b>';
     }
@@ -996,44 +1020,13 @@ sub outrow {
 	     'N','O','P','Q','R','S','T','U','V','W','X','Y','Z',
 	     'a','b','c','d','e','f','g','h','i','j','k','l','m',
 	     'n','o','p','q','r','s','t','u','v','w','x','y','z') {
-        my $fm=$f{$_.$n};
-        $fm=~s/[\'\"]/\&\#34;/g;
-        push(@cols,"'$_$n','$fm'".'___eq___'.$sheet_values{$_.$n});
-    }
-    return @cols;
-}
-
-sub exportrowa {
-    my @exportarray=();
-    foreach ('A','B','C','D','E','F','G','H','I','J','K','L','M',
-	     'N','O','P','Q','R','S','T','U','V','W','X','Y','Z') {
-	push(@exportarray,$sheet_values{$_.'0'});
-    } 
-    return @exportarray;
-}
-
-sub templaterow {
-    my @cols=();
-    $cols[0]='<b><font size=+1>Template</font></b>';
-    foreach ('A','B','C','D','E','F','G','H','I','J','K','L','M',
-	     'N','O','P','Q','R','S','T','U','V','W','X','Y','Z',
-	     'a','b','c','d','e','f','g','h','i','j','k','l','m',
-	     'n','o','p','q','r','s','t','u','v','w','x','y','z') {
-        my $fm=$f{'template_'.$_};
+        my $fm=$sheet->{'f'}->{$_.$n};
         $fm=~s/[\'\"]/\&\#34;/g;
-        push(@cols,"'template_$_','$fm'".'___eq___'.$fm);
+        push(@cols,"'$_$n','$fm'".'___eq___'.$sheet->{'values'}->{$_.$n});
     }
     return @cols;
 }
 
-
-# ------------------------------------------- End of "Inside of the safe space"
-ENDDEFS
-    $safeeval->reval($code);
-    return $safeeval;
-}
-
-
 # ------------------------------------------------ Add or change formula values
 sub setformulas {
     my ($sheet)=shift;
@@ -1043,6 +1036,12 @@ sub setformulas {
 # ------------------------------------------------ Add or change formula values
 sub setconstants {
     my ($sheet)=shift;
+    my ($constants) = @_;
+    if (! ref($constants)) {
+        my %tmp = @_;
+        $constants = \%tmp;
+    }
+    $sheet->{'constants'} = $constants;
     return %{$sheet->{'safe'}->varglob('c')}=%{$sheet->{'constants'}};
 }
 
@@ -1058,13 +1057,20 @@ sub setothersheets {
 # ------------------------------------------------ Add or change formula values
 sub setrowlabels {
     my $sheet=shift;
-    %{$sheet->{'safe'}->varglob('rowlabel')}=%{$sheet->{'rowlabel'}};
+    my ($rowlabel) = @_;
+    if (! ref($rowlabel)) {
+        my %tmp = @_;
+        $rowlabel = \%tmp;
+    }
+    $sheet->{'rowlabel'}=$rowlabel;
 }
 
 # ------------------------------------------------------- Calculate spreadsheet
 sub calcsheet {
     my $sheet=shift;
-    return $sheet->{'safe'}->reval('&calc();');
+    my $result =  $sheet->{'safe'}->reval('&calc();');
+    %{$sheet->{'values'}} = %{$sheet->{'safe'}->varglob('sheet_values')};
+    return $result;
 }
 
 # ---------------------------------------------------------------- Get formulas
@@ -1083,10 +1089,14 @@ sub getfa {
 # ------------------------------------------------------------- Export of A-row
 sub exportdata {
     my $sheet=shift;
-    return $sheet->{'safe'}->reval('&exportrowa()');
+    my @exportarray=();
+    foreach ('A','B','C','D','E','F','G','H','I','J','K','L','M',
+	     'N','O','P','Q','R','S','T','U','V','W','X','Y','Z') {
+	push(@exportarray,$sheet->{'values'}->{$_.'0'});
+    } 
+    return @exportarray;
 }
 
-
 # ========================================================== End of Spreadsheet
 # =============================================================================
 
@@ -1111,27 +1121,32 @@ sub rown {
         $rowdata.="\n".'"'.$n.'"';
     }
     my $showf=0;
-    my $proc;
+    #
+    # Determine how many pink (uneditable) cells there are in this sheet.
     my $maxred=1;
     my $sheettype=$sheet->{'sheettype'};
     if ($sheettype eq 'studentcalc') {
-        $proc='&outrowassess';
         $maxred=26;
-    } else {
-        $proc='&outrow';
-    }
-    if ($sheettype eq 'assesscalc') {
+    } elsif ($sheettype eq 'assesscalc') {
         $maxred=1;
     } else {
         $maxred=26;
     }
-    if (&getfa($sheet,$n)=~/^[\~\-]/) { $maxred=1; }
+    $maxred=1 if (&getfa($sheet,$n)=~/^[\~\-]/);
+    #
+    # Get the proper row
+    my @rowdata;
     if ($n eq '-') { 
-        $proc='&templaterow'; 
+        @rowdata = &templaterow($sheet);
         $n=-1; 
         $dataflag=1; 
+    } elsif ($sheettype eq 'studentcalc') {
+        @rowdata = &outrowassess($sheet,$n);
+    } else {
+        @rowdata = &outrow($sheet,$n);
     }
-    foreach ($sheet->{'safe'}->reval($proc.'('.$n.')')) {
+    #
+    foreach (@rowdata) {
         my $bgcolor=$defaultbg.((($showf-1)/5==int(($showf-1)/5))?'99':'DD');
         my ($fm,$vl)=split(/\_\_\_eq\_\_\_/,$_);
         if ((($vl ne '') || ($vl eq '0')) &&
@@ -1601,6 +1616,40 @@ sub parmval {
     return &Apache::lonnet::metadata($fn,$rwhat.'.default');
 }
 
+sub format_rowlabel {
+    my $rowlabel = shift;
+    my ($type,$labeldata) = split(':',$rowlabel,2);
+    my $result = '';
+    if ($type eq 'symb') {
+        my ($symb,$uname,$udom,$title) = split(':',$labeldata);
+        $symb = &Apache::lonnet::unescape($symb);
+        if ($ENV{'form.showcsv'}) {
+            $result = $title;
+        } else {
+            $result = '<a href="/adm/assesscalc?usymb='.$symb.
+                '&uname='.$uname.'&udom='.$udom.'">'.$title.'</a>';
+        }
+    } elsif ($type eq 'student') {
+        my ($sname,$sdom,$fullname,$section,$id) = split(':',$labeldata);
+        if ($ENV{'form.showcsv'}) {
+            $result = '"'.
+                join('","',($sname,$sdom,$fullname,$section,$id).'"');
+        } else {
+            $result ='<a href="/adm/studentcalc?uname='.$sname.
+                '&udom='.$sdom.'">';
+            $result.=$section.'&nbsp;'.$id."&nbsp;".$fullname.'</a>';
+        }
+    } elsif ($type eq 'parameter') {
+        if ($ENV{'form.showcsv'}) {
+            $labeldata =~ s/<br>/ /g;
+        }
+        $result = $labeldata;
+    } else {
+        &Apache::lonnet::logthis("lonspreadsheet:bogus rowlabel type: $type");
+    }
+    return $result;
+}
+
 # ---------------------------------------------- Update rows for course listing
 sub updateclasssheet {
     my ($sheet) = @_;
@@ -1623,17 +1672,9 @@ sub updateclasssheet {
         my ($studentDomain,$studentName,$end,$start,$id,$studentSection,
             $fullname,$status)   =   @{$classlist->{$student}};
         if ($ENV{'form.Status'} eq $status || $ENV{'form.Status'} eq 'Any') {
-            my $rowlabel='';
-            if ($ENV{'form.showcsv'}) {
-                $rowlabel= '"'.join('","',($studentName,$studentDomain,
-                                           $fullname,$studentSection,$id).'"');
-            } else {
-                $rowlabel='<a href="/adm/studentcalc?uname='.$studentName.
-                    '&udom='.$studentDomain.'">';
-                $rowlabel.=$studentSection.'&nbsp;'.$id."&nbsp;".$fullname;
-                $rowlabel.='</a>';
-            }
-            $currentlist{$student}=$rowlabel;
+            $currentlist{$student}=join(':',('student',$studentName,
+                                             $studentDomain,$fullname,
+                                             $studentSection,$id));
         }
     }
     #
@@ -1653,7 +1694,7 @@ sub updateclasssheet {
             }
             $existing{$f{$_}}=1;
             unless ((defined($currentlist{$f{$_}})) || (!$1) ||
-                    ($f{$_}=~/^(\~\~\~|\-\-\-)/)) {
+                    ($f{$_}=~/^(~~~|---)/)) {
                 $f{$_}='!!! Obsolete';
                 $changed=1;
             }
@@ -1673,8 +1714,7 @@ sub updateclasssheet {
         &setformulas($sheet,%f); 
     }
     #
-    $sheet->{'rowlabel'} = \%currentlist;
-    &setrowlabels($sheet);
+    &setrowlabels($sheet,\%currentlist);
 }
 
 # ----------------------------------- Update rows for student and assess sheets
@@ -1698,28 +1738,28 @@ sub updatestudentassesssheet {
             return 'Could not access course data';
         }
         # Get all assessments
+        #
+        # allkeys is used in the assessment sheets to provide labels
+        # for the parameters.
         my %allkeys=('timestamp' => 
-                     'Timestamp of Last Transaction<br>timestamp',
+                     'parameter:Timestamp of Last Transaction<br>timestamp',
                      'subnumber' =>
-                     'Number of Submissions<br>subnumber',
+                     'parameter:Number of Submissions<br>subnumber',
                      'tutornumber' =>
-                     'Number of Tutor Responses<br>tutornumber',
+                     'parameter:Number of Tutor Responses<br>tutornumber',
                      'totalpoints' =>
-                     'Total Points Granted<br>totalpoints');
+                     'parameter:Total Points Granted<br>totalpoints');
         my $adduserstr='';
         if (($uname ne $ENV{'user.name'}) || ($udom ne $ENV{'user.domain'})){
             $adduserstr='&uname='.$uname.'&udom='.$udom;
         }
-        my %allassess =
-            ('_feedback' =>'<a href="/adm/assesscalc?usymb=_feedback'.
-             $adduserstr.'">Feedback</a>',
-             '_evaluation' =>'<a href="/adm/assesscalc?usymb=_evaluation'.
-             $adduserstr.'">Evaluation</a>',
-             '_tutoring' =>'<a href="/adm/assesscalc?usymb=_tutoring'.
-             $adduserstr.'">Tutoring</a>',
-             '_discussion' =>'<a href="/adm/assesscalc?usymb=_discussion'.
-             $adduserstr.'">Discussion</a>'
-             );
+        #
+        # allassess holds the descriptions of all assessments
+        my %allassess;
+        foreach ('Feedback','Evaluation','Tutoring','Discussion') {
+            my $symb = '_'.lc($_);
+            $allassess{$symb} = join(':',('symb',$symb,$uname,$udom,$_));
+        }
         while (($_,undef) = each(%bighash)) {
             next if ($_!~/^src\_(\d+)\.(\d+)$/);
             my $mapid=$1;
@@ -1730,9 +1770,8 @@ sub updatestudentassesssheet {
                 my $symb=
                     &Apache::lonnet::declutter($bighash{'map_id_'.$mapid}).
                         '___'.$resid.'___'.&Apache::lonnet::declutter($srcf);
-                $allassess{$symb}=
-                    '<a href="/adm/assesscalc?usymb='.$symb.$adduserstr.'">'.
-                        $bighash{'title_'.$id}.'</a>';
+                $allassess{$symb}='symb:'.&Apache::lonnet::escape($symb).':'
+                    .$uname.':'.$udom.':'.$bighash{'title_'.$id};
                 next if ($stype ne 'assesscalc');
                 foreach my $key (split(/\,/,
                                        &Apache::lonnet::metadata($srcf,'keys')
@@ -1745,7 +1784,7 @@ sub updatestudentassesssheet {
                             &Apache::lonnet::metadata($srcf,$key.'.name');
                     }
                     $display.='<br>'.$key;
-                    $allkeys{$key}=$display;
+                    $allkeys{$key}='parameter:'.$display;
                 } # end of foreach
             }
         } # end of foreach (keys(%bighash))
@@ -1780,7 +1819,7 @@ sub updatestudentassesssheet {
         $existing{$usy}=1;
         unless ((exists($sheet->{'rowlabel'}->{$usy}) && 
                  (defined($sheet->{'rowlabel'}->{$usy})) || (!$1) ||
-                ($f{$_}=~/^(\~\~\~|\-\-\-)/))){
+                ($f{$_}=~/^(~~~|---)/))){
             $f{$_}='!!! Obsolete';
             $changed=1;
         } elsif ($ufn) {
@@ -1800,7 +1839,6 @@ sub updatestudentassesssheet {
         $sheet->{'f'} = \%f;
         &setformulas($sheet); 
     }
-    &setrowlabels($sheet);
     #
     undef %existing;
 }
@@ -1851,9 +1889,8 @@ sub loadstudent {
     $cachedassess='';
     undef %cachedstores;
     $sheet->{'f'} = \%f;
-    $sheet->{'constants'} = \%c;
     &setformulas($sheet);
-    &setconstants($sheet);
+    &setconstants($sheet,\%c);
 }
 
 # --------------------------------------------------- Load data for one student
@@ -1913,9 +1950,8 @@ ENDPOP
         }
     }
     $sheet->{'f'}=\%f;
-    $sheet->{'constants'}=\%c;
     &setformulas($sheet);
-    &setconstants($sheet);
+    &setconstants($sheet,\%c);
     $r->print('<script>popwin.close()</script>');
     $r->rflush(); 
 }
@@ -2019,28 +2055,27 @@ sub loadassessment {
     if (tie(%parmhash,'GDBM_File',
             $sheet->{'coursefilename'}.'_parms.db',&GDBM_READER(),0640)) {
         my %f=&getformulas($sheet);
-        foreach (keys(%f))  {
-            next if ($_!~/^A/);
-            next if  ($f{$_}=~/^[\!\~\-]/);
-            if ($f{$_}=~/^parameter/) {
-                if ($thisassess{$f{$_}}) {
-                    my $val=&parmval($f{$_},$sheet);
-                    $c{$_}=$val;
-                    $c{$f{$_}}=$val;
+        foreach my $cell (keys(%f))  {
+            next if ($cell !~ /^A/);
+            next if  ($f{$cell} =~/^[\!\~\-]/);
+            if ($f{$cell}=~/^parameter/) {
+                if (defined($thisassess{$f{$cell}})) {
+                    my $val       = &parmval($f{$cell},$sheet);
+                    $c{$cell}     = $val;
+                    $c{$f{$cell}} = $val;
                 }
             } else {
-                my $key=$f{$_};
+                my $key=$f{$cell};
                 my $ckey=$key;
                 $key=~s/^stores\_/resource\./;
                 $key=~s/\_/\./g;
-                $c{$_}=$returnhash{$key};
+                $c{$cell}=$returnhash{$key};
                 $c{$ckey}=$returnhash{$key};
             }
         }
         untie(%parmhash);
     }
-    $sheet->{'constants'}=\%c;
-    &setconstants($sheet);
+    &setconstants($sheet,\%c);
 }
 
 # --------------------------------------------------------- Various form fields
@@ -2113,7 +2148,7 @@ sub forcedrecalc {
     if ($ENV{'form.forcerecalc'}) { return 1; }
     unless ($time) { return 1; }
     if ($stype eq 'assesscalc') {
-        my $map=(split(/\_\_\_/,$usymb))[0];
+        my $map=(split(/___/,$usymb))[0];
         if (&checkthis('::assesscalc:',$time) ||
             &checkthis('::assesscalc:'.$map,$time) ||
             &checkthis('::assesscalc:'.$usymb,$time) ||
@@ -2132,14 +2167,10 @@ sub forcedrecalc {
 }
 
 # ============================================================== Export handler
-#
-# Non-interactive call from with program
-#
-
 sub exportsheet {
     my ($uname,$udom,$stype,$usymb,$fn)=@_;
     my @exportarr=();
-    if (($usymb=~/^\_(\w+)/) && (!$fn)) {
+    if (defined($usymb) && ($usymb=~/^\_(\w+)/) && (!$fn)) {
         $fn='default_'.$1;
     }
     #
@@ -2148,8 +2179,8 @@ sub exportsheet {
     my $key=$uname.':'.$udom.':'.$stype.':'.$usymb;
     my $found='';
     if ($oldsheets{$key}) {
-        foreach (split(/\_\_\_\&\_\_\_/,$oldsheets{$key})) {
-            my ($name,$value)=split(/\_\_\_\=\_\_\_/,$_);
+        foreach (split(/___&\___/,$oldsheets{$key})) {
+            my ($name,$value)=split(/___=___/,$_);
             if ($name eq $fn) {
                 $found=$value;
             }
@@ -2158,8 +2189,8 @@ sub exportsheet {
     unless ($found) {
         &cachedssheets($uname,$udom,&Apache::lonnet::homeserver($uname,$udom));
         if ($oldsheets{$key}) {
-            foreach (split(/\_\_\_\&\_\_\_/,$oldsheets{$key})) {
-                my ($name,$value)=split(/\_\_\_\=\_\_\_/,$_);
+            foreach (split(/___&\___/,$oldsheets{$key})) {
+                my ($name,$value)=split(/___=___/,$_);
                 if ($name eq $fn) {
                     $found=$value;
                 }
@@ -2178,72 +2209,65 @@ sub exportsheet {
         #
         # Return what was cached
         #
-        @exportarr=split(/\_\_\_\;\_\_\_/,$found);
-    } else {
-        #
-        # Not cached
-        #        
-        my ($sheet)=&makenewsheet($uname,$udom,$stype,$usymb);
-        &readsheet($sheet,$fn);
-        &updatesheet($sheet);
-        &loadrows($sheet);
-        &calcsheet($sheet); 
-        @exportarr=&exportdata($sheet);
-        #
-        # Store now
-        #
-        my $cid=$ENV{'request.course.id'}; 
-        my $current='';
-        if ($stype eq 'studentcalc') {
-            $current=&Apache::lonnet::reply('get:'.
-                                            $ENV{'course.'.$cid.'.domain'}.':'.
-                                            $ENV{'course.'.$cid.'.num'}.
-                                            ':nohist_calculatedsheets:'.
-                                            &Apache::lonnet::escape($key),
-                                            $ENV{'course.'.$cid.'.home'});
-        } else {
-            $current=&Apache::lonnet::reply('get:'.$sheet->{'udom'}.':'.
-                                            $sheet->{'uname'}.
-                                            ':nohist_calculatedsheets_'.
-                                            $ENV{'request.course.id'}.':'.
-                                            &Apache::lonnet::escape($key),
-                                            $sheet->{'uhome'});
-        }
-        my %currentlystored=();
-        unless ($current=~/^error\:/) {
-            foreach (split(/___&\___/,&Apache::lonnet::unescape($current))) {
-                my ($name,$value)=split(/___=___/,$_);
-                $currentlystored{$name}=$value;
-            }
-        }
-        $currentlystored{$fn}=join('___;___',@exportarr);
-        #
-        my $newstore='';
-        foreach (keys(%currentlystored)) {
-            if ($newstore) { $newstore.='___&___'; }
-            $newstore.=$_.'___=___'.$currentlystored{$_};
-        }
-        my $now=time;
-        if ($stype eq 'studentcalc') {
-            &Apache::lonnet::reply('put:'.
-                                   $ENV{'course.'.$cid.'.domain'}.':'.
-                                   $ENV{'course.'.$cid.'.num'}.
-                                   ':nohist_calculatedsheets:'.
-                                   &Apache::lonnet::escape($key).'='.
-                                   &Apache::lonnet::escape($newstore).'&'.
-                                   &Apache::lonnet::escape($key).'.time='.$now,
-                                   $ENV{'course.'.$cid.'.home'});
-        } else {
-            &Apache::lonnet::reply('put:'.
-                                   $sheet->{'udom'}.':'.
-                                   $sheet->{'uname'}.
-                                   ':nohist_calculatedsheets_'.
-                                   $ENV{'request.course.id'}.':'.
-                                   &Apache::lonnet::escape($key).'='.
-                                   &Apache::lonnet::escape($newstore).'&'.
-                                   &Apache::lonnet::escape($key).'.time='.$now,
-                                   $sheet->{'uhome'});
-        }
+        @exportarr=split(/___;___/,$found);
+        return @exportarr;
+    }
+    #
+    # Not cached
+    #        
+    my ($sheet)=&makenewsheet($uname,$udom,$stype,$usymb);
+    &readsheet($sheet,$fn);
+    &updatesheet($sheet);
+    &loadrows($sheet);
+    &calcsheet($sheet); 
+    @exportarr=&exportdata($sheet);
+    #
+    # Store now
+    #
+    my $cid=$ENV{'request.course.id'}; 
+    my $current='';
+    if ($stype eq 'studentcalc') {
+        $current=&Apache::lonnet::reply('get:'.
+                                        $ENV{'course.'.$cid.'.domain'}.':'.
+                                        $ENV{'course.'.$cid.'.num'}.
+                                        ':nohist_calculatedsheets:'.
+                                        &Apache::lonnet::escape($key),
+                                        $ENV{'course.'.$cid.'.home'});
+    } else {
+        $current=&Apache::lonnet::reply('get:'.$sheet->{'udom'}.':'.
+                                        $sheet->{'uname'}.
+                                        ':nohist_calculatedsheets_'.
+                                        $ENV{'request.course.id'}.':'.
+                                        &Apache::lonnet::escape($key),
+                                        $sheet->{'uhome'});
+    }
+    my %currentlystored=();
+    unless ($current=~/^error\:/) {
+        foreach (split(/___&\___/,&Apache::lonnet::unescape($current))) {
+            my ($name,$value)=split(/___=___/,$_);
+            $currentlystored{$name}=$value;
+        }
+    }
+    $currentlystored{$fn}=join('___;___',@exportarr);
+    #
+    my $newstore='';
+    foreach (keys(%currentlystored)) {
+        if ($newstore) { $newstore.='___&___'; }
+        $newstore.=$_.'___=___'.$currentlystored{$_};
+    }
+    my $now=time;
+    if ($stype eq 'studentcalc') {
+        &Apache::lonnet::put('nohist_calculatedsheets',
+                             { $key => $newstore,
+                               $key.time => $now },
+                             $ENV{'course.'.$cid.'.domain'},
+                             $ENV{'course.'.$cid.'.num'})
+    } else {
+        &Apache::lonnet::put('nohist_calculatedsheets_'.$sheet->{'cid'},
+                             { $key => $newstore,
+                               $key.time => $now },
+                             $sheet->{'udom'},
+                             $sheet->{'uname'})
     }
     return @exportarr;
 }
@@ -2451,7 +2475,7 @@ ENDSCRIPT
         &readsheet($sheet,$ENV{'form.ufn'});
     }
     # Print out user information
-    unless ($sheet->{'sheettype'} eq 'classcalc') {
+    if ($sheet->{'sheettype'} ne 'classcalc') {
         $r->print('<p><b>User:</b> '.$sheet->{'uname'}.
                   '<br><b>Domain:</b> '.$sheet->{'udom'});
         $r->print('<br><b>Section/Group:</b> '.$sheet->{'csec'});
@@ -2564,7 +2588,7 @@ ENDSCRIPT
         $r->print('<br>Show rows with empty A column: ');
     } else {
         $r->print('<br>Show empty rows: ');
-    } 
+    }
     #
     $r->print(&hiddenfield('userselhidden','true').
               '<input type="checkbox" name="showall" onClick="submit()"');
@@ -2581,19 +2605,19 @@ ENDSCRIPT
         }
     }
     $r->print('>');
+    #
+    # CSV format checkbox (classcalc sheets only)
+    $r->print(' Output CSV format: <input type="checkbox" '.
+              'name="showcsv" onClick="submit()"');
+    $r->print(' checked') if ($ENV{'form.showcsv'});
+    $r->print('>');
     if ($sheet->{'sheettype'} eq 'classcalc') {
-        #
-        # CSV format checkbox (classcalc sheets only)
-        $r->print(' Output CSV format: <input type="checkbox" '.
-                  'name="showcsv" onClick="submit()"');
-        if ($ENV{'form.showcsv'}) { $r->print(' checked'); }
-        $r->print('>');
-        #
-        # Buttons to insert rows
         $r->print('&nbsp;Student Status: '.
                   &Apache::lonhtmlcommon::StatusOptions
                   ($ENV{'form.Status'},'sheet'));
     }
+    #
+    # Buttons to insert rows
     $r->print(<<ENDINSERTBUTTONS);
 <br>
 <input type='button' onClick='insertrow("top");'