Closed Bug 67482 Opened 24 years ago Closed 24 years ago

real names with parens ( ) confuse bugzilla

Categories

(Bugzilla :: Bugzilla-General, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Bugzilla 2.12

People

(Reporter: st.n, Assigned: jacob)

References

()

Details

(Whiteboard: 2.12)

Attachments

(1 file)

Real names with parens ( ) confuse bugzilla. For example, look at any bug assigned to Leaf, like bug 7487. Scroll down a bit, and two lines above the "commit" button it reads "Reassign bug to leaf@netscape.com (Daniel" instead of "Reassign bug to leaf@netscape.com". This can cause strange things, see my comment dated 2000-10-27 06:38 in bug 20092: | I managed to change bug 36358 from assigned to new although I'm not allowed | to do something like this. This is how I did that: | | I wasn't logged in to Bugzilla when viewing that bug. Then I added my | comment, and logged in in another Mozilla window. Then I hit commit. | Bugzilla complained that I tried to change the owner or something from | "leaf@netscape.com" to "leaf@netscape.com (Daniel" (without the quotes). So | I hit the back button and deleted the " (Daniel" from the "assign to" | field. I hit commit again and this time Bugzilla was happy with it. Maybe it's even one of the reasons for bug 20092. And if not, it's at least confusing. I really thought this bug would have been filed already, but I just couldn't find it. By the way, this is the reason why I put my IRC nick inside [ ] instead of ( ).
Bug 63524 has fixes this (kinda as a side effect). The problem is in this code block (line 447 of bugform.pl): if ( $bug{'assigned_to'} =~ /(.*)\((.*)\)/ ) { $bug{'assigned_to'} = $1; chop($bug{'assigned_to'}); } What I did in bug 63524 was put the e-mail adress in $bug{assigned_to_email} and used that at line 451.
*** Bug 55786 has been marked as a duplicate of this bug. ***
Depends on: 63524
Whiteboard: 2.14
No longer depends on: 63524
Depends on: 63524
I'll take the relevent stuff out of bug 63524 and attach it here. It's really fairly simple.
Assignee: tara → jake
Attached patch Patch (deleted) — Splinter Review
Status: NEW → ASSIGNED
Keywords: patch, review
Whiteboard: 2.14 → 2.16
This bug has a simple fix (thanks Jake!) and really should be fixed since it causes lots of trouble (see my initial comments). So renominating for 2.12. Also removing dependency to bug 63524 since I don't see the reason any more why that bug would block this one. I'll test the patch out myself soon (hopefully).
No longer depends on: 63524
Whiteboard: 2.16 → 2.12
checked in.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Sorry for the spam, but I needed to be able to query for all of these correctly.
Target Milestone: --- → Bugzilla 2.12
VERIFIED. Gerv
Status: RESOLVED → VERIFIED
Moving closed bugs to Bugzilla product
Component: Bugzilla → Bugzilla-General
Product: Webtools → Bugzilla
Version: other → unspecified
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: