Fix typo in default html head rss line, and adjust tests.

[matthijs/upstream/blosxom.git] / blosxom.cgi
diff --git a/blosxom.cgi b/blosxom.cgi

index 7c669dde565a2798204ff7179dd75d1183c54388..638830fe39a4a966c9c59902d44699b9c9097375 100755 (executable)
--- a/blosxom.cgi
+++ b/blosxom.cgi
@@ -251,13 +251,19 @@ my @plugin_list = ();
  my %plugin_hash = ();
  
  # If $plugin_list is set, read plugins to use from that file
-if ( $plugin_list and -r $plugin_list and $fh->open("< $plugin_list") ) {
-    @plugin_list = map { chomp $_; $_ } grep { /\S/ && !/^#/ } <$fh>;
-    $fh->close;
+if ( $plugin_list ) {
+    if ( -r $plugin_list and $fh->open("< $plugin_list") ) {
+        @plugin_list = map { chomp $_; $_ } grep { /\S/ && !/^#/ } <$fh>;
+        $fh->close;
+    }
+    else {
+        warn "unable to read or open plugin_list '$plugin_list': $!";
+        $plugin_list = '';
+    }
  }
  
  # Otherwise walk @plugin_dirs to get list of plugins to use
-elsif (@plugin_dirs) {
+if ( ! @plugin_list && @plugin_dirs ) {
      for my $plugin_dir (@plugin_dirs) {
          next unless -d $plugin_dir;
          if ( opendir PLUGINS, $plugin_dir ) {
@@ -283,18 +289,21 @@ elsif (@plugin_dirs) {
  unshift @INC, @plugin_dirs;
  foreach my $plugin (@plugin_list) {
      my ( $plugin_name, $off ) = $plugin =~ /^\d*([\w:]+?)(_?)$/;
+    my $plugin_file = $plugin_list ? $plugin_name : $plugin;
      my $on_off = $off eq '_' ? -1 : 1;
  
      # Allow perl module plugins
-    if ( $plugin =~ m/::/ && -z $plugin_hash{$plugin} ) {
+    # The -z test is a hack to allow a zero-length placeholder file in a 
+    #   $plugin_path directory to indicate an @INC module should be loaded
+    if ( $plugin =~ m/::/ && ( $plugin_list || -z $plugin_hash{$plugin} ) ) {
  
       # For Blosxom::Plugin::Foo style plugins, we need to use a string require
-        eval "require $plugin_name";
+        eval "require $plugin_file";
      }
      else
      { # we try first to load from $plugin_dir before attempting from $plugin_path
-        eval        { require "$plugin_dir/$plugin" }
-            or eval { require $plugin };
+        eval        { require "$plugin_dir/$plugin_file" }
+            or eval { require $plugin_file };
      }
  
      if ($@) {
@@ -717,7 +726,7 @@ html content_type text/html; charset=$blog_encoding
  html head <html>
  html head     <head>
  html head         <meta http-equiv="content-type" content="text/html;charset=$blog_encoding" />
-html head         <link rel="alternate" type="type="application/rss+xml" title="RSS" href="$url/index.rss" />
+html head         <link rel="alternate" type="application/rss+xml" title="RSS" href="$url/index.rss" />
  html head         <title>$blog_title $path_info_da $path_info_mo $path_info_yr
  html head         </title>
  html head     </head>