--- loncom/homework/imageresponse.pm	2001/06/02 03:59:59	1.8
+++ loncom/homework/imageresponse.pm	2003/09/22 20:49:01	1.35
@@ -1,33 +1,71 @@
 # The LearningOnline Network with CAPA
-# iimage click response style
-
-#FIXME assumes multiple possbile submissions but only one is possible currently
+# image click response style
+#
+# $Id: imageresponse.pm,v 1.35 2003/09/22 20:49:01 albertel Exp $
+#
+# Copyright Michigan State University Board of Trustees
+#
+# This file is part of the LearningOnline Network with CAPA (LON-CAPA).
+#
+# LON-CAPA is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# LON-CAPA is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with LON-CAPA; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+#
+# /home/httpd/html/adm/gpl.txt
+#
+# http://www.lon-capa.org/
+#
+# July,August 2003 H. K. Ng
+#
+#FIXME LATER assumes multiple possible submissions but only one is possible 
+#currently
 
 package Apache::imageresponse;
+use Apache::randomlylabel;
 use strict;
+use Image::Magick;
+use GD;
 
-sub BEGIN {
+BEGIN {
   &Apache::lonxml::register('Apache::imageresponse',('imageresponse'));
 }
 
 sub start_imageresponse {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
+  my $result;
   #when in a radiobutton response use these
   &Apache::lonxml::register('Apache::imageresponse',('foilgroup','foil','text','image','rectangle','conceptgroup'));
   push (@Apache::lonxml::namespace,'imageresponse');
   my $id = &Apache::response::start_response($parstack,$safeeval);
-  return '';
+  if ($target eq 'meta') {
+    $result=&Apache::response::meta_package_write('imageresponse');
+  }
+  return $result;
 }
 
 sub end_imageresponse {
-  &Apache::response::end_response;
-  pop @Apache::lonxml::namespace;
-  return '';
+    my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
+    &Apache::response::end_response;
+    pop @Apache::lonxml::namespace;
+    &Apache::lonxml::deregister('Apache::imageresponse',('foilgroup','foil','text','image','rectangle','conceptgroup'));
+    my $result;
+    if ($target eq 'edit') { $result=&Apache::edit::end_table(); }
+    return $result;
 }
 
-%Apache::response::foilgroup={};
+%Apache::response::foilgroup=();
 sub start_foilgroup {
-  %Apache::response::foilgroup={};
+  %Apache::response::foilgroup=();
   $Apache::imageresponse::conceptgroup=0;
   &Apache::response::setrandomnumber();
   return '';
@@ -39,16 +77,23 @@ sub getfoilcounts {
   my $max = &Apache::lonxml::get_param('max',$parstack,$safeeval,'-2');
   # +1 since instructors will count from 1
   my $count = $#{ $Apache::response::foilgroup{'names'} }+1;
+  if (&Apache::response::showallfoils()) { $max=$count; }
   return ($count,$max);
 }
 
 sub whichfoils {
   my ($max)=@_;
+  if (!defined(@{ $Apache::response::foilgroup{'names'} })) { return; }
   my @names = @{ $Apache::response::foilgroup{'names'} };
   my @whichopt =();
   while ((($#whichopt+1) < $max) && ($#names > -1)) {
     &Apache::lonxml::debug("Have $#whichopt max is $max");
-    my $aopt=int(rand($#names+1));
+    my $aopt;
+    if (&Apache::response::showallfoils()) {
+      $aopt=0;
+    } else {
+      $aopt=int(&Math::Random::random_uniform() * ($#names+1));
+    }
     &Apache::lonxml::debug("From $#names elms, picking $aopt");
     $aopt=splice(@names,$aopt,1);
     &Apache::lonxml::debug("Picked $aopt");
@@ -58,17 +103,28 @@ sub whichfoils {
 }
 
 sub displayfoils {
-  my (@whichopt) = @_;
+  my ($target,@whichopt) = @_;
   my $result ='';
   my $name;
   my $temp=1;
   foreach $name (@whichopt) {
-    $result.=$Apache::response::foilgroup{"$name.text"}."<br />\n";
+    $result.=$Apache::response::foilgroup{"$name.text"};
+    &Apache::lonxml::debug("Text is $result");
+    if ($target eq 'tex') {$result.="\\vskip 0 mm \n";} else {$result.="<br />\n";}
     my $image=$Apache::response::foilgroup{"$name.image"};
-    if ($Apache::lonhomework::history{"resource.$Apache::inputtags::part.solved"} =~ /^correct/ ) {
-      $result.="<img src=\"$image\"/> <br />\n";
+    &Apache::lonxml::debug("image is $image");
+    if ( &Apache::response::show_answer() ) {
+      if ($target eq 'tex') {
+	$result.=$Apache::response::foilgroup{"$name.image"}."\\vskip 0 mm \n";
+      } else {
+        $result.="<img src=\"$image\"/> <br />\n";
+      }
     } else {
-      $result.="<input type=\"image\" name=\"HWVAL_$Apache::inputtags::response['-1']:$temp\" src=\"$image\"/> <br />\n";
+      if ($target eq 'tex') {
+	$result.=$Apache::response::foilgroup{"$name.image"}."\\vskip 0 mm \n";
+      } else {
+        $result.="<input type=\"image\" name=\"HWVAL_$Apache::inputtags::response['-1']:$temp\" src=\"$image\"/> <br />\n";
+      }
     }
     $temp++;
   }
@@ -77,7 +133,6 @@ sub displayfoils {
 
 sub gradefoils {
   my (@whichopt) = @_;
-  my $result='';
   my $x;
   my $y;
   my $result;
@@ -87,25 +142,28 @@ sub gradefoils {
     $x=$ENV{"form.HWVAL_$id:$temp.x"};
     $y=$ENV{"form.HWVAL_$id:$temp.y"};
     &Apache::lonxml::debug("Got a x of $x and a y of $y for $name");
-    my @areas = @{ $Apache::response::foilgroup{"$name.area"} };
-    my $grade="INCORRECT";
-    foreach my $area (@areas) {
-      &Apache::lonxml::debug("Area is $area for $name");
-      $area =~ m/([a-z]*):/;
-      &Apache::lonxml::debug("Area of type $1");
-      if ($1 eq 'rectangle') {
-	$grade=&grade_rectangle($area,$x,$y);
-      } else {
-	&Apache::lonxml::error("Unknown area style $area");
+    if (defined($x) && defined($y) &&
+	defined(@{ $Apache::response::foilgroup{"$name.area"} })) {
+      my @areas = @{ $Apache::response::foilgroup{"$name.area"} };
+      my $grade="INCORRECT";
+      foreach my $area (@areas) {
+	&Apache::lonxml::debug("Area is $area for $name");
+	$area =~ m/([a-z]*):/;
+	&Apache::lonxml::debug("Area of type $1");
+	if ($1 eq 'rectangle') {
+	  $grade=&grade_rectangle($area,$x,$y);
+	} else {
+	  &Apache::lonxml::error("Unknown area style $area");
+	}
+	&Apache::lonxml::debug("Area said $grade");
+	if ($grade eq 'APPROX_ANS') { last; }
       }
-      &Apache::lonxml::debug("Area said $grade");
-      if ($grade eq 'APPROX_ANS') { last; }
+      &Apache::lonxml::debug("Foil was $grade");
+      if ($grade eq 'INCORRECT') { $result= 'INCORRECT'; }
+      if (($grade eq 'APPROX_ANS') && ($result ne 'APPROX_ANS')) { $result=$grade; }
+      &Apache::lonxml::debug("Question is $result");
+      $temp++;
     }
-    &Apache::lonxml::debug("Foil was $grade");
-    if ($grade eq 'INCORRECT') { $result= 'INCORRECT'; }
-    if (($grade eq 'APPROX_ANS') && ($result ne 'APPROX_ANS')) { $result=$grade; }
-    &Apache::lonxml::debug("Question is $result");
-    $temp++;
   }
   $Apache::lonhomework::results{"resource.$Apache::inputtags::part.$id.submission"}="$x:$y";
   $Apache::lonhomework::results{"resource.$Apache::inputtags::part.$id.awarddetail"}=$result;
@@ -116,54 +174,71 @@ sub end_foilgroup {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
   my $result='';
   my @whichopt;
-  if ($target eq 'web' || $target eq 'grade') {
+  if ($target eq 'web' || $target eq 'grade' || $target eq 'tex') {
     my ($count,$max) = &getfoilcounts($parstack,$safeeval);
     if ($count>$max) { $count=$max }
     &Apache::lonxml::debug("Count is $count from $max");
     @whichopt = &whichfoils($max);
-  }
-  if ($target eq 'web') {
-    $result=&displayfoils(@whichopt);
-  }
-  if ($target eq 'grade') {
-    if ( defined $ENV{'form.submitted'}) {
-      &gradefoils(@whichopt);
-    }
+    if ($target eq 'web' || $target eq 'tex') {
+	$result=&displayfoils($target,@whichopt);
+    } elsif ($target eq 'grade') {
+	if ( defined $ENV{'form.submitted'}) { &gradefoils(@whichopt); }
+    } 
+  } elsif ($target eq 'edit') {
+      $result=&Apache::edit::end_table();
   }
   return $result;
 }
 
 sub start_conceptgroup {
   $Apache::imageresponse::conceptgroup=1;
-  %Apache::response::conceptgroup={};
+  %Apache::response::conceptgroup=();
   return '';
 }
 
 sub end_conceptgroup {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  $Apache::imageresponse::conceptgroup=0;  
-  if ($target eq 'web' || $target eq 'grade') {
-    my @names = @{ $Apache::response::conceptgroup{'names'} };
-    my $pick=int(rand($#names+1));
-    my $name=$names[$pick];
-    push @{ $Apache::response::foilgroup{'names'} }, $name;
-    $Apache::response::foilgroup{"$name.text"} = $Apache::response::conceptgroup{"$name.text"};
-    $Apache::response::foilgroup{"$name.image"} = $Apache::response::conceptgroup{"$name.image"};
-    push(@{ $Apache::response::foilgroup{"$name.area"} }, @{ $Apache::response::conceptgroup{"$name.area"} });
-    my $concept = &Apache::lonxml::get_param('concept',$parstack,$safeeval);
-    $Apache::response::foilgroup{"$name.concept"} = $concept;
-    &Apache::lonxml::debug("Selecting $name in $concept");
+  $Apache::imageresponse::conceptgroup=0;
+  my $result;
+  if ($target eq 'web' || $target eq 'grade' || $target eq 'tex') {
+    if (defined(@{ $Apache::response::conceptgroup{'names'} })) {
+      my @names = @{ $Apache::response::conceptgroup{'names'} };
+      my $pick=int(&Math::Random::random_uniform() * ($#names+1));
+      my $name=$names[$pick];
+      if (defined(@{ $Apache::response::conceptgroup{"$name.area"} })) {
+	push @{ $Apache::response::foilgroup{'names'} }, $name;
+	$Apache::response::foilgroup{"$name.text"} = $Apache::response::conceptgroup{"$name.text"};
+	$Apache::response::foilgroup{"$name.image"} = $Apache::response::conceptgroup{"$name.image"};
+	push(@{ $Apache::response::foilgroup{"$name.area"} }, @{ $Apache::response::conceptgroup{"$name.area"} });
+	my $concept = &Apache::lonxml::get_param('concept',$parstack,$safeeval);
+	$Apache::response::foilgroup{"$name.concept"} = $concept;
+	&Apache::lonxml::debug("Selecting $name in $concept");
+      }
+    }
+  } elsif ($target eq 'edit') {
+      $result=&Apache::edit::end_table();
   }
-  return '';
+  return $result;
+}
+
+sub insert_foil {
+    return '
+       <foil>
+           <image></image>
+           <text></text>
+           <rectangle></rectangle>
+       </foil>
+';
 }
 
 $Apache::imageresponse::curname='';
 sub start_foil {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  if ($target eq 'web' || $target eq 'grade') {
+  if ($target eq 'web' || $target eq 'grade' || $target eq 'tex') {
     my $name = &Apache::lonxml::get_param('name',$parstack,$safeeval);
     if ($name eq '') { $name=$Apache::lonxml::curdepth; }
-    if ( $Apache::imageresponse::conceptgroup ) {
+    if ( $Apache::imageresponse::conceptgroup
+	 && !&Apache::response::showallfoils()) {
       push(@{ $Apache::response::conceptgroup{'names'} }, $name);
     } else {
       push(@{ $Apache::response::foilgroup{'names'} }, $name);
@@ -174,54 +249,210 @@ sub start_foil {
 }
 
 sub end_foil {
-  my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  return '';
+    my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
+    my $result;
+    if ($target eq 'edit') {
+	$result=&Apache::edit::end_table();
+    }
+    return $result;
 }
 
 sub start_text {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  if ($target eq 'web') { &Apache::lonxml::startredirection; }
-  return '';
+  my $result='';
+  if ($target eq 'web' || $target eq 'tex') { 
+     &Apache::lonxml::startredirection; 
+  } elsif ($target eq 'edit') {
+    my $descr=&Apache::lonxml::get_all_text('/text',$parser);
+    $result=&Apache::edit::tag_start($target,$token,'Task Description').
+	&Apache::edit::editfield($token->[1],$descr,'Text',60,2).
+        &Apache::edit::end_row();
+  } elsif ($target eq "modified") {
+    my $descr=&Apache::lonxml::get_all_text('/text',$parser);
+    $result=$token->[4].&Apache::edit::modifiedfield($token);
+    &Apache::lonxml::debug($result);
+  }
+  return $result;
 }
 
 sub end_text {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  if ($target eq 'web') {
+  my $result;
+  if ($target eq 'web' || $target eq 'tex') {
     my $name = $Apache::imageresponse::curname;
-    if ( $Apache::imageresponse::conceptgroup ) {
+    if ( $Apache::imageresponse::conceptgroup
+       && !&Apache::response::showallfoils() ) {
       $Apache::response::conceptgroup{"$name.text"} = &Apache::lonxml::endredirection;
     } else {
       $Apache::response::foilgroup{"$name.text"} = &Apache::lonxml::endredirection;
     }
+  } elsif ($target eq 'edit') {
+      $result=&Apache::edit::end_table();
   }
-  return '';
+  return $result;
 }
 
 sub start_image {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  if ($target eq 'web') { &Apache::lonxml::startredirection; }
-  return '';
+  my $result='';
+  if ($target eq 'web' || $target eq 'tex') { 
+      &Apache::lonxml::startredirection; 
+  } elsif ($target eq 'edit') {
+    my $bgimg=&Apache::lonxml::get_all_text('/image',$parser);
+    $Apache::edit::bgimgsrc=$bgimg;
+    $Apache::edit::bgimgsrcdepth=$Apache::lonxml::curdepth;
+
+    $result=&Apache::edit::tag_start($target,$token,'Clickable Image').
+	&Apache::edit::editline($token->[1],$bgimg,'Image Source File',40);
+    $result.=&Apache::edit::browse(undef,'textnode').' ';
+    $result.=&Apache::edit::search(undef,'textnode').
+        &Apache::edit::end_row();
+  } elsif ($target eq "modified") {
+    my $bgimg=&Apache::lonxml::get_all_text('/image',$parser);
+    $result=$token->[4].&Apache::edit::modifiedfield($token);
+    &Apache::lonxml::debug($result);
+  }
+  return $result;
 }
 
 sub end_image {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
+  my $result;
+  my $name = $Apache::imageresponse::curname;
   if ($target eq 'web') {
-    my $name = $Apache::imageresponse::curname;
     my $image = &Apache::lonxml::endredirection;
-    &Apache::lonxml::debug("out is $image");
-    if ( $Apache::imageresponse::conceptgroup ) {
+    &Apache::lonxml::debug("original image is $image");
+    my $id=$Apache::inputtags::response['-1'];
+    my $temp=1;
+    my $x=$ENV{"form.HWVAL_$id:$temp.x"};
+    my $y=$ENV{"form.HWVAL_$id:$temp.y"};
+    if (defined ($x) && defined ($y)) {
+	&Apache::lonxml::debug("x and y defined as $x,$y");
+	my $currentImage = &Apache::randomlylabel::get_image('/home/httpd/html'.$image,1);
+	if (! defined($currentImage)) {
+	    &Apache::lonnet::logthis('Unable to create image object for '.$image);
+	    return '';
+	}
+	my $red;
+	if (!($red = $currentImage->colorResolve(255,0,0))) {
+	    $red = $currentImage->colorClosestHWB(255,0,0);
+	}
+	my $length = 6;
+	$currentImage->line($x-$length,$y-$length,$x+$length,$y+$length,$red);
+	$currentImage->line($x-$length,$y+$length,$x+$length,$y-$length,$red);
+
+	my ($nameWOext) = ($image =~ /^.*\/(.*)\..*$/);
+	&Apache::lonxml::debug("graph name $nameWOext");
+	my $webImageName = "/prtspool/$ENV{'user.name'}_$ENV{'user.domain'}_".
+	    $nameWOext.'.png'; #needs to be more random or specific
+	my $newImageName = '/home/httpd'.$webImageName;
+
+	my $imgfh = Apache::File->new('>'.$newImageName); 
+	print $imgfh $currentImage->png;
+	$image = $webImageName;
+    }
+    &Apache::lonxml::debug("out image is $image");
+    if ( $Apache::imageresponse::conceptgroup
+	 && !&Apache::response::showallfoils()) {
       $Apache::response::conceptgroup{"$name.image"} = $image;
     } else {
       $Apache::response::foilgroup{"$name.image"} = $image;
     }
-  }
-  return '';
+  } elsif ($target eq 'edit') {
+      $result=&Apache::edit::end_table();
+  } elsif ($target eq 'tex') {
+    my $src = &Apache::lonxml::endredirection;
+    $src=&Apache::lonnet::filelocation($Apache::lonxml::pwd[-1],$src);
+    my $width_param = '';
+    my $height_param = '';
+    my $scaling = .3;
+    my $image = Image::Magick->new;
+    my $current_figure = $image->Read($src);
+    $width_param = $image->Get('width') * $scaling;;
+    $height_param = $image->Get('height') * $scaling;;
+    undef $image;
+    my $epssrc = $src;
+    $epssrc =~ s/(\.gif|\.jpg)$/\.eps/i;
+    if (not -e $epssrc) {
+	my $localfile = $epssrc;
+	$localfile =~ s/.*(\/res)/$1/;	
+	my $file;
+	my $path;	
+	if ($localfile =~ m!(.*)/([^/]*)$!) {
+	    $file = $2;
+	    $path = $1.'/'; 
+	}	
+	my $signal_eps = 0;
+	my @content_directory = &Apache::lonnet::dirlist($path);
+	for (my $iy=0;$iy<=$#content_directory;$iy++) {
+	    my @tempo_array = split(/&/,$content_directory[$iy]);
+	    $content_directory[$iy] = $tempo_array[0];
+	    if ($file eq $tempo_array[0]) {
+		$signal_eps = 1;
+		last;
+	    }
+	}
+	if ($signal_eps) {
+	    my $eps_file = &Apache::lonnet::getfile($localfile);
+	} else {
+	    $localfile = $src;
+	    $localfile =~ s/.*(\/res)/$1/;	
+	    my $as = &Apache::lonnet::getfile($src);		      
+	}
+    }
+    my $file;
+    my $path;	
+    if ($src =~ m!(.*)/([^/]*)$!) {
+	$file = $2;
+	$path = $1.'/'; 
+    }
+    my $newsrc = $src;
+    $newsrc =~ s/(\.gif|\.jpg|\.jpeg)$/\.eps/i;
+    $file=~s/(\.gif|\.jpg|\.jpeg)$/\.eps/i;
+    #do we have any specified size of the picture?
+    my $TeXwidth = &Apache::lonxml::get_param('TeXwidth',$parstack,$safeeval); 
+    my $TeXheight = &Apache::lonxml::get_param('TeXheight',$parstack,$safeeval); 
+    my $width = &Apache::lonxml::get_param('width',$parstack,$safeeval);
+    if ($TeXwidth ne '') { 
+	$width_param = $TeXwidth; 
+    } elsif ($TeXheight ne '') { 
+	$width_param = $TeXheight/$height_param*$width_param;
+    } elsif ($width ne '') {
+	$width_param = $width*$scaling;      
+    }
+    #where can we find the picture?
+    if (-e $newsrc) {
+	if ($path) {
+	  $Apache::response::foilgroup{"$name.image"} ='\vskip 0 mm \noindent\graphicspath{{'.$path.'}}\includegraphics[width='.$width_param.' mm]{'.$file.'} ';
+	}
+    } else {
+	my $temp_file;
+	my $filename = "/home/httpd/prtspool/$ENV{'user.name'}_$ENV{'user.domain'}_printout.dat";
+	$temp_file = Apache::File->new('>>'.$filename); 
+	print $temp_file "$src\n";
+$Apache::response::foilgroup{"$name.image"} ='\vskip 0 mm \graphicspath{{/home/httpd/prtspool/}}\includegraphics[width='.$width_param.' mm]{'.$file.'} ';
+    }
+  } 
+  return $result;
 }
 
 sub start_rectangle {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  if ($target eq 'web' || $target eq 'grade') { &Apache::lonxml::startredirection; }
-  return '';
+  my $result='';
+  if ($target eq 'web' || $target eq 'grade' || $target eq 'tex') { 
+     &Apache::lonxml::startredirection; 
+  } elsif ($target eq 'edit') {
+    my $coords=&Apache::lonxml::get_all_text('/rectangle',$parser);
+    $result=&Apache::edit::tag_start($target,$token,'Rectangle').
+	&Apache::edit::editline($token->[1],$coords,'Coordinate Pairs',40).
+        &Apache::edit::entercoordpair(undef,'textnode').
+        &Apache::edit::end_row();
+  } elsif ($target eq "modified") {
+    my $coords=&Apache::lonxml::get_all_text('/rectangle',$parser);
+    $result=$token->[4].&Apache::edit::modifiedfield($token);
+    &Apache::lonxml::debug($result);
+  }
+  return $result;
 }
 
 sub grade_rectangle {
@@ -235,23 +466,29 @@ sub grade_rectangle {
   &Apache::lonxml::debug("Point $x1,$y1,$x2,$y2");
   if ($x1 > $x2) { my $temp=$x1;$x1=$x2;$x2=$temp; }
   if ($y1 > $y2) { my $temp=$y1;$y1=$y2;$y2=$temp; }
-  if ($x => $x1) { if ($x <= $x2) { if ($y => $y1) { if ($y <= $y2) { return 'APPROX_ANS'; } } } }
+  if (($x >= $x1) && ($x <= $x2) && ($y >= $y1) && ($y <= $y2)) {
+    return 'APPROX_ANS';
+  }
   return 'INCORRECT';
 }
 
 sub end_rectangle {
   my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  if ($target eq 'web' || $target eq 'grade') {
+  my $result;
+  if ($target eq 'web' || $target eq 'grade' || $target eq 'tex') {
     my $name = $Apache::imageresponse::curname;
     my $area = &Apache::lonxml::endredirection;
     &Apache::lonxml::debug("out is $area for $name");
-    if ( $Apache::imageresponse::conceptgroup ) {
+    if ( $Apache::imageresponse::conceptgroup
+	 && !&Apache::response::showallfoils()) {
       push @{ $Apache::response::conceptgroup{"$name.area"} },"rectangle:$area";
     } else {
       push @{ $Apache::response::foilgroup{"$name.area"} },"rectangle:$area";
     }
+  } elsif ($target eq 'edit') {
+      $result=&Apache::edit::end_table();
   }
-  return '';
+  return $result;
 }
 1;
 __END__