--- loncom/xml/londefdef.pm	2002/10/08 20:35:19	1.94
+++ loncom/xml/londefdef.pm	2003/02/18 22:19:30	1.117
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Tags Default Definition Module 
 #
-# $Id: londefdef.pm,v 1.94 2002/10/08 20:35:19 sakharuk Exp $
+# $Id: londefdef.pm,v 1.117 2003/02/18 22:19:30 www Exp $
 # 
 #
 # Copyright Michigan State University Board of Trustees
@@ -49,10 +49,12 @@ use strict;
 use Apache::lonxml;
 use Apache::File();
 use Image::Magick;
+use Apache::lonmenu;
+#use Apache::lonmeta;
 
 BEGIN {
 
-    &Apache::lonxml::register('Apache::londefdef',('a','abbr','acronym','address','allow','applet','area','b','base','basefont','bgo','bgsound','big','blink','blockquote','blankspace','body','br','button','caption','center','cite','code','col','colgroup','dd','del','dfn','dir','div','dl','dt','em','embed','externallink','fieldset','font','form','frame','frameset','h1','h2','h3','h4','h5','h6','head','hr','html','i','iframe','img','input','ins','insert','isindex','kbd','keygen','label','layer','legend','li','link','m','map','marquee','menu','meta','multicol','nobr','noembed','noframes','nolayer','noscript','object','ol','optgroup','option','output','p','param','pre','q','s','samp','select','server','small','spacer','span','strike','strong','sub','sup','table','tbody','td','textarea','tfoot','th','thead','title','tr','tt','u','ul','var','wbr'));
+    &Apache::lonxml::register('Apache::londefdef',('a','abbr','acronym','address','allow','applet','area','b','base','basefont','bgo','bgsound','big','blink','blockquote','blankspace','body','br','button','caption','center','cite','code','col','colgroup','dd','del','dfn','dir','div','dl','dt','em','embed','externallink','fieldset','font','form','frame','frameset','h1','h2','h3','h4','h5','h6','head','hr','html','i','iframe','img','input','ins','insert','isindex','kbd','keygen','label','layer','legend','li','link','m','map','marquee','menu','meta','multicol','nobr','noembed','noframes','nolayer','noscript','object','ol','optgroup','option','output','p','param','pre','q','s','samp','select','server','small','spacer','span','strike','strong','sub','sup','table','tbody','td','textarea','tfoot','th','thead','title','tr','tt','tthoption','u','ul','var','wbr'));
 
 }
 
@@ -108,29 +110,61 @@ sub end_m {
   }
   return $currentstring;
 }
+
+sub start_tthoption {
+  my ($target,$token,$tagstack,$parstack,$parser,$safeeval) = @_;
+  my $result;
+  if ($target eq 'web') {
+      my $inside = &Apache::lonxml::get_all_text("/tthoption",$parser);
+      $inside=~s/^\s*//;
+      if ($ENV{'browser.mathml'}) {
+	  &tth::ttmoptions($inside);
+      } else {
+	  &tth::tthoptions($inside);
+      }
+  }
+}
+
+sub end_tthoption {
+  my ($target,$token) = @_;
+  my $result;
+  return $result;
+}
+
 #-- <html> tag    
