User facing errors shouldn't talk about integers and should be consistent

RESOLVED WONTFIX

Status

()

Bugzilla
Bugzilla-General
--
trivial
RESOLVED WONTFIX
12 years ago
4 years ago

People

(Reporter: timeless, Assigned: sjoshi)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
[% ELSIF error == "votes_must_be_nonnegative" %]
    [% title = "Votes Must Be Non-negative" %]
    [% admindocslinks = {'voting.html' => 'Setting up the voting feature'} %]
    Only use non-negative numbers for your [% terms.bug %] votes.
...
  [% ELSIF error == "illegal_user_id" %]
    [% title = "Illegal User ID" %]
    User ID '[% userid FILTER html %]' is not valid integer.

expected results:
don't talk to end users about integers, especially "valid" ones. If you mean non-negative, then it seems that's ok w/ some other part of the error file ...

Updated

12 years ago
Severity: normal → trivial
OS: MacOS X → All
Hardware: Macintosh → All
(Assignee)

Comment 1

4 years ago
Created attachment 778961 [details] [diff] [review]
Patch-v1

Just mentioning "valid" is enough,IMO.
Assignee: general → joshi_sunil
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #778961 - Flags: review?(justdave)

Comment 2

4 years ago
Comment on attachment 778961 [details] [diff] [review]
Patch-v1

>     [% title = "Illegal User ID" %]
>-    User ID '[% userid FILTER html %]' is not valid integer.
>+    User ID '[% userid FILTER html %]' is not valid.

"is not valid" is too vague as it could also mean "there is no user with such an ID" or "you are not allowed to view this account" even in the case of a valid integer. This error is thrown only when an admin didn't pass an integer, so the error message is accurate as is really about integers, nothing else. I see no reasons to reword it.

My vote to WONTFIX this bug.
Attachment #778961 - Flags: review?(justdave) → review-

Comment 3

4 years ago
(In reply to timeless from comment #0)
> don't talk to end users about integers

This message is only displayed to admins, not end users. So WONTFIX per my previous comment.
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.