If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Wrong error message when entering unknown / "invalid" URLs into "See also" field

RESOLVED FIXED in Bugzilla 3.6

Status

()

Bugzilla
Creating/Changing Bugs
--
minor
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Thomas D., Assigned: Max Kanat-Alexander)

Tracking

(Blocks: 1 bug)

unspecified
Bugzilla 3.6
Bug Flags:
approval +
approval4.0 +
blocking4.0 +
approval3.6 +

Details

Attachments

(1 attachment, 1 obsolete attachment)

v2
915 bytes, patch
glob
: review+
Details | Diff | Splinter Review
(Reporter)

Description

7 years ago
STR

1 enter an URL into See Also field that is not accepted yet (like http://support.delicious.com/forum/comments.php?DiscussionID=549)

2 Look at error msg

Actual

Bugzilla shows this error msg (where {} indicates variables in the msg):

"{http://support.delicious.com/forum/comments.php?DiscussionID=549} is not a valid URL to a bug. Bug URLs should point to {show_bug.cgi} in a Bugzilla installation."

That msg is wrong and confusing, as we are already accepting bugs from installations other than Bugzilla (currently google code, debian).

Expected

The error message must be updated, otherwise it will confuse the user which values are allowed or not:

Apart from Bugzilla installations, see-also now accepts bugs from some different installations, and more to come/requested:

bug 532350 (fixed) Can't add Debian bug URLs to a bug using "See Also"
bug 533121 (fixed) Support Google Code for see_also urls
bug 543667 Support trac URLs in the "See Also" field
bug 558784 support JIRA links in see also
bug 571740 Cannot add getsatisfaction links/URLs to "See also" field

So the error msg should correctly reflect accepted values, at best something like this (or less):

"{http://support.delicious.com/forum/comments.php?DiscussionID=549} is not a valid URL to a bug. Bug URLs must point to one of the following bug installations:
{http://code.google.com/*
http://bugs.debian.org/*}"

If that looks complex, blame it on the complex concept of the "See also" field.
Things would be a lot easier if "See also" would just accept *any* URLs. Which would not prevent us from implementing automation (input shortcuts, friendly outputs, cross-system notifications) for those few systems that are explicitly supported.
(Reporter)

Updated

7 years ago
Blocks: 231429
(Reporter)

Comment 1

7 years ago
(In reply to comment #0)
> So the error msg should correctly reflect accepted values, at best something
> like this (or less):
> "{http://support.delicious.com/forum/comments.php?DiscussionID=549} is not a
> valid URL to a bug. Bug URLs must point to one of the following bug
> installations:
> {http://code.google.com/*
> http://bugs.debian.org/*}"

Where I obviously missed the basic use case (bugzilla installations), and I have no way of knowing what exactly the patterns are that you check for google and debian, so just take the above as a sketch to refine.

Updated

7 years ago
Severity: normal → minor
(Assignee)

Updated

7 years ago
Target Milestone: --- → Bugzilla 3.6
(Assignee)

Updated

7 years ago
Flags: blocking4.0+
(Assignee)

Comment 2

7 years ago
Created attachment 463973 [details] [diff] [review]
v1
Assignee: create-and-change → mkanat
Status: NEW → ASSIGNED
Attachment #463973 - Flags: review?(bugzilla)
Comment on attachment 463973 [details] [diff] [review]
v1

>+        <li>A [% terms.bug %] on launchpad.net</li>
>+        <li>An issue on code.google.com.</li>
>+        <li>A [% terms.bug %] on b[% %]ugs.debian.org.</li>

we shouldn't be using terms.bug to refer to external entities, instead we should reference them by the same terminology that the external system uses.

launchpad calls them 'bugs', as does bugs.debian.org.
Attachment #463973 - Flags: review?(bugzilla) → review-
(Assignee)

Comment 4

7 years ago
Created attachment 472579 [details] [diff] [review]
v2

That's a really good point. :-)
Attachment #463973 - Attachment is obsolete: true
Attachment #472579 - Flags: review?(bugzilla)
Comment on attachment 472579 [details] [diff] [review]
v2

r=glob
Attachment #472579 - Flags: review?(bugzilla) → review+
(Assignee)

Updated

7 years ago
Flags: approval4.0+
Flags: approval3.6+
Flags: approval+
(Assignee)

Comment 6

7 years ago
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/                       
modified template/en/default/global/user-error.html.tmpl
Committed revision 7472.

Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/4.0/                         
modified template/en/default/global/user-error.html.tmpl
Committed revision 7402.

Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/3.6/                         
modified template/en/default/global/user-error.html.tmpl
Committed revision 7173.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.