--- loncom/publisher/lonpublisher.pm	2001/05/09 17:38:13	1.28
+++ loncom/publisher/lonpublisher.pm	2001/08/07 21:27:06	1.33
@@ -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 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'}; 
 	      }
@@ -308,6 +316,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 +340,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 +520,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 +727,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 +757,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',