Closed Bug 302582 Opened 19 years ago Closed 19 years ago

"add component/version" doesn't include action=add on enter_bug.cgi

Categories

(Bugzilla :: Bugzilla-General, defect)

2.20
defect
Not set
trivial

Tracking

()

RESOLVED FIXED
Bugzilla 2.20

People

(Reporter: LpSolit, Assigned: emmanuel)

Details

Attachments

(1 file, 1 obsolete file)

When trying to enter a bug into a product which has no component, we
get:

Sorry, the product xxxxxx has to have at least one component in order for
you to enter a bug into it.

Create a new component. 


But the link "Create a new component" brings you to
editcomponents.cgi?product=sans_comp

instead of
editcomponents.cgi?action=add&product=sans_comp


We already know there is no componement for this product, so the first link is
useless; we should directly go to the last one, i.e. with action=add.

The same comment applies to a missing version.
Status: NEW → ASSIGNED
Target Milestone: --- → Bugzilla 2.20
Attached patch Patch to go directly to the "add" page (obsolete) β€” β€” Splinter Review
Comment on attachment 190931 [details] [diff] [review]
Patch to go directly to the "add" page

Man, I need coffee. Asking for review
Attachment #190931 - Flags: review?(LpSolit)
Comment on attachment 190931 [details] [diff] [review]
Patch to go directly to the "add" page

>-      <a href="editcomponents.cgi?product=[% product FILTER url_quote %]">Create
>+      <a href="editcomponents.cgi?action=add&product=[% product FILTER url_quote %]">Create

>-      <a href="editversions.cgi?product=[% product FILTER url_quote %]">Create
>+      <a href="editversions.cgi?action=add&product=[% product FILTER url_quote %]">Create

ampersands must be escaped &amp;
Attachment #190931 - Flags: review?(LpSolit) → review-
Assignee: LpSolit → eseyman
Status: ASSIGNED → NEW
Attachment #190931 - Attachment is obsolete: true
Attachment #190935 - Flags: review?(LpSolit)
Comment on attachment 190935 [details] [diff] [review]
Taking into account LpSolit's review

r=LpSolit
Attachment #190935 - Flags: review?(LpSolit) → review+
Status: NEW → ASSIGNED
Flags: approval?
Flags: approval2.20?
Flags: approval?
Flags: approval2.20?
Flags: approval2.20+
Flags: approval+
tip:

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.119; previous revision: 1.118
done


2.20rc1:

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.115.2.3; previous revision: 1.115.2.2
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.

Attachment

General

Created:
Updated:
Size: