aboutsummaryrefslogtreecommitdiff
path: root/doc/plugins/contrib
diff options
context:
space:
mode:
authorhttp://smcv.pseudorandom.co.uk/ <http://smcv.pseudorandom.co.uk/@web>2010-03-26 14:09:04 +0000
committerJoey Hess <joey@finch.kitenet.net>2010-03-26 14:09:04 +0000
commitb9958fe3cf1a27b58d3e2bd59c76553eb0046782 (patch)
tree957db169996f3ae9289a687b7431c265cc3de359 /doc/plugins/contrib
parentfffd7e7e026f61e03b2fa052acdf7130889f6343 (diff)
downloadikiwiki-b9958fe3cf1a27b58d3e2bd59c76553eb0046782.tar
ikiwiki-b9958fe3cf1a27b58d3e2bd59c76553eb0046782.tar.gz
brief review of report
Diffstat (limited to 'doc/plugins/contrib')
-rw-r--r--doc/plugins/contrib/report/discussion.mdwn23
1 files changed, 23 insertions, 0 deletions
diff --git a/doc/plugins/contrib/report/discussion.mdwn b/doc/plugins/contrib/report/discussion.mdwn
new file mode 100644
index 000000000..6b1e28f6a
--- /dev/null
+++ b/doc/plugins/contrib/report/discussion.mdwn
@@ -0,0 +1,23 @@
+Wow, this plugin does a lot... it seems to be `inline` (but without the feeds
+or the ability to not have `archive="yes"`), plus part of
+[[plugins/contrib/trail]], plus some sorting, plus an ingenious workaround
+for template evaluation being relatively stateless.
+
+A large part of this plugin would just fall off if one of the versions of
+[[todo/allow_add_plugins_to_add_sorting_methods]] was merged, which was a
+large part of the idea of that feature request :-) To make use of that
+you'd have to use `pagespec_match_list` in the trail case too, but that's
+easy enough - just add `list => [@the_trail_pages]` to the arguments.
+
+Another large part would fall off if this plugin required, and internally
+invoked, `inline` (like my `comments` plugin does) - `inline` runs
+`pagetemplate` hooks, and in particular, it'll run the `field` hook.
+Alternatively, this plugin could invoke `pagetemplate` hooks itself,
+removing the special case for `field`.
+
+Perhaps the `headers` thing could migrate into inline somehow? That might
+lead to making inline too big, though.
+
+Is the intention that the `trail` part is a performance hack, or a way
+to select pages? How does it relate to [[todo/wikitrails]] or
+[[plugins/contrib/trail]]? --[[smcv]]