Changeset 24403 for main/trunk/greenstone2/perllib/plugins/BasePlugin.pm

Timestamp:

2011-08-12T18:10:11+12:00 (13 years ago)

Author:

ak19

Message:

Dr Bainbridge has fixed the conflict between OAIPlugin and EmbeddedMetadataPlugin which resulted in the oai tutorial (with the JCDL pictures) going wrong: meta was not attached to the images. Dr Bainbridge solved the problem by introducing a new method in BasePlugin: can_process_this_file_for_metadata, which by default returns undef so that things should work by default mostly. This method has been overridden in OAIPlugin and EmbeddedMetadataPlugin now to do the right thing there.

File:

• main/trunk/greenstone2/perllib/plugins/BasePlugin.pm (modified) (2 diffs)

main/trunk/greenstone2/perllib/plugins/BasePlugin.pm

@@ -528 +528 @@
 }
 
+# Even if a plugin can extract metadata in its metadata_read pass,
+# make the default return 'undef' so processing of the file continues
+# down the pipeline, so other plugins can also have the opportunity to
+# locate metadata and set it up in the extrametakeys variables that
+# are passed around.
+
+sub can_process_this_file_for_metadata {
+    my $self = shift(@_);
+
+    return undef;
+}
+
+
 # just converts path as is to utf8.
 sub filepath_to_utf8 {
@@ -1131 +1144 @@
     # can we process this file??
     my ($filename_full_path, $filename_no_path) = &util::get_full_filenames($base_dir, $file);
-    return undef unless $self->can_process_this_file($filename_full_path);
+    return undef unless $self->can_process_this_file_for_metadata($filename_full_path);
 
     return 1; # we recognise the file, but don't actually do anything with it