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 12 years ago by
| Status: | new → accepted |
|---|
comment:3 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | accepted → closed |
Assuming upstream change will be accepted, closing this ticket for now.
comment:4 Changed 12 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
urlfromsetup.pyviainfo.get('home_page')to extract theComponentand 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.