Modify ↓
#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 58 58 <input py:if="url == trac.homepage.strip('/')" type="hidden" name="version" 59 59 value="${trac.version.split('-', 1)[0] if 'dev' in trac.version else trac.version}" /> 60 60 <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']}" /> 61 62 <textarea name="description" rows="3" cols="10"> 62 63 ${description_en if url else description}</textarea> 63 64 <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 11 years ago by
Status: | new → accepted |
---|
comment:3 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
Assuming upstream change will be accepted, closing this ticket for now.
comment:4 Changed 11 years ago by
Fixed in trac:#11434 for milestone:0.12.6 / milestone:1.0.2 / milestone:1.1.2.
Note: See
TracTickets for help on using
tickets.
A better idea I think is to use
url
fromsetup.py
viainfo.get('home_page')
to extract theComponent
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.