-      sub start_html {
-	    my ($target,$token) = @_;
-            my $currentstring = '';
-	    if ($ENV{'browser.mathml'}) {
-	      &tth::ttminit();
-	    } else {
-	      &tth::tthinit();
-	    }
-            if ($target eq 'web') {
-              $currentstring = &Apache::lonxml::xmlbegin().
-                               &Apache::lonxml::fontsettings();     
-	    } elsif ($target eq 'tex') {
-	      @Apache::londefdef::table = ();
-	      $currentstring .= '\documentclass[letterpaper]{article}
-                                 \newcommand{\keephidden}[1]{}
-                                 \renewcommand{\deg}{$^{\circ}$}
-                                 \usepackage[dvips]{graphicx}
-                                 \usepackage{epsfig}\usepackage{calc}
-                                 \newenvironment{choicelist}{\begin{enumerate}}{\end{enumerate}}';
-	    }
-	   return $currentstring;
+sub start_html {
+    my ($target,$token) = @_;
+    my $currentstring = '';
+    if ($ENV{'browser.mathml'}) {
+	&tth::ttminit();
+	if ($ENV{'browser.unicode'}) {
+	    &tth::ttmoptions('-L -u1');
+	} else {
+	    &tth::ttmoptions('-L -u0');
+	}
+    } else {
+	&tth::tthinit();
+	if ($ENV{'browser.unicode'}) {
+	    &tth::tthoptions('-L -u1');
+	} else {
+	    &tth::tthoptions('-L -u0');
 	}
+    }
+    if ($target eq 'web') {
+	$currentstring = &Apache::lonxml::xmlbegin().
+	    &Apache::lonxml::fontsettings();     
+    } elsif ($target eq 'tex') {
+	@Apache::londefdef::table = ();
+	$currentstring .= '\documentclass[letterpaper]{article}
+                           \newcommand{\keephidden}[1]{}
+                           \renewcommand{\deg}{$^{\circ}$}
+                           \usepackage{textcomp}
+                           \usepackage[dvips]{graphicx}
+                           \usepackage{epsfig}\usepackage{calc}
+\newenvironment{choicelist}{\begin{list}{}{\setlength{\rightmargin}{0in}\setlength{\leftmargin}{0.13in}\setlength{\topsep}{0.05in}\setlength{\itemsep}{0.022in}\setlength{\parsep}{0in}\setlength{\belowdisplayskip}{0.04in}\setlength{\abovedisplayskip}{0.05in}\setlength{\abovedisplayshortskip}{-0.04in}\setlength{\belowdisplayshortskip}{0.04in}}}{\end{list}}';
+    }
+    return $currentstring;
+}
         sub end_html {
 	    my ($target,$token) = @_;
             my $currentstring = '';
@@ -152,7 +186,7 @@ sub end_m {
 	    my ($target,$token) = @_;
             my $currentstring = '';
             if ($target eq 'web') {
-              $currentstring = &Apache::lonxml::registerurl(undef,$target).
+              $currentstring = &Apache::lonmenu::registerurl(undef,$target).
                                $token->[2];    
             } 
 	   return $currentstring;
@@ -296,7 +330,7 @@ sub end_m {
 	      my $args='';
 	      if ( $#$parstack > -1 ) { $args=$$parstack[$#$parstack]; }
 	      if ($args eq '') {
-		&Apache::lonxml::get_all_text("/meta",$$parser[$#$parser]);
+		&Apache::lonxml::get_all_text("/meta",$parser);
 	      } else {
 		$currentstring = $token->[4];
 	      }
@@ -334,7 +368,7 @@ sub end_m {
             if ($target eq 'web') {
 	      if (!$Apache::lonxml::registered) {
 		$currentstring.='<head>'.
-		    &Apache::lonxml::registerurl(undef,$target).'</head>';
+		    &Apache::lonmenu::registerurl(undef,$target).'</head>';
 	      }
 	      my $onLoad='';
 	      foreach my $key (keys(%{$token->[2]})) {
@@ -343,7 +377,7 @@ sub end_m {
 		  delete($token->[2]->{$key});
 		}
 	      }
-	      $token->[2]->{'onLoad'}=&Apache::lonxml::loadevents().
+	      $token->[2]->{'onload'}=&Apache::lonmenu::loadevents().
 		                       ';'.$onLoad;
 	      my $onUnload='';
 	      foreach my $key (keys(%{$token->[2]})) {
@@ -352,7 +386,7 @@ sub end_m {
 		  delete($token->[2]->{$key});
 		}
 	      }
-	      $token->[2]->{'onUnload'}=&Apache::lonxml::unloadevents().
+	      $token->[2]->{'onunload'}=&Apache::lonmenu::unloadevents().
 		                         ';'.$onUnload;
 
 	      $currentstring .= '<'.$token->[1];
@@ -366,7 +400,9 @@ sub end_m {
 		<input type="submit" name="editmode" value="Edit" />
 		</form>
 EDITBUTTON
-	      }
+              } else {
+		$currentstring.=&Apache::lonmenu::menubuttons(undef,$target,1);
+              }
        	    } elsif ($target eq 'tex') {
               $currentstring = '\begin{document}';  
 	    } 
@@ -845,7 +881,7 @@ EDITBUTTON
 	       $currentstring .= $token->[4];
 	    } elsif ($target eq 'tex') {
 		if ($$tagstack[-2] ne 'sub' && $$tagstack[-2] ne 'sup') {
-		    $currentstring .= '\vskip 0 mm';
+		    $currentstring .= '\vskip 0.2 mm';
 		}
 	    } elsif ($target eq 'latexsource') {
 		$currentstring .= '\\';
@@ -955,7 +991,7 @@ EDITBUTTON
             if ($target eq 'web') {
 	       $currentstring .= $token->[4];
 	    } elsif ($target eq 'tex') {
-		$currentstring .= '\underline{';
+	       &Apache::lonxml::startredirection();
 	    } 
            return $currentstring;
 	}
@@ -965,7 +1001,10 @@ EDITBUTTON
             if ($target eq 'web') {
 	       $currentstring .= $token->[2];
 	    } elsif ($target eq 'tex') {
-	        $currentstring .= '}';
+	       $currentstring=&Apache::lonxml::endredirection();
+	       $currentstring=~s/(\S)(\s+)(\S)/$1}$2\\underline{$3/g;
+	       $currentstring=~s/^\s*(\S)/\\underline{$1/;
+	       $currentstring=~s/(\S)\s*$/$1}/;	 	       
             }
            return $currentstring;
 	}
@@ -976,7 +1015,7 @@ EDITBUTTON
             if ($target eq 'web') {
 	       $currentstring .= $token->[4];
 	    } elsif ($target eq 'tex') {
-		$currentstring .= '\underline{';
+	       &Apache::lonxml::startredirection();
 	    } 
            return $currentstring;
 	}
@@ -986,7 +1025,10 @@ EDITBUTTON
             if ($target eq 'web') {
 	       $currentstring .= $token->[2];
 	    } elsif ($target eq 'tex') {
-	        $currentstring .= '}';
+	       $currentstring=&Apache::lonxml::endredirection();
+	       $currentstring=~s/(\S)(\s+)(\S)/$1}$2\\underline{$3/g;
+	       $currentstring=~s/^\s*(\S)/\\underline{$1/;
+	       $currentstring=~s/(\S)\s*$/$1}/;	
             }
            return $currentstring;
 	}
@@ -1132,7 +1174,7 @@ EDITBUTTON
             if ($target eq 'web') {
 	       $currentstring .= $token->[4];
 	    } elsif ($target eq 'tex') {
-		$currentstring .= '\underline{';
+	       &Apache::lonxml::startredirection();
 	    } 
            return $currentstring;
 	}
@@ -1142,7 +1184,10 @@ EDITBUTTON
             if ($target eq 'web') {
 	       $currentstring .= $token->[2];
 	    } elsif ($target eq 'tex') {
-	        $currentstring .= '}';
+	       $currentstring=&Apache::lonxml::endredirection();
+	       $currentstring=~s/(\S)(\s+)(\S)/$1}$2\\underline{$3/g;
+	       $currentstring=~s/^\s*(\S)/\\underline{$1/;
+	       $currentstring=~s/(\S)\s*$/$1}/;		
             }
            return $currentstring;
 	}
@@ -1300,7 +1345,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 '; 
@@ -1314,7 +1359,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; }
@@ -1412,8 +1457,12 @@ sub start_table {
 		if ($signature) {
 		    my $NumberEmptyLength = $how_many_columns - $filled_columns;		
 		    my $SpacePerColumn = '(\textwidth '.$available_space.')/'.$NumberEmptyLength;
-		    $output =~ s/\$SpacePerColumn/$SpacePerColumn/g;
+		    my $shorthand = ($filled_columns+1)*4;
+		    $output =~ s/\$SpacePerColumn/$SpacePerColumn - $shorthand mm/g;
 		}	
+		if ($how_many_columns==1) {  #start of block with width correction for one column table
+		  $output=~s/^\s*\\parbox{([^}]*)}/\\parbox{\\textwidth - 1 cm}/;
+		}                            #end of block with width correction for one column table
 		$Apache::londefdef::table[-1]{'output'} .= $header_of_table.$output.$Apache::londefdef::table[-1]{'hinc'}.'\end{tabular}\vskip 0 mm ';
 		if ($#Apache::londefdef::table > 0) {
 		    $inmemory = $Apache::londefdef::table[-1]{'output'};
@@ -1438,7 +1487,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 {
@@ -1467,15 +1516,14 @@ sub start_table {
             my $currentstring = '';
             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);
+	    } elsif ($target eq 'tex') {
+		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);;
 		}
 		    push @{ $Apache::londefdef::table[-1]{'columns'} }, $what_to_push;
 		$Apache::londefdef::table[-1]{'counter_columns'}++;
 		&Apache::lonxml::startredirection();
