Bug 240733 - Can't edit IP log
Summary: Can't edit IP log
Status: CLOSED FIXED
Alias: None
Product: Community
Classification: Eclipse Foundation
Component: Process (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Bjorn Freeman-Benson CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-07-14 17:46 EDT by John Arthorne CLA
Modified: 2008-08-21 13:06 EDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description John Arthorne CLA 2008-07-14 17:46:07 EDT
I am unable to edit the auto-generated IP log for either Equinox or the Eclipse Platform project. I am a committer on both projects, and when I get directed to log in through friends of Eclipse I am able to successfully login. I am only viewing the IP log for one project at a time. Based on the info on the wiki page, the other possibility is that the foundation has a different email address on record for me than my bugzilla email.
Comment 1 Bjorn Freeman-Benson CLA 2008-07-14 20:50:20 EDT
You can check the email address that the Foundation database has for you via the portal.
You already know the email address that bugzilla uses for you because you use that to login.
Do these two emails match?
What email address are you logging in with on the Friends of Eclipse page?

I tested the ip_log.php again and the login is working for me so the code is obviously working.
Comment 2 John Arthorne CLA 2008-07-15 11:38:02 EDT
I confirmed the email addresses are the same. I also asked DJ to try it to ensure it wasn't just an issue with my account. It looks to me like it's not recognizing the login. I click the "login" button on the left hand side of the page. After going through the login successfully, I click "Return to previous page", and it takes me back to the IP log page.  Back on the IP log page, I can still see the "login" button on the left hand side, which suggests to me that it doesn't know I'm logged in. DJ sees the same behaviour. Note that we are not "friends of eclipse", if that makes a difference.
Comment 3 John Arthorne CLA 2008-07-15 11:45:26 EDT
To answer the other question in comment #1, I'm logging in with "john_arthorne@ca.ibm.com", which matches my bugzilla email and the email address in the portal.
Comment 4 Matthew Hall CLA 2008-07-15 11:47:25 EDT
Same here.  I follow the login link, enter my credentials and login successfully.  But when I click "Return to previous page" the IP Log still says that I need to login.

I've verified that my bugzilla e-mail address is the says as the address in portal.
Comment 5 John Arthorne CLA 2008-07-15 11:48:54 EDT
From a quick poll of other committers on #eclipse-dev, it only seems to work for people who are friends of eclipse. So, I guess I need to pay you to keep my IP log up to date ;)
Comment 6 Bjorn Freeman-Benson CLA 2008-07-17 11:07:21 EDT
Denis - could you look into this? It's supposed to work for anyone, not just Friends even though it uses the Friends login.
Comment 7 Matthew Hall CLA 2008-07-17 11:33:18 EDT
That or there should be a login page for Casual Acquaintances of Eclipse
Comment 8 Denis Roy CLA 2008-07-21 16:38:29 EDT
> Denis - could you look into this? 

Sure .. where is the code, or what's the URL?
Comment 9 Bjorn Freeman-Benson CLA 2008-07-21 16:55:58 EDT
(In reply to comment #8)
> Sure .. where is the code, or what's the URL?

http://www.eclipse.org/projects/ip_log.php?projectid=eclipse.equinox
Comment 10 Denis Roy CLA 2008-07-22 14:50:09 EDT
(In reply to comment #6)
> Denis - could you look into this? It's supposed to work for anyone, not just
> Friends even though it uses the Friends login.
> 

Here is the faulty code:

if( $Friend->getFriendID() == 0 ) {

If you simply want to verify for a logged in user (Friend or not) change that to:

if( $Session->getBugzillaID() == 0 ) {

If you need to check for a committer, that code is not there yet, but I can hack it in real quick.


Also, I added:
header("Cache-control: no-cache");

As newly logged-in folks would get the stale page upon hitting Return To Previous Page.
Comment 11 Bjorn Freeman-Benson CLA 2008-07-29 17:12:29 EDT
I changed the code a la Denis' guidance (comment #10). I am unable to test if this solves the problem because I never experienced the problem using my own login. I am assuming that the problem is solved. Please reopen (with details) if the problem still exists. Sorry about my faulty code causing you trouble and thanks for your patience.
Comment 12 John Arthorne CLA 2008-08-07 10:22:18 EDT
I still can't edit the log. The difference now is that I see this message at the bottom of the IP log page after logging in:

internal error 2: no bugzilla_id for friend_id
Comment 13 John Arthorne CLA 2008-08-07 10:24:58 EDT
Also from comment #10, Denis suggests there is no way to currently check for a logged in committer. Since we presumably don't want everyone with a bugzilla id to be able to edit IP logs, doesn't this support need to be added?
Comment 14 Karl Matthias CLA 2008-08-07 12:07:16 EDT
(In reply to comment #13)
> Also from comment #10, Denis suggests there is no way to currently check for a
> logged in committer. Since we presumably don't want everyone with a bugzilla id
> to be able to edit IP logs, doesn't this support need to be added?

The code does check this.  It only works for project leads, pmc members and leads, and committers on the project as far as I can tell.
Comment 15 Bjorn Freeman-Benson CLA 2008-08-08 14:01:39 EDT
Ok, I think (for the second time) that I've fixed this. It's a bit hard to test for the failure because it always seems to work for me, but I did find a second place in the code that might have been the cause.  I'm marking as FIXED again, but of course REOPEN it if I am wrong again.
Comment 16 John Arthorne CLA 2008-08-11 13:16:35 EDT
Works for me now, thanks.
Comment 17 Bjorn Freeman-Benson CLA 2008-08-21 13:06:02 EDT
Closing.