Done

On 04/02/13 15:42, Matthew Buckett wrote:
>
> In impl/src/main/java/uk/ac/ox/oucs/vle/XcriOxCapPopulatorImpl.java:
>
> >  			
> > -				if (validCourse(data, id, title, departmentCode, subunitCode, description,
> > -						departmentName, subunitName, visibility,
> > -						supervisorApproval, administratorApproval,
> > -						divisionEmail, regulations,
> > -						(Set<String>) administrators,
> > -						(Set<String>) divisionSuperUsers,
> > -						(Set<String>) otherDepartments,
> > +				if (updateCourse(data, myCourse,
> >  						(Set<Subject>) researchCategories,
>
> Would be better to define these as Set to begin with.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/ox-it/wl-course-signup/pull/40/files#r2877709>.
>