<div dir="ltr"><div class="gmail_default" style="font-family:trebuchet ms,sans-serif">Hi Julien,</div><div class="gmail_default" style="font-family:trebuchet ms,sans-serif"><br></div><div class="gmail_default" style="font-family:trebuchet ms,sans-serif">

So this means that CDash is expecting "author" to be the user name (pnavarro) and not the full name (Pedro Navarro), right? Wouldn't it make more sense, then, to add a new "Full Name" field that, if not set by the VCS would be a duplicate of the "author" field? For the daily updates view (or build update view) the mailto: link would display "full name" but in other places, especially SQL joins the "author" field would be used as it is now.</div>
<div class="gmail_default" style="font-family:trebuchet ms,sans-serif"><br></div><div class="gmail_default" style="font-family:trebuchet ms,sans-serif">Pedro</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Tue, Nov 5, 2013 at 8:27 AM, Julien Jomier <span dir="ltr"><<a href="mailto:julien.jomier@kitware.com" target="_blank">julien.jomier@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Pedro,<br>
<br>
The main issue is that some VCS don't provide the email field.<br>
<br>
I think we should change the SQL request to add a match on the dailyupdatefile.email with user2repository.credential along with the usual request we have right now.<br>
<br>
Could you log a bug report and we'll try to address that.<br>
<br>
Thanks,<br>
Julien<div class="im"><br>
<br>
On 04/11/2013 22:33, Pedro Navarro wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Hi,<br>
<br>
When CDash gathers the list of updates from Perforce the users are<br>
returned as author="Full Name", email="<a href="mailto:user@company.com" target="_blank">user@company.com</a><br></div>
<mailto:<a href="mailto:user@company.com" target="_blank">user@company.com</a>>". The problem is that on the Manage Users page<div class="im"><br>
we get lots of messages like:<br>
<br>
"Pedro Navarro is not registerd for this project but has been submitting<br>
in the past month".<br>
<br>
The issue is that in manageProjectRoles.php page, the sql query to get<br>
unregistered users has:<br>
<br>
...LEFT JOIN user2repository ON<br>
(dailyupdatefile.author=<u></u>user2repository.credential ...<br>
<br>
It's matching the author of the commit with the repository credentials!<br>
In this case dailyupdatefile.author="Pedro Navarro" and<br>
user2repository.credential="<a href="mailto:pnavarro@netflix.com" target="_blank">pn<u></u>avarro@netflix.com</a><br></div>
<mailto:<a href="mailto:pnavarro@netflix.com" target="_blank">pnavarro@netflix.com</a>>"<u></u>. If I add "Pedro Navarro" as a repository<div class="im"><br>
credential everything works fine but this doesn't seem right and leads<br>
me to the following questions:<br>
<br>
- Is Perforce the only VCS that's returning full user names? Should we<br>
not do that?<br>
- Wouldn't it be better to match dailyupdatefile.email with<br>
user2repository.credential? After all the user's E-Mail is always added<br>
as a credential by default.<br>
<br>
Pedro<br>
<br>
<br>
<br>
<br>
<br></div>
______________________________<u></u>_________________<br>
Cdash mailing list<br>
<a href="mailto:Cdash@public.kitware.com" target="_blank">Cdash@public.kitware.com</a><br>
<a href="http://public.kitware.com/cgi-bin/mailman/listinfo/cdash" target="_blank">http://public.kitware.com/cgi-<u></u>bin/mailman/listinfo/cdash</a><br>
<br>
</blockquote>
</blockquote></div><br></div>