Merge branch 'master' of ssh://git.ikiwiki.info/srv/git/ikiwiki.info
authorJoey Hess <joey@gnu.kitenet.net>
Tue, 21 Jul 2009 07:57:08 +0000 (09:57 +0200)
committerJoey Hess <joey@gnu.kitenet.net>
Tue, 21 Jul 2009 07:57:08 +0000 (09:57 +0200)
doc/bugs/map_fails_to_close_ul_element_for_empty_list.mdwn
doc/plugins/map/discussion.mdwn

index 940e56cab4651a26b815ee129b0477be67ed426d..e565b8035dad0ee543bab89fd74c402d16f0925b 100644 (file)
@@ -51,3 +51,30 @@ Patch:
 >>> The patch in [[map/discussion|plugins/map/discussion]] has the same
 >>> problem, but does suggest a simpler approach to solving it (bail out
 >>> early if the map has no items at all). --[[smcv]]
+
+>>>> Thanks for pointing out the problem. I guess this patch should solve it.
+>>>> --[[harishcm]]
+
+Patch:
+
+    --- /usr/local/share/perl/5.8.8/IkiWiki/Plugin/map.pm
+    +++ map.pm
+    @@ -80,7 +80,17 @@
+       my $indent=0;
+       my $openli=0;
+       my $addparent="";
+    -  my $map = "<div class='map'>\n<ul>\n";
+    +  my $map = "<div class='map'>\n";
+    +
+    +  # Return empty div if %mapitems is empty
+    +  if (!scalar(keys %mapitems)) {
+    +          $map .= "</div>\n";
+    +          return $map; 
+    +  } 
+    +  else { # continue populating $map
+    +          $map .= "<ul>\n";
+    +  }
+    +
+       foreach my $item (sort keys %mapitems) {
+               my @linktext = (length $mapitems{$item} ? (linktext => $mapitems{$item}) : ());
+               $item=~s/^\Q$common_prefix\E\///
index 78f6802a7d88e0b6fbc58cb14c0a71fbe141196c..2f7b140d6e6859369dd89d3fcf6a59ac63de7bc7 100644 (file)
@@ -44,3 +44,6 @@ that seems to work on the examples I tried. I am a beginner so please help me ou
 > This was also reported as [[bugs/map_fails_to_close_ul_element_for_empty_list]];
 > this patch is simpler than the one there, but has the same problem (it emits
 > `<ul></ul>`, which technically isn't valid HTML either). --[[smcv]]
+
+>> Thanks for the tip, I added another patch addressing the issue at
+>> [[bugs/map_fails_to_close_ul_element_for_empty_list]]. --[[harishcm]]