Changeset 32511 for main/trunk/greenstone2/bin/script/plugoutinfo.pl

Timestamp:

2018-10-10T21:25:13+13:00 (6 years ago)

Author:

ak19

Message:

Running plugoutinfo.pl with describeall or listall flag would break on FedoraMETSPlugout when either FEDORA_HOME or FEDORA_VERSION aren't set (as is often the case), as there's a die statement in the BEGIN of FedoraMETSPlugout. Needed to run die if either FEDORA env var is not set only if the plugout is NOT in info_only mode in plugout constructor. However, info_only mode was never set in any of the plugouts, so had to add set up the infrastructure for it in plugoutinfo.pl and plugout.pm. Then added the info_only test to all teh plugouts, even though it's redundant in most of them for making sure future changes to any plugout's constructors does not break plugoutinfo.pl.

File:

• main/trunk/greenstone2/bin/script/plugoutinfo.pl (modified) (2 diffs)

main/trunk/greenstone2/bin/script/plugoutinfo.pl

@@ -177 +177 @@
         my $plugout;
         push @$plugout,$plugout_name; 
-        my $plugoutobj = &plugout::load_plugout ($plugout);
+        my $plugoutobj = &plugout::load_plugout ($plugout, "info_only");
         if ($describeall) {
         $plugoutobj->print_xml_usage(0);
@@ -202 +202 @@
     push @$plugout,$plugout_name;
 
-    my $plugoutobj = &plugout::load_plugout($plugout);
+    my $plugoutobj = &plugout::load_plugout($plugout, "info_only");
 
     if ($xml) {