--- loncom/metadata_database/searchcat.pl	2007/01/03 04:01:32	1.70.2.3
+++ loncom/metadata_database/searchcat.pl	2006/12/29 19:15:27	1.71
@@ -2,7 +2,7 @@
 # The LearningOnline Network
 # searchcat.pl "Search Catalog" batch script
 #
-# $Id: searchcat.pl,v 1.70.2.3 2007/01/03 04:01:32 albertel Exp $
+# $Id: searchcat.pl,v 1.71 2006/12/29 19:15:27 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -241,11 +241,11 @@ foreach my $dom (@domains) {
             my ($group,$url,$fullpath);
             if ($is_course) {
                 ($group, my ($path)) = ($file =~ /^(\w+)(\/.+)$/);
-                $fullpath = $pathstart.'/groups/'.$group.'/portfolio'.$path;
+                $fullpath = $pathstart.'/groups/'.$group.'/portfolio/'.$path;
                 $url = $urlstart.'/groups/'.$group.'/portfolio'.$path;
             } else {
                 $fullpath = $pathstart.'/portfolio'.$file;
-                $url = $urlstart.'/portfolio'.$file;
+                $url .= $urlstart.'/portfolio'.$file;
             }
             if (ref($access{$file}) eq 'HASH') {
                 &process_portfolio_access_data($url,$access{$file});
@@ -330,12 +330,11 @@ sub process_portfolio_access_data {
         $acc_data->{keynum} = $key;
         my ($num,$scope,$end,$start) =
                         ($key =~ /^([^:]+):([a-z]+)_(\d*)_?(\d*)$/);
-        next if (($scope ne 'public') && ($scope ne 'guest'));
         $acc_data->{scope} = $scope;
         if ($end != 0) {
-            $acc_data->{end} = &sqltime($end);
+            $acc_data->{end} = &LONCAPA::lonmetadata::sqltime($end);
         }
-        $acc_data->{start} = &sqltime($start);
+        $acc_data->{start} = &LONCAPA::lonmetadata::sqltime($start);
         if (! $simulate) {
             my ($count,$err) =
               &LONCAPA::lonmetadata::store_metadata($dbh,
@@ -613,7 +612,7 @@ sub portfolio_metadata {
         $metacache{'owner'} = $uname.':'.$dom;
         $metacache{'domain'} = $dom;
         $metacache{'mime'} = $mime;
-        if ($group ne '') {
+        if (defined($group)) {
             $metacache{'keys'} .= ',courserestricted';
             $metacache{'courserestricted'} = 'course.'.$dom.'_'.$uname;
         } 
@@ -693,8 +692,10 @@ sub getfiledates {
         $ref->{'lastrevisiondate'} =~ /^\s*$/) {
         $ref->{'lastrevisiondate'} = (stat($target))[9];
     }
-    $ref->{'creationdate'}     = &sqltime($ref->{'creationdate'});
-    $ref->{'lastrevisiondate'} = &sqltime($ref->{'lastrevisiondate'});
+    $ref->{'creationdate'}     = 
+        &LONCAPA::lonmetadata::sqltime($ref->{'creationdate'});
+    $ref->{'lastrevisiondate'} = 
+        &LONCAPA::lonmetadata::sqltime($ref->{'lastrevisiondate'});
 }
 
 ########################################################
@@ -889,41 +890,6 @@ sub propath {
 } 
 
 ##
-## &sqltime($timestamp)
-##
-## Convert perl $timestamp to MySQL time.  MySQL expects YYYY-MM-DD HH:MM:SS
-##
-sub sqltime {
-    my ($time) = @_;
-    my $mysqltime;
-    if ($time =~ 
-        /(\d+)-(\d+)-(\d+) # YYYY-MM-DD
-        \s                 # a space
-        (\d+):(\d+):(\d+)  # HH:MM::SS
-        /x ) { 
-        # Some of the .meta files have the time in mysql
-        # format already, so just make sure they are 0 padded and
-        # pass them back.
-        $mysqltime = sprintf('%04d-%02d-%02d %02d:%02d:%02d',
-                             $1,$2,$3,$4,$5,$6);
-    } elsif ($time =~ /^\d+$/) {
-        my @TimeData = gmtime($time);
-        # Alter the month to be 1-12 instead of 0-11
-        $TimeData[4]++;
-        # Alter the year to be from 0 instead of from 1900
-        $TimeData[5]+=1900;
-        $mysqltime = sprintf('%04d-%02d-%02d %02d:%02d:%02d',
-                             @TimeData[5,4,3,2,1,0]);
-    } elsif (! defined($time) || $time == 0) {
-        $mysqltime = 0;
-    } else {
-        &log(0,"    sqltime:Unable to decode time ".$time);
-        $mysqltime = 0;
-    }
-    return $mysqltime;
-}
-
-##
 ## &declutter($filename)
 ##   Given a filename, returns a url for the filename.
 sub declutter {