--- loncom/publisher/lonpublisher.pm	2001/05/08 01:26:48	1.27
+++ loncom/publisher/lonpublisher.pm	2001/08/11 18:51:40	1.35
@@ -10,6 +10,8 @@
 # 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 Gerd Kortemeyer
 
 package Apache::lonpublisher;
 
@@ -47,9 +49,15 @@ sub metaeval {
            if ($token->[0] eq 'S') {
 	      my $entry=$token->[1];
               my $unikey=$entry;
+              if (defined($token->[2]->{'package'})) { 
+                  $unikey.='_package_'.$token->[2]->{'package'};
+              } 
               if (defined($token->[2]->{'part'})) { 
                  $unikey.='_'.$token->[2]->{'part'}; 
 	      }
+              if (defined($token->[2]->{'id'})) { 
+                  $unikey.='_'.$token->[2]->{'id'};
+              } 
               if (defined($token->[2]->{'name'})) { 
                  $unikey.='_'.$token->[2]->{'name'}; 
 	      }
@@ -125,6 +133,26 @@ sub selectbox {
 
 # -------------------------------------------------------- Publication Step One
 
+sub makeallowed {
+}
+
+sub urlfixup {
+    my ($url,$target)=@_;
+    my ($host)=($url=~/(?:http\:\/\/)*([^\/]+)/);
+    map {
+	if ($_ eq $host) {
+	    $url=~s/^http\:\/\///;
+            $url=~s/^$host//;
+        }
+    } values %Apache::lonnet::hostname;
+    $url=~s/\~$cuname/res\/$cudom\/$cuname/;
+    if ($target) {
+	$target=~s/\/[^\/]+$//;
+       $url=&Apache::lonnet::hreflocation($target,$url);
+    }
+    return $url;
+}
+
 sub publish {
 
     my ($source,$target,$style)=@_;
@@ -192,42 +220,62 @@ sub publish {
           print $logfile "Needs ID and/or index fixup\n".
 	        "Max ID   : $maxid (min 10)\n".
                 "Max Index: $maxindex (min 10)\n";
-
+      }
           my $outstring='';
           my $parser=HTML::TokeParser->new(\$content);
           my $token;
           while ($token=$parser->get_token) {
               if ($token->[0] eq 'S') {
-                  my $counter;
-		  if ($counter=$addid{$token->[1]}) {
+                my $counter;
+                my $tag=$token->[1];
+                unless ($tag eq 'allow') {  
+                  my %parms=%{$token->[2]};
+		  if ($counter=$addid{$tag}) {
 		      if ($counter eq 'id') {
-			  if (defined($token->[2]->{'id'})) {
-			      $outstring.=$token->[4];
-			  } else {
+			  unless (defined($parms{'id'})) {
                               $maxid++;
-                              my $thisid=' id="'.$maxid.'"';
-			      my $fixup=$token->[4];
-                              $fixup=~s/(\<\w+)/$1$thisid/;
-                              $outstring.=$fixup;
-                              print $logfile 'ID: '.$fixup."\n";
+                              $parms{'id'}=$maxid;
+                              print $logfile 'ID: '.$tag.':'.$maxid."\n";
                           }
-                      } else {
- 			  if (defined($token->[2]->{'index'})) {
-			      $outstring.=$token->[4];
-			  } else {
+                      } elsif ($counter eq 'index') {
+ 			  unless (defined($parms{'index'})) {
                               $maxindex++;
-                              my $thisindex=' index="'.$maxindex.'"';
-			      my $fixup=$token->[4];
-                              $fixup=~s/(\<\w+)/$1$thisindex/;
-                              $outstring.=$fixup;
-                              print $logfile 'Index: '.$fixup."\n";
+                              $parms{'index'}=$maxindex;
+                              print $logfile 'Index: '.$tag.':'.$maxindex."\n";
 			  }
 		      }
-		  } else {
-                      $outstring.=$token->[4];
-                  }
+		  } 
+                  
+                  map {
+                      if (defined($parms{$_})) {
+			  my $oldurl=$parms{$_};
+                          my $newurl=&urlfixup($oldurl,$target);
+                          if ($newurl ne $oldurl) {
+			      $parms{$_}=$newurl;
+                              print $logfile 'URL: '.$tag.':'.$oldurl.' - '.
+				  $newurl."\n";
+			  }
+                          &makeallowed($newurl);
+                      }
+                  } ('src','href','codebase');
+
+                  my $newparmstring='';
+                  my $endtag='';
+                  map {
+                    if ($_ eq '/') {
+                      $endtag=' /';
+                    } else { 
+                      my $quote=($parms{$_}=~/\"/?"'":'"');
+                      $newparmstring.=' '.$_.'='.$quote.$parms{$_}.$quote;
+		    }
+                  } keys %parms;
+  
+		  $outstring.='<'.$tag.$newparmstring.$endtag.'>';
+	        }
               } elsif ($token->[0] eq 'E') {
-                  $outstring.=$token->[2];
+                  unless ($token->[1] eq 'allow') {
+                     $outstring.=$token->[2];
+		  }
               } else {
                   $outstring.=$token->[1];
               }
@@ -242,6 +290,8 @@ sub publish {
           print $org $outstring;
         }
 	  $content=$outstring;
+
+      if ($needsfixup) {
           print $logfile "End of ID and/or index fixup\n".
 	        "Max ID   : $maxid (min 10)\n".
                 "Max Index: $maxindex (min 10)\n";
@@ -308,6 +358,7 @@ sub publish {
 # -------------------------------------------------- Parse content for metadata
 
         $allmeta=Apache::lonxml::xmlparse('meta',$content);
+
         &metaeval($allmeta);
 
 # ---------------- Find and document discrepancies in the parameters and stores
@@ -331,7 +382,8 @@ sub publish {
         my $chparms='';
         map {
 	    if (($_=~/^parameter/) || ($_=~/^stores/)) {
-                unless (($metadatafields{$_.'.name'}) || ($_=~/\.\w+$/)) {
+                unless (($metadatafields{$_.'.name'}) ||
+                        ($metadatafields{$_.'.package'}) || ($_=~/\.\w+$/)) {
 		    print $logfile 'Obsolete: '.$_."\n";
                     $chparms.=$_.' ';
                 }
@@ -510,44 +562,55 @@ sub phasetwo {
     }
     close(CONFIG);
 
+    my $warning;
     my $dbh;
     {
 	unless (
 		$dbh = DBI->connect("DBI:mysql:loncapa","www",$perlvar{'lonSqlAccess'},{ RaiseError =>0,PrintError=>0})
 		) { 
-	    return '<font color=red>Cannot connect to database!</font>';
+	    $warning='<font color=red>WARNING: Cannot connect to '.
+		'database!</font>';
+	}
+	else {
+	    my %sqldatafields;
+	    $sqldatafields{'url'}=$distarget;
+	    my $sth=$dbh->prepare(
+				  'delete from metadata where url like binary'.
+				  '"'.$sqldatafields{'url'}.'"');
+	    $sth->execute();
+	    map {my $field=$metadatafields{$_}; $field=~s/\"/\'\'/g; 
+		 $sqldatafields{$_}=$field;}
+	    ('title','author','subject','keywords','notes','abstract',
+	     'mime','language','creationdate','lastrevisiondate','owner',
+	     'copyright');
+	    
+	    $sth=$dbh->prepare('insert into metadata values ('.
+			       '"'.delete($sqldatafields{'title'}).'"'.','.
+			       '"'.delete($sqldatafields{'author'}).'"'.','.
+			       '"'.delete($sqldatafields{'subject'}).'"'.','.
+			       '"'.delete($sqldatafields{'url'}).'"'.','.
+			       '"'.delete($sqldatafields{'keywords'}).'"'.','.
+			       '"'.'current'.'"'.','.
+			       '"'.delete($sqldatafields{'notes'}).'"'.','.
+			       '"'.delete($sqldatafields{'abstract'}).'"'.','.
+			       '"'.delete($sqldatafields{'mime'}).'"'.','.
+			       '"'.delete($sqldatafields{'language'}).'"'.','.
+			       '"'.
+			       sqltime(delete($sqldatafields{'creationdate'}))
+			       .'"'.','.
+			       '"'.
+			       sqltime(delete(
+			       $sqldatafields{'lastrevisiondate'})).'"'.','.
+			       '"'.delete($sqldatafields{'owner'}).'"'.','.
+			       '"'.delete(
+			       $sqldatafields{'copyright'}).'"'.')');
+	    $sth->execute();
+	    $dbh->disconnect;
+	    $scrout.='<p>Synchronized SQL metadata database';
+	    print $logfile "\nSynchronized SQL metadata database";
 	}
     }
 
-    my %sqldatafields;
-    $sqldatafields{'url'}=$distarget;
-    my $sth=$dbh->prepare("delete from metadata where url like binary \"".
-		       $sqldatafields{'url'}."\"");
-    $sth->execute();
-    map {my $field=$metadatafields{$_}; $field=~s/\"/\'\'/g; 
-	 $sqldatafields{$_}=$field;}
-    ('title','author','subject','keywords','notes','abstract',
-     'mime','language','creationdate','lastrevisiondate','owner','copyright');
-
-    $sth=$dbh->prepare('insert into metadata values ('.
-			  '"'.delete($sqldatafields{'title'}).'"'.','.
-			  '"'.delete($sqldatafields{'author'}).'"'.','.
-			  '"'.delete($sqldatafields{'subject'}).'"'.','.
-			  '"'.delete($sqldatafields{'url'}).'"'.','.
-			  '"'.delete($sqldatafields{'keywords'}).'"'.','.
-			  '"'.'current'.'"'.','.
-			  '"'.delete($sqldatafields{'notes'}).'"'.','.
-			  '"'.delete($sqldatafields{'abstract'}).'"'.','.
-			  '"'.delete($sqldatafields{'mime'}).'"'.','.
-			  '"'.delete($sqldatafields{'language'}).'"'.','.
-			  '"'.sqltime(delete($sqldatafields{'creationdate'})).'"'.','.
-			  '"'.sqltime(delete($sqldatafields{'lastrevisiondate'})).'"'.','.
-			  '"'.delete($sqldatafields{'owner'}).'"'.','.
-			  '"'.delete($sqldatafields{'copyright'}).'"'.')');
-    $sth->execute();
-    $dbh->disconnect;
-    $scrout.='<p>Synchronized SQL metadata database';
-    print $logfile "\nSynchronized SQL metadata database";
 
 # ----------------------------------------------------------- Copy old versions
    
@@ -706,7 +769,7 @@ if (-e $target) {
     $thissrcdir=~s/\/[^\/]+$/\//;
 
 
-    return $scrout.
+    return $warning.$scrout.
       '<hr><a href="'.$thisdistarget.'"><font size=+2>View Target</font></a>'.
       '<p><a href="'.$thissrc.'"><font size=+2>Back to Source</font></a>'.
       '<p><a href="'.$thissrcdir.
@@ -736,8 +799,9 @@ sub handler {
      return HTTP_NOT_FOUND;
   } 
 
-  unless (($cuname,$cudom)=
-    &Apache::loncacc::constructaccess($fn,$r->dir_config('lonDefDomain'))) {
+  ($cuname,$cudom)=
+    &Apache::loncacc::constructaccess($fn,$r->dir_config('lonDefDomain'));
+  unless (($cuname) && ($cudom)) {
      $r->log_reason($cuname.' at '.$cudom.
          ' trying to publish file '.$ENV{'form.filename'}.
          ' ('.$fn.') - not authorized', 
@@ -861,7 +925,8 @@ unless ($ENV{'form.phase'} eq 'two') {
       }
 
       if (&Apache::lonnet::fileembstyle($thistype) eq 'ssi') {
-          $r->print('<br><a href="/adm/diff?filename='.$thisdisfn.
+          $r->print('<br><a href="/adm/diff?filename=/~'.$cuname.'/'.
+                    $thisdisfn.
   	  '&versionone=priv" target=cat>Diffs with Current Version</a><p>');
       }