--- loncom/xml/londefdef.pm	2002/11/09 22:05:33	1.100
+++ loncom/xml/londefdef.pm	2002/11/18 20:47:51	1.105
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Tags Default Definition Module 
 #
-# $Id: londefdef.pm,v 1.100 2002/11/09 22:05:33 albertel Exp $
+# $Id: londefdef.pm,v 1.105 2002/11/18 20:47:51 sakharuk Exp $
 # 
 #
 # Copyright Michigan State University Board of Trustees
@@ -1310,7 +1310,7 @@ sub start_table {
 	push @Apache::londefdef::table, $aa; 
 	$Apache::londefdef::table[-1]{'row_number'} = -1;
 	$Apache::londefdef::table[-1]{'output'} = ' \noindent \begin{tabular} ';
-	my $border = &Apache::lonxml::get_param('border',$parstack,$safeeval);
+	my $border = &Apache::lonxml::get_param('border',$parstack,$safeeval,undef,1); 
 	unless (defined $border) { $border = 0; }
 	if ($border) { 
 	    $Apache::londefdef::table[-1]{'hinc'} = '\hline '; 
@@ -1324,7 +1324,7 @@ sub start_table {
 	my $width;
 	foreach my $key (keys(%{$token->[2]})) {
 	    if ($key =~ /^width$/i) {
-		$width = &Apache::lonxml::get_param($key,$parstack,$safeeval);
+		$width = &Apache::lonxml::get_param($key,$parstack,$safeeval,undef,1);
 	    }
 	}
 	if (defined($width)) { $Apache::londefdef::table[-1]{'width'}=$width; }
@@ -1422,7 +1422,7 @@ sub start_table {
 		if ($signature) {
 		    my $NumberEmptyLength = $how_many_columns - $filled_columns;		
 		    my $SpacePerColumn = '(\textwidth '.$available_space.')/'.$NumberEmptyLength;
-		    my $shorthand = ($NumberEmptyLength+1)*4;
+		    my $shorthand = ($filled_columns+1)*4;
 		    $output =~ s/\$SpacePerColumn/$SpacePerColumn - $shorthand mm/g;
 		}	
 		$Apache::londefdef::table[-1]{'output'} .= $header_of_table.$output.$Apache::londefdef::table[-1]{'hinc'}.'\end{tabular}\vskip 0 mm ';
@@ -1449,7 +1449,7 @@ sub start_table {
               $currentstring = $token->[4];     
 	    } elsif ($target eq 'tex') {
 		$Apache::londefdef::table[-1]{'row_number'}++;
-		my $alignchar = substr(&Apache::lonxml::get_param('align',$parstack,$safeeval),0,1);
+		my $alignchar = substr(&Apache::lonxml::get_param('align',$parstack,$safeeval,undef,1),0,1);
 		if ($alignchar ne '') {
 		    push @ {$Apache::londefdef::table[-1]{'rows'} }, $alignchar;
 		} else {
@@ -1479,7 +1479,7 @@ sub start_table {
             if ($target eq 'web') {
               $currentstring = $token->[4];     
     } elsif ($target eq 'tex') {
-		my $what_to_push = substr(&Apache::lonxml::get_param('align',$parstack,$safeeval),0,1);
+		my $what_to_push = substr(&Apache::lonxml::get_param('align',$parstack,$safeeval,undef,1),0,1);
 		if ($what_to_push eq '') {
 		    $what_to_push = substr($Apache::londefdef::table[-1]{'rows'}[0],0,1);;
 		}
@@ -1505,7 +1505,7 @@ sub start_table {
 		} else {
 		    if (length($data)<5) {
 			$Apache::londefdef::table[-1]{'length'} .= '0 mm,';
-			$tempolen = '5 mm';
+			$tempolen = '6 mm';
 		    } else {
 			$Apache::londefdef::table[-1]{'length'} .= '0 mm,';
 			$tempolen = 'TOBECHANGEDONNUMBER';
@@ -1522,7 +1522,7 @@ sub start_table {
             if ($target eq 'web') {
               $currentstring = $token->[4];     
 	    } elsif ($target eq 'tex') {
-		my $what_to_push = substr(&Apache::lonxml::get_param('align',$parstack,$safeeval),0,1);
+		my $what_to_push = substr(&Apache::lonxml::get_param('align',$parstack,$safeeval,undef,1),0,1);
 		if ($what_to_push eq '') {
 		    $what_to_push = substr($Apache::londefdef::table[-1]{'rows'}[0],0,1);;
 		}
@@ -1566,7 +1566,7 @@ sub start_table {
 	    } elsif ($target eq 'tex') {
 	      &image_replication($src);
 	      $src=&Apache::lonnet::filelocation($Apache::lonxml::pwd[-1],$src);
-              #if original gif/jpg file exist do following:
+              #if original gif/jpg/png file exist do following:
               if (-e $src) {          
                   #defines the default size of image
 		  my $image = Image::Magick->new;
@@ -1575,9 +1575,10 @@ sub start_table {
 		  $height_param = $image->Get('height') * $scaling;;
 		  undef $image;
 		  #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);
+		  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,
+                                                         undef,1);
 		  if ($TeXwidth ne '') {  
 		      if ($TeXwidth=~/(\d+)\s*\%/) {
 			  $width_param = $1*$ENV{'form.textwidth'}/100;
@@ -1596,8 +1597,8 @@ sub start_table {
 		      $path = $1.'/'; 
 		  } 
 		  my $newsrc = $src;
-		  $newsrc =~ s/(\.gif|\.jpg)$/\.eps/i;
-		  $file=~s/(\.gif|\.jpg)$/\.eps/i;
+		  $newsrc =~ s/\.(gif|jpg|png)$/.eps/i;
+		  $file=~s/\.(gif|jpg|png)$/.eps/i;
 		  #where can we find the picture?
 		  if (-e $newsrc) {
                       #eps counterpart for image exist 
@@ -1624,7 +1625,7 @@ sub start_table {
 		  }
 	      } else {
 		  #original image file doesn't exist so check the alt attribute
-		  my $alt = &Apache::lonxml::get_param('alt',$parstack,$safeeval);
+		  my $alt = &Apache::lonxml::get_param('alt',$parstack,$safeeval,undef,1);
                   if ($alt) {
 		      $currentstring .= ' '.$alt.' ';
 		  } else {
@@ -1729,7 +1730,7 @@ sub start_allow {
     my ($target,$token,$tagstack,$parstack,$parser,$safeeval) = @_;
     $Apache::lonxml::extlinks[$#Apache::lonxml::extlinks+1]=
 	$token->[2]->{'src'};
-    my $src = &Apache::lonxml::get_param('src',$parstack,$safeeval);
+    my $src = &Apache::lonxml::get_param('src',$parstack,$safeeval,undef,1);
     &image_replication($src);
     my $result;
     if ($target eq 'edit') {
@@ -1796,7 +1797,7 @@ sub end_allow {
 	    my ($target,$token,$tagstack,$parstack,$parser,$safeeval) = @_;
             my $currentstring = '';
             if ($target eq 'web') {
-		my $display = &Apache::lonxml::get_param('display',$parstack,$safeeval);
+		my $display = &Apache::lonxml::get_param('display',$parstack,$safeeval,undef,1);
 	       $currentstring .= '<b>'.$display.'</b>';;
 	    }
            return $currentstring;
@@ -1814,7 +1815,7 @@ sub end_allow {
 	    my ($target,$token,$tagstack,$parstack,$parser,$safeeval) = @_;
             my $currentstring = '';
             if ($target eq 'web') {
-		my $display = &Apache::lonxml::get_param('display',$parstack,$safeeval);
+		my $display = &Apache::lonxml::get_param('display',$parstack,$safeeval,undef,1);
 	       $currentstring .= '<b>'.$display.'</b>';;
 	    }
            return $currentstring;
@@ -1832,7 +1833,7 @@ sub end_allow {
 	    my ($target,$token,$tagstack,$parstack,$parser,$safeeval) = @_;
             my $currentstring = '';
             if ($target eq 'tex') {
-		my $howmuch = &Apache::lonxml::get_param('heigth',$parstack,$safeeval);
+		my $howmuch = &Apache::lonxml::get_param('heigth',$parstack,$safeeval,undef,1);
 		$currentstring .= '\vskip '.$howmuch.' ';
 	    }
            return $currentstring;
@@ -2550,7 +2551,7 @@ sub image_replication {
 	&Apache::lonnet::repcopy('/home/httpd/html'.$src);
 	#replicates eps or ps 
 	my $newsrc = $src;
-	$newsrc =~ s/(.gif|.jpg)$/.eps/;
+	$newsrc =~ s/\.(gif|jpg|png)$/.eps/;
 	if (not-e $newsrc && &Apache::lonnet::repcopy('/home/httpd/html'.$newsrc) ne 'OK') {
 	    $newsrc =~ s/\.ps$/\.eps/;
 	    &Apache::lonnet::repcopy('/home/httpd/html'.$newsrc);