Modify

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#11479 closed enhancement (fixed)

Improve plugin association in defect reporting

Reported by: Ryan J Ollos Owned by: Ryan J Ollos
Priority: normal Component: TracHacks
Severity: normal Keywords:
Cc: Steffen Hoffmann, Michael Renzmann, Jun Omae Trac Release:

Description

On tickets that are populated by clicking create on an error page, the Component is not set and so we rely on the user to set the Component field. Frequently they do not. I briefly investigated ways to solve this. We could add Component to the GET request:

  • trac/templates/error.html

    diff --git a/trac/templates/error.html b/trac/templates/error.html
    index 6e6a6e6..6c4fc34 100644
    a b  
    5858    <input py:if="url == trac.homepage.strip('/')" type="hidden" name="version"
    5959           value="${trac.version.split('-', 1)[0] if 'dev' in trac.version else trac.version}" />
    6060    <input type="hidden" name="summary" value="$message" />
     61    <input py:if="'trac' not in faulty_plugins[0]['info']" type="hidden" name="component" value="${faulty_plugins[0]['name']}" />
    6162    <textarea name="description" rows="3" cols="10">
    6263${description_en if url else description}</textarea>
    6364    <span class="inlinebuttons">

That would only work in cases that the plugin name matches the project name. Frequently they do not, for example: SumStatsPlugin and TracSumStats. Perhaps we could use IRequestHandler to handle some common mappings, such as TracSumStats -> SumStatsPlugin.

Attachments (0)

Change History (4)

comment:1 Changed 10 years ago by Ryan J Ollos

Status: newaccepted

A better idea I think is to use url from setup.py via info.get('home_page') to extract the Component and pass it as a parameter in the GET request: t:browser:/branches/1.0-stable/trac/web/main.py?rev=12315&marks=561,586-587#L553.

comment:2 Changed 10 years ago by Ryan J Ollos

See proposed change in trac:#11434.

comment:3 Changed 10 years ago by Ryan J Ollos

Resolution: fixed
Status: acceptedclosed

Assuming upstream change will be accepted, closing this ticket for now.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


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

 
Note: See TracTickets for help on using tickets.