loncom/publisher/lonpublisher.pm - diff

Return to lonpublisher.pm CVS log

Up to [LON-CAPA] / loncom / publisher

Diff for /loncom/publisher/lonpublisher.pm between versions 1.165 and 1.292

-version 1.165, 2004/03/31 05:24:00
+version 1.292, 2014/08/03 13:52:59
  Line 66  invocation by F<loncapa_apache.conf>:
  =head1 OVERVIEW
- Authors can only write-access the C</~authorname/> space. They can
+ Authors can only write-access the C</priv/domain/authorname/> space.
- copy resources into the resource area through the publication step,
+ They can copy resources into the resource area through the
- and move them back through a recover step. Authors do not have direct
+ publication step, and move them back through a recover step.
- write-access to their resource space.
+ Authors do not have direct write-access to their resource space.
  During the publication step, several events will be
  triggered. Metadata is gathered, where a wizard manages default
  Line 102  to publication space.
  Many of the undocumented subroutines implement various magical
  parsing shortcuts.
- =over 4
  =cut
  ######################################################################
- Line 118  use Apache::File;
+ Line 116  use Apache::File;
  use File::Copy;
  use Apache::Constants qw(:common :http :methods);
  use HTML::LCParser;
+ use HTML::Entities;
+ use Encode::Encoder;
  use Apache::lonxml;
- use Apache::loncacc;
  use DBI;
- use Apache::lonnet();
+ use Apache::lonnet;
  use Apache::loncommon();
+ use Apache::lonhtmlcommon;
  use Apache::lonmysql;
  use Apache::lonlocal;
  use Apache::loncfile;
- use Apache::lonmeta;
+ use LONCAPA::lonmetadata;
  use Apache::lonmsg;
  use vars qw(%metadatafields %metadatakeys);
+ use LONCAPA qw(:DEFAULT :match);
  my %addid;
  my %nokey;
- Line 138  my $docroot;
+ Line 140  my $docroot;
  my $cuname;
  my $cudom;
+ my $registered_cleanup;
+ my $modified_urls;
+ my $lock;
  =pod
+ =over 4
  =item B<metaeval>
  Evaluates a string that contains metadata.  This subroutine
