]> sipb.mit.edu Git - ikiwiki.git/blobdiff - doc/todo/else_parameter_for_map_plugin.mdwn
thinking loud
[ikiwiki.git] / doc / todo / else_parameter_for_map_plugin.mdwn
index e6cd1adc7d0c4bf34005358e8f34b51cdf7dc4bc..859b027e799e77db8f9a67ace35ac79ee53fa71c 100644 (file)
@@ -2,8 +2,36 @@
 
 [[plugins/map]] (and I) could benefit from a bonus parameter:
 
-       else="Display this if no page matches the [[ikiwiki/PageSpec]]"
+       else="Display this if no page matches the PageSpec"
 
 This was quite simple, so I implemented this (branch "map" in my
 ikiwiki repo, see my user page for the up-to-date URL). Not patched the
 documentation yet, I'm waiting for feedback first, but I'll do it for sure. -- [[intrigeri]]
+
+> Can't a [[plugins/conditional]] be for this?
+> --[[Joey]]
+
+>> Hmmm, what do you mean? Adding a syntax such as the one below?
+>> Or something else?
+
+        \[[if test="map(" then="..." else="..."]]
+
+>> What would you write in the `then` clause?
+>> I'm not opposed at all to rewrite my two-liner, but I don't understand.
+>> --[[intrigeri]]
+
+       \[[if  test="foo/*" then="""
+       [[map pages="foo/*"]]
+       """ else="no pages"]]
+
+--[[Joey]]
+
+>>> I'm not convinced: the syntax you're proposing implies to duplicate
+>>> the pagespec (once in the test clause, and once in the map query), which I find
+>>> not only inelegant, which I can live with, but also tiring and unpractical:
+>>> my `else` suggestion
+>>> finds its roots in map queries with rather long pagespecs. On the other
+>>> hand, if I'm the only one using map in such a way, I can live with this
+>>> heavy duplicated syntax without bloating the map plugin with features
+>>> no-one but me needs. On the other other hand, the patch is a 3-liner.
+>>> I'm not fixed yet, I'll think about it. --[[intrigeri]]