Closed Bug 284185 Opened 20 years ago Closed 20 years ago

Incorrect parameter in the "attachment_already_obsolete" error message

Categories

(Bugzilla :: Attachments & Requests, defect)

2.19.2
defect
Not set
minor

Tracking

()

RESOLVED FIXED
Bugzilla 2.18

People

(Reporter: LpSolit, Assigned: LpSolit)

Details

Attachments

(1 file)

The "attachment_already_obsolete" error message in code-error.html.tmpl uses
"attachid" for the attachment ID while "attach_id" is used in attachment.cgi.
Consequently, the attachment ID is not displayed when obsoleting an already
obsolete attachment.
Attached patch trivial fix, v1Splinter Review
replace attachid by attach_id in code-error.html.tmpl.
Attachment #175883 - Flags: review?(wurblzap)
Status: NEW → ASSIGNED
Target Milestone: --- → Bugzilla 2.20
Comment on attachment 175883 [details] [diff] [review]
trivial fix, v1

r=wurblzap by inspection.
Attachment #175883 - Flags: review?(wurblzap) → review+
In 2.18, the relevant parts of attachment.cgi and code-error.html.tmpl are
identical.
Flags: approval?
Flags: approval2.18?
Flags: approval?
Flags: approval2.18?
Flags: approval2.18+
Flags: approval+
Target Milestone: Bugzilla 2.20 → Bugzilla 2.18
2.18:
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.39.2.4; previous revision: 1.39.2.3
done

Tip:
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.46; previous revision: 1.45
done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: