Bug 457704 - mouse-over other users shows my email address
Summary: mouse-over other users shows my email address
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Bugzilla
Classification: Community
Component: User Interface
Version: 3.2
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Noura El hawary
QA Contact:
URL:
Whiteboard:
: 457729 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-08-04 02:48 UTC by Jens Petersen
Modified: 2025-10-16 23:18 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2008-08-04 16:25:45 UTC
Embargoed:


Attachments (Terms of Use)
v1 patch to fix login names appearing when mouseover usernames in bug reports (758 bytes, patch)
2008-08-04 13:09 UTC, Noura El hawary
dkl: review+
Details | Diff

Description Jens Petersen 2008-08-04 02:48:09 UTC
Description of problem:
With the new bugzilla when I bring the mouse pointer over the name of a bugzilla user
(eg in the Assigned To or CC fields etc) the mouse-over text (tooltip) shows
my email address not the address of the user under the pointer.

How reproducible:
every time

Steps to Reproduce:
1. Point to a username mailto link in a bug.
  
Actual results:
1. My email address is shown.

Expected results:
1. The user's email address to be shown.

Comment 1 Noura El hawary 2008-08-04 13:09:03 UTC
Created attachment 313358 [details]
v1 patch to fix login names appearing when mouseover usernames in bug reports

Thanks for reporting the issue Jens.

The attached patch should fix the problem , I have applied it to bz-web2 and seems to work fine there so now whenever you move the mouse over a name it should display the login name for that user not for the logged in user.

Cheers,
Noura

Comment 2 Noura El hawary 2008-08-04 13:30:57 UTC
*** Bug 457729 has been marked as a duplicate of this bug. ***

Comment 3 David Lawrence 2008-08-04 15:18:08 UTC
Comment on attachment 313358 [details]
v1 patch to fix login names appearing when mouseover usernames in bug reports

Looks good Noura. We need to verify that this is also an upstream bug and submit a bug for them as well. I don't believe we changed anything in this part of the code ourselves.

Dave

Comment 4 Noura El hawary 2008-08-04 16:25:45 UTC
Thanks for the review Dave, committed to cvs.

Noura

Comment 5 Frédéric Buclin 2008-08-04 17:38:46 UTC
(In reply to comment #3)
> (From update of attachment 313358 [details])
> Looks good Noura. We need to verify that this is also an upstream bug

Not an upstream bug. We don't use bug_user in this template. I cannot reproduce this bug in 3.2 RC1.

Comment 6 David Lawrence 2008-08-04 17:48:07 UTC
(In reply to comment #5)
> (In reply to comment #3)
> > (From update of attachment 313358 [details] [details])
> > Looks good Noura. We need to verify that this is also an upstream bug
> 
> Not an upstream bug. We don't use bug_user in this template. I cannot reproduce
> this bug in 3.2 RC1.

Yeah we figured out that it was a typo in the portion of our code that hides the email addresses for non-logged in users. Should be fixed now.

Dave

Comment 7 Jens Petersen 2008-08-04 23:58:53 UTC
Thanks!  Looks good now. :)


Note You need to log in before you can comment on or make changes to this bug.