Changeset 15918

Timestamp:

2008-06-09T13:36:30+12:00 (16 years ago)

Author:

kjdon

Message:

tidied up new method to match other plugins

Location:

gsdl/trunk/perllib/plugins

Files:

• AcronymExtractor.pm (modified) (1 diff)
• ExcelPlugin.pm (modified) (1 diff)
• UnknownPlugin.pm (modified) (1 diff)

gsdl/trunk/perllib/plugins/AcronymExtractor.pm

@@ -31 +31 @@
     push(@$pluginlist, $class);
 
-    if(defined $arguments){ push(@{$hashArgOptLists->{"ArgList"}},@{$arguments});}
-    if(defined $options) { push(@{$hashArgOptLists->{"OptList"}},$options)};
+    push(@{$hashArgOptLists->{"ArgList"}},@{$arguments});
+    push(@{$hashArgOptLists->{"OptList"}},$options);
 
     my $self = new PrintInfo($pluginlist, $inputargs, $hashArgOptLists,1);

gsdl/trunk/perllib/plugins/ExcelPlugin.pm

@@ -56 +56 @@
     push(@$pluginlist, $class);
 
-    if(defined $arguments){ push(@{$hashArgOptLists->{"ArgList"}},@{$arguments});}
-    if(defined $options) { push(@{$hashArgOptLists->{"OptList"}},$options)};
+    push(@{$hashArgOptLists->{"ArgList"}},@{$arguments});
+    push(@{$hashArgOptLists->{"OptList"}},$options);
     
     my $self = new ConvertBinaryFile($pluginlist, $inputargs, $hashArgOptLists);

gsdl/trunk/perllib/plugins/UnknownPlugin.pm

@@ -104 +104 @@
     push(@$pluginlist, $class);
 
-    if(defined $arguments){ push(@{$hashArgOptLists->{"ArgList"}},@{$arguments});}
-    if(defined $options) { push(@{$hashArgOptLists->{"OptList"}},$options)};
+    push(@{$hashArgOptLists->{"ArgList"}},@{$arguments});
+    push(@{$hashArgOptLists->{"OptList"}},$options);
 
     my $self = new BasePlugin($pluginlist, $inputargs, $hashArgOptLists);