Bug 339137

Description Wayne Beaton 2011-03-07 14:25:49 EST

We have been finding some committer records missing from the Foundation database. In the two most recent cases, the person being elected to the project has an existing role in the project. Specifically, in these two most recent cases, the individual being elected as a committer was already the project lead of the corresponding project. That this bug has surfaced now may be the result of a slight change in the way we execute project provisioning: i.e. in some cases, an over-vigilant EMO has designated the PL before the project provisioning request comes in. In this case, the individual already has a relationship with the project and the committer record is not created.

I can see exactly where this is probably happening in the portal code; specifically in committer_vote_summary.class.php#set_sent_to_webmaster()

The check code that determines whether or not a relationship pre-exists should be updated to determine if a 'CM' relationship pre-exists. Even better would be to just rely on the database's ability to manage its own primary keys and do an "insert ignore"