-;
 	    } 
 	   return $currentstring;
 	}        
@@ -1494,7 +1542,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';
@@ -1511,7 +1559,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);;
 		}
@@ -1542,20 +1590,31 @@ sub start_table {
 #-- <img> tag
         sub start_img {
 	    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,
+						 undef,1);
+            $Apache::lonxml::extlinks[$#Apache::lonxml::extlinks+1]=$src;
             my $currentstring = '';
 	    my $width_param = '';
 	    my $height_param = '';
 	    my $scaling = .3;
 
 	    if ($target eq 'web') {
-              $currentstring = $token->[4];     
+              if ($ENV{'browser.imagesuppress'} ne 'on') {
+                 $currentstring = $token->[4];
+	      } else {
+		  my $alttag= &Apache::lonxml::get_param
+                               ('alt',$parstack,$safeeval,undef,1);
+                  unless ($alttag) {
+		    $alttag=&Apache::lonmeta::alttag
+                                   ($Apache::lonxml::pwd[-1],$src);
+                  }
+                  $currentstring='[IMAGE: '.$alttag.']';
+	      }
 	    } elsif ($target eq 'tex') {
-		my $src = &Apache::lonxml::get_param('src',$parstack,$safeeval);
-		&image_replication($src);
-	      $src=&Apache::lonnet::filelocation($Apache::lonxml::pwd[-1],$src);
-              #if original gif/jpg file exist do following:
+ 	     $src=&Apache::lonnet::filelocation($Apache::lonxml::pwd[-1],$src);
+	     &image_replication($src);
+
+              #if original gif/jpg/png file exist do following:
               if (-e $src) {          
                   #defines the default size of image
 		  my $image = Image::Magick->new;
@@ -1564,9 +1623,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;
@@ -1585,13 +1645,13 @@ 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 
 		      if ($path) {
-			  $currentstring .= '\noindent\graphicspath{{'.$path.'}}\fbox{\includegraphics[width='.$width_param.' mm]{'.$file.'}} ';
+			  $currentstring .= '\vskip 1 mm \noindent\graphicspath{{'.$path.'}}\includegraphics[width='.$width_param.' mm]{'.$file.'} ';
 		      }
 		  } else {
                       #there is no eps counterpart for image - check for ps one
@@ -1600,7 +1660,7 @@ sub start_table {
 			  #ps counterpart for image exist 
 			  $file =~ s/\.eps$/\.ps/;
 			  if ($path) {
-			      $currentstring .= '\noindent\graphicspath{{'.$path.'}}\fbox{\includegraphics[width='.$width_param.' mm]{'.$file.'}} ';
+			      $currentstring .= '\vskip 1 mm \noindent\graphicspath{{'.$path.'}}\includegraphics[width='.$width_param.' mm]{'.$file.'} ';
 			  }
 		      } else {
 			  #there aren't eps or ps - so create eps 
@@ -1608,12 +1668,12 @@ sub start_table {
 			  my $filename = "/home/httpd/prtspool/$ENV{'user.name'}_$ENV{'user.domain'}_printout.dat";
 			  $temp_file = Apache::File->new('>>'.$filename); 
 			  print $temp_file "$src\n";
-			  $currentstring .= '\graphicspath{{/home/httpd/prtspool/}}\fbox{\includegraphics[width='.$width_param.' mm]{'.$file.'}} ';
+			  $currentstring .= '\vskip 1 mm \graphicspath{{/home/httpd/prtspool/}}\includegraphics[width='.$width_param.' mm]{'.$file.'} ';
 		      }
 		  }
 	      } 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 {
@@ -1643,7 +1703,16 @@ sub start_table {
                                         $token->[2]->{'archive'};
             my $currentstring = '';
             if ($target eq 'web') {
-              $currentstring = $token->[4];
+              if ($ENV{'browser.appletsuppress'} ne 'on') {
+                 $currentstring = $token->[4];
+	      } else {
+		  my $alttag=$token->[2]->{'alt'};
+                  unless ($alttag) {
+                     $alttag=&Apache::lonmeta::alttag($ENV{'REQUEST_URI'},
+                                                      $token->[2]->{'code'});
+                  }
+                  $currentstring='[APPLET: '.$alttag.']';
+	      }
 	    } elsif ($target eq 'tex') {
               $currentstring = " \\begin{figure} ";
 	    } 
@@ -1668,7 +1737,16 @@ sub start_embed {
 	$token->[2]->{'src'};
     my $currentstring = '';
     if ($target eq 'web') {
-	$currentstring = $token->[4];
+       if ($ENV{'browser.embedsuppress'} ne 'on') {
+          $currentstring = $token->[4];
+       } else {
+	  my $alttag=$token->[2]->{'alt'};
+          unless ($alttag) {
+             $alttag=&Apache::lonmeta::alttag($ENV{'REQUEST_URI'},
+                                              $token->[2]->{'src'});
+          }
+          $currentstring='[EMBED: '.$alttag.']';
+      }
     } elsif ($target eq 'tex') {
 	$currentstring = " \\begin{figure} ";  
     } 
@@ -1714,17 +1792,30 @@ sub start_embed {
 	   return $currentstring;
 	}
 #-- <allow> tag
-        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);
-	    &image_replication($src);
-	    return '';
-	}
-        sub end_allow {
-	   return '';
-	}
+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,undef,1);
+    &image_replication($src);
+    my $result;
+    if ($target eq 'edit') {
+	$result .=&Apache::edit::tag_start($target,$token);
+	$result .=&Apache::edit::text_arg('File Spec:','src',$token,70);
+	$result .=&Apache::edit::end_row();#.&Apache::edit::start_spanning_row();
+    } elsif ($target eq 'modified') {
+	my $constructtag=&Apache::edit::get_new_args($token,$parstack,
+						     $safeeval,'src');
+	if ($constructtag) { $result = &Apache::edit::rebuild_tag($token); }
+    }
+    return $result;
+}
+
+sub end_allow {
+    my ($target,$token,$tagstack,$parstack,$parser,$safeeval) = @_;
+    if ( $target eq 'edit') { return (&Apache::edit::end_table()); }
+    return '';
+}
 #-- Frames
 	sub start_frameset {
 	  my ($target,$token) = @_;
@@ -1732,7 +1823,7 @@ sub start_embed {
 	  if ($target eq 'web') { 
 	    if (!$Apache::lonxml::registered) {
 	      $currentstring.='<head>'.
-		  &Apache::lonxml::registerurl(undef,$target).'</head>';
+		  &Apache::lonmenu::registerurl(undef,$target).'</head>';
 	    }
 	    $currentstring .= $token->[4];
 	  }
@@ -1772,7 +1863,7 @@ sub start_embed {
 	    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;
@@ -1790,7 +1881,7 @@ sub start_embed {
 	    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;
@@ -1808,7 +1899,7 @@ sub start_embed {
 	    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;
@@ -2526,9 +2617,9 @@ 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/;
+	    $newsrc =~ s/\.eps$/\.ps/;
 	    &Apache::lonnet::repcopy('/home/httpd/html'.$newsrc);
 	}
     }