Opened 15 years ago
Closed 15 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 , 15 years ago
comment:1 by , 15 years ago
| Summary: | Genshi UnicodeDecodeError (Admin > General > Plugin) → [PATCH] Genshi UnicodeDecodeError (Admin > General > Plugin) |
|---|
follow-up: 4 comment:2 by , 15 years ago
comment:4 by , 15 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 , 15 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 , 15 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 , 15 years ago
The patch has been applied in [9925], but still needs to be merged forward.
comment:9 by , 15 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!