--- loncom/publisher/lonpublisher.pm	2002/10/07 13:50:36	1.99
+++ loncom/publisher/lonpublisher.pm	2003/03/17 16:58:09	1.119
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Publication Handler
 #
-# $Id: lonpublisher.pm,v 1.99 2002/10/07 13:50:36 www Exp $
+# $Id: lonpublisher.pm,v 1.119 2003/03/17 16:58:09 www Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -33,18 +33,14 @@
 # 11/28,11/29,11/30,12/01,12/02,12/04,12/23 Gerd Kortemeyer
 # 03/23 Guy Albertelli
 # 03/24,03/29,04/03 Gerd Kortemeyer
-# 04/16/2001 Scott Harrison
 # 05/03,05/05,05/07 Gerd Kortemeyer
-# 05/28/2001 Scott Harrison
 # 06/23,08/07,08/11,8/13,8/17,8/18,8/24,9/26,10/16 Gerd Kortemeyer
 # 12/04,12/05 Guy Albertelli
 # 12/05 Gerd Kortemeyer
 # 12/05 Guy Albertelli
 # 12/06,12/07 Gerd Kortemeyer
-# 12/15,12/16 Scott Harrison
 # 12/25 Gerd Kortemeyer
 # YEAR=2002
-# 1/16,1/17 Scott Harrison
 # 1/17 Gerd Kortemeyer
 #
 ###
@@ -121,27 +117,21 @@ use File::Copy;
 use Apache::Constants qw(:common :http :methods);
 use HTML::LCParser;
 use Apache::lonxml;
-use Apache::lonhomework;
 use Apache::loncacc;
 use DBI;
 use Apache::lonnet();
 use Apache::loncommon();
 use Apache::lonmysql;
+use vars qw(%metadatafields %metadatakeys);
 
 my %addid;
 my %nokey;
 
-my %metadatafields;
-my %metadatakeys;
-
 my $docroot;
 
 my $cuname;
 my $cudom;
 
-#########################################
-#########################################
-
 =pod
 
 =item B<metaeval>
