diff options
author | chrysn <chrysn@fsfe.org> | 2014-07-03 04:36:51 +0200 |
---|---|---|
committer | chrysn <chrysn@fsfe.org> | 2014-07-03 04:36:51 +0200 |
commit | a6f4dcd4959f0b460f8e10480d67e8bdf50efafa (patch) | |
tree | 7a1e74797c2e1aca17dc592bf1b8c8ebb80a3ccb /doc/bugs | |
parent | ee72b1e03294b0efbfbf107de6046a68947293a8 (diff) | |
download | ikiwiki-a6f4dcd4959f0b460f8e10480d67e8bdf50efafa.tar ikiwiki-a6f4dcd4959f0b460f8e10480d67e8bdf50efafa.tar.gz |
respond to smcv's comment
Diffstat (limited to 'doc/bugs')
-rw-r--r-- | doc/bugs/pythonproxy-utf8_again.mdwn | 28 |
1 files 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]] |