I believe there was only one error. I'll refactor it out tomorrow just to
be sure...


On Mon, Feb 11, 2013 at 6:31 PM, Michael de Jong
<notifications@github.com>wrote:

> In
> devhub-server/web/src/main/java/nl/tudelft/ewi/dea/jaxrs/api/projects/provisioner/ProvisionTask.java:
>
> > @@ -173,7 +173,7 @@ private void createContinuousIntegrationJob(Project project, User creator) throw
> >  		BuildProject buildRequest = new BuildProject(buildId, project.getSourceCodeUrl());
> >  		for (ProjectMembership membership : project.getMembers()) {
> >  			User member = membership.getUser();
> > -			ServiceUser user = new ServiceUser(member.getNetId(), member.getEmail(), member.getDisplayName());
> > +			ServiceUser user = new ServiceUser(member.getNetId(), member.getDisplayName(), member.getEmail());
>
> I see more ServiceUser objects are created in this class, with mixed
> orders of arguments. I suspect that these might also be incorrect then...
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/devhub-tud/DevHub/pull/103/files#r2965050>.
>
>