Modify

Opened 5 years ago

Closed 4 years ago

#4685 closed defect (invalid)

`[[DepGraph]]` macro fails, when there is non ascii characters in ticket or ticket priorities

Reported by: ilmars@… Owned by: ftiede
Priority: normal Component: TracTicketDepgraphPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

I have checkouted latest TicketDepgraphPlugin [5302], which fixes issues with coloring, when there are ticket priorities with non ascii chars. Then I got next problem, when I included [[DepGraph]] macro in wiki page

Trac detected an internal error:
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc4 in position 62: ordinal not in range(128)

See stack trace for more detailed info in attachment stacktrace.log

Attachments (1)

stacktrace.log (7.9 KB) - added by ilmars@… 5 years ago.
Stack trace from trac.log

Download all attachments as: .zip

Change History (5)

Changed 5 years ago by ilmars@…

Stack trace from trac.log

comment:1 Changed 5 years ago by ftiede

Do you have any tickets with quotation marks (', ") in their summaries which might be included in depgraph?

comment:2 follow-up: Changed 5 years ago by ilmars@…

Yes, there are some tickets with ' and " in summaries. I don't know if they are included in DepGraph call, because they doesn't depend or block any other tickets. I have two tickets with dependencies, but they doesn't contain ' or " in summaries.

comment:3 in reply to: ↑ 2 Changed 5 years ago by ftiede

Replying to ilmars@delibero.lv:

Yes, there are some tickets with ' and " in summaries. I don't know if they are included in DepGraph call, because they doesn't depend or block any other tickets. I have two tickets with dependencies, but they doesn't contain ' or " in summaries.

They should not for added orphan check in an earlier revision, but I'm not sure if that really works the way intended. Anyway single quotes might pose a problem due to escaping. I'll look into it.
Only thing I can say so far is that a test ticket with almost every character above 0x7f and including ' in its summary works quite well here.

comment:4 Changed 4 years ago by ricardo.ichizo@…

  • Resolution set to invalid
  • Status changed from new to closed

This is a bug of Graphviz plugin. There is a patch here.

Add Comment

Modify Ticket

Action
as closed .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from ftiede. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.