Closed
Bug 310982
Opened 19 years ago
Closed 19 years ago
Remove the 'browserbugmessage' parameter completely
Categories
(Bugzilla :: Bugzilla-General, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.22
People
(Reporter: LpSolit, Assigned: LpSolit)
Details
Attachments
(1 file)
|
2.48 KB,
patch
|
bugreport
:
review+
|
Details | Diff | Splinter Review |
This parameter is only used in code-error.html.tmpl and user-error.html.tmpl. It
doesn't give any useful information and editing these two templates directly
would be even better than setting this parameter.
(22:38:25) LpSolit: justdave: we could even remove the 'browserbugmessage' param
completely from the code
(22:38:37) justdave: probably
(22:38:55) justdave: it doesn't seem to have much use except for historical
purposes :)
[...]
(22:45:05) LpSolit: joel: but I think you agree with me that
Param('browserbugmessage') doesn't help and is useless?
(22:45:22) joel: yes| Assignee | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → Bugzilla 2.22
Comment 1•19 years ago
|
||
LpSolit would you approve a patch which removes it? ;) joel LpSolit: I think I would reather see it assume that, if the knob is not defined, the user wanted to leave the status alone.
Comment 3•19 years ago
|
||
Comment on attachment 198351 [details] [diff] [review] patch, v1 r=joel by inspection
Attachment #198351 -
Flags: review?(bugreport) → review+
| Assignee | ||
Updated•19 years ago
|
Flags: approval?
Updated•19 years ago
|
Flags: approval? → approval+
| Assignee | ||
Comment 4•19 years ago
|
||
Checking in defparams.pl; /cvsroot/mozilla/webtools/bugzilla/defparams.pl,v <-- defparams.pl new revision: 1.167; previous revision: 1.166 done Checking in template/en/default/global/code-error.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/global/code-error.html.tmpl,v <-- code-error.html.tmpl new revision: 1.58; previous revision: 1.57 done Checking in template/en/default/global/user-error.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/global/user-error.html.tmpl,v <-- user-error.html.tmpl new revision: 1.130; previous revision: 1.129 done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•