Edgewall Software
Modify

Opened 11 years ago

Closed 11 years ago

#9417 closed defect (fixed)

[PATCH] Genshi UnicodeDecodeError (Admin > General > Plugin)

Reported by: recena@… Owned by: Manuel Jesús Recena Soto <recena@…>
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)

9417.diff (987 bytes ) - added by Manuel Jesús Recena Soto <recena@…> 11 years ago.

Download all attachments as: .zip

Change History (10)

by Manuel Jesús Recena Soto <recena@…>, 11 years ago

Attachment: 9417.diff added

comment:1 by Manuel Jesús Recena Soto <recena@…>, 11 years ago

Summary: Genshi UnicodeDecodeError (Admin > General > Plugin)[PATCH] Genshi UnicodeDecodeError (Admin > General > Plugin)

comment:2 by Christian Boos, 11 years ago

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!

comment:3 by Remy Blank, 11 years ago

This should go into 0.12 as well, right?

in reply to:  2 comment:4 by Manuel Jesús Recena Soto <recena@…>, 11 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,

in reply to:  3 comment:5 by Manuel Jesús Recena Soto <recena@…>, 11 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:6 by Christian Boos, 11 years ago

Owner: set to Manuel Jesús Recena Soto <recena@…>

I'll apply the patch soon.

in reply to:  6 comment:7 by anonymous, 11 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 Remy Blank, 11 years ago

The patch has been applied in [9925], but still needs to be merged forward.

comment:9 by Remy Blank, 11 years ago

Resolution: fixed
Status: newclosed

[9925] has been merged to both 0.12-stable and trunk.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Manuel Jesús Recena Soto <recena@…>.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Manuel Jesús Recena Soto <recena@…> to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.