diff options
| author | Joey Hess <joey@kitenet.net> | 2010-07-04 15:15:34 -0400 |
|---|---|---|
| committer | Joey Hess <joey@kitenet.net> | 2010-07-04 15:15:34 -0400 |
| commit | 6160238439c5df335944add13b00a0a80d1b663b (patch) | |
| tree | b6c6c849ef717cc392d9d22f61ceffb525b0a587 | |
| parent | 1b14a849fff50888e6a48d7299c23c7d5821572a (diff) | |
| download | ikiwiki-6160238439c5df335944add13b00a0a80d1b663b.tar ikiwiki-6160238439c5df335944add13b00a0a80d1b663b.tar.gz | |
comments; my filter-full branch should fix this
| -rw-r--r-- | doc/bugs/po_vs_templates.mdwn | 22 |
1 files changed, 21 insertions, 1 deletions
diff --git a/doc/bugs/po_vs_templates.mdwn b/doc/bugs/po_vs_templates.mdwn index ce0ac3577..7bc56676e 100644 --- a/doc/bugs/po_vs_templates.mdwn +++ b/doc/bugs/po_vs_templates.mdwn @@ -18,6 +18,26 @@ This has been fixed in my po branch. -- [[intrigeri]] - +>> Hmm. Don't like adding a fourth positional parameter to that (or +>> any really) function. +>> +>> I think it's quite possible that some of the directives that are +>> calling filter do so unnecessarily. For example, conditional, +>> cutpaste, more, and toggle each re-filter text that comes from the +>> page and so has already been filtered. They could probably drop +>> the filtering. template likewise does not need to filter the +>> parameters passed into it. Does it need to filter the template output? +>> Well, it allows the (deprecated) embed plugin to work on template +>> content, but that's about it. +>> +>> Note also that the only other plugin to provide a filter, txt, +>> could also run into similar problems as po has, in theory (it looks at +>> the page parameter and assumes the content is for the whole page). +>> +>> [[!template id=gitbranch branch=origin/filter-full author="[[joey]]"]] +>> So, I've made a filter-full branch, where I attempt to fix this +>> by avoiding unnecessary filtering. Can you check it and merge it into +>> your po branch and remove your other workarounds so I can merge? +>> --[[Joey]] [[!tag patch]] |