@@ -267,48 +257,27 @@ sub metaread {
 #########################################
 #########################################
 
-=pod
-
-=item B<sqltime>
-
-Convert 'time' format into a datetime sql format
-
-Parameters:
-
-=over 4
-
-=item I<$timef>
-
-Seconds since 00:00:00 UTC, January 1, 1970.
-
-=back
-
-Returns:
-
-=over 4
-
-=item Scalar string
-
-MySQL-compatible datetime string.
-
-=back
-
-=cut
-
-#########################################
-#########################################
-sub sqltime {
-    my $timef=shift @_;
-    my ($sec,$min,$hour,$mday,$mon,$year,$wday,$yday,$isdst) =
-	localtime($timef);
-    $mon++; $year+=1900;
-    return "$year-$mon-$mday $hour:$min:$sec";
+sub coursedependencies {
+    my $url=&Apache::lonnet::declutter(shift);
+    $url=~s/\.meta$//;
+    my ($adomain,$aauthor)=($url=~/^(\w+)\/(\w+)\//);
+    my $regexp=$url;
+    $regexp=~s/(\W)/\\$1/g;
+    $regexp='___'.$regexp.'___course';
+    my %evaldata=&Apache::lonnet::dump('nohist_resevaldata',$adomain,
+				       $aauthor,$regexp);
+    my %courses=();
+    foreach (keys %evaldata) {
+	if ($_=~/^([a-zA-Z0-9]+_[a-zA-Z0-9]+)___.+___course$/) {
+	    $courses{$1}=1;
+        }
+    }
+    return %courses;
 }
-
-
 #########################################
 #########################################
 
+
 =pod
 
 =item Form-field-generating subroutines.
@@ -467,7 +436,7 @@ sub get_subscribed_hosts {
     my $srcf=$2;
     opendir(DIR,$1);
     while ($filename=readdir(DIR)) {
-	if ($filename=~/$srcf\.(\w+)$/) {
+	if ($filename=~/\Q$srcf\E\.(\w+)$/) {
 	    my $subhost=$1;
 	    if (($subhost ne 'meta' && $subhost ne 'subscription') &&
                 ($subhost ne $Apache::lonnet::perlvar{'lonHostID'})) {
@@ -492,7 +461,6 @@ sub get_subscribed_hosts {
     } else {
 	&Apache::lonnet::logthis("Unable to open $target.subscription");
     }
-    &Apache::lonnet::logthis("Got list of ".join(':',@subscribed));
     return @subscribed;
 }
 
@@ -515,6 +483,10 @@ sub get_max_ids_indices {
     my $maxindex=10;
     my $maxid=10;
     my $needsfixup=0;
+    my $duplicateids=0;
+
+    my %allids;
+    my %duplicatedids;
 
     my $parser=HTML::LCParser->new($content);
     my $token;
@@ -525,6 +497,12 @@ sub get_max_ids_indices {
 		if ($counter eq 'id') {
 		    if (defined($token->[2]->{'id'})) {
 			$maxid=($token->[2]->{'id'}>$maxid)?$token->[2]->{'id'}:$maxid;
+			if (exists($allids{$token->[2]->{'id'}})) {
+			    $duplicateids=1;
+			    $duplicatedids{$token->[2]->{'id'}}=1;
+			} else {
+			    $allids{$token->[2]->{'id'}}=1;
+			}
 		    } else {
 			$needsfixup=1;
 		    }
@@ -538,7 +516,8 @@ sub get_max_ids_indices {
 	    }
 	}
     }
-    return ($needsfixup,$maxid,$maxindex);
+    return ($needsfixup,$maxid,$maxindex,$duplicateids,
+	    (keys(%duplicatedids)));
 }
 
 #########################################
@@ -570,7 +549,7 @@ sub get_all_text_unbalanced {
 	} elsif ($token->[0] eq 'E')  {
 	    $result.=$token->[2];
 	}
-	if ($result =~ /(.*)$tag(.*)/) {
+	if ($result =~ /(.*)\Q$tag\E(.*)/s) {
 	    #&Apache::lonnet::logthis('Got a winner with leftovers ::'.$2);
 	    #&Apache::lonnet::logthis('Result is :'.$1);
 	    $result=$1;
@@ -607,8 +586,16 @@ sub fix_ids_and_indices {
 	$content=join('',<$org>);
     }
 
-    my ($needsfixup,$maxid,$maxindex)=&get_max_ids_indices(\$content);
+    my ($needsfixup,$maxid,$maxindex,$duplicateids,@duplicatedids)=
+	&get_max_ids_indices(\$content);
 
+    print $logfile ("Got $needsfixup,$maxid,$maxindex,$duplicateids--".
+			   join(', ',@duplicatedids));
+    if ($duplicateids) {
+	print $logfile "Duplicate ID(s) exist, ".join(', ',@duplicatedids)."\n";
+	my $outstring='<font color="red">Unable to publish file, it contains duplicated ID(s), ID(s) need to be unique. The duplicated ID(s) are: '.join(', ',@duplicatedids).'</font>';
+	return ($outstring,1);
+    }
     if ($needsfixup) {
 	print $logfile "Needs ID and/or index fixup\n".
 	    "Max ID   : $maxid (min 10)\n".
@@ -732,7 +719,7 @@ sub fix_ids_and_indices {
 	print $logfile "Does not need ID and/or index fixup\n";
     }
 
-    return ($outstring,%allow);
+    return ($outstring,0,%allow);
 }
 
 #########################################
@@ -802,6 +789,10 @@ This is the workhorse function of this m
 backup copies, performs any automatic processing (prior to publication,
 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
+(no error occurred)
+
 I<Additional documentation needed.>
 
 =cut
@@ -818,13 +809,12 @@ sub publish {
     my %allow=();
 
     unless ($logfile=Apache::File->new('>>'.$source.'.log')) {
-	return 
-         '<font color=red>No write permission to user directory, FAIL</font>';
+	return ('<font color="red">No write permission to user directory, FAIL</font>',1);
     }
     print $logfile 
 "\n\n================= Publish ".localtime()." Phase One  ================\n";
 
-    if (($style eq 'ssi') || ($style eq 'rat')) {
+    if (($style eq 'ssi') || ($style eq 'rat') || ($style eq 'prv')) {
 # ------------------------------------------------------- This needs processing
 
 # ----------------------------------------------------------------- Backup Copy
@@ -833,12 +823,14 @@ 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>";
+	    return ("<font color=\"red\">Failed to write backup copy, $!,FAIL</font>",1);
         }
 # ------------------------------------------------------------- IDs and indices
 	
-	my $outstring;
-	($outstring,%allow)=&fix_ids_and_indices($logfile,$source,$target);
+	my ($outstring,$error);
+	($outstring,$error,%allow)=&fix_ids_and_indices($logfile,$source,
+							$target);
+	if ($error) { return ($outstring,$error); }
 # ------------------------------------------------------------ Construct Allows
     
 	$scrout.='<h3>Dependencies</h3>';
@@ -883,9 +875,8 @@ sub publish {
           my $org;
           unless ($org=Apache::File->new('>'.$source)) {
              print $logfile "No write permit to $source\n";
-             return 
-		 '<font color="red">No write permission to '.$source.
-		 ', FAIL</font>';
+             return ('<font color="red">No write permission to '.$source.
+		     ', FAIL</font>',1);
 	  }
           print($org $outstring);
         }
@@ -955,7 +946,7 @@ sub publish {
     }
 
 # -------------------------------------------------- Parse content for metadata
-    if ($style eq 'ssi') {
+    if (($style eq 'ssi') || ($style eq 'prv')) {
         my $oldenv=$ENV{'request.uri'};
 
         $ENV{'request.uri'}=$target;
@@ -966,90 +957,87 @@ sub publish {
     }
 # ---------------- Find and document discrepancies in the parameters and stores
 
-        my $chparms='';
-        foreach (sort keys %metadatafields) {
-	    if (($_=~/^parameter/) || ($_=~/^stores/)) {
-                unless ($_=~/\.\w+$/) { 
-                   unless ($oldparmstores{$_}) {
-		      print $logfile 'New: '.$_."\n";
-                      $chparms.=$_.' ';
-                   }
-	        }
-            }
-        }
-        if ($chparms) {
-	    $scrout.='<p><b>New parameters or stored values:</b> '.
-                     $chparms;
-        }
+    my $chparms='';
+    foreach (sort keys %metadatafields) {
+	if (($_=~/^parameter/) || ($_=~/^stores/)) {
+	    unless ($_=~/\.\w+$/) { 
+		unless ($oldparmstores{$_}) {
+		    print $logfile 'New: '.$_."\n";
+		    $chparms.=$_.' ';
+		}
+	    }
+	}
+    }
+    if ($chparms) {
+	$scrout.='<p><b>New parameters or stored values:</b> '.$chparms;
+    }
 
-        $chparms='';
-        foreach (sort keys %oldparmstores) {
-	    if (($_=~/^parameter/) || ($_=~/^stores/)) {
-                unless (($metadatafields{$_.'.name'}) ||
-                        ($metadatafields{$_.'.package'}) || ($_=~/\.\w+$/)) {
-		    print $logfile 'Obsolete: '.$_."\n";
-                    $chparms.=$_.' ';
-                }
-            }
-        }
-        if ($chparms) {
-	    $scrout.='<p><b>Obsolete parameters or stored values:</b> '.
-                     $chparms;
-        }
+    $chparms='';
+    foreach (sort keys %oldparmstores) {
+	if (($_=~/^parameter/) || ($_=~/^stores/)) {
+	    unless (($metadatafields{$_.'.name'}) ||
+		    ($metadatafields{$_.'.package'}) || ($_=~/\.\w+$/)) {
+		print $logfile 'Obsolete: '.$_."\n";
+		$chparms.=$_.' ';
+	    }
+	}
+    }
+    if ($chparms) {
+	$scrout.='<p><b>Obsolete parameters or stored values:</b> '.
+	    $chparms;
+    }
 
 # ------------------------------------------------------- Now have all metadata
 
-        my %keywords=();
+    my %keywords=();
         
-	if (length($content)<500000) {
-	    my $textonly=$content;
-            $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;
-                } 
-            }
-        }
+    if (length($content)<500000) {
+	my $textonly=$content;
+	$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;
+	    } 
+	}
+    }
 
             
-            foreach (split(/\W+/,$metadatafields{'keywords'})) {
-		$keywords{$_}=1;
-            }
+    foreach (split(/\W+/,$metadatafields{'keywords'})) {
+	$keywords{$_}=1;
+    }
 # --------------------------------------------------- Now we also have keywords
 # =============================================================================
 # INTERACTIVE MODE
 #
-   unless ($batch) {
+    unless ($batch) {
         $scrout.=
-     '<form name="pubform" action="/adm/publish" method="post">'.
-       '<p><input type="submit" value="Finalize Publication" /></p>'.
-          &hiddenfield('phase','two').
-          &hiddenfield('filename',$ENV{'form.filename'}).
-	  &hiddenfield('allmeta',&Apache::lonnet::escape($allmeta)).
-          &hiddenfield('dependencies',join(',',keys %allow)).
-          &textfield('Title','title',$metadatafields{'title'}).
-          &textfield('Author(s)','author',$metadatafields{'author'}).
-	  &textfield('Subject','subject',$metadatafields{'subject'});
+	    '<form name="pubform" action="/adm/publish" method="post">'.
+            '<p><input type="submit" value="Finalize Publication" /></p>'.
+            &hiddenfield('phase','two').
+            &hiddenfield('filename',$ENV{'form.filename'}).
+	    &hiddenfield('allmeta',&Apache::lonnet::escape($allmeta)).
+            &hiddenfield('dependencies',join(',',keys %allow)).
+            &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)
-{
+function checkAll(field) {
     for (i = 0; i < field.length; i++)
         field[i].checked = true ;
 }
 
-function uncheckAll(field)
-{
+function uncheckAll(field) {
     for (i = 0; i < field.length; i++)
         field[i].checked = false ;
 }
@@ -1059,110 +1047,116 @@ function uncheckAll(field)
 <input type="button" value="uncheck all" onclick="javascript:uncheckAll(document.pubform.keywords)"> 
 <br />
 END
-        $keywordout.='<table border=2><tr>';
-        my $colcount=0;
+	$keywordout.='<table border=2><tr>';
+	my $colcount=0;
+
+	foreach (sort keys %keywords) {
+	    $keywordout.='<td><input type=checkbox name="keywords" value="'.$_.'"';
+	    if ($metadatafields{'keywords'}) {
+		if ($metadatafields{'keywords'}=~/$_/) {
+		    $keywordout.=' checked';
+		}
+	    } elsif (&Apache::loncommon::keyword($_)) {
+		$keywordout.=' checked';
+	    }
+	    $keywordout.='>'.$_.'</td>';
+	    if ($colcount>10) {
+		$keywordout.="</tr><tr>\n";
+		$colcount=0;
+	    }
+	    $colcount++;
+	}
 
-            foreach (sort keys %keywords) {
-                $keywordout.='<td><input type=checkbox name="keywords" value="'.$_.'"';
-                if ($metadatafields{'keywords'}) {
-                   if ($metadatafields{'keywords'}=~/$_/) { 
-                      $keywordout.=' checked'; 
-                   }
-	        } elsif (&Apache::loncommon::keyword($_)) {
-	            $keywordout.=' checked';
-                } 
-                $keywordout.='>'.$_.'</td>';
-                if ($colcount>10) {
-		    $keywordout.="</tr><tr>\n";
-                    $colcount=0;
-                }
-                $colcount++;
-            }
-        
 	$keywordout.='</tr></table>';
 
-        $scrout.=$keywordout;
+	$scrout.=$keywordout;
 
-        $scrout.=&textfield('Additional Keywords','addkey','');
+	$scrout.=&textfield('Additional Keywords','addkey','');
 
-        $scrout.=&textfield('Notes','notes',$metadatafields{'notes'});
+	$scrout.=&textfield('Notes','notes',$metadatafields{'notes'});
 
-        $scrout.=
-             '<p><b>Abstract:</b><br><textarea cols=80 rows=5 name=abstract>'.
-              $metadatafields{'abstract'}.'</textarea>';
+	$scrout.=
+	    '<p><b>Abstract:</b><br><textarea cols=80 rows=5 name=abstract>'.
+	    $metadatafields{'abstract'}.'</textarea>';
 
 	$source=~/\.(\w+)$/;
 
 	$scrout.=&hiddenfield('mime',$1);
 
-        $scrout.=&selectbox('Language','language',
-                            $metadatafields{'language'},
+	$scrout.=&selectbox('Language','language',
+			    $metadatafields{'language'},
 			    \&Apache::loncommon::languagedescription,
 			    (&Apache::loncommon::languageids),
-			     );
+			   );
 
-        unless ($metadatafields{'creationdate'}) {
+	unless ($metadatafields{'creationdate'}) {
 	    $metadatafields{'creationdate'}=time;
-        }
-        $scrout.=&hiddenfield('creationdate',$metadatafields{'creationdate'});
+	}
+	$scrout.=&hiddenfield('creationdate',
+			      &Apache::loncommon::unsqltime($metadatafields{'creationdate'}));
+
+	$scrout.=&hiddenfield('lastrevisiondate',time);
 
-        $scrout.=&hiddenfield('lastrevisiondate',time);
 
-			   
 	$scrout.=&textfield('Publisher/Owner','owner',
-                            $metadatafields{'owner'});
+			    $metadatafields{'owner'});
 
 # -------------------------------------------------- Correct copyright for rat.
-    if ($style eq 'rat') {
-	if ($metadatafields{'copyright'} eq 'public') { 
-	    delete $metadatafields{'copyright'};
+	unless ($style eq 'prv') {
+	    if ($style eq 'rat') {
+		if ($metadatafields{'copyright'} eq 'public') { 
+		    delete $metadatafields{'copyright'};
+		}
+		$scrout.=&selectbox('Copyright/Distribution','copyright',
+				    $metadatafields{'copyright'},
+				    \&Apache::loncommon::copyrightdescription,
+				    (grep !/^public$/,(&Apache::loncommon::copyrightids)));
+	    } else {
+		$scrout.=&selectbox('Copyright/Distribution','copyright',
+				    $metadatafields{'copyright'},
+				    \&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;
+	} else {
+	    $scrout.=&hiddenfield('copyright','private');
 	}
-        $scrout.=&selectbox('Copyright/Distribution','copyright',
-                            $metadatafields{'copyright'},
-			    \&Apache::loncommon::copyrightdescription,
-		     (grep !/^public$/,(&Apache::loncommon::copyrightids)));
-    }
-    else {
-        $scrout.=&selectbox('Copyright/Distribution','copyright',
-                            $metadatafields{'copyright'},
-			    \&Apache::loncommon::copyrightdescription,
-			     (&Apache::loncommon::copyrightids));
-    }
-
-    my $copyright_help =
-        Apache::loncommon::help_open_topic('Publishing_Copyright');
-    $scrout =~ s/DISTRIBUTION:/'DISTRIBUTION: ' . $copyright_help/ge;
-    return $scrout.
-        '<p><input type="submit" value="Finalize Publication" /></p></form>';
+	return ($scrout.'<p><input type="submit" value="Finalize Publication" /></p></form>',0);
 # =============================================================================
 # BATCH MODE
 #
-  } else {
+    } else {
 # Transfer metadata directly to environment for stage 2
-    foreach (keys %metadatafields) {
-	$ENV{'form.'.$_}=$metadatafields{$_};
+	foreach (keys %metadatafields) {
+	    $ENV{'form.'.$_}=$metadatafields{$_};
+	}
+	$ENV{'form.addkey'}='';
+	$ENV{'form.keywords'}='';
+	foreach (keys %keywords) {
+	    if ($metadatafields{'keywords'}) {
+		if ($metadatafields{'keywords'}=~/$_/) { 
+		    $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);
     }
-    $ENV{'form.addkey'}='';
-    $ENV{'form.keywords'}='';
-    foreach (keys %keywords) {
-        if ($metadatafields{'keywords'}) {
-           if ($metadatafields{'keywords'}=~/$_/) { 
-              $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;
-  }
 }
 
 #########################################
@@ -1196,249 +1190,271 @@ Returns:
 =item Scalar string
 
 String contains status (errors and warnings) and information associated with
-the server's attempts at publication.
+the server's attempts at publication.     
 
 =cut
 
+#'stupid emacs
 #########################################
 #########################################
 sub phasetwo {
 
-    my ($source,$target,$style,$distarget,$batch)=@_;
+    my ($r,$source,$target,$style,$distarget,$batch)=@_;
+    $source=~s/\/+/\//g;
+    $target=~s/\/+/\//g;
+
+    if ($target=~/\_\_\_/) {
+	$r->print(
+ '<font color="red">Unsupported character combination "<tt>___</tt>" in filename, FAIL</font>');
+        return 0;
+    }
+    $distarget=~s/\/+/\//g;
     my $logfile;
-    my $scrout='';
     unless ($logfile=Apache::File->new('>>'.$source.'.log')) {
-	return 
-         '<font color=red>No write permission to user directory, FAIL</font>';
+	$r->print(
+        '<font color="red">No write permission to user directory, FAIL</font>');
+        return 0;
     }
     print $logfile 
-"\n================= Publish ".localtime()." Phase Two  ================\n";
-
-     %metadatafields=();
-     %metadatakeys=();
-
-     &metaeval(&Apache::lonnet::unescape($ENV{'form.allmeta'}));
-
-     $metadatafields{'title'}=$ENV{'form.title'};
-     $metadatafields{'author'}=$ENV{'form.author'};
-     $metadatafields{'subject'}=$ENV{'form.subject'};
-     $metadatafields{'notes'}=$ENV{'form.notes'};
-     $metadatafields{'abstract'}=$ENV{'form.abstract'};
-     $metadatafields{'mime'}=$ENV{'form.mime'};
-     $metadatafields{'language'}=$ENV{'form.language'};
-     $metadatafields{'creationdate'}=
-         &sqltime($ENV{'form.creationdate'});
-     $metadatafields{'lastrevisiondate'}=
-         &sqltime($ENV{'form.lastrevisiondate'});
-     $metadatafields{'owner'}=$ENV{'form.owner'};
-     $metadatafields{'copyright'}=$ENV{'form.copyright'};
-     $metadatafields{'dependencies'}=$ENV{'form.dependencies'};
-
-     my $allkeywords=$ENV{'form.addkey'};
-     if (exists($ENV{'form.keywords'})) {
-         if (ref($ENV{'form.keywords'})) {
-             $allkeywords .= ','.join(',',@{$ENV{'form.keywords'}});
-         } else {
-             $allkeywords .= ','.$ENV{'form.keywords'};
-         }
-     }
-     $allkeywords=~s/\W+/\,/;
-     $allkeywords=~s/^\,//;
-     $metadatafields{'keywords'}=$allkeywords;
- 
-     {
-       print $logfile "\nWrite metadata file for ".$source;
-       my $mfh;
-       unless ($mfh=Apache::File->new('>'.$source.'.meta')) {
-	return 
-         '<font color=red>Could not write metadata, FAIL</font>';
-       }
-       foreach (sort keys %metadatafields) {
-	 unless ($_=~/\./) {
-           my $unikey=$_;
-           $unikey=~/^([A-Za-z]+)/;
-           my $tag=$1;
-           $tag=~tr/A-Z/a-z/;
-           print $mfh "\n\<$tag";
-           foreach (split(/\,/,$metadatakeys{$unikey})) {
-               my $value=$metadatafields{$unikey.'.'.$_};
-               $value=~s/\"/\'\'/g;
-               print $mfh ' '.$_.'="'.$value.'"';
-           }
-	   print $mfh '>'.
-	     &HTML::Entities::encode($metadatafields{$unikey})
-	       .'</'.$tag.'>';
-         }
-       }
-       $scrout.='<p>Wrote Metadata';
-       print $logfile "\nWrote metadata";
-     }
-
+        "\n================= Publish ".localtime()." Phase Two  ================\n";
+    
+    %metadatafields=();
+    %metadatakeys=();
+    
+    &metaeval(&Apache::lonnet::unescape($ENV{'form.allmeta'}));
+    
+    $metadatafields{'title'}=$ENV{'form.title'};
+    $metadatafields{'author'}=$ENV{'form.author'};
+    $metadatafields{'subject'}=$ENV{'form.subject'};
+    $metadatafields{'notes'}=$ENV{'form.notes'};
+    $metadatafields{'abstract'}=$ENV{'form.abstract'};
+    $metadatafields{'mime'}=$ENV{'form.mime'};
+    $metadatafields{'language'}=$ENV{'form.language'};
+    $metadatafields{'creationdate'}=$ENV{'form.creationdate'};
+    $metadatafields{'lastrevisiondate'}=$ENV{'form.lastrevisiondate'};
+    $metadatafields{'owner'}=$ENV{'form.owner'};
+    $metadatafields{'copyright'}=$ENV{'form.copyright'};
+    $metadatafields{'customdistributionfile'}=
+                                 $ENV{'form.customdistributionfile'};
+    $metadatafields{'dependencies'}=$ENV{'form.dependencies'};
+    
+    my $allkeywords=$ENV{'form.addkey'};
+    if (exists($ENV{'form.keywords'})) {
+        if (ref($ENV{'form.keywords'})) {
+            $allkeywords .= ','.join(',',@{$ENV{'form.keywords'}});
+        } else {
+            $allkeywords .= ','.$ENV{'form.keywords'};
+        }
+    }
+    $allkeywords=~s/\W+/\,/;
+    $allkeywords=~s/^\,//;
+    $metadatafields{'keywords'}=$allkeywords;
+    
+    {
+        print $logfile "\nWrite metadata file for ".$source;
+        my $mfh;
+        unless ($mfh=Apache::File->new('>'.$source.'.meta')) {
+            return 
+                '<font color="red">Could not write metadata, FAIL</font>';
+        }
+        foreach (sort keys %metadatafields) {
+            unless ($_=~/\./) {
+                my $unikey=$_;
+                $unikey=~/^([A-Za-z]+)/;
+                my $tag=$1;
+                $tag=~tr/A-Z/a-z/;
+                print $mfh "\n\<$tag";
+                foreach (split(/\,/,$metadatakeys{$unikey})) {
+                    my $value=$metadatafields{$unikey.'.'.$_};
+                    $value=~s/\"/\'\'/g;
+                    print $mfh ' '.$_.'="'.$value.'"';
+                }
+                print $mfh '>'.
+                    &HTML::Entities::encode($metadatafields{$unikey})
+                        .'</'.$tag.'>';
+            }
+        }
+        $r->print('<p>Wrote Metadata');
+        print $logfile "\nWrote metadata";
+    }
+    
 # -------------------------------- Synchronize entry with SQL metadata database
-    my $warning;
+
     $metadatafields{'url'} = $distarget;
     $metadatafields{'version'} = 'current';
     unless ($metadatafields{'copyright'} eq 'priv') {
         my ($error,$success) = &store_metadata(\%metadatafields);
         if ($success) {
-            $scrout.='<p>Synchronized SQL metadata database';
+            $r->print('<p>Synchronized SQL metadata database');
             print $logfile "\nSynchronized SQL metadata database";
         } else {
-            $warning.=$error;
+            $r->print($error);
             print $logfile "\n".$error;
         }
     } else {
-        $scrout.='<p>Private Publication - did not synchronize database';
+        $r->print('<p>Private Publication - did not synchronize database');
         print $logfile "\nPrivate: Did not synchronize data into ".
             "SQL metadata database";
     }
 # ----------------------------------------------------------- Copy old versions
    
-if (-e $target) {
-    my $filename;
-    my $maxversion=0;
-    $target=~/(.*)\/([^\/]+)\.(\w+)$/;
-    my $srcf=$2;
-    my $srct=$3;
-    my $srcd=$1;
-    unless ($srcd=~/^\/home\/httpd\/html\/res/) {
-	print $logfile "\nPANIC: Target dir is ".$srcd;
-        return "<font color=red>Invalid target directory, FAIL</font>";
-    }
-    opendir(DIR,$srcd);
-    while ($filename=readdir(DIR)) {
-      if (-l $srcd.'/'.$filename) {
-       unlink($srcd.'/'.$filename);
-       unlink($srcd.'/'.$filename.'.meta');
-      } else {
-       if ($filename=~/$srcf\.(\d+)\.$srct$/) {
-	   $maxversion=($1>$maxversion)?$1:$maxversion;
-       }
-      }
-    }
-    closedir(DIR);
-    $maxversion++;
-    $scrout.='<p>Creating old version '.$maxversion;
-    print $logfile "\nCreating old version ".$maxversion;
-
-    my $copyfile=$srcd.'/'.$srcf.'.'.$maxversion.'.'.$srct;
-
+    if (-e $target) {
+        my $filename;
+        my $maxversion=0;
+        $target=~/(.*)\/([^\/]+)\.(\w+)$/;
+        my $srcf=$2;
+        my $srct=$3;
+        my $srcd=$1;
+        unless ($srcd=~/^\/home\/httpd\/html\/res/) {
+            print $logfile "\nPANIC: Target dir is ".$srcd;
+            return "<font color=\"red\">Invalid target directory, FAIL</font>";
+        }
+        opendir(DIR,$srcd);
+        while ($filename=readdir(DIR)) {
+            if (-l $srcd.'/'.$filename) {
+                unlink($srcd.'/'.$filename);
+                unlink($srcd.'/'.$filename.'.meta');
+            } else {
+                if ($filename=~/\Q$srcf\E\.(\d+)\.\Q$srct\E$/) {
+                    $maxversion=($1>$maxversion)?$1:$maxversion;
+                }
+            }
+        }
+        closedir(DIR);
+        $maxversion++;
+        $r->print('<p>Creating old version '.$maxversion);
+        print $logfile "\nCreating old version ".$maxversion;
+        
+        my $copyfile=$srcd.'/'.$srcf.'.'.$maxversion.'.'.$srct;
+        
         if (copy($target,$copyfile)) {
 	    print $logfile "Copied old target to ".$copyfile."\n";
-            $scrout.='<p>Copied old target file';
+            $r->print('<p>Copied old target file');
         } else {
 	    print $logfile "Unable to write ".$copyfile.':'.$!."\n";
-           return "<font color=red>Failed to copy old target, $!, FAIL</font>";
+            return "<font color=\"red\">Failed to copy old target, $!, FAIL</font>";
         }
-
+        
 # --------------------------------------------------------------- Copy Metadata
 
 	$copyfile=$copyfile.'.meta';
-
+        
         if (copy($target.'.meta',$copyfile)) {
 	    print $logfile "Copied old target metadata to ".$copyfile."\n";
-            $scrout.='<p>Copied old metadata';
+            $r->print('<p>Copied old metadata')
         } else {
 	    print $logfile "Unable to write metadata ".$copyfile.':'.$!."\n";
             if (-e $target.'.meta') {
-               return 
-       "<font color=red>Failed to write old metadata copy, $!, FAIL</font>";
+                return 
+                    "<font color=\"red\">Failed to write old metadata copy, $!, FAIL</font>";
 	    }
         }
-
-
-} else {
-    $scrout.='<p>Initial version';
-    print $logfile "\nInitial version";
-}
+        
+        
+    } else {
+        $r->print('<p>Initial version');
+        print $logfile "\nInitial version";
+    }
 
 # ---------------------------------------------------------------- Write Source
-	my $copyfile=$target;
-
-           my @parts=split(/\//,$copyfile);
-           my $path="/$parts[1]/$parts[2]/$parts[3]/$parts[4]";
-
-           my $count;
-           for ($count=5;$count<$#parts;$count++) {
-               $path.="/$parts[$count]";
-               if ((-e $path)!=1) {
-                   print $logfile "\nCreating directory ".$path;
-                   $scrout.='<p>Created directory '.$parts[$count];
-		   mkdir($path,0777);
-               }
-           }
-
-        if (copy($source,$copyfile)) {
-	    print $logfile "\nCopied original source to ".$copyfile."\n";
-            $scrout.='<p>Copied source file';
-        } else {
-	    print $logfile "\nUnable to write ".$copyfile.':'.$!."\n";
-            return "<font color=red>Failed to copy source, $!, FAIL</font>";
+    my $copyfile=$target;
+    
+    my @parts=split(/\//,$copyfile);
+    my $path="/$parts[1]/$parts[2]/$parts[3]/$parts[4]";
+    
+    my $count;
+    for ($count=5;$count<$#parts;$count++) {
+        $path.="/$parts[$count]";
+        if ((-e $path)!=1) {
+            print $logfile "\nCreating directory ".$path;
+            $r->print('<p>Created directory '.$parts[$count]);
+            mkdir($path,0777);
         }
-
+    }
+    
+    if (copy($source,$copyfile)) {
+        print $logfile "\nCopied original source to ".$copyfile."\n";
+        $r->print('<p>Copied source file');
+    } else {
+        print $logfile "\nUnable to write ".$copyfile.':'.$!."\n";
+        return "<font color=\"red\">Failed to copy source, $!, FAIL</font>";
+    }
+    
 # --------------------------------------------------------------- Copy Metadata
 
-        $copyfile=$copyfile.'.meta';
-
-        if (copy($source.'.meta',$copyfile)) {
-	    print $logfile "\nCopied original metadata to ".$copyfile."\n";
-            $scrout.='<p>Copied metadata';
-        } else {
-	    print $logfile "\nUnable to write metadata ".$copyfile.':'.$!."\n";
-            return 
-          "<font color=red>Failed to write metadata copy, $!, FAIL</font>";
-        }
-
+    $copyfile=$copyfile.'.meta';
+    
+    if (copy($source.'.meta',$copyfile)) {
+        print $logfile "\nCopied original metadata to ".$copyfile."\n";
+        $r->print('<p>Copied metadata');
+    } else {
+        print $logfile "\nUnable to write metadata ".$copyfile.':'.$!."\n";
+        return 
+            "<font color=\"red\">Failed to write metadata copy, $!, FAIL</font>";
+    }
+    $r->rflush;
 # --------------------------------------------------- Send update notifications
 
     my @subscribed=&get_subscribed_hosts($target);
     foreach my $subhost (@subscribed) {
-	$scrout.='<p>Notifying host '.$subhost.':';
+	$r->print('<p>Notifying host '.$subhost.':');$r->rflush;
 	print $logfile "\nNotifying host ".$subhost.':';
 	my $reply=&Apache::lonnet::critical('update:'.$target,$subhost);
-	$scrout.=$reply;
+	$r->print($reply.'<br />');$r->rflush;
 	print $logfile $reply;
     }
-
+    
 # ---------------------------------------- Send update notifications, meta only
 
     my @subscribedmeta=&get_subscribed_hosts("$target.meta");
     foreach my $subhost (@subscribedmeta) {
-	$scrout.='<p>Notifying host for metadata only '.$subhost.':';
+	$r->print('<p>Notifying host for metadata only '.$subhost.':');$r->rflush;
 	print $logfile "\nNotifying host for metadata only ".$subhost.':';
 	my $reply=&Apache::lonnet::critical('update:'.$target.'.meta',
 					    $subhost);
-	$scrout.=$reply;
+	$r->print($reply.'<br />');$r->rflush;
+	print $logfile $reply;
+    }
+    
+# --------------------------------------------------- Notify subscribed courses
+    my %courses=&coursedependencies($target);
+    my $now=time;
+    foreach (keys %courses) {
+	$r->print('<p>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.'<br />');$r->rflush;
 	print $logfile $reply;
     }
-
 # ------------------------------------------------ Provide link to new resource
-  unless ($batch) {
-    my $thisdistarget=$target;
-    $thisdistarget=~s/^$docroot//;
-
-    my $thissrc=$source;
-    $thissrc=~s/^\/home\/(\w+)\/public_html/\/priv\/$1/;
-
-    my $thissrcdir=$thissrc;
-    $thissrcdir=~s/\/[^\/]+$/\//;
-
-
-    return $warning.$scrout.
-      '<hr><a href="'.$thisdistarget.'"><font size="+2">'.
-      'View Published Version</font></a>'.
-      '<p><a href="'.$thissrc.'"><font size=+2>Back to Source</font></a>'.
-      '<p><a href="'.$thissrcdir.
-      '"><font size="+2">Back to Source Directory</font></a>';
-  } else {
-    return $warning.$scrout;
-  }
+    unless ($batch) {
+        my $thisdistarget=$target;
+        $thisdistarget=~s/^$docroot//;
+        
+        my $thissrc=$source;
+        $thissrc=~s/^\/home\/(\w+)\/public_html/\/priv\/$1/;
+        
+        my $thissrcdir=$thissrc;
+        $thissrcdir=~s/\/[^\/]+$/\//;
+        
+        
+        $r->print(
+           '<hr><a href="'.$thisdistarget.'"><font size="+2">'.
+           'View Published Version</font></a>'.
+           '<p><a href="'.$thissrc.'"><font size=+2>Back to Source</font></a>'.
+           '<p><a href="'.$thissrcdir.
+                   '"><font size="+2">Back to Source Directory</font></a>');
+    }
 }
 
 #########################################
 
 sub batchpublish {
     my ($r,$srcfile,$targetfile)=@_;
+    $srcfile=~s/\/+/\//g;
+    $targetfile=~s/\/+/\//g;
     my $thisdisfn=$srcfile;
     $thisdisfn=~s/\/home\/korte\/public_html\///;
     $srcfile=~s/\/+/\//g;
@@ -1462,12 +1478,16 @@ sub batchpublish {
 
 # phase one takes
 #  my ($source,$target,$style,$batch)=@_;
-    $r->print('<p>'.&publish($srcfile,$targetfile,$thisembstyle,1).'</p>');
+    my ($outstring,$error)=&publish($srcfile,$targetfile,$thisembstyle,1);
+    $r->print('<p>'.$outstring.'</p>');
 # phase two takes
 # my ($source,$target,$style,$distarget,batch)=@_;
 # $ENV{'form.allmeta'},$ENV{'form.title'},$ENV{'form.author'},...
-    $r->print(
-'<p>'.&phasetwo($srcfile,$targetfile,$thisembstyle,$thisdistarget,1).'</p>');
+    if (!$error) {
+	$r->print('<p>');
+	&phasetwo($r,$srcfile,$targetfile,$thisembstyle,$thisdistarget,1);
+	$r->print('</p>');
+    }
     return '';
 }
 
@@ -1475,10 +1495,12 @@ sub batchpublish {
 
 sub publishdirectory {
     my ($r,$fn,$thisdisfn)=@_;
+    $fn=~s/\/+/\//g;
+    $thisdisfn=~s/\/+/\//g;
     my $resdir=
-    $Apache::lonnet::perlvar{'lonDocRoot'}.'/res/'.$cudom.'/'.$cuname.
+    $Apache::lonnet::perlvar{'lonDocRoot'}.'/res/'.$cudom.'/'.$cuname.'/'.
       $thisdisfn;
-      $r->print('<h1>Directory <tt>'.$thisdisfn.'/</tt></h1>'.
+      $r->print('<h1>Directory <tt>'.$thisdisfn.'</tt></h1>'.
                 'Target: <tt>'.$resdir.'</tt><br />');
 
       my $dirptr=16384;		# Mask indicating a directory in stat.cmode.
@@ -1580,7 +1602,7 @@ sub handler {
 
 # -------------------------------------------------------------- Check filename
 
-  my $fn=$ENV{'form.filename'};
+  my $fn=&Apache::lonnet::unescape($ENV{'form.filename'});
 
   
   unless ($fn) { 
@@ -1666,6 +1688,8 @@ unless ($ENV{'form.phase'} eq 'two') {
 
   $r->print('<html><head><title>LON-CAPA Publishing</title></head>');
   $r->print(&Apache::loncommon::bodytag('Resource Publication'));
+
+
   my $thisfn=$fn;
 
   my $thistarget=$thisfn;
@@ -1702,20 +1726,18 @@ unless ($ENV{'form.phase'} eq 'two') {
       if (&Apache::loncommon::fileembstyle($thistype) eq 'ssi') {
           $r->print('<br /><a href="/adm/diff?filename=/~'.$cuname.'/'.
                     $thisdisfn.
-  	  '&versionone=priv" target="cat">Diffs with Current Version</a><p>');
+  	  '&versiontwo=priv" target="cat">Diffs with Current Version</a><p>');
       }
   
 # ------------------ Publishing from $thisfn to $thistarget with $thisembstyle.
 
        unless ($ENV{'form.phase'} eq 'two') {
-         $r->print(
-          '<hr />'.&publish($thisfn,$thistarget,$thisembstyle));
+	   my ($outstring,$error)=&publish($thisfn,$thistarget,$thisembstyle);
+	   $r->print('<hr />'.$outstring);
        } else {
-         $r->print(
-          '<hr />'.&phasetwo($thisfn,$thistarget,
-			     $thisembstyle,$thisdistarget)); 
-       }  
-
+           $r->print('<hr />');
+           &phasetwo($r,$thisfn,$thistarget,$thisembstyle,$thisdistarget); 
+       }
   }
   $r->print('</body></html>');