From ee72b1e03294b0efbfbf107de6046a68947293a8 Mon Sep 17 00:00:00 2001 From: "https://www.google.com/accounts/o8/id?id=AItOawkickHAzX_uVJMd_vFJjae6SLs2G38URPU" Date: Wed, 2 Jul 2014 17:53:10 -0400 Subject: repo links and test site for trail depends bug --- doc/plugins/trail/discussion.mdwn | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/doc/plugins/trail/discussion.mdwn b/doc/plugins/trail/discussion.mdwn index d37e5f7f9..731b8c790 100644 --- a/doc/plugins/trail/discussion.mdwn +++ b/doc/plugins/trail/discussion.mdwn @@ -152,3 +152,21 @@ I have removed a similar comment from the album discussion. > tend to be more personal/private than typical wikis, so you don't > necessarily want to link the real thing - that's why my album demos > tend to use dummy data). --[[smcv]] + +>> I was expecting the same depends pattern you describe. +>> My photo wikis are mostly public so I've set up a publicly accessible repo +>> (update-server-info type, git clone the first link below), a low-res copy of +>> the underlay and a quick sanitized setup file. + +>>* [[http://www.kalleswork.net/downloads/stockholm/.git]] +>>* [[http://www.kalleswork.net/downloads/stockholm.underlay.tar.gz]] +>>* [[http://www.kalleswork.net/downloads/stockholm.setup]] + +>> It might be a bit unwieldly and the site itself at [[http://stockholm.kalleswork.net]] +>> uses a few tweaks to the album templates and css, but I don't currently +>> have access to the machine where I setup a cleaner debug wiki to test. +>> (travelling atm). The images will likely be distorted due to the up scaling +>> bug in the [[img]] plugin but other than that it should work. + +>> Let me know if you need anything else. Would be great to hear it works +>> as expected for everyone else ;) --[[kjs]] -- cgit v1.2.3 From a6f4dcd4959f0b460f8e10480d67e8bdf50efafa Mon Sep 17 00:00:00 2001 From: chrysn Date: Thu, 3 Jul 2014 04:36:51 +0200 Subject: respond to smcv's comment --- doc/bugs/pythonproxy-utf8_again.mdwn | 28 +++++++++++++++++----------- 1 file changed, 17 insertions(+), 11 deletions(-) diff --git a/doc/bugs/pythonproxy-utf8_again.mdwn b/doc/bugs/pythonproxy-utf8_again.mdwn index b5564d6c1..fa702a22c 100644 --- a/doc/bugs/pythonproxy-utf8_again.mdwn +++ b/doc/bugs/pythonproxy-utf8_again.mdwn @@ -18,17 +18,17 @@ patch. > update 2014-06-29: the problem persists, but i found it is not trivial to > reproduce. to demonstrate, use this test plugin: > -> #!/usr/bin/env python -> # -*- coding: utf-8 -*- -> -> from proxy import IkiWikiProcedureProxy -> -> def preprocess(self, proxy, *args): -> return repr(self.rpc('pagetype', 'schön')) -> -> proxy = IkiWikiProcedureProxy(__name__) -> proxy.hook('preprocess', preprocess, id='testdirective') -> proxy.run() +> #!/usr/bin/env python +> # -*- coding: utf-8 -*- +> +> from proxy import IkiWikiProcedureProxy +> +> def preprocess(self, proxy, *args): +> return repr(self.rpc('pagetype', 'schön')) +> +> proxy = IkiWikiProcedureProxy(__name__) +> proxy.hook('preprocess', preprocess, id='testdirective') +> proxy.run() > > note that when the 'schön' is stored in a variable, the exception changes -- > it seems to me that the issue is related to the way exceptions are encoded. @@ -46,3 +46,9 @@ patch. >> >> Other than that it looks good to me. I like the use of `repr` in debug >> messages. --[[smcv]] + +>>> afaict, encode is fine there -- the relevant methods in python2 are +>>> `unicode.encode` which gives a `str`, and `str.decode` which usually gives +>>> a `unicode`. (i'd happily ditch python2 and port all plugins to python3, +>>> where this is all easier, but my [[todo/vCard rendering]] still uses an +>>> ancient module.) --[[chrysn]] -- cgit v1.2.3 From fb4cf0790ad93791040a99d6e183cfae274dd7f2 Mon Sep 17 00:00:00 2001 From: smcv Date: Thu, 3 Jul 2014 15:03:01 -0400 Subject: comment (not a review yet) --- doc/bugs/linkmap_displays_underscore_escapes.mdwn | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/doc/bugs/linkmap_displays_underscore_escapes.mdwn b/doc/bugs/linkmap_displays_underscore_escapes.mdwn index 62cfb3471..abcbfb9ed 100644 --- a/doc/bugs/linkmap_displays_underscore_escapes.mdwn +++ b/doc/bugs/linkmap_displays_underscore_escapes.mdwn @@ -17,6 +17,17 @@ the attached [[!taglink patch]] fixes this; from its commit message: the output will look much better (at least in my wikis) with the "[[bugs/pagetitle function does not respect meta titles]]" issue fixed. +> I agree that it's (likely to be) a bug to not use `pagetitle()`. I +> haven't reviewed this particular implementation yet but I'll try to +> do that soon. +> +> I don't think it's correct for `pagetitle()` to output `\[[!meta title]]` +> though, as discussed on the linked bug: it appears in an assortment of +> contexts where the full formal title of the page seems inappropriate. +> If you want linkmap to use `\[[!meta title]]`, I think it would be +> better to give it a `show` parameter, like `\[[!map]]` has? +> --[[smcv]] + the patch is stored in [[the patch.pl]] as created by git-format-patch, and can be pulled from the abovementioned branch. -- cgit v1.2.3 From 7760b8c72878336d45d00ce456697a3a250b7761 Mon Sep 17 00:00:00 2001 From: chrysn Date: Thu, 3 Jul 2014 21:54:30 +0200 Subject: reply to smcv's comment --- doc/bugs/linkmap_displays_underscore_escapes.mdwn | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/bugs/linkmap_displays_underscore_escapes.mdwn b/doc/bugs/linkmap_displays_underscore_escapes.mdwn index abcbfb9ed..75b917163 100644 --- a/doc/bugs/linkmap_displays_underscore_escapes.mdwn +++ b/doc/bugs/linkmap_displays_underscore_escapes.mdwn @@ -28,6 +28,9 @@ the output will look much better (at least in my wikis) with the "[[bugs/pagetit > better to give it a `show` parameter, like `\[[!map]]` has? > --[[smcv]] +>> sounds good; i'll have a look at it the next time i touch the linkmap +>> plugin. the patch at hand would be a starting point for that. --[[chrysn]] + the patch is stored in [[the patch.pl]] as created by git-format-patch, and can be pulled from the abovementioned branch. -- cgit v1.2.3