Ticket #10231 (closed defect: fixed)

Opened 10 months ago

Last modified 8 months ago

Slash in Milestone name prevents displaying dependency graph

Reported by: omandel Assigned to: rjollos
Priority: low Component: MasterTicketsPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.12

Description

Description

When viewing a milestone with a forward slash ("/") in its name and a few tickets with dependencies, the link "Depgraph" in the upper right corner is shown correctly. But the linked page contains only an empty image without content.

Anlysis

It seems that when parsing the URL, only the part of the Milestone name up until the first forward slash is taken as the ticket milestone, resulting in an empty list of tickets.

E.g. for a milestone called "GUI/HW separation", I see this query like this in the log file:

SELECT t.id AS id,t.summary AS summary,t.owner AS owner,t.type AS type,
t.status AS status,t.priority AS priority,t.component AS component,
t.time AS time,t.changetime AS changetime,t.milestone AS milestone,
priority.value AS priority_value
FROM ticket AS t LEFT OUTER JOIN enum AS priority
ON (priority.type='priority' AND priority.name=priority)
WHERE ((COALESCE(t.milestone,'')='GUI'));

Note that the 'GUI' in the WHERE clause should have been 'GUI/HW separation'

Note

I use the distributor supplied version of this plugin, but I had a look at the diff between that and the "official" download: https://github.com/coderanger/trac-mastertickets/tarball/master . The differences are minimal and do not seem to include changes to the URL parsing.

Attachments

Change History

(in reply to: ↑ description ; follow-up: ↓ 2 ) 10/08/12 14:43:21 changed by rjollos

Replying to omandel:

... I use the distributor supplied version of this plugin, but I had a look at the diff between that and the "official" download: https://github.com/coderanger/trac-mastertickets/tarball/master . The differences are minimal and do not seem to include changes to the URL parsing.

Just curious, who is the distributor?

(in reply to: ↑ 1 ) 10/08/12 15:03:42 changed by omandel

Replying to rjollos:

Replying to omandel:

[ ... ] I use the distributor supplied version of this plugin, but I had a look at the diff between that and the "official" download: [ they ] are minimal and do not seem to include changes to the URL parsing.

Just curious, who is the distributor?

Fedora. The specific distribution is Fedora 17 Desktop 64-bit.

10/08/12 17:49:54 changed by rjollos

  • status changed from new to assigned.
  • owner changed from coderanger to rjollos.

10/08/12 17:50:09 changed by rjollos

  • status changed from assigned to closed.
  • resolution set to fixed.

Add/Change #10231 (Slash in Milestone name prevents displaying dependency graph)




Change Properties
Action