Opened 14 years ago
Closed 14 years ago
#9417 closed defect (fixed)
[PATCH] Genshi UnicodeDecodeError (Admin > General > Plugin)
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | high | Milestone: | 0.11.8 |
Component: | admin/web | Version: | 0.11.7 |
Severity: | minor | Keywords: | |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
I have the following problem:
500 Trac Error (Genshi UnicodeDecodeError error while rendering template '/usr/lib/python2.5/site-packages/Trac-0.11.6-py2.5.egg/trac/admin/templates/admin_plugins.html', line 54, char -1)
You can see that I'm using Python 2.5. This problem happens when I try to install CmisTracPlugin (a new plugin for trac). This plugin contains the following configuration in setup.py.
I have special characters. Viewing trac code I have found that only unicode transforms the attribute name and not others (i.e. description): http://trac.edgewall.org/browser/tags/trac-0.11.7/trac/admin/web_ui.py#L525
Between lines 542 - 548.
I have prepared a patch.
Attachments (1)
Change History (10)
by , 14 years ago
comment:1 by , 14 years ago
Summary: | Genshi UnicodeDecodeError (Admin > General > Plugin) → [PATCH] Genshi UnicodeDecodeError (Admin > General > Plugin) |
---|
follow-up: 4 comment:2 by , 14 years ago
comment:4 by , 14 years ago
Replying to cboos:
I'm getting crazy, I'm quite sure this came up before and recently, but I can't find the ticket anymore. In that ticket I can't find, it was said that we should do something similar than what we did for the author in #8444…
Anyway, thanks for the patch!
Hi cboos:
Certainly the bugs are related. That patch did not solve the problem completely.
Regards,
comment:5 by , 14 years ago
Replying to rblank:
This should go into 0.12 as well, right?
Hi rblank:
Are you very saturated with work? Maybe you need more commiters ;)
Regards,
comment:7 by , 14 years ago
Replying to cboos:
I'll apply the patch soon.
Hi cboos,
I'm using the patch and it seems to work correctly. Ok. I'll wait.
Regars,
comment:8 by , 14 years ago
The patch has been applied in [9925], but still needs to be merged forward.
comment:9 by , 14 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
[9925] has been merged to both 0.12-stable and trunk.
I'm getting crazy, I'm quite sure this came up before and recently, but I can't find the ticket anymore. In that ticket I can't find, it was said that we should do something similar than what we did for the author in #8444…
Anyway, thanks for the patch!