Changeset 24403 for main/trunk/greenstone2/perllib/plugins/EmbeddedMetadataPlugin.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/EmbeddedMetadataPlugin.pm (modified) (2 diffs)

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

@@ -121 +121 @@
 #}
 
+# plugins that rely on more than process_exp (eg XML plugins) can override this method
+sub can_process_this_file {
+    my $self = shift(@_);
+
+    # we process metadata, not the file 
+    return 0;    
+}
+
+# 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(@_);
+
+    # this plugin will look for metadata in any file through its
+    # metadata_read() returning undef here means anything else further
+    # down the pipeline can do the same
+
+    return undef;
+}
+
 
 sub extractEmbeddedMetadata()
@@ -285 +309 @@
 }
 
+sub read
+{
+    return undef;
+}
 
 sub process()