Avoid "conditional and" for checking after plugins "start" and stuffing @plugins

[matthijs/upstream/blosxom.git] / blosxom.cgi

diff --git a/blosxom.cgi b/blosxom.cgi
index 0c483e9c4c5b17e7fc3a530c3a4cb7373664567d..c51ca2a9d2cc1f21b98988085c68a0380f22aa88 100755 (executable)
--- a/blosxom.cgi
+++ b/blosxom.cgi
@@ -250,8 +250,15 @@ foreach my $plugin ( @plugin_list ) {
   else {
     eval { require $plugin };
   }
-  $@ and warn "$@ error finding or loading blosxom plugin $plugin_name - skipping\n" and next;
-  $plugin_name->start() and ( $plugins{$plugin_name} = $on_off ) and push @plugins, $plugin_name;
+
+  if ($@) {
+      warn "error finding or loading blosxom plugin '$plugin_name': $@";
+      next;
+  }
+  if ( $plugin_name->start() and ( $plugins{$plugin_name} = $on_off ) ) {
+      push @plugins, $plugin_name;
+  }
+
 }
 shift @INC foreach @plugin_dirs;
 
@@ -388,7 +395,11 @@ else {
 }
 
 # Plugins: End
-foreach my $plugin ( @plugins ) { $plugins{$plugin} > 0 and $plugin->can('end') and $entries = $plugin->end() }
+foreach my $plugin (@plugins) {
+    if ( $plugins{$plugin} > 0 and $plugin->can('end') ) {
+        $entries = $plugin->end();
+    }
+}
 
 # Generate 
 sub generate {
@@ -500,7 +511,7 @@ sub generate {
       ($dw,$mo,$mo_num,$da,$ti,$yr,$utc_offset) = nice_date($files{"$path_file"});
       ($hr,$min) = split /:/, $ti;
       ($hr12, $ampm) = $hr >= 12 ? ($hr - 12,'pm') : ($hr, 'am'); 
-      $hr12 =~ s/^0//; $hr12 == 0 and $hr12 = 12;
+      $hr12 =~ s/^0//; if ($hr12 == 0) {$hr12 = 12};
   
       # Only stories from the right date
       my($path_info_yr,$path_info_mo_num, $path_info_da) = split /\//, $date;