- Line 178  sub metaeval {
+ Line 187  sub metaeval {
  	if ($token->[0] eq 'S') {
  	    my $entry=$token->[1];
  	    my $unikey=$entry;
+ 	    next if ($entry =~ m/^(?:parameter|stores)_/);
  	    if (defined($token->[2]->{'package'})) {
- 		$unikey.='_package_'.$token->[2]->{'package'};
+ 		$unikey.="\0package\0".$token->[2]->{'package'};
  	    }
  	    if (defined($token->[2]->{'part'})) {
- 		$unikey.='_'.$token->[2]->{'part'};
+ 		$unikey.="\0".$token->[2]->{'part'};
  	    }
  	    if (defined($token->[2]->{'id'})) {
- 		$unikey.='_'.$token->[2]->{'id'};
+ 		$unikey.="\0".$token->[2]->{'id'};
  	    }
  	    if (defined($token->[2]->{'name'})) {
- 		$unikey.='_'.$token->[2]->{'name'};
+ 		$unikey.="\0".$token->[2]->{'name'};
  	    }
  	    foreach (@{$token->[3]}) {
  		$metadatafields{$unikey.'.'.$_}=$token->[2]->{$_};
- Line 199  sub metaeval {
+ Line 209  sub metaeval {
  		}
  	    }
  	    my $newentry=$parser->get_text('/'.$entry);
- 	    if ($entry eq 'customdistributionfile') {
+ 	    if (($entry eq 'customdistributionfile') ||
+ 		($entry eq 'sourcerights')) {
  		$newentry=~s/^\s*//;
  		if ($newentry !~m|^/res|) { $newentry=$prefix.$newentry; }
  	    }
- Line 254  sub metaread {
+ Line 265  sub metaread {
      my ($logfile,$fn,$prefix)=@_;
      unless (-e $fn) {
  	print($logfile 'No file '.$fn."\n");
-         return '<br /><b>'.&mt('No file').':</b> <tt>'.
+         return '<p class="LC_warning">'
- 	    &Apache::loncfile::display($fn).'</tt>';
+               .&mt('No file: [_1]',&Apache::loncfile::display($fn))
+               .'</p>';
      }
      print($logfile 'Processing '.$fn."\n");
      my $metastring;
- Line 264  sub metaread {
+ Line 276  sub metaread {
  	$metastring=join('',<$metafh>);
      }
      &metaeval($metastring,$prefix);
-     return '<br /><b>'.&mt('Processed file').':</b> <tt>'.
+     return '<p class="LC_info">'
- 	&Apache::loncfile::display($fn).'</tt>';
+           .&mt('Processed file: [_1]',&Apache::loncfile::display($fn))
+           .'</p>';
  }
  #########################################
- Line 274  sub metaread {
+ Line 287  sub metaread {
  sub coursedependencies {
      my $url=&Apache::lonnet::declutter(shift);
      $url=~s/\.meta$//;
-     my ($adomain,$aauthor)=($url=~/^(\w+)\/(\w+)\//);
+     my ($adomain,$aauthor)=($url=~ m{^($match_domain)/($match_username)/});
-     my $regexp=$url;
+     my $regexp=quotemeta($url);
-     $regexp=~s/(\W)/\\$1/g;
      $regexp='___'.$regexp.'___course';
      my %evaldata=&Apache::lonnet::dump('nohist_resevaldata',$adomain,
  				       $aauthor,$regexp);
- Line 317  string which presents the form field (fo
+ Line 329  string which presents the form field (fo
  #########################################
  #########################################
  sub textfield {
-     my ($title,$name,$value)=@_;
+     my ($title,$name,$value,$noline)=@_;
      $value=~s/^\s+//gs;
      $value=~s/\s+$//gs;
      $value=~s/\s+/ /gs;
      $title=&mt($title);
-     return "\n<p><font color=\"#800000\" face=\"helvetica\"><b>$title:".
+     $env{'form.'.$name}=$value;
-            "</b></font></p><br />".
+     return "\n".&Apache::lonhtmlcommon::row_title($title)
-            '<input type="text" name="'.$name.'" size=80 value="'.$value.'" />';
+            .'<input type="text" name="'.$name.'" size="80" value="'.$value.'" />'
+            .&Apache::lonhtmlcommon::row_closure($noline);
+ }
+ sub text_with_browse_field {
+     my ($title,$name,$value,$restriction,$noline)=@_;
+     $value=~s/^\s+//gs;
+     $value=~s/\s+$//gs;
+     $value=~s/\s+/ /gs;
+     $title=&mt($title);
+     $env{'form.'.$name}=$value;
+     return "\n".&Apache::lonhtmlcommon::row_title($title)
+           .'<input type="text" name="'.$name.'" size="80" value="'.$value.'" />'
+           .'<br />'
+ 	  .'<a href="javascript:openbrowser(\'pubform\',\''.$name.'\',\''.$restriction.'\');">'
+           .&mt('Select')
+           .'</a>&nbsp;'
+ 	  .'<a href="javascript:opensearcher(\'pubform\',\''.$name.'\');">'
+           .&mt('Search')
+           .'</a>'
+           .&Apache::lonhtmlcommon::row_closure($noline);
  }
  sub hiddenfield {
      my ($name,$value)=@_;
+     $env{'form.'.$name}=$value;
      return "\n".'<input type="hidden" name="'.$name.'" value="'.$value.'" />';
  }
+ sub checkbox {
+     my ($name,$text)=@_;
+     return "\n<br /><label><input type='checkbox' name='$name' /> ".
+ 	&mt($text)."</label>";
+ }
  sub selectbox {
      my ($title,$name,$value,$functionref,@idlist)=@_;
      $title=&mt($title);
      $value=(split(/\s*,\s*/,$value))[-1];
-     my $selout="\n<p><font color=\"#800000\" face=\"helvetica\"><b>$title:".
+     if (defined($value)) {
- 	'</b></font></p><br /><select name="'.$name.'">';
+ 	$env{'form.'.$name}=$value;
+     } else {
+ 	$env{'form.'.$name}=$idlist[0];
+     }
+     my $selout="\n".&Apache::lonhtmlcommon::row_title($title)
+               .'<select name="'.$name.'">';
      foreach (@idlist) {
-         $selout.='<option value=\''.$_.'\'';
+         $selout.='<option value="'.$_.'"';
          if ($_ eq $value) {
- 	    $selout.=' selected>'.&{$functionref}($_).'</option>';
+ 	    $selout.=' selected="selected"';
- 	}
+         }
-         else {$selout.='>'.&{$functionref}($_).'</option>';}
+         $selout.='>'.&{$functionref}($_).'</option>';
      }
-     return $selout.'</select>';
+     $selout.='</select>'.&Apache::lonhtmlcommon::row_closure();
+     return $selout;
  }
+ sub select_level_form {
+     my ($value,$name)=@_;
+     $env{'form.'.$name}=$value;
+     if (!defined($value)) { $env{'form.'.$name}=0; }
+     return  &Apache::loncommon::select_level_form($value,$name);
+ }
  #########################################
  #########################################
- Line 369  sub urlfixup {
+ Line 420  sub urlfixup {
      if ($url =~ /^mailto:/i) { return $url; }
      #internal document links need no fixing
      if ($url =~ /^\#/) { return $url; }
-     my ($host)=($url=~/(?:http\:\/\/)*([^\/]+)/);
+     my ($host)=($url=~m{(?:(?:http|https|ftp)://)*([^/]+)});
-     foreach (values %Apache::lonnet::hostname) {
+     my @lonids = &Apache::lonnet::machine_ids($host);
- 	if ($_ eq $host) {
+     if (@lonids) {
- 	    $url=~s/^http\:\/\///;
+ 	$url=~s{^(?:http|https|ftp)://}{};
-             $url=~s/^$host//;
+ 	$url=~s/^\Q$host\E//;
-         }
      }
-     if ($url=~/^http\:\/\//) { return $url; }
+     if ($url=~m{^(?:http|https|ftp)://}) { return $url; }
-     $url=~s/\~$cuname/res\/$cudom\/$cuname/;
+     $url=~s{\Q~$cuname\E}{res/$cudom/$cuname};
      return $url;
  }
- Line 418  Currently undocumented
+ Line 468  Currently undocumented
  #########################################
  #########################################
  sub set_allow {
-     my ($allow,$logfile,$target,$tag,$oldurl)=@_;
+     my ($allow,$logfile,$target,$tag,$oldurl,$type)=@_;
      my $newurl=&urlfixup($oldurl,$target);
      my $return_url=$oldurl;
      print $logfile 'GUYURL: '.$tag.':'.$oldurl.' - '.$newurl."\n";
- Line 428  sub set_allow {
+ Line 478  sub set_allow {
      }
      if (($newurl !~ /^javascript:/i) &&
  	($newurl !~ /^mailto:/i) &&
- 	($newurl !~ /^http:/i) &&
+ 	($newurl !~ /^(?:http|https|ftp):/i) &&
  	($newurl !~ /^\#/)) {
+         if (($type eq 'src') || ($type eq 'href')) {
+             if ($newurl =~ /^([^?]+)\?[^?]*$/) {
+                 $newurl = $1;
+             }
+         }
  	$$allow{&absoluteurl($newurl,$target)}=1;
      }
-     return $return_url
+     return $return_url;
  }
  #########################################
- Line 455  sub get_subscribed_hosts {
+ Line 510  sub get_subscribed_hosts {
      $target=~/(.*)\/([^\/]+)$/;
      my $srcf=$2;
      opendir(DIR,$1);
+     # cycle through listed files, subscriptions used to exist
+     # as "filename.lonid"
      while ($filename=readdir(DIR)) {
- 	if ($filename=~/\Q$srcf\E\.(\w+)$/) {
+ 	if ($filename=~/\Q$srcf\E\.($match_lonid)$/) {
  	    my $subhost=$1;
- 	    if (($subhost ne 'meta' && $subhost ne 'subscription') &&
+ 	    if (($subhost ne 'meta'
+ 		 && $subhost ne 'subscription'
+ 		 && $subhost ne 'meta.subscription'
+ 		 && $subhost ne 'tmp') &&
                  ($subhost ne $Apache::lonnet::perlvar{'lonHostID'})) {
  		push(@subscribed,$subhost);
  	    }
- Line 467  sub get_subscribed_hosts {
+ Line 527  sub get_subscribed_hosts {
      closedir(DIR);
      my $sh;
      if ( $sh=Apache::File->new("$target.subscription") ) {
- 	&Apache::lonnet::logthis("opened $target.subscription");
  	while (my $subline=<$sh>) {
- 	    &Apache::lonnet::logthis("Trying $subline");
+ 	    if ($subline =~ /^($match_lonid):/) {
- 	    if ($subline =~ /(^\w+):/) {
                  if ($1 ne $Apache::lonnet::perlvar{'lonHostID'}) {
                     push(@subscribed,$1);
  	        }
-             } else {
- 		&Apache::lonnet::logthis("No Match for $subline");
  	    }
  	}
-     } else {
- 	&Apache::lonnet::logthis("Unable to open $target.subscription");
      }
      return @subscribed;
  }
- Line 509  sub get_max_ids_indices {
+ Line 563  sub get_max_ids_indices {
      my %duplicatedids;
      my $parser=HTML::LCParser->new($content);
+     $parser->xml_mode(1);
      my $token;
      while ($token=$parser->get_token) {
  	if ($token->[0] eq 'S') {
  	    my $counter;
  	    if ($counter=$addid{$token->[1]}) {
  		if ($counter eq 'id') {
- 		    if (defined($token->[2]->{'id'})) {
+ 		    if (defined($token->[2]->{'id'}) &&
+ 			$token->[2]->{'id'} !~ /^\s*$/) {
  			$maxid=($token->[2]->{'id'}>$maxid)?$token->[2]->{'id'}:$maxid;
  			if (exists($allids{$token->[2]->{'id'}})) {
  			    $duplicateids=1;
- Line 527  sub get_max_ids_indices {
+ Line 583  sub get_max_ids_indices {
  			$needsfixup=1;
  		    }
  		} else {
- 		    if (defined($token->[2]->{'index'})) {
+ 		    if (defined($token->[2]->{'index'}) &&
+ 			$token->[2]->{'index'} !~ /^\s*$/) {
  			$maxindex=($token->[2]->{'index'}>$maxindex)?$token->[2]->{'index'}:$maxindex;
  		    } else {
  			$needsfixup=1;
- Line 569  sub get_all_text_unbalanced {
+ Line 626  sub get_all_text_unbalanced {
  	} elsif ($token->[0] eq 'E')  {
  	    $result.=$token->[2];
  	}
- 	if ($result =~ /(.*)\Q$tag\E(.*)/s) {
+ 	if ($result =~ /\Q$tag\E/s) {
+ 	    ($result,my $redo)=$result =~ /(.*)\Q$tag\E(.*)/is;
  	    #&Apache::lonnet::logthis('Got a winner with leftovers ::'.$2);
  	    #&Apache::lonnet::logthis('Result is :'.$1);
- 	    $result=$1;
+ 	    $redo=$tag.$redo;
- 	    my $redo=$tag.$2;
  	    push (@$pars,HTML::LCParser->new(\$redo));
  	    $$pars[-1]->xml_mode('1');
  	    last;
- Line 613  sub fix_ids_and_indices {
+ Line 670  sub fix_ids_and_indices {
  			   join(', ',@duplicatedids));
      if ($duplicateids) {
  	print $logfile "Duplicate ID(s) exist, ".join(', ',@duplicatedids)."\n";
- 	my $outstring='<font color="red">'.&mt('Unable to publish file, it contains duplicated ID(s), ID(s) need to be unique. The duplicated ID(s) are').': '.join(', ',@duplicatedids).'</font>';
+ 	my $outstring='<span class="LC_error">'.&mt('Unable to publish file, it contains duplicated ID(s), ID(s) need to be unique. The duplicated ID(s) are').': '.join(', ',@duplicatedids).'</span>';
  	return ($outstring,1);
      }
      if ($needsfixup) {
- Line 622  sub fix_ids_and_indices {
+ Line 679  sub fix_ids_and_indices {
                  "Max Index: $maxindex (min 10)\n";
      }
      my $outstring='';
+     my $responsecounter=1;
      my @parser;
      $parser[0]=HTML::LCParser->new(\$content);
      $parser[-1]->xml_mode(1);
- Line 636  sub fix_ids_and_indices {
+ Line 694  sub fix_ids_and_indices {
  		    $allow{$token->[2]->{'src'}}=1;
  		    next;
  		}
+ 		if ($lctag eq 'base') { next; }
+                 if (($lctag eq 'part') || ($lctag eq 'problem')) {
+                     $responsecounter=0;
+                 }
+                 if ($lctag=~/response$/) { $responsecounter++; }
+                 if ($lctag eq 'import') { $responsecounter++; }
  		my %parms=%{$token->[2]};
  		$counter=$addid{$tag};
  		if (!$counter) { $counter=$addid{$lctag}; }
  		if ($counter) {
  		    if ($counter eq 'id') {
- 			unless (defined($parms{'id'})) {
+ 			unless (defined($parms{'id'}) &&
+ 				$parms{'id'}!~/^\s*$/) {
  			    $maxid++;
  			    $parms{'id'}=$maxid;
- 			    print $logfile 'ID: '.$tag.':'.$maxid."\n";
+ 			    print $logfile 'ID(new) : '.$tag.':'.$maxid."\n";
+ 			} else {
+ 			    print $logfile 'ID(kept): '.$tag.':'.$parms{'id'}."\n";
  			}
  		    } elsif ($counter eq 'index') {
- 			unless (defined($parms{'index'})) {
+ 			unless (defined($parms{'index'}) &&
+ 				$parms{'index'}!~/^\s*$/) {
  			    $maxindex++;
  			    $parms{'index'}=$maxindex;
  			    print $logfile 'Index: '.$tag.':'.$maxindex."\n";
  			}
  		    }
  		}
- 		foreach my $type ('src','href','background','bgimg') {
+                 unless ($parms{'type'} eq 'zombie') {
- 		    foreach my $key (keys(%parms)) {
+ 		    foreach my $type ('src','href','background','bgimg') {
- 			if ($key =~ /^$type$/i) {
+ 			foreach my $key (keys(%parms)) {
- 			    $parms{$key}=&set_allow(\%allow,$logfile,
+ 			    if ($key =~ /^$type$/i) {
- 						    $target,$tag,
+                                 next if (($lctag eq 'img') && ($type eq 'src') &&
- 						    $parms{$key});
+                                          ($parms{$key} =~ m{^data\:image/gif;base64,}));
+ 				$parms{$key}=&set_allow(\%allow,$logfile,
+ 							$target,$tag,
+ 							$parms{$key},$type);
+ 			    }
  			}
  		    }
  		}
- Line 670  sub fix_ids_and_indices {
+ Line 742  sub fix_ids_and_indices {
  		    ($lctag eq 'image')) {
  		    my $next_token=$parser[-1]->get_token();
  		    if ($next_token->[0] eq 'T') {
+                         $next_token->[1] =~ s/[\n\r\f]+//g;
  			$next_token->[1]=&set_allow(\%allow,$logfile,
  						    $target,$tag,
  						    $next_token->[1]);
- Line 724  sub fix_ids_and_indices {
+ Line 797  sub fix_ids_and_indices {
  		}
  		if (!$endtag) { if ($token->[4]=~m:/>$:) { $endtag=' /'; }; }
  		$outstring.='<'.$tag.$newparmstring.$endtag.'>';
- 		if ($lctag eq 'm' || $lctag eq 'script'
+ 		if ($lctag eq 'm' || $lctag eq 'answer' || $lctag eq 'display' ||
-                     || $lctag eq 'display' || $lctag eq 'tex') {
+                     $lctag eq 'tex') {
  		    $outstring.=&get_all_text_unbalanced('/'.$lctag,\@parser);
- 		}
+                 } elsif ($lctag eq 'script') {
+                     if ($parms{'type'} eq 'loncapa/perl') {
+                         $outstring.=&get_all_text_unbalanced('/'.$lctag,\@parser);
+                     } else {
+                         my $script = &get_all_text_unbalanced('/'.$lctag,\@parser);
+                         if ($script =~ m{\.set\w+(Src|Swf)\(["']}i) {
+                             my @srcs = split(/\.set/i,$script);
+                             if (scalar(@srcs) > 1) {
+                                 foreach my $item (@srcs) {
+                                     if ($item =~ m{^(FlashPlayerSwf|MediaSrc|XMPSrc|ConfigurationSrc|PosterImageSrc)\((['"])(?:(?!\2).)+\2\)}is) {
+                                         my $srctype = $1;
+                                         my $quote = $2;
+                                         my ($url) = ($item =~ m{^\Q$srctype($quote\E([^$quote]+)\Q$quote)\E});
+                                         $url = &urlfixup($url);
+                                         unless ($url=~m{^(?:http|https|ftp)://}) {
+                                             $allow{&absoluteurl($url,$target)}=1;
+                                             if ($srctype eq 'ConfigurationSrc') {
+                                                 if ($url =~ m{^(.+/)configuration_express\.xml$}) {
+ #
+ # Camtasia 8.1: express_show/spritesheet.png needed, and included in zip archive.
+ # Not referenced directly in <main>.html or <main>_player.html files,
+ # so add this file to %allow (where <main> is name user gave to file/archive).
+ #
+                                                     my $spritesheet = $1.'express_show/spritesheet.png';
+                                                     $allow{&absoluteurl($spritesheet,$target)}=1;
+                                                 }
+                                             } elsif ($srctype eq 'PosterImageSrc') {
+                                                 if ($url =~ m{^(.+)_First_Frame\.png$}) {
+                                                     my $prefix = $1;
+ #
+ # Camtasia 8.1: <main>_Thumbnails.png needed, and included in zip archive.
+ # Not referenced directly in <main>.html or <main>_player.html files,
+ # so add this file to %allow (where <main> is name user gave to file/archive).
+ #
+                                                     my $thumbnail = $prefix.'_Thumbnails.png';
+                                                     $allow{&absoluteurl($thumbnail,$target)}=1;
+                                                 }
+                                             }
+                                         }
+                                     }
+                                 }
+                             }
+                         }
+                         if ($script =~ /\(document,\s*(['"])script\1,\s*\[([^\]]+)\]\);/s) {
+                             my $scriptslist = $2;
+                             my @srcs = split(/\s*,\s*/,$scriptslist);
+                             foreach my $src (@srcs) {
+                                 if ($src =~ /(["'])(?:(?!\1).)+\.js\1/) {
+                                     my $quote = $1;
+                                     my ($url) = ($src =~ m/\Q$quote\E([^$quote]+)\Q$quote\E/);
+                                     $url = &urlfixup($url);
+                                     unless ($url=~m{^(?:http|https|ftp)://}) {
+                                         $allow{&absoluteurl($url,$target)}=1;
+                                     }
+                                 }
+                             }
+                         }
+                         $outstring .= $script;
+                     }
+                 }
  	    } elsif ($token->[0] eq 'E') {
  		if ($token->[2]) {
  		    unless ($token->[1] eq 'allow') {
  			$outstring.='</'.$token->[1].'>';
  		    }
- 		}
+                 }
+                 if ((($token->[1] eq 'part') || ($token->[1] eq 'problem'))
+                     && (!$responsecounter)) {
+                     my $outstring='<span class="LC_error">'.&mt('Found [_1] without responses. This resource cannot be published.',$token->[1]).'</span>';
+                     return ($outstring,1);
+                 }
  	    } else {
  		$outstring.=$token->[1];
  	    }
- Line 776  sub store_metadata {
+ Line 913  sub store_metadata {
      # Determine if the table exists
      my $status = &Apache::lonmysql::check_table('metadata');
      if (! defined($status)) {
-         $error='<font color="red">WARNING: Cannot connect to '.
+         $error='<span class="LC_error">'
-             'database!</font>';
+               .&mt('WARNING: Cannot connect to database!')
+               .'</span>';
          &Apache::lonnet::logthis($error);
          return ($error,undef);
      }
      if ($status == 0) {
          # It would be nice to actually create the table....
-         $error ='<font color="red">WARNING: The metadata table does not '.
+         $error ='<span class="LC_error">'
-             'exist in the LON-CAPA database.</font>';
+                .&mt('WARNING: The metadata table does not exist in the LON-CAPA database!')
+                .'</span>';
          &Apache::lonnet::logthis($error);
          return ($error,undef);
      }
-     if (($metadata{'obsolete'}) || ($metadata{'copyright'} eq 'priv') ||
+     my $dbh = &Apache::lonmysql::get_dbh();
- 	($metadata{'copyright'} eq 'custom')) {
+     if (($metadata{'obsolete'}) || ($metadata{'copyright'} eq 'priv')) {
- # remove this entry
+         # remove this entry
- 	$status=&Apache::lonmysql::remove_from_table
+ 	my $delitem = 'url = '.$dbh->quote($metadata{'url'});
- 	    ('metadata','url',$metadata{'url'});
+ 	$status = &LONCAPA::lonmetadata::delete_metadata($dbh,undef,$delitem);
      } else {
- # store new data
+         $status = &LONCAPA::lonmetadata::update_metadata($dbh,undef,undef,
- # adjust some values to metadatadatabase (e.g., "usage" is a reserved word)
+                                                          \%metadata);
- 	$metadata{'creationdate'}=
- 	    &Apache::lonmysql::sqltime($metadata{'creationdate'});
- 	$metadata{'lastrevisiondate'}=
- 	    &Apache::lonmysql::sqltime($metadata{'lastrevisiondate'});
- 	$metadata{'sequsage'}=$metadata{'usage'};
- 	$metadata{'sequsage_list'}=$metadata{'usage_list'};
- 	my %newmetadata=();
- # see if we have old entries
- 	my @oldmeta=&Apache::lonmysql::get_rows('metadata',
- 						"url LIKE BINARY '".
- 						$metadata{'url'}."'");
- 	if ($#oldmeta==0) {
- # yes, there is one old entry, transfer to newmetadata
- 	    %newmetadata=&Apache::lonmeta::metadata_col_to_hash(@{$oldmeta[0]});
- # remove old entry
- 	    $status=&Apache::lonmysql::remove_from_table
- 		('metadata','url',$metadata{'url'});
- 	} elsif ($#oldmeta>0) {
- # more than one entry fit - how did that happen?
- 	    $error='<font color="red">Error occured retrieving old values in '.
- 	      'metadata table in LON-CAPA database: '.$#oldmeta.
- 	      ' matches</font>';
- 	    &Apache::lonnet::logthis($error);
- 	    return ($error,undef);
- 	}
- # store new data on top of it
- 	foreach (keys %metadata) {
- 	    $newmetadata{$_}=$metadata{$_};
- 	}
- 	$status = &Apache::lonmysql::store_row('metadata',\%newmetadata);
      }
-     if (! defined($status)) {
+     if (defined($status) && $status ne '') {
-         $error='<font color="red">Error occured storing new values in '.
+         $error='<span class="LC_error">'
-             'metadata table in LON-CAPA database</font>';
+               .&mt('Error occurred saving new values in metadata table in LON-CAPA database!')
+               .'</span>';
          &Apache::lonnet::logthis($error);
+         &Apache::lonnet::logthis($status);
          return ($error,undef);
      }
-     return (undef,$status);
+     return (undef,'success');
  }
+ # ========================================== Parse file for errors and warnings
+ sub checkonthis {
+     my ($r,$source)=@_;
+     my $uri=&Apache::lonnet::hreflocation($source);
+     $uri=~s/\/$//;
+     my $result=&Apache::lonnet::ssi_body($uri,
+ 					 ('grade_target'=>'web',
+ 					  'return_only_error_and_warning_counts' => 1));
+     my ($errorcount,$warningcount)=split(':',$result);
+     if (($errorcount) || ($warningcount)) {
+         $r->print('<h3>'.&mt('Warnings and Errors').'</h3>');
+         $r->print('<tt>'.$uri.'</tt>:');
+         $r->print('<ul>');
+         if ($warningcount) {
+             $r->print('<li><div class="LC_warning">'
+                      .&mt('[quant,_1,warning]',$warningcount)
+                      .'</div></li>');
+         }
+         if ($errorcount) {
+             $r->print('<li><div class="LC_error">'
+                      .&mt('[quant,_1,error]',$errorcount)
+                      .' <img src="/adm/lonMisc/bomb.gif" />'
+                      .'</div></li>');
+         }
+         $r->print('</ul>');
+     } else {
+ 	#$r->print('<font color="green">'.&mt('ok').'</font>');
+     }
+     $r->rflush();
+     return ($warningcount,$errorcount);
+ }
  # ============================================== Parse file itself for metadata
  #
  # parses a file with target meta, sets global %metadatafields %metadatakeys
- Line 868  backup copies, performs any automatic pr
+ Line 1013  backup copies, performs any automatic pr
  especially for rat and ssi files),
  Returns a 2 element array, the first is the string to be shown to the
- user, the second is an error code, either 1 (an error occured) or 0
+ user, the second is an error code, either 1 (an error occurred) or 0
  (no error occurred)
  I<Additional documentation needed.>
- Line 887  sub publish {
+ Line 1032  sub publish {
      my %allow=();
      unless ($logfile=Apache::File->new('>>'.$source.'.log')) {
- 	return ('<font color="red">'.&mt('No write permission to user directory, FAIL').'</font>',1);
+ 	return ('<span class="LC_error">'.&mt('No write permission to user directory, FAIL').'</span>',1);
      }
      print $logfile
- "\n\n================= Publish ".localtime()." Phase One  ================\n".$ENV{'user.name'}.'@'.$ENV{'user.domain'}."\n";
+ "\n\n================= Publish ".localtime()." Phase One  ================\n".$env{'user.name'}.':'.$env{'user.domain'}."\n";
      if (($style eq 'ssi') || ($style eq 'rat') || ($style eq 'prv')) {
  # ------------------------------------------------------- This needs processing
- Line 901  sub publish {
+ Line 1046  sub publish {
  	    print $logfile "Copied original file to ".$copyfile."\n";
          } else {
  	    print $logfile "Unable to write backup ".$copyfile.':'.$!."\n";
- 	    return ("<font color=\"red\">Failed to write backup copy, $!,FAIL</font>",1);
+ 	    return ("<span class=\"LC_error\">".&mt("Failed to write backup copy, [_1], FAIL",$1)."</span>",1);
          }
  # ------------------------------------------------------------- IDs and indices
- Line 911  sub publish {
+ Line 1056  sub publish {
  	if ($error) { return ($outstring,$error); }
  # ------------------------------------------------------------ Construct Allows
- 	$scrout.='<h3>'.&mt('Dependencies').'</h3>';
+         my $outdep=''; # Collect dependencies output data
          my $allowstr='';
-         foreach (sort(keys(%allow))) {
+         foreach my $thisdep (sort(keys(%allow))) {
- 	   my $thisdep=$_;
  	   if ($thisdep !~ /[^\s]/) { next; }
+            if ($thisdep =~/\$/) {
+               $outdep.='<div class="LC_warning">'
+                        .&mt('The resource depends on another resource with variable filename, i.e., [_1].','<tt>'.$thisdep.'</tt>').'<br />'
+                        .&mt('You likely need to explicitly allow access to all possible dependencies using the [_1]-tag.','<tt>&lt;allow&gt;</tt>')
+                        ."</div>\n";
+            }
             unless ($style eq 'rat') {
                $allowstr.="\n".'<allow src="'.$thisdep.'" />';
  	   }
-            $scrout.='<br />';
+           $outdep.='<div>';
-            if ($thisdep!~/\*/ && $thisdep!~m|^/adm/|) {
+            if ($thisdep!~/[\*\$]/ && $thisdep!~m|^/adm/|) {
- 	       $scrout.='<a href="'.$thisdep.'">';
+ 	       $outdep.='<a href="'.$thisdep.'">';
             }
-            $scrout.='<tt>'.$thisdep.'</tt>';
+            $outdep.='<tt>'.$thisdep.'</tt>';
-            if ($thisdep!~/\*/ && $thisdep!~m|^/adm/|) {
+            if ($thisdep!~/[\*\$]/ && $thisdep!~m|^/adm/|) {
- 	       $scrout.='</a>';
+ 	       $outdep.='</a>';
                 if (
         &Apache::lonnet::getfile($Apache::lonnet::perlvar{'lonDocRoot'}.'/'.
                                              $thisdep.'.meta') eq '-1') {
- 		   $scrout.= ' - <font color="red">'.&mt('Currently not available').
+ 		   $outdep.= ' - <span class="LC_error">'.&mt('Currently not available').
- 		       '</font>';
+ 		       '</span>';
                 } else {
+ #
+ # Store the fact that the dependency has been used by the target file
+ # Unfortunately, usage is erroneously named sequsage in lonmeta.pm
+ # The translation happens in lonmetadata.pm
+ #
                     my %temphash=(&Apache::lonnet::declutter($target).'___'.
                               &Apache::lonnet::declutter($thisdep).'___usage'
                                   => time);
-                    $thisdep=~/^\/res\/(\w+)\/(\w+)\//;
+                    $thisdep=~m{^/res/($match_domain)/($match_username)/};
                     if ((defined($1)) && (defined($2))) {
                        &Apache::lonnet::put('nohist_resevaldata',\%temphash,
  					   $1,$2);
  		   }
  	       }
             }
+            $outdep.='</div><br />';
+         }
+         if ($outdep) {
+             $scrout.='<h3>'.&mt('Dependencies').'</h3>'
+                     .$outdep
          }
-         $outstring=~s/\n*(\<\/[^\>]+\>)\s*$/$allowstr\n$1\n/s;
+         $outstring=~s/\n*(\<\/[^\>]+\>[^<]*)$/$allowstr\n$1\n/s;
  # ------------------------------------------------------------- Write modified.
- Line 951  sub publish {
+ Line 1112  sub publish {
            my $org;
            unless ($org=Apache::File->new('>'.$source)) {
               print $logfile "No write permit to $source\n";
-              return ('<font color="red">'.&mt('No write permission to').
+              return ('<span class="LC_error">'.&mt('No write permission to').
  		     ' '.$source.
- 		     ', '.&mt('FAIL').'</font>',1);
+ 		     ', '.&mt('FAIL').'</span>',1);
  	  }
            print($org $outstring);
          }
- Line 971  sub publish {
+ Line 1132  sub publish {
       my %oldparmstores=();
      unless ($batch) {
-      $scrout.='<h3>'.&mt('Metadata Information').' ' .
+      $scrout.='<h3>'.&mt('Metadata').' ' .
-        Apache::loncommon::help_open_topic("Metadata_Description")
+        &Apache::loncommon::help_open_topic("Metadata_Description")
         . '</h3>';
      }
  # ------------------------------------------------ First, check out environment
-      unless (-e $source.'.meta') {
+      if ((!(-e $source.'.meta')) || ($env{'form.forceoverride'})) {
-         $metadatafields{'author'}=$ENV{'environment.firstname'}.' '.
+         $metadatafields{'author'}=$env{'environment.firstname'}.' '.
- 	                          $ENV{'environment.middlename'}.' '.
+ 	                          $env{'environment.middlename'}.' '.
- 		                  $ENV{'environment.lastname'}.' '.
+ 		                  $env{'environment.lastname'}.' '.
- 		                  $ENV{'environment.generation'};
+ 		                  $env{'environment.generation'};
          $metadatafields{'author'}=~s/\s+/ /g;
          $metadatafields{'author'}=~s/\s+$//;
-         $metadatafields{'owner'}=$cuname.'@'.$cudom;
+         $metadatafields{'owner'}=$cuname.':'.$cudom;
  # ------------------------------------------------ Check out directory hierachy
          my $thisdisfn=$source;
-         $thisdisfn=~s/^\/home\/\Q$cuname\E\///;
-         my @urlparts=split(/\//,$thisdisfn);
+         $thisdisfn=~s/^\Q$docroot\E\/priv\/\Q$cudom\E\/\Q$cuname\E\///;
+         my @urlparts=('.',split(/\//,$thisdisfn));
          $#urlparts--;
-         my $currentpath='/home/'.$cuname.'/';
+         my $currentpath=$docroot.'/priv/'.$cudom.'/'.$cuname.'/';
  	my $prefix='../'x($#urlparts);
-         foreach (@urlparts) {
+         foreach my $subdir (@urlparts) {
- 	    $currentpath.=$_.'/';
+ 	    $currentpath.=$subdir.'/';
              $scrout.=&metaread($logfile,$currentpath.'default.meta',$prefix);
  	    $prefix=~s|^\.\./||;
          }
  # ----------------------------------------------------------- Parse file itself
  # read %metadatafields from file itself
- Line 1026  sub publish {
+ Line 1188  sub publish {
  		delete $metadatafields{$_};
              }
          }
+ # ------------------------------------------------------------- Save some stuff
+         my %savemeta=();
+         foreach ('title') {
+             if ($metadatafields{$_}) { $savemeta{$_}=$metadatafields{$_}; }
+ 	}
  # ------------------------------------------ See if anything new in file itself
  	$allmeta=&parseformeta($source,$style);
+ # ----------------------------------------------------------- Restore the stuff
+         foreach (keys %savemeta) {
+ 	    $metadatafields{$_}=$savemeta{$_};
+ 	}
     }
- Line 1040  sub publish {
+ Line 1210  sub publish {
  	if (($_=~/^parameter/) || ($_=~/^stores/)) {
  	    unless ($_=~/\.\w+$/) {
  		unless ($oldparmstores{$_}) {
- 		    print $logfile 'New: '.$_."\n";
+ 		    my $disp_key = $_;
- 		    $chparms.=$_.' ';
+ 		    $disp_key =~ tr/\0/_/;
+ 		    print $logfile ('New: '.$disp_key."\n");
+ 		    $chparms .= $disp_key.' ';
  		}
  	    }
  	}
      }
      if ($chparms) {
- 	$scrout.='<p><b>'.&mt('New parameters or stored values').
+ 	$scrout.='<p><b>'.&mt('New parameters or saved values').
  	    ':</b> '.$chparms.'</p>';
      }
- Line 1056  sub publish {
+ Line 1228  sub publish {
  	if (($_=~/^parameter/) || ($_=~/^stores/)) {
  	    unless (($metadatafields{$_.'.name'}) ||
  		    ($metadatafields{$_.'.package'}) || ($_=~/\.\w+$/)) {
- 		print $logfile 'Obsolete: '.$_."\n";
+ 		my $disp_key = $_;
- 		$chparms.=$_.' ';
+ 		$disp_key =~ tr/\0/_/;
+ 		print $logfile ('Obsolete: '.$disp_key."\n");
+ 		$chparms.=$disp_key.' ';
  	    }
  	}
      }
      if ($chparms) {
- 	$scrout.='<p><b>'.&mt('Obsolete parameters or stored values').':</b> '.
+         $scrout.='<p><b>'.&mt('Obsolete parameters or saved values').':</b> '
- 	    $chparms.'</p><h1><font color="red">'.&mt('Warning!').
+ 	        .$chparms.'</p>'
- 	    '</font></h1><p><font color="red" size="+1">'.
+                 .'<p class="LC_warning"><b>'.&mt('Warning!').'</b><br />'
- 	    &mt('If this resource is in active use, student performance data from the previous version may become inaccessible.').'</font></p><hr />';
+                 .&mt('If this resource is in active use, student performance data from the previous version may become inaccessible.')
+                 .'</p><hr />';
+     }
+     if ($metadatafields{'copyright'} eq 'priv') {
+         $scrout.='<p class="LC_warning"><b>'.&mt('Warning!').'</b><br />'
+                 .&mt('Copyright/distribution option "Private" is no longer supported. Select another option from below. Consider "Custom Rights" for maximum control over the usage of your resource.')
+                 .'</p><hr />';
      }
  # ------------------------------------------------------- Now have all metadata
- Line 1077  sub publish {
+ Line 1257  sub publish {
  	$textonly=~s/\<script[^\<]+\<\/script\>//g;
  	$textonly=~s/\<m\>[^\<]+\<\/m\>//g;
  	$textonly=~s/\<[^\>]*\>//g;
- 	$textonly=~tr/A-Z/a-z/;
- 	$textonly=~s/[\$\&][a-z]\w*//g;
- 	$textonly=~s/[^a-z\s]//g;
- 	foreach ($textonly=~m/(\w+)/g) {
- 	    unless ($nokey{$_}) {
- 		$keywords{$_}=1;
- 	    }
- 	}
-     }
+         #this is a work simplification for german authors for present
+         $textonly=HTML::Entities::decode($textonly);           #decode HTML-character
+         $textonly=Encode::Encoder::encode('utf8', $textonly);  #encode to perl internal unicode
+         $textonly=~tr/A-ZÜÄÖ/a-züäö/;      #add lowercase rule for german "Umlaute"
+         $textonly=~s/[\$\&][a-z]\w*//g;
+         $textonly=~s/[^a-z^ü^ä^ö^ß\s]//g;  #dont delete german "Umlaute"
+         foreach ($textonly=~m/[^\s]+/g) {  #match all but whitespaces
+             unless ($nokey{$_}) {
+                 $keywords{$_}=1;
+             }
+         }
+     }
-     foreach (split(/\W+/,$metadatafields{'keywords'})) {
+     foreach my $addkey (split(/[\"\'\,\;]/,$metadatafields{'keywords'})) {
- 	$keywords{$_}=1;
+ 	$addkey=~s/\s+/ /g;
+ 	$addkey=~s/^\s//;
+ 	$addkey=~s/\s$//;
+ 	if ($addkey=~/\w/) {
+ 	    $keywords{$addkey}=1;
+ 	}
      }
  # --------------------------------------------------- Now we also have keywords
  # =============================================================================
- # INTERACTIVE MODE
+ # interactive mode html goes into $intr_scrout
- #
+ # batch mode throws away this HTML
-     unless ($batch) {
+ # additionally all of the field functions have a by product of setting
-         $scrout.=
+ #   $env{'from.'..} so that it can be used by the phase two handler in
- 	    '<form name="pubform" action="/adm/publish" method="post">'.
+ #    batch mode
-             '<p><input type="submit" value="'.&mt('Finalize Publication').'" /></p>'.
-             &hiddenfield('phase','two').
+     my $intr_scrout.='<br />'
-             &hiddenfield('filename',$ENV{'form.filename'}).
+                     .'<form name="pubform" action="/adm/publish" method="post">';
- 	    &hiddenfield('allmeta',&Apache::lonnet::escape($allmeta)).
+     unless ($env{'form.makeobsolete'}) {
-             &hiddenfield('dependencies',join(',',keys %allow)).
+        $intr_scrout.='<p class="LC_warning">'
-             &textfield('Title','title',$metadatafields{'title'}).
+                     .&mt('Searching for your resource will be based on the following metadata. Please provide as much data as possible.')
-             &textfield('Author(s)','author',$metadatafields{'author'}).
+                     .'</p>'
- 	    &textfield('Subject','subject',$metadatafields{'subject'});
+                     .'<p><input type="submit" value="'
+                     .&mt('Finalize Publication')
- # --------------------------------------------------- Scan content for keywords
+                     .'" /> <a href="'.&Apache::loncfile::url($source).'">'.&mt('Cancel').'</a></p>';
+     }
-         my $keywords_help = Apache::loncommon::help_open_topic("Publishing_Keywords");
+     $intr_scrout.=&Apache::lonhtmlcommon::start_pick_box();
-         my $KEYWORDS=&mt('Keywords');
+     $intr_scrout.=
- 	my $CheckAll=&mt('check all');
+ 	&hiddenfield('phase','two').
- 	my $UncheckAll=&mt('uncheck all');
+ 	&hiddenfield('filename',$env{'form.filename'}).
- 	my $keywordout=<<"END";
+ 	&hiddenfield('allmeta',&escape($allmeta)).
+ 	&hiddenfield('dependencies',join(',',keys %allow));
+     unless ($env{'form.makeobsolete'}) {
+        $intr_scrout.=
+ 	&textfield('Title','title',$metadatafields{'title'}).
+ 	&textfield('Author(s)','author',$metadatafields{'author'}).
+ 	&textfield('Subject','subject',$metadatafields{'subject'});
+  # --------------------------------------------------- Scan content for keywords
+     my $keywords_help = &Apache::loncommon::help_open_topic("Publishing_Keywords");
+     my $keywordout=<<"END";
  <script>
  function checkAll(field) {
      for (i = 0; i < field.length; i++)
- Line 1126  function uncheckAll(field) {
+ Line 1326  function uncheckAll(field) {
          field[i].checked = false ;
  }
  </script>
- <p><font color="#800000" face="helvetica"><b>$KEYWORDS:</b></font>
-  $keywords_help</b>
- <input type="button" value="$CheckAll" onclick="javascript:checkAll(document.pubform.keywords)" />
- <input type="button" value="$UncheckAll" onclick="javascript:uncheckAll(document.pubform.keywords)" />
- </p>
- <br />
  END
- 	$keywordout.='<table border="2"><tr>';
+     $keywordout.="\n".&Apache::lonhtmlcommon::row_title(&mt('Keywords'))
- 	my $colcount=0;
+                 .$keywords_help
+                 .'<input type="button" value="'.&mt('check all').'" onclick="javascript:checkAll(document.pubform.keywords)" />'
- 	foreach (sort keys %keywords) {
+                 .'<input type="button" value="'.&mt('uncheck all').'" onclick="javascript:uncheckAll(document.pubform.keywords)" />'
- 	    $keywordout.='<td><input type="checkbox" name="keywords" value="'.$_.'"';
+                 .'</p><br />'
- 	    if ($metadatafields{'keywords'}) {
+                 .&Apache::loncommon::start_data_table();
- 		if ($metadatafields{'keywords'}=~/\Q$_\E/) {
+     my $cols_per_row = 10;
- 		    $keywordout.=' checked="on"';
+     my $colcount=0;
- 		}
+     my $wordcount=0;
- 	    } elsif (&Apache::loncommon::keyword($_)) {
+     my $numkeywords = scalar(keys(%keywords));
- 		$keywordout.=' checked="on"';
- 	    }
+     foreach my $word (sort(keys(%keywords))) {
- 	    $keywordout.=' />'.$_.'</td>';
+         if ($colcount == 0) {
- 	    if ($colcount>10) {
+             $keywordout .= &Apache::loncommon::start_data_table_row();
- 		$keywordout.="</tr><tr>\n";
+         }
- 		$colcount=0;
+         $colcount++;
- 	    }
+         $wordcount++;
- 	    $colcount++;
+         if (($wordcount == $numkeywords) && ($colcount < $cols_per_row)) {
- 	}
+             my $colspan = 1+$cols_per_row-$colcount;
+             $keywordout .= '<td colspan="'.$colspan.'">';
- 	$keywordout.='</tr></table>';
+         } else {
+             $keywordout .= '<td>';
- 	$scrout.=$keywordout;
+         }
+         $keywordout.='<label><input type="checkbox" name="keywords" value="'.$word.'"';
- 	$scrout.=&textfield('Additional Keywords','addkey','');
+         if ($metadatafields{'keywords'}) {
+             if ($metadatafields{'keywords'}=~/\Q$word\E/) {
+                 $keywordout.=' checked="checked"';
+                 $env{'form.keywords'}.=$word.',';
+             }
+         } elsif (&Apache::loncommon::keyword($word)) {
+             $keywordout.=' checked="checked"';
+             $env{'form.keywords'}.=$word.',';
+         }
+         $keywordout.=' />'.$word.'</label></td>';
+         if ($colcount == $cols_per_row) {
+             $keywordout.=&Apache::loncommon::end_data_table_row();
+             $colcount=0;
+         }
+     }
+     if ($colcount > 0) {
+         $keywordout .= &Apache::loncommon::end_data_table_row();
+     }
- 	$scrout.=&textfield('Notes','notes',$metadatafields{'notes'});
+     $env{'form.keywords'}=~s/\,$//;
- 	$scrout.=
+     $keywordout.=&Apache::loncommon::end_data_table_row()
- 	    "\n<p><font color=\"#800000\" face=\"helvetica\"><b>".&mt('Abstract').":".
+                  .&Apache::loncommon::end_data_table()
- 	    "</b></font></p><br />".
+                  .&Apache::lonhtmlcommon::row_closure();
- 	    '<textarea cols="80" rows="5" name="abstract">'.
- 	    $metadatafields{'abstract'}.'</textarea></p>';
- 	$source=~/\.(\w+)$/;
+     $intr_scrout.=$keywordout;
+     $intr_scrout.=&textfield('Additional Keywords','addkey','');
- 	$scrout.=
+     $intr_scrout.=&textfield('Notes','notes',$metadatafields{'notes'});
- 	    "\n<p><font color=\"#800000\" face=\"helvetica\"><b>".
- 	    &mt('Lowest Grade Level').':'.
-            "</b></font></p><br />".
- 	   &Apache::loncommon::select_level_form($metadatafields{'lowestgradelevel'},'lowestgradelevel').
- 	    "\n<p><font color=\"#800000\" face=\"helvetica\"><b>".
- 	    &mt('Highest Grade Level').':'.
-            "</b></font></p><br />".
- 	   &Apache::loncommon::select_level_form($metadatafields{'highestgradelevel'},'highestgradelevel').
-            &textfield('Standards','standards',$metadatafields{'standards'});
+     $intr_scrout.="\n".&Apache::lonhtmlcommon::row_title(&mt('Abstract'))
+                  .'<textarea cols="80" rows="5" name="abstract">'
+                  .$metadatafields{'abstract'}
+                  .'</textarea>'
+                  .&Apache::lonhtmlcommon::row_closure();
+     $source=~/\.(\w+)$/;
+     $intr_scrout.="\n".&Apache::lonhtmlcommon::row_title(&mt('Grade Levels'))
+                  .&mt('Lowest Grade Level:').'&nbsp;'
+                  .&select_level_form($metadatafields{'lowestgradelevel'},'lowestgradelevel')
+ #                .&Apache::lonhtmlcommon::row_closure();
+ #   $intr_scrout.="\n".&Apache::lonhtmlcommon::row_title(&mt('Highest Grade Level'))
+                  .' '.&mt('Highest Grade Level:').'&nbsp;'
+                  .&select_level_form($metadatafields{'highestgradelevel'},'highestgradelevel')
+                  .&Apache::lonhtmlcommon::row_closure();
- 	$scrout.=&hiddenfield('mime',$1);
+     $intr_scrout.=&textfield('Standards','standards',$metadatafields{'standards'});
- 	my $defaultlanguage=$metadatafields{'language'};
+     $intr_scrout.=&hiddenfield('mime',$1);
- 	$defaultlanguage =~ s/\s*notset\s*//g;
- 	$defaultlanguage =~ s/^,\s*//g;
- 	$defaultlanguage =~ s/,\s*$//g;
- 	$scrout.=&selectbox('Language','language',
+     my $defaultlanguage=$metadatafields{'language'};
- 			    $defaultlanguage,
+     $defaultlanguage =~ s/\s*notset\s*//g;
- 			    \&Apache::loncommon::languagedescription,
+     $defaultlanguage =~ s/^,\s*//g;
- 			    (&Apache::loncommon::languageids),
+     $defaultlanguage =~ s/,\s*$//g;
- 			   );
- 	unless ($metadatafields{'creationdate'}) {
+     $intr_scrout.=&selectbox('Language','language',
- 	    $metadatafields{'creationdate'}=time;
+ 			     $defaultlanguage,
- 	}
+ 			     \&Apache::loncommon::languagedescription,
- 	$scrout.=&hiddenfield('creationdate',
+ 			     (&Apache::loncommon::languageids),
- 			      &Apache::lonmysql::unsqltime($metadatafields{'creationdate'}));
+ 			     );
- 	$scrout.=&hiddenfield('lastrevisiondate',time);
+     unless ($metadatafields{'creationdate'}) {
+ 	$metadatafields{'creationdate'}=time;
+     }
+     $intr_scrout.=&hiddenfield('creationdate',
+ 			       &Apache::lonmysql::unsqltime($metadatafields{'creationdate'}));
+     $intr_scrout.=&hiddenfield('lastrevisiondate',time);
- 	$scrout.=&textfield('Publisher/Owner','owner',
+     my $pubowner_last;
- 			    $metadatafields{'owner'});
+     if ($style eq 'prv') {
+         $pubowner_last = 1;
+     }
+     $intr_scrout.=&textfield('Publisher/Owner','owner',
+ 			     $metadatafields{'owner'},$pubowner_last);
+ # ---------------------------------------------- Retrofix for unused copyright
+     if ($metadatafields{'copyright'} eq 'free') {
+ 	$metadatafields{'copyright'}='default';
+ 	$metadatafields{'sourceavail'}='open';
+     }
+     if ($metadatafields{'copyright'} eq 'priv') {
+         $metadatafields{'copyright'}='domain';
+     }
+ # ------------------------------------------------ Dial in reasonable defaults
+     my $defaultoption=$metadatafields{'copyright'};
+     unless ($defaultoption) { $defaultoption='default'; }
+     my $defaultsourceoption=$metadatafields{'sourceavail'};
+     unless ($defaultsourceoption) { $defaultsourceoption='closed'; }
+     unless ($style eq 'prv') {
  # -------------------------------------------------- Correct copyright for rat.
-         my $defaultoption=$metadatafields{'copyright'};
+ 	if ($style eq 'rat') {
-         unless ($defaultoption) { $defaultoption='default'; }
+ # -------------------------------------- Retrofix for non-applicable copyright
- 	unless ($style eq 'prv') {
+ 	    if ($metadatafields{'copyright'} eq 'public') {
- 	    if ($style eq 'rat') {
+ 		delete $metadatafields{'copyright'};
- 		if ($metadatafields{'copyright'} eq 'public') {
+ 		$defaultoption='default';
- 		    delete $metadatafields{'copyright'};
+ 	    }
-                     $defaultoption='default';
+ 	    $intr_scrout.=&selectbox('Copyright/Distribution','copyright',
- 		}
+ 				     $defaultoption,
- 		$scrout.=&selectbox('Copyright/Distribution','copyright',
+ 				     \&Apache::loncommon::copyrightdescription,
- 				    $defaultoption,
+ 				    (grep !/^(public|priv)$/,(&Apache::loncommon::copyrightids)));
- 				    \&Apache::loncommon::copyrightdescription,
- 				    (grep !/^public$/,(&Apache::loncommon::copyrightids)));
- 	    } else {
- 		$scrout.=&selectbox('Copyright/Distribution','copyright',
- 				    $defaultoption,
- 				    \&Apache::loncommon::copyrightdescription,
- 				    (&Apache::loncommon::copyrightids));
- 	    }
- 	    my $copyright_help =
- 		Apache::loncommon::help_open_topic('Publishing_Copyright');
- 	    $scrout =~ s/DISTRIBUTION:/'DISTRIBUTION: ' . $copyright_help/ge;
- 	    $scrout.=&textfield('Custom Distribution File','customdistributionfile',
- 				$metadatafields{'customdistributionfile'}).
- 				    $copyright_help;
- 	    my $uctitle=&mt('Obsolete');
-             $scrout.=
- 		"\n<p><font color=\"#800000\" face=\"helvetica\"><b>$uctitle:".
- 		'</b></font> <input type="checkbox" name="obsolete" ';
- 	    if ($metadatafields{'obsolete'}) {
- 		$scrout.=' checked="1" ';
- 	    }
- 	    $scrout.='/ ></p>'.
- 		&textfield('Suggested Replacement for Obsolete File',
- 				    'obsoletereplacement',
- 				    $metadatafields{'obsoletereplacement'});
  	} else {
- 	    $scrout.=&hiddenfield('copyright','private');
+ 	    $intr_scrout.=&selectbox('Copyright/Distribution','copyright',
- 	}
+ 				     $defaultoption,
- 	return ($scrout.'<p><input type="submit" value="'.
+ 				     \&Apache::loncommon::copyrightdescription,
- 		&mt('Finalize Publication').'" /></p></form>',0);
+ 				     (grep !/^priv$/,(&Apache::loncommon::copyrightids)));
- # =============================================================================
+ 	}
- # BATCH MODE
+ 	my $copyright_help =
- #
+ 	    &Apache::loncommon::help_open_topic('Publishing_Copyright');
+         my $replace=&mt('Copyright/Distribution:');
+ 	$intr_scrout =~ s/$replace/$replace.' '.$copyright_help/ge;
+ 	$intr_scrout.=&text_with_browse_field('Custom Distribution File','customdistributionfile',$metadatafields{'customdistributionfile'},'rights');
+ 	$intr_scrout.=&selectbox('Source Distribution','sourceavail',
+ 				 $defaultsourceoption,
+ 				 \&Apache::loncommon::source_copyrightdescription,
+ 				 (&Apache::loncommon::source_copyrightids));
+ #	$intr_scrout.=&text_with_browse_field('Source Custom Distribution File','sourcerights',$metadatafields{'sourcerights'},'rights');
+ 	my $uctitle=&mt('Obsolete');
+         my $obsolete_checked=($metadatafields{'obsolete'})?' checked="checked"':'';
+         $intr_scrout.="\n".&Apache::lonhtmlcommon::row_title($uctitle)
+                      .'<input type="checkbox" name="obsolete"'.$obsolete_checked.' />'
+                      .&Apache::lonhtmlcommon::row_closure(1);
+         $intr_scrout.=&text_with_browse_field('Suggested Replacement for Obsolete File',
+ 				    'obsoletereplacement',
+ 				    $metadatafields{'obsoletereplacement'},'',1);
      } else {
- # Transfer metadata directly to environment for stage 2
+ 	$intr_scrout.=&hiddenfield('copyright','private');
- 	foreach (keys %metadatafields) {
- 	    $ENV{'form.'.$_}=$metadatafields{$_};
- 	}
- 	$ENV{'form.addkey'}='';
- 	$ENV{'form.keywords'}='';
- 	foreach (keys %keywords) {
- 	    if ($metadatafields{'keywords'}) {
- 		if ($metadatafields{'keywords'}=~/\Q$_\E/) {
- 		    $ENV{'form.keywords'}.=$_.',';
- 		}
- 	    } elsif (&Apache::loncommon::keyword($_)) {
- 		$ENV{'form.keywords'}.=$_.',';
- 	    }
- 	}
- 	$ENV{'form.keywords'}=~s/\,$//;
- 	unless ($ENV{'form.creationdate'}) { $ENV{'form.creationdate'}=time; }
- 	$ENV{'form.lastrevisiondate'}=time;
- 	if ((($style eq 'rat') && ($ENV{'form.copyright'} eq 'public')) ||
- 	    (!$ENV{'form.copyright'})) {
- 	    $ENV{'form.copyright'}='default';
- 	}
- 	$ENV{'form.allmeta'}=&Apache::lonnet::escape($allmeta);
- 	return ($scrout,0);
      }
+    } else {
+        $intr_scrout.=
+ 	&hiddenfield('title',$metadatafields{'title'}).
+ 	&hiddenfield('author',$metadatafields{'author'}).
+ 	&hiddenfield('subject',$metadatafields{'subject'}).
+ 	&hiddenfield('keywords',$metadatafields{'keywords'}).
+ 	&hiddenfield('abstract',$metadatafields{'abstract'}).
+ 	&hiddenfield('notes',$metadatafields{'notes'}).
+ 	&hiddenfield('mime',$metadatafields{'mime'}).
+ 	&hiddenfield('creationdate',$metadatafields{'creationdate'}).
+ 	&hiddenfield('lastrevisiondate',time).
+ 	&hiddenfield('owner',$metadatafields{'owner'}).
+ 	&hiddenfield('lowestgradelevel',$metadatafields{'lowestgradelevel'}).
+ 	&hiddenfield('standards',$metadatafields{'standards'}).
+ 	&hiddenfield('highestgradelevel',$metadatafields{'highestgradelevel'}).
+ 	&hiddenfield('language',$metadatafields{'language'}).
+ 	&hiddenfield('copyright',$metadatafields{'copyright'}).
+ 	&hiddenfield('sourceavail',$metadatafields{'sourceavail'}).
+ 	&hiddenfield('customdistributionfile',$metadatafields{'customdistributionfile'}).
+ 	&hiddenfield('obsolete',1).
+ 	&text_with_browse_field('Suggested Replacement for Obsolete File',
+ 				    'obsoletereplacement',
+ 				    $metadatafields{'obsoletereplacement'},'',1);
+    }
+     if (!$batch) {
+ 	$scrout.=$intr_scrout
+             .&Apache::lonhtmlcommon::end_pick_box()
+             .'<p><input type="submit" value="'
+ 	    .&mt($env{'form.makeobsolete'}?'Make Obsolete':'Finalize Publication')
+             .'" /></p>'
+             .'</form>';
+     }
+     return($scrout,0);
  }
  #########################################
- Line 1310  Returns:
+ Line 1545  Returns:
  =over 4
- =item Scalar string
+ =item integer
- String contains status (errors and warnings) and information associated with
+: fail
- the server's attempts at publication.
+: success
+ =back
  =cut
- Line 1325  sub phasetwo {
+ Line 1562  sub phasetwo {
      my ($r,$source,$target,$style,$distarget,$batch)=@_;
      $source=~s/\/+/\//g;
      $target=~s/\/+/\//g;
+ #
-     if ($target=~/\_\_\_/) {
+ # Unless trying to get rid of something, check name validity
- 	$r->print(
+ #
-  '<font color="red">'.&mt('Unsupported character combination').
+     unless ($env{'form.obsolete'}) {
- 		  ' "<tt>___</tt>" '.&mt('in filename, FAIL').'</font>');
+ 	if ($target=~/(\_\_\_|\&\&\&|\:\:\:)/) {
-         return 0;
+ 	    $r->print('<span class="LC_error">'.
+ 		      &mt('Unsupported character combination [_1] in filename, FAIL.',"<tt>'.$1.'</tt>").
+ 		      '</span>');
+ 	    return 0;
+ 	}
+ 	unless ($target=~/\.(\w+)$/) {
+ 	    $r->print('<span class="LC_error">'.&mt('No valid extension found in filename, FAIL').'</span>');
+ 	    return 0;
+ 	}
+ 	if ($target=~/\.(\d+)\.(\w+)$/) {
+ 	    $r->print('<span class="LC_error">'.&mt('Filename of resource contains internal version number. Cannot publish such resources, FAIL').'</span>');
+ 	    return 0;
+ 	}
      }
+ #
+ # End name check
+ #
      $distarget=~s/\/+/\//g;
      my $logfile;
      unless ($logfile=Apache::File->new('>>'.$source.'.log')) {
  	$r->print(
-         '<font color="red">'.
+         '<span class="LC_error">'.
- 		&mt('No write permission to user directory, FAIL').'</font>');
+ 		&mt('No write permission to user directory, FAIL').'</span>');
          return 0;
      }
+     if ($source =~ /\.rights$/) {
+ 	$r->print('<p><span class="LC_warning">'.&mt('Warning: It can take up to 1 hour for rights changes to fully propagate.').'</span></p>');
+     }
      print $logfile
-         "\n================= Publish ".localtime()." Phase Two  ================\n".$ENV{'user.name'}.'@'.$ENV{'user.domain'}."\n";
+         "\n================= Publish ".localtime()." Phase Two  ================\n".$env{'user.name'}.':'.$env{'user.domain'}."\n";
      %metadatafields=();
      %metadatakeys=();
+     &metaeval(&unescape($env{'form.allmeta'}));
-     &metaeval(&Apache::lonnet::unescape($ENV{'form.allmeta'}));
+     $metadatafields{'title'}=$env{'form.title'};
+     $metadatafields{'author'}=$env{'form.author'};
-     $metadatafields{'title'}=$ENV{'form.title'};
+     $metadatafields{'subject'}=$env{'form.subject'};
-     $metadatafields{'author'}=$ENV{'form.author'};
+     $metadatafields{'notes'}=$env{'form.notes'};
-     $metadatafields{'subject'}=$ENV{'form.subject'};
+     $metadatafields{'abstract'}=$env{'form.abstract'};
-     $metadatafields{'notes'}=$ENV{'form.notes'};
+     $metadatafields{'mime'}=$env{'form.mime'};
-     $metadatafields{'abstract'}=$ENV{'form.abstract'};
+     $metadatafields{'language'}=$env{'form.language'};
-     $metadatafields{'mime'}=$ENV{'form.mime'};
+     $metadatafields{'creationdate'}=$env{'form.creationdate'};
-     $metadatafields{'language'}=$ENV{'form.language'};
+     $metadatafields{'lastrevisiondate'}=$env{'form.lastrevisiondate'};
-     $metadatafields{'creationdate'}=$ENV{'form.creationdate'};
+     $metadatafields{'owner'}=$env{'form.owner'};
-     $metadatafields{'lastrevisiondate'}=$ENV{'form.lastrevisiondate'};
+     $metadatafields{'copyright'}=$env{'form.copyright'};
-     $metadatafields{'owner'}=$ENV{'form.owner'};
+     $metadatafields{'standards'}=$env{'form.standards'};
-     $metadatafields{'copyright'}=$ENV{'form.copyright'};
+     $metadatafields{'lowestgradelevel'}=$env{'form.lowestgradelevel'};
-     $metadatafields{'standards'}=$ENV{'form.standards'};
+     $metadatafields{'highestgradelevel'}=$env{'form.highestgradelevel'};
-     $metadatafields{'lowestgradelevel'}=$ENV{'form.lowestgradelevel'};
-     $metadatafields{'highestgradelevel'}=$ENV{'form.highestgradelevel'};
      $metadatafields{'customdistributionfile'}=
-                                  $ENV{'form.customdistributionfile'};
+                                  $env{'form.customdistributionfile'};
-     $metadatafields{'obsolete'}=$ENV{'form.obsolete'};
+     $metadatafields{'sourceavail'}=$env{'form.sourceavail'};
+     $metadatafields{'obsolete'}=$env{'form.obsolete'};
      $metadatafields{'obsoletereplacement'}=
- 	                        $ENV{'form.obsoletereplacement'};
+ 	                        $env{'form.obsoletereplacement'};
-     $metadatafields{'dependencies'}=$ENV{'form.dependencies'};
+     $metadatafields{'dependencies'}=$env{'form.dependencies'};
-     $metadatafields{'modifyinguser'}=$ENV{'user.name'}.'@'.
+     $metadatafields{'modifyinguser'}=$env{'user.name'}.':'.
- 	                                 $ENV{'user.domain'};
+ 	                                 $env{'user.domain'};
-     $metadatafields{'authorspace'}=$cuname.'@'.$cudom;
+     $metadatafields{'authorspace'}=$cuname.':'.$cudom;
+     $metadatafields{'domain'}=$cudom;
-     my $allkeywords=$ENV{'form.addkey'};
+     my $allkeywords=$env{'form.addkey'};
-     if (exists($ENV{'form.keywords'})) {
+     if (exists($env{'form.keywords'})) {
-         if (ref($ENV{'form.keywords'})) {
+         if (ref($env{'form.keywords'})) {
-             $allkeywords .= ','.join(',',@{$ENV{'form.keywords'}});
+             $allkeywords .= ','.join(',',@{$env{'form.keywords'}});
          } else {
-             $allkeywords .= ','.$ENV{'form.keywords'};
+             $allkeywords .= ','.$env{'form.keywords'};
          }
      }
-     $allkeywords=~s/\W+/\,/;
+     $allkeywords=~s/[\"\']//g;
-     $allkeywords=~s/^\,//;
+     $allkeywords=~s/\s*[\;\,]\s*/\,/g;
+     $allkeywords=~s/\s+/ /g;
+     $allkeywords=~s/^[ \,]//;
+     $allkeywords=~s/[ \,]$//;
      $metadatafields{'keywords'}=$allkeywords;
  # check if custom distribution file is specified
      if ($metadatafields{'copyright'} eq 'custom') {
  	my $file=$metadatafields{'customdistributionfile'};
  	unless ($file=~/\.rights$/) {
-             return
+             $r->print(
-                 '<font color="red">'.&mt('No valid custom distribution rights file specified, FAIL').
+                 '<span class="LC_error">'.&mt('No valid custom distribution rights file specified, FAIL').
- 		'</font>';
+ 		'</span>');
+ 	    return 0;
          }
      }
      {
          print $logfile "\nWrite metadata file for ".$source;
          my $mfh;
          unless ($mfh=Apache::File->new('>'.$source.'.meta')) {
-             return
+             $r->print(
-                 '<font color="red">'.&mt('Could not write metadata, FAIL').
+                 '<span class="LC_error">'.&mt('Could not write metadata, FAIL').
- 		'</font>';
+ 		'</span>');
+ 	    return 0;
          }
          foreach (sort keys %metadatafields) {
              unless ($_=~/\./) {
- Line 1448  sub phasetwo {
+ Line 1713  sub phasetwo {
          my $srcf=$2;
          my $srct=$3;
          my $srcd=$1;
-         unless ($srcd=~/^\/home\/httpd\/html\/res/) {
+         my $docroot = $Apache::lonnet::perlvar{'lonDocRoot'};
+         unless ($srcd=~/^\Q$docroot\E\/res/) {
              print $logfile "\nPANIC: Target dir is ".$srcd;
-             return "<font color=\"red\">Invalid target directory, FAIL</font>";
+             $r->print(
+ 	 "<span class=\"LC_error\">".&mt('Invalid target directory, FAIL')."</span>");
+ 	    return 0;
          }
          opendir(DIR,$srcd);
          while ($filename=readdir(DIR)) {
- Line 1458  sub phasetwo {
+ Line 1726  sub phasetwo {
                  unlink($srcd.'/'.$filename);
                  unlink($srcd.'/'.$filename.'.meta');
              } else {
-                 if ($filename=~/\Q$srcf\E\.(\d+)\.\Q$srct\E$/) {
+                 if ($filename=~/^\Q$srcf\E\.(\d+)\.\Q$srct\E$/) {
                      $maxversion=($1>$maxversion)?$1:$maxversion;
                  }
              }
          }
          closedir(DIR);
          $maxversion++;
-         $r->print('<p>Creating old version '.$maxversion.'</p>');
+         $r->print('<p>'.&mt('Creating old version [_1]',$maxversion).'</p>');
          print $logfile "\nCreating old version ".$maxversion."\n";
          my $copyfile=$srcd.'/'.$srcf.'.'.$maxversion.'.'.$srct;
          if (copy($target,$copyfile)) {
  	    print $logfile "Copied old target to ".$copyfile."\n";
-             $r->print('<p>'.&mt('Copied old target file').'</p>');
+             $r->print(&Apache::lonhtmlcommon::confirm_success(&mt('Copied old target file')));
          } else {
  	    print $logfile "Unable to write ".$copyfile.':'.$!."\n";
-             return "<font color=\"red\">".&mt('Failed to copy old target').
+             $r->print(&Apache::lonhtmlcommon::confirm_success(&mt('Failed to copy old target').", $!",1));
- 		", $!, ".&mt('FAIL')."</font>";
+ 	    return 0;
          }
  # --------------------------------------------------------------- Copy Metadata
- Line 1485  sub phasetwo {
+ Line 1753  sub phasetwo {
          if (copy($target.'.meta',$copyfile)) {
  	    print $logfile "Copied old target metadata to ".$copyfile."\n";
-             $r->print('<p>'.&mt('Copied old metadata').'</p>')
+             $r->print(&Apache::lonhtmlcommon::confirm_success(&mt('Copied old metadata')));
          } else {
  	    print $logfile "Unable to write metadata ".$copyfile.':'.$!."\n";
              if (-e $target.'.meta') {
-                 return
+                 $r->print(&Apache::lonhtmlcommon::confirm_success(
-                     "<font color=\"red\">".
+                            &mt('Failed to write old metadata copy').", $!",1));
- &mt('Failed to write old metadata copy').", $!, ".&mt('FAIL')."</font>";
+ 		return 0;
  	    }
          }
- Line 1512  sub phasetwo {
+ Line 1780  sub phasetwo {
          $path.="/$parts[$count]";
          if ((-e $path)!=1) {
              print $logfile "\nCreating directory ".$path;
-             $r->print('<p>'.&mt('Created directory').' '.$parts[$count].'</p>');
              mkdir($path,0777);
+             $r->print('<p>'
+                      .&mt('Created directory [_1]'
+                          ,'<span class="LC_filename">'.$parts[$count].'</span>')
+                      .'</p>'
+             );
          }
      }
      if (copy($source,$copyfile)) {
          print $logfile "\nCopied original source to ".$copyfile."\n";
-         $r->print('<p>'.&mt('Copied source file').'</p>');
+         $r->print(&Apache::lonhtmlcommon::confirm_success(&mt('Copied source file')));
      } else {
          print $logfile "\nUnable to write ".$copyfile.':'.$!."\n";
-         return "<font color=\"red\">".
+         $r->print(&Apache::lonhtmlcommon::confirm_success(
- 	    &mt('Failed to copy source').", $!, ".&mt('FAIL')."</font>";
+ 	    &mt('Failed to copy source').", $!",1));
+ 	return 0;
      }
+ # ---------------------------------------------- Delete local tmp-preview files
+     unlink($copyfile.'.tmp');
  # --------------------------------------------------------------- Copy Metadata
      $copyfile=$copyfile.'.meta';
      if (copy($source.'.meta',$copyfile)) {
          print $logfile "\nCopied original metadata to ".$copyfile."\n";
-         $r->print('<p>'.&mt('Copied metadata').'</p>');
+         $r->print(&Apache::lonhtmlcommon::confirm_success(&mt('Copied metadata')));
      } else {
          print $logfile "\nUnable to write metadata ".$copyfile.':'.$!."\n";
-         return
+         $r->print(&Apache::lonhtmlcommon::confirm_success(
-             "<font color=\"red\">".&mt('Failed to write metadata copy').", $!, ".&mt('FAIL')."</font>";
+                   &mt('Failed to write metadata copy').", $!",1));
+ 	return 0;
      }
      $r->rflush;
- # --------------------------------------------------- Send update notifications
-     my @subscribed=&get_subscribed_hosts($target);
+ # ------------------------------------------------------------- Trigger updates
-     foreach my $subhost (@subscribed) {
+     push(@{$modified_urls},[$target,$source]);
- 	$r->print('<p>'.&mt('Notifying host').' '.$subhost.':');$r->rflush;
+     unless ($registered_cleanup) {
- 	print $logfile "\nNotifying host ".$subhost.':';
+         my $handlers = $r->get_handlers('PerlCleanupHandler');
- 	my $reply=&Apache::lonnet::critical('update:'.$target,$subhost);
+         $r->set_handlers('PerlCleanupHandler' => [\&notify,@{$handlers}]);
- 	$r->print($reply.'</p><br />');$r->rflush;
+ 	$registered_cleanup=1;
- 	print $logfile $reply;
      }
- # ---------------------------------------- Send update notifications, meta only
-     my @subscribedmeta=&get_subscribed_hosts("$target.meta");
+ # ---------------------------------------------------------- Clear local caches
-     foreach my $subhost (@subscribedmeta) {
+     my $thisdistarget=$target;
- 	$r->print('<p>'.
+     $thisdistarget=~s/^\Q$docroot\E//;
- &mt('Notifying host for metadata only').' '.$subhost.':');$r->rflush;
+     &Apache::lonnet::devalidate_cache_new('resversion',$target);
- 	print $logfile "\nNotifying host for metadata only ".$subhost.':';
+     &Apache::lonnet::devalidate_cache_new('meta',
- 	my $reply=&Apache::lonnet::critical('update:'.$target.'.meta',
+ 			 &Apache::lonnet::declutter($thisdistarget));
- 					    $subhost);
- 	$r->print($reply.'</p><br />');$r->rflush;
+ # ------------------------------------------------------------- Everything done
- 	print $logfile $reply;
+     $logfile->close();
-     }
+     $r->print('<p class="LC_success">'.&mt('Done').'</p>');
- # --------------------------------------------------- Notify subscribed courses
-     my %courses=&coursedependencies($target);
-     my $now=time;
-     foreach (keys %courses) {
- 	$r->print('<p>'.&mt('Notifying course').' '.$_.':');$r->rflush;
- 	print $logfile "\nNotifying host ".$_.':';
-         my ($cdom,$cname)=split(/\_/,$_);
- 	my $reply=&Apache::lonnet::cput
-                   ('versionupdate',{$target => $now},$cdom,$cname);
- 	$r->print($reply.'</p><br />');$r->rflush;
- 	print $logfile $reply;
-     }
  # ------------------------------------------------ Provide link to new resource
      unless ($batch) {
-         my $thisdistarget=$target;
-         $thisdistarget=~s/^\Q$docroot\E//;
-         my $thissrc=$source;
-         $thissrc=~s/^\/home\/(\w+)\/public_html/\/priv\/$1/;
+         my $thissrc=&Apache::loncfile::url($source);
          my $thissrcdir=$thissrc;
          $thissrcdir=~s/\/[^\/]+$/\//;
          $r->print(
-            '<hr /><a href="'.$thisdistarget.'"><font size="+2">'.
+             &Apache::lonhtmlcommon::actionbox([
-            &mt('View Published Version').'</font></a>'.
+                 '<a href="'.$thisdistarget.'">'.
-            '<p><a href="'.$thissrc.'"><font size=+2>'.
+                 &mt('View Published Version').
- 		  &mt('Back to Source').'</font></a></p>'.
+                 '</a>',
-            '<p><a href="'.$thissrcdir.
+                 '<a href="'.$thissrc.'">'.
-                    '"><font size="+2">'.
+                 &mt('Back to Source').
- 		  &mt('Back to Source Directory').'</font></a></p>');
+                 '</a>',
+                 '<a href="'.$thissrcdir.'">'.
+                 &mt('Back to Source Directory').
+                 '</a>'])
+         );
+     }
+     return 1;
+ }
+ # =============================================================== Notifications
+ sub notify {
+ # --------------------------------------------------- Send update notifications
+     foreach my $targetsource (@{$modified_urls}){
+ 	my ($target,$source)=@{$targetsource};
+ 	my $logfile=Apache::File->new('>>'.$source.'.log');
+ 	print $logfile "\nCleanup phase: Notifications\n";
+ 	my @subscribed=&get_subscribed_hosts($target);
+ 	foreach my $subhost (@subscribed) {
+ 	    print $logfile "\nNotifying host ".$subhost.':';
+ 	    my $reply=&Apache::lonnet::critical('update:'.$target,$subhost);
+ 	    print $logfile $reply;
+ 	}
+ # ---------------------------------------- Send update notifications, meta only
+ 	my @subscribedmeta=&get_subscribed_hosts("$target.meta");
+ 	foreach my $subhost (@subscribedmeta) {
+ 	    print $logfile "\nNotifying host for metadata only ".$subhost.':';
+ 	    my $reply=&Apache::lonnet::critical('update:'.$target.'.meta',
+ 						$subhost);
+ 	    print $logfile $reply;
+ 	}
+ # --------------------------------------------------- Notify subscribed courses
+ 	my %courses=&coursedependencies($target);
+ 	my $now=time;
+ 	foreach (keys %courses) {
+ 	    print $logfile "\nNotifying course ".$_.':';
+ 	    my ($cdom,$cname)=split(/\_/,$_);
+ 	    my $reply=&Apache::lonnet::cput
+ 		('versionupdate',{$target => $now},$cdom,$cname);
+ 	    print $logfile $reply;
+ 	}
+ 	print $logfile "\n============ Done ============\n";
+ 	$logfile->close();
      }
-     return '<p><font color="green">'.&mt('Done').'</font></p>';
+     if ($lock) { &Apache::lonnet::remove_lock($lock); }
+     return OK;
  }
  #########################################
  sub batchpublish {
      my ($r,$srcfile,$targetfile)=@_;
-     #publication pollutes %ENV with form.* values
+     #publication pollutes %env with form.* values
-     my %oldENV=%ENV;
+     my %oldenv=%env;
      $srcfile=~s/\/+/\//g;
      $targetfile=~s/\/+/\//g;
-     my $thisdisfn=$srcfile;
-     $thisdisfn=~s/\/home\/korte\/public_html\///;
      $srcfile=~s/\/+/\//g;
      my $docroot=$r->dir_config('lonDocRoot');
- Line 1624  sub batchpublish {
+ Line 1919  sub batchpublish {
      my $thisembstyle=&Apache::loncommon::fileembstyle($thistype);
-     $r->print('<h2>'.&mt('Publishing').' <tt>'.$thisdisfn.'</tt></h2>');
+     $r->print('<h2>'
+              .&mt('Publishing [_1]',&Apache::loncfile::display($srcfile))
+              .'</h2>'
+     );
  # phase one takes
  #  my ($source,$target,$style,$batch)=@_;
- Line 1632  sub batchpublish {
+ Line 1930  sub batchpublish {
      $r->print('<p>'.$outstring.'</p>');
  # phase two takes
  # my ($source,$target,$style,$distarget,batch)=@_;
- # $ENV{'form.allmeta'},$ENV{'form.title'},$ENV{'form.author'},...
+ # $env{'form.allmeta'},$env{'form.title'},$env{'form.author'},...
      if (!$error) {
  	$r->print('<p>');
  	&phasetwo($r,$srcfile,$targetfile,$thisembstyle,$thisdistarget,1);
  	$r->print('</p>');
      }
-     %ENV=%oldENV;
+     %env=%oldenv;
      return '';
  }
- Line 1648  sub publishdirectory {
+ Line 1946  sub publishdirectory {
      my ($r,$fn,$thisdisfn)=@_;
      $fn=~s/\/+/\//g;
      $thisdisfn=~s/\/+/\//g;
-     my $resdir=
+     my $thisdisresdir=$thisdisfn;
- 	$Apache::lonnet::perlvar{'lonDocRoot'}.'/res/'.$cudom.'/'.$cuname.'/'.
+     $thisdisresdir=~s/^\/priv\//\/res\//;
- 	$thisdisfn;
+     my $resdir = $r->dir_config('lonDocRoot').$thisdisresdir;
-     $r->print('<h1>'.&mt('Directory').' <tt>'.$thisdisfn.'</tt></h1>'.
+     $r->print('<form name="pubdirpref" method="post" action="">'
- 	      &mt('Target').': <tt>'.$resdir.'</tt><br />');
+              .&Apache::lonhtmlcommon::start_pick_box()
+              .&Apache::lonhtmlcommon::row_title(&mt('Directory'))
+             .'<span class="LC_filename">'.$thisdisfn.'</span>'
+             .&Apache::lonhtmlcommon::row_closure()
+             .&Apache::lonhtmlcommon::row_title(&mt('Target'))
+             .'<span class="LC_filename">'.$thisdisresdir.'</span>'
+     );
      my $dirptr=16384;		# Mask indicating a directory in stat.cmode.
+     unless ($env{'form.phase'} eq 'two') {
-     opendir(DIR,$fn);
+ # ask user what they want
-     my @files=sort(readdir(DIR));
+         $r->print(&Apache::lonhtmlcommon::row_closure()
-     foreach my $filename (@files) {
+                  .&Apache::lonhtmlcommon::row_title(&mt('Options'))
- 	my ($cdev,$cino,$cmode,$cnlink,
+         );
-             $cuid,$cgid,$crdev,$csize,
+         $r->print(&hiddenfield('phase','two').
-             $catime,$cmtime,$cctime,
+ 		  &hiddenfield('filename',$env{'form.filename'}).
-             $cblksize,$cblocks)=stat($fn.'/'.$filename);
+ 		  &checkbox('pubrec','include subdirectories').
+ 		  &checkbox('forcerepub','force republication of previously published files').
- 	my $extension='';
+                   &checkbox('obsolete','make file(s) obsolete').
- 	if ($filename=~/\.(\w+)$/) { $extension=$1; }
+ 		  &checkbox('forceoverride','force directory level metadata over existing')
- 	if ($cmode&$dirptr) {
+         );
- 	    if (($filename!~/^\./) && ($ENV{'form.pubrec'})) {
+         $r->print(&Apache::lonhtmlcommon::row_closure(1)
- 		&publishdirectory($r,$fn.'/'.$filename,$thisdisfn.'/'.$filename);
+                  .&Apache::lonhtmlcommon::end_pick_box()
- 	    }
+                  .'<br /><input type="submit" value="'.&mt('Publish Directory').'" /></form>'
- 	} elsif ((&Apache::loncommon::fileembstyle($extension) ne 'hdn') &&
+         );
- 		 ($filename!~/^[\#\.]/) && ($filename!~/\~$/)) {
+         $lock=0;
+     } else {
+         $r->print(&Apache::lonhtmlcommon::row_closure(1)
+                  .&Apache::lonhtmlcommon::end_pick_box()
+         );
+         unless ($lock) { $lock=&Apache::lonnet::set_lock(&mt('Publishing [_1]',$fn)); }
+ # actually publish things
+ 	opendir(DIR,$fn);
+ 	my @files=sort(readdir(DIR));
+ 	foreach my $filename (@files) {
+ 	    my ($cdev,$cino,$cmode,$cnlink,
+ 		$cuid,$cgid,$crdev,$csize,
+ 		$catime,$cmtime,$cctime,
+ 		$cblksize,$cblocks)=stat($fn.'/'.$filename);
+ 	    my $extension='';
+ 	    if ($filename=~/\.(\w+)$/) { $extension=$1; }
+ 	    if ($cmode&$dirptr) {
+ 		if (($filename!~/^\./) && ($env{'form.pubrec'})) {
+ 		    &publishdirectory($r,$fn.'/'.$filename,$thisdisfn.'/'.$filename);
+ 		}
+ 	    } elsif ((&Apache::loncommon::fileembstyle($extension) ne 'hdn') &&
+ 		     ($filename!~/^[\#\.]/) && ($filename!~/\~$/)) {
  # find out publication status and/or exiting metadata
- 	    my $publishthis=0;
+ 		my $publishthis=0;
- 	    if (-e $resdir.'/'.$filename) {
+ 		if (-e $resdir.'/'.$filename) {
- 	        my ($rdev,$rino,$rmode,$rnlink,
+ 		    my ($rdev,$rino,$rmode,$rnlink,
- 		    $ruid,$rgid,$rrdev,$rsize,
+ 			$ruid,$rgid,$rrdev,$rsize,
- 		    $ratime,$rmtime,$rctime,
+ 			$ratime,$rmtime,$rctime,
- 		    $rblksize,$rblocks)=stat($resdir.'/'.$filename);
+ 			$rblksize,$rblocks)=stat($resdir.'/'.$filename);
- 	        if (($rmtime<$cmtime) || ($ENV{'form.forcerepub'})) {
+ 		    if (($rmtime<$cmtime) || ($env{'form.forcerepub'})) {
  # previously published, modified now
- 		    $publishthis=1;
+ 			$publishthis=1;
-                 }
+ 		    }
- 	    } else {
+ 		    my $meta_cmtime = (stat($fn.'/'.$filename.'.meta'))[9];
+ 		    my $meta_rmtime = (stat($resdir.'/'.$filename.'.meta'))[9];
+ 		    if ( $meta_rmtime<$meta_cmtime ) {
+ 			$publishthis=1;
+ 		    }
+ 		} else {
  # never published
- 		$publishthis=1;
+ 		    $publishthis=1;
- 	    }
+ 		}
- 	    if ($publishthis) {
-                 &batchpublish($r,$fn.'/'.$filename,$resdir.'/'.$filename);
+ 		if ($publishthis) {
- 	    } else {
+ 		    &batchpublish($r,$fn.'/'.$filename,$resdir.'/'.$filename);
- 		$r->print('<br />'.&mt('Skipping').' '.$filename.'<br />');
+ 		} else {
+ 		    $r->print('<br />'.&mt('Skipping').' '.$filename.'<br />');
+ 		}
+ 		$r->rflush();
  	    }
- 	    $r->rflush();
  	}
+ 	closedir(DIR);
      }
-     closedir(DIR);
  }
  #########################################
- Line 1703  sub publishdirectory {
+ Line 2036  sub publishdirectory {
  sub defaultmetapublish {
      my ($r,$fn,$cuname,$cudom)=@_;
-     $fn=~s/^\/\~$cuname\//\/home\/$cuname\/public_html\//;
      unless (-e $fn) {
         return HTTP_NOT_FOUND;
      }
      my $target=$fn;
-     $target=~s/^\/home\/$cuname\/public_html\//$Apache::lonnet::perlvar{'lonDocRoot'}\/res\/$cudom\/$cuname\//;
+     $target=~s/^\Q$Apache::lonnet::perlvar{'lonDocRoot'}\E\/priv\//\Q$Apache::lonnet::perlvar{'lonDocRoot'}\E\/res\//;
      &Apache::loncommon::content_type($r,'text/html');
      $r->send_http_header;
-     $r->print('<html><head><title>LON-CAPA Publishing</title></head>');
+     $r->print(&Apache::loncommon::start_page('Metadata Publication'));
-     $r->print(&Apache::loncommon::bodytag('Catalog Information Publication'));
  # ---------------------------------------------------------------- Write Source
      my $copyfile=$target;
- Line 1727  sub defaultmetapublish {
+ Line 2058  sub defaultmetapublish {
      for ($count=5;$count<$#parts;$count++) {
          $path.="/$parts[$count]";
          if ((-e $path)!=1) {
-             $r->print('<p>'.&mt('Created directory').' '.$parts[$count].'</p>');
              mkdir($path,0777);
+             $r->print('<p>'
+                      .&mt('Created directory [_1]'
+                          ,'<span class="LC_filename">'.$parts[$count].'</span>')
+                      .'</p>'
+             );
          }
      }
      if (copy($fn,$copyfile)) {
          $r->print('<p>'.&mt('Copied source file').'</p>');
      } else {
-         return "<font color=\"red\">".
+         return "<span class=\"LC_error\">".
- 	    &mt('Failed to copy source').", $!, ".&mt('FAIL')."</font>";
+ 	    &mt('Failed to copy source').", $!, ".&mt('FAIL')."</span>";
      }
  # --------------------------------------------------- Send update notifications
- Line 1748  sub defaultmetapublish {
+ Line 2083  sub defaultmetapublish {
  	$r->print($reply.'</p><br />');$r->rflush;
      }
  # ------------------------------------------------------------------- Link back
-     my $link=$fn;
+     $r->print("<a href='".&Apache::loncfile::display($fn)."'>".&mt('Back to Metadata').'</a>');
-     $link=~s/^\/home\/$cuname\/public_html\//\/priv\/$cuname\//;
+     $r->print(&Apache::loncommon::end_page());
-     $r->print("<a href='$link'>".&mt('Back to Catalog Information').'</a>');
-     $r->print('</body></html>');
      return OK;
  }
  #########################################
- Line 1808  sub handler {
+ Line 2141  sub handler {
      &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},
                                              ['filename']);
+ # -------------------------------------- Flag and buffer for registered cleanup
+     $registered_cleanup=0;
+     @{$modified_urls}=();
  # -------------------------------------------------------------- Check filename
-     my $fn=&Apache::lonnet::unescape($ENV{'form.filename'});
+     my $fn=&unescape($env{'form.filename'});
+     ($cuname,$cudom)=&Apache::lonnet::constructaccess($fn);
+ # ----------------------------------------------------- Do we have permissions?
+      unless (($cuname) && ($cudom)) {
+        $r->log_reason($env{'user.name'}.' at '.$env{'user.domain'}.
+                       ' trying to publish file '.$env{'form.filename'}.
+                       ' - not authorized',
+                       $r->filename);
+        return HTTP_NOT_ACCEPTABLE;
+      }
+ # ----------------------------------------------------------------- Get docroot
+     $docroot=$r->dir_config('lonDocRoot');
-     ($cuname,$cudom)=
- 	&Apache::loncacc::constructaccess($fn,$r->dir_config('lonDefDomain'));
  # special publication: default.meta file
      if ($fn=~/\/default.meta$/) {
  	return &defaultmetapublish($r,$fn,$cuname,$cudom);
      }
      $fn=~s/\.meta$//;
+ # sanity test on the filename
      unless ($fn) {
  	$r->log_reason($cuname.' at '.$cudom.
  		       ' trying to publish empty filename', $r->filename);
  	return HTTP_NOT_FOUND;
      }
-     unless (($cuname) && ($cudom)) {
+     unless (-e $docroot.$fn) {
- 	$r->log_reason($cuname.' at '.$cudom.
- 		       ' trying to publish file '.$ENV{'form.filename'}.
- 		       ' ('.$fn.') - not authorized',
- 		       $r->filename);
- 	return HTTP_NOT_ACCEPTABLE;
-     }
-     my $home=&Apache::lonnet::homeserver($cuname,$cudom);
-     my $allowed=0;
-     my @ids=&Apache::lonnet::current_machine_ids();
-     foreach my $id (@ids) { if ($id eq $home) { $allowed = 1; }  }
-     unless ($allowed) {
- 	$r->log_reason($cuname.' at '.$cudom.
- 		       ' trying to publish file '.$ENV{'form.filename'}.
- 		       ' ('.$fn.') - not homeserver ('.$home.')',
- 		       $r->filename);
- 	return HTTP_NOT_ACCEPTABLE;
-     }
-     $fn=~s/^http\:\/\/[^\/]+//;
-     $fn=~s/^\/\~(\w+)/\/home\/$1\/public_html/;
-     my $targetdir='';
-     $docroot=$r->dir_config('lonDocRoot');
-     if ($1 ne $cuname) {
- 	$r->log_reason($cuname.' at '.$cudom.
- 		       ' trying to publish unowned file '.
- 		       $ENV{'form.filename'}.' ('.$fn.')',
- 		       $r->filename);
- 	return HTTP_NOT_ACCEPTABLE;
-     } else {
- 	$targetdir=$docroot.'/res/'.$cudom;
-     }
-     unless (-e $fn) {
  	$r->log_reason($cuname.' at '.$cudom.
  		       ' trying to publish non-existing file '.
- 		       $ENV{'form.filename'}.' ('.$fn.')',
+ 		       $env{'form.filename'}.' ('.$fn.')',
  		       $r->filename);
  	return HTTP_NOT_FOUND;
      }
-     unless ($ENV{'form.phase'} eq 'two') {
  # -------------------------------- File is there and owned, init lookup tables.
- 	%addid=();
+     %addid=();
- 	{
+     {
- 	    my $fh=Apache::File->new($r->dir_config('lonTabDir').'/addid.tab');
+ 	my $fh=Apache::File->new($r->dir_config('lonTabDir').'/addid.tab');
- 	    while (<$fh>=~/(\w+)\s+(\w+)/) {
+ 	while (<$fh>=~/(\w+)\s+(\w+)/) {
- 		$addid{$1}=$2;
+ 	    $addid{$1}=$2;
- 	    }
  	}
+     }
- 	%nokey=();
+     %nokey=();
- 	{
+     {
- 	    my $fh=Apache::File->new($r->dir_config('lonIncludes').'/un_keyword.tab');
+ 	my $fh=Apache::File->new($r->dir_config('lonIncludes').'/un_keyword.tab');
- 	    while (<$fh>) {
+ 	while (<$fh>) {
- 		my $word=$_;
+ 	    my $word=$_;
- 		chomp($word);
+ 	    chomp($word);
- 		$nokey{$word}=1;
+ 	    $nokey{$word}=1;
- 	    }
  	}
      }
  # ---------------------------------------------------------- Start page output.
      &Apache::loncommon::content_type($r,'text/html');
      $r->send_http_header;
-     $r->print('<html><head><title>LON-CAPA Publishing</title></head>');
+     # Breadcrumbs
-     $r->print(&Apache::loncommon::bodytag('Resource Publication'));
+     &Apache::lonhtmlcommon::clear_breadcrumbs();
+     &Apache::lonhtmlcommon::add_breadcrumb({
+         'text'  => 'Authoring Space',
-     my $thisfn=$fn;
+         'href'  => &Apache::loncommon::authorspace($fn),
+     });
-     my $thistarget=$thisfn;
+     &Apache::lonhtmlcommon::add_breadcrumb({
+         'text'  => 'Resource Publication',
-     $thistarget=~s/^\/home/$targetdir/;
+         'href'  => '',
-     $thistarget=~s/\/public\_html//;
+     });
-     my $thisdistarget=$thistarget;
+     my $js='<script type="text/javascript">'.
-     $thisdistarget=~s/^\Q$docroot\E//;
+ 	&Apache::loncommon::browser_and_searcher_javascript().
+ 	'</script>';
-     my $thisdisfn=$thisfn;
+     $r->print(&Apache::loncommon::start_page('Resource Publication',$js)
-     $thisdisfn=~s/^\/home\/\Q$cuname\E\/public_html\///;
+              .&Apache::lonhtmlcommon::breadcrumbs()
+              .&Apache::loncommon::head_subbox(
+                   &Apache::loncommon::CSTR_pageheader($docroot.$fn))
+     );
+     my $thisdisfn=&HTML::Entities::encode($fn,'<>&"');
+     my $thistarget=$fn;
+     $thistarget=~s/^\/priv\//\/res\//;
+     my $thisdistarget=&HTML::Entities::encode($thistarget,'<>&"');
      if ($fn=~/\/$/) {
  # -------------------------------------------------------- This is a directory
- 	&publishdirectory($r,$fn,$thisdisfn);
+ 	&publishdirectory($r,$docroot.$fn,$thisdisfn);
- 	$r->print('<hr><font size="+2">'.&mt('Done').'</font><br><a href="/priv/'
+         $r->print(
- 		  .$cuname.'/'.$thisdisfn
+             '<br /><br />'.
- 		  .'">'.&mt('Return to Directory').'</a>');
+             &Apache::lonhtmlcommon::actionbox([
+                 '<a href="'.$thisdisfn.'">'.&mt('Return to Directory').'</a>']));
      } else {
  # ---------------------- Evaluate individual file, and then output information.
- 	$thisfn=~/\.(\w+)$/;
+ 	$fn=~/\.(\w+)$/;
  	my $thistype=$1;
  	my $thisembstyle=&Apache::loncommon::fileembstyle($thistype);
- 	$r->print('<h2>'.&mt('Publishing').' '.
+         if ($thistype eq 'page') {  $thisembstyle = 'rat'; }
- 		  &Apache::loncommon::filedescription($thistype).' <tt>');
+         $r->print('<h2>'
+                  .&mt('Publishing [_1]'
+                      ,'<span class="LC_filename">'.$thisdisfn.'</span>')
+                  .'</h2>'
+         );
+         $r->print('<h3>'.&mt('Resource Details').'</h3>');
+         $r->print(&Apache::lonhtmlcommon::start_pick_box());
+         $r->print(&Apache::lonhtmlcommon::row_title(&mt('Type'))
+                  .&Apache::loncommon::filedescription($thistype)
+                  .&Apache::lonhtmlcommon::row_closure()
+                  );
+         $r->print(&Apache::lonhtmlcommon::row_title(&mt('Link to Resource'))
+                  .'<tt>'
+                  );
  	$r->print(<<ENDCAPTION);
- <a href='javascript:void(window.open("/~$cuname/$thisdisfn","cat","height=300,width=500,scrollbars=1,resizable=1,menubar=0,location=1"))'>
+ <a href='javascript:void(window.open("$thisdisfn","cat","height=300,width=500,scrollbars=1,resizable=1,menubar=0,location=1"))'>
  $thisdisfn</a>
  ENDCAPTION
-         $r->print('</tt></h2><b>'.&mt('Target').':</b> <tt>'.
+         $r->print('</tt>'
- 		  $thisdistarget.'</tt><br />');
+                  .&Apache::lonhtmlcommon::row_closure()
+                  );
- 	if (($cuname ne $ENV{'user.name'})||($cudom ne $ENV{'user.domain'})) {
- 	    $r->print('<h3><font color="red">'.&mt('Co-Author').': '.
+         $r->print(&Apache::lonhtmlcommon::row_title(&mt('Target'))
- 		      $cuname.&mt(' at ').$cudom.'</font></h3>');
+                  .'<tt>'.$thisdistarget.'</tt>'
+                  );
+ 	if (($cuname ne $env{'user.name'})||($cudom ne $env{'user.domain'})) {
+             $r->print(&Apache::lonhtmlcommon::row_closure()
+                      .&Apache::lonhtmlcommon::row_title(&mt('Co-Author'))
+                      .'<span class="LC_warning">'
+ 		     .&Apache::loncommon::plainname($cuname,$cudom) .' ('.$cuname.':'.$cudom.')'
+                      .'</span>'
+                      );
  	}
  	if (&Apache::loncommon::fileembstyle($thistype) eq 'ssi') {
+             $r->print(&Apache::lonhtmlcommon::row_closure()
+                      .&Apache::lonhtmlcommon::row_title(&mt('Diffs')));
  	    $r->print(<<ENDDIFF);
- <br />
+ <a href='javascript:void(window.open("/adm/diff?filename=$thisdisfn&amp;versiontwo=priv","cat","height=300,width=500,scrollbars=1,resizable=1,menubar=0,location=1"))'>
- <a href='javascript:void(window.open("/adm/diff?filename=/~$cuname/$thisdisfn&versiontwo=priv","cat","height=300,width=500,scrollbars=1,resizable=1,menubar=0,location=1"))'>
  ENDDIFF
-             $r->print(&mt('Diffs with Current Version').'</a><br />');
+             $r->print(&mt('Diffs with Current Version').'</a>');
  	}
+         $r->print(&Apache::lonhtmlcommon::row_closure(1)
+                  .&Apache::lonhtmlcommon::end_pick_box()
+                  );
- # ------------------ Publishing from $thisfn to $thistarget with $thisembstyle.
+ # ---------------------- Publishing from $fn to $thistarget with $thisembstyle.
- 	unless ($ENV{'form.phase'} eq 'two') {
+ 	unless ($env{'form.phase'} eq 'two') {
- 	    my ($outstring,$error)=&publish($thisfn,$thistarget,$thisembstyle);
+ # ---------------------------------------------------------- Parse for problems
- 	    $r->print('<hr />'.$outstring);
+ 	    my ($warningcount,$errorcount);
+ 	    if ($thisembstyle eq 'ssi') {
+ 		($warningcount,$errorcount)=&checkonthis($r,$fn);
+ 	    }
+ 	    unless ($errorcount) {
+ 		my ($outstring,$error)=
+ 		    &publish($docroot.$fn,$docroot.$thistarget,$thisembstyle);
+ 		$r->print($outstring);
+ 	    } else {
+ 		$r->print('<h3 class="LC_error">'.
+ 			  &mt('The document contains errors and cannot be published.').
+ 			  '</h3>');
+ 	    }
  	} else {
- 	    $r->print('<hr />'.
+ 	    &phasetwo($r,$docroot.$fn,$docroot.$thistarget,$thisembstyle,$thisdistarget);
- 	    &phasetwo($r,$thisfn,$thistarget,$thisembstyle,$thisdistarget));
  	}
      }
-     $r->print('</body></html>');
+     $r->print(&Apache::loncommon::end_page());
      return OK;
  }
- Line 1977  __END__
+ Line 2332  __END__
  =back
- =back
  =cut

FreeBSD-CVSweb <freebsd-cvsweb@FreeBSD.org>

Removed from v.1.165
changed lines
	Added in v.1.292