problem with the patch
authorJoey Hess <joey@kodama.kitenet.net>
Sat, 23 Aug 2008 16:13:38 +0000 (12:13 -0400)
committerJoey Hess <joey@kodama.kitenet.net>
Sat, 23 Aug 2008 16:13:38 +0000 (12:13 -0400)
doc/bugs/map_is_inconsistent_about_bare_directories.mdwn

index dc2479e73c42b38f1bf2685e8c25ff04ea5fa83a..c78a8c68e8360bab1957d62e84ed22383ef38407 100644 (file)
@@ -76,6 +76,16 @@ One solution could also use the [[plugins/autoindex]] plugin to make sure that p
 
 Note: This patch adds items to a map while it is in a foreach loop over a sorted list of keys from that same map.  Changing a map while iterating through it is normally problematic.  I'm assuming the sort insulates the code from this - I do not need to iterate over any of the newly added elements.
 
+> This patch causes a small bug in the case where the map includes pages in
+> two subdirs, and does not explicitly include either subdir. In that case,
+> the map is supposed to show both subdirs, but use special styling an no
+> link for either. With the patch, which otherwise seems to work ok, some
+> of the non-included subdirs are displayed as if included. (I
+> assume because the loop you added does in fact include them..)
+> 
+> Example of the problem: \[[!map pages="plugins/contrib/* or plugins/type/*"]]
+> --[[Joey]]
+
     diff --git a/IkiWiki/Plugin/map.pm b/IkiWiki/Plugin/map.pm
     index 5b6a843..16de45e 100644
     --- a/IkiWiki/Plugin/map.pm