comments; my filter-full branch should fix this
authorJoey Hess <joey@kitenet.net>
Sun, 4 Jul 2010 19:15:34 +0000 (15:15 -0400)
committerJoey Hess <joey@kitenet.net>
Sun, 4 Jul 2010 19:15:34 +0000 (15:15 -0400)
doc/bugs/po_vs_templates.mdwn

index ce0ac3577a6c5d6ffe0a05f2e8fbe003b5c422f7..7bc56676ea46edf286ad1b17243f4b71d92b95de 100644 (file)
@@ -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]]