Custom message if "Open for bug entry" not enabled (instead of "Entering a bug has been disabled.")

NEW
Unassigned

Status

()

Bugzilla
Administration
--
enhancement
5 years ago
4 years ago

People

(Reporter: Andre Klapper, Unassigned)

Tracking

Details

Attachments

(1 obsolete attachment)

(Reporter)

Description

5 years ago
1. Have some external doc pages linking to /enter_bug.cgi?product=foo
2. Go to /editproducts.cgi?action=edit&product=foo
3. Uncheck "Open for bug entry" for various reasons (deprecated product, 
   superseded by product foo2, restructured Bugzilla organization)
4. Go to /enter_bug.cgi?product=foo
5. Get a generic "Sorry, entering a bug into the product foo has been disabled."

Thinking about usability:
Confused average users that will either not file their report at all or file it in a random place instead.

Proposal:
Add a non-mandatory setting under /editproducts.cgi?action=edit&product=foo which is only shown/enabled if "Open for bug entry" is NOT enabled.
Make it a text field to enter a custom message replacing or added to the current string, so I could turn it into something more helpful, like (random example):

"Sorry, entering a bug into the product 'Mobile' has been disabled. 
Please enter a bug about our mobile application in the product 'MobileApp' instead, or a bug about our mobile website under product 'Website' and component 'Mobile'."

Updated

5 years ago
Assignee: administration → hugo.seabrook
Status: NEW → ASSIGNED

Comment 1

5 years ago
Created attachment 704211 [details] [diff] [review]
v1 patch
Attachment #704211 - Flags: review?

Updated

5 years ago
Target Milestone: --- → Bugzilla 5.0

Updated

5 years ago
Attachment #704211 - Flags: review? → review?(LpSolit)
I like this idea, but I think this would be better implemented as a template hook.  Then you can drop a file into your template directory with your message in it (and an if statement checking which product if necessary) and use the template system to its fullest advantage to put whatever you wanted in there instead of just what you can fit in a text box.
Template doc hooks are here: http://www.bugzilla.org/docs/tip/en/html/api/Bugzilla/Extension.html#Template_Hooks

We'd likely need to identify the correct spot in the existing template that includes this error message to put the hook.

Comment 4

4 years ago
Comment on attachment 704211 [details] [diff] [review]
v1 patch

I'm honestly not interested in that feature, because I don't think it would be used a lot. Most products are active, and when a product becomes inactive, people will simply not see it. "Broken" links should rather be fixed to point to the new product.

For DB performance, I would prefer to not see another text field in the products table. And MEDIUMTEXT is IMO too large for such a field.
From a UI point of view, I would prefer to not see another text field in the admin page by default, as this field is useless as long as the product is active. So at the very minimum, JavaScript should hide this field by default till the product is marked as closed for new bugs.

I will let justdave have the final word on this.
Attachment #704211 - Flags: review?(LpSolit) → review-

Updated

4 years ago
Target Milestone: Bugzilla 5.0 → ---
+1 for template hook
(In reply to Frédéric Buclin from comment #4)
> I don't think it would be used a lot.

Whis is exactly what makes it a good template hook rather than a DB field.

> "Broken" links should rather be fixed to point to the new product.

In a perfect world, yes.  But the world isn't perfect, and trying to make it perfect is often an exercise in frustration.  Better to make things easy on the users when you can.

Comment 7

4 years ago
(In reply to Frédéric Buclin from comment #4)
> I will let justdave have the final word on this.

So Dave, WONTFIX or commit?

Regards,
Hugo.
Flags: needinfo?(justdave)
(In reply to Hugo Seabrook from comment #7)
> So Dave, WONTFIX or commit?

If I'm reading the bug correctly, neither. Thanks for the patch but we'd need a new one that does the template hook thingy that people think is better suited to fix the reported problem.
OS: Linux → All
Hardware: x86 → All

Comment 9

4 years ago
(In reply to Teemu Mannermaa (:wicked) from comment #8)
> If I'm reading the bug correctly, neither. Thanks for the patch but we'd
> need a new one that does the template hook thingy that people think is
> better suited to fix the reported problem.

No problems. I'll reassign this to the default assignee for someone else to work on. There are probably more important bugs that need attention.

Regards,
Hugo
Assignee: hugo.seabrook → administration
Status: ASSIGNED → NEW
Flags: needinfo?(justdave)

Updated

4 years ago
Attachment #704211 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.