Changeset 24547 for main/trunk/greenstone2/perllib/plugins/MARCPlugin.pm

Timestamp:

2011-09-02T16:28:57+12:00 (13 years ago)

Author:

ak19

Message:

Added new abstract plugin MetadataRead that defines can_process_this_file_for_metadata that MetadataPlugin subclasses can inherit (if MetadataRead is listed first in the ISA inheritance list) and which will then override the one defined in BasePlugin. For now committing MARC, ISIS and OAIPlugins which now additionally inherit from MetadataRead. Other metadataPlugins also need to be committed.

File:

• main/trunk/greenstone2/perllib/plugins/MARCPlugin.pm (modified) (3 diffs)

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

@@ -28 +28 @@
 
 use SplitTextFile;
+use MetadataRead;
 
 use unicode;
@@ -36 +37 @@
 no strict 'refs'; # allow filehandles to be variables and viceversa
 
+# methods defined in superclasses that have the same signature take
+# precedence in the order given in the ISA list. We want MetaPlugins to
+# call MetadataRead's can_process_this_file_for_metadata(), rather than
+# calling BasePlugin's version of the same method, so list inherited 
+# superclasses in this order.
 sub BEGIN {
-    @MARCPlugin::ISA = ('SplitTextFile');
+    @MARCPlugin::ISA = ('MetadataRead', 'SplitTextFile'); 
     unshift (@INC, "$ENV{'GSDLHOME'}/perllib/cpan");
 }
@@ -84 +90 @@
     push(@{$hashArgOptLists->{"ArgList"}},@{$arguments});
     push(@{$hashArgOptLists->{"OptList"}},$options);
-
+ 
+    # this does nothing yet, but if member vars are ever added
+    # to MetadataRead, will need to do this anyway:
+    #new MetadataRead($pluginlist, $inputargs, $hashArgOptLists);
     my $self = new SplitTextFile($pluginlist, $inputargs, $hashArgOptLists);