Changeset 19747

Show
Ignore:
Timestamp:
05.06.2009 17:14:11 (10 years ago)
Author:
ak19
Message:

Undoing yesterday's commit: Katherine explains that the document-prefix is not simply the path shared by documents, but may also end on the shared filename prefix, in which case we should not be forcing a slash at the end after all (where the user did not provide one).

Files:
1 modified

Legend:

Unmodified
Added
Removed
  • gsdl/trunk/bin/script/explode_metadata_database.pl

    r19742 r19747  
    405405        # Does this metadata element specify a document to obtain? 
    406406        if ($field eq $document_field) { 
    407         # if the document prefix provided didn't have the requisite slash at the end, we make sure it does now 
    408         # by first of all stripping any trailing slash and then always ensuring one is used through filename_cat 
    409         $document_prefix =~ s/(\/|\\)$//; 
    410         my $document_file_full = &util::filename_cat($document_prefix, "$value$document_suffix"); 
     407        # shouldn't force a slash at the end after all, since the doc prefix may already 
     408        # contain the prefix of the actual filename itself (that's shared over several docs)  
     409        my $document_file_full = $document_prefix . $value . $document_suffix; 
    411410 
    412411        # this either downloads/copies the document, or creates a nul file for it.