WL-3308

Import location information on sessions and display it.
Yeah the old code was just crap, I just added a test so we didn't ever get
a array out of bounds exception.


On 2 July 2014 12:35, Ben Holmes <notifications@github.com> wrote:

> In impl/src/main/java/uk/ac/ox/oucs/vle/XcriOxCapPopulatorImpl.java:
>
> > @@ -981,10 +983,33 @@ private boolean updateComponent(PopulatorContext context,
> >  			
> >  			componentDao.getComponentSessions().clear();
> >  			for (Session session : sessions) {
> > +
> > +				if (session.getIdentifiers().length < 1) {
> > +					logMe(context, "Skipped session because no identifier found ["
> > +							+myPresentation.getPresentationId()+":"+myPresentation.getTitle()+"]");
> > +					continue;
> > +				}
> > +
> > +				String identifier = session.getIdentifiers()[0].getValue();
>
> I don't know much about the context of this code but just to check:
>
> We're definitely not interested in any other session identifiers than the
> first one?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ox-it/wl-course-signup/pull/131/files#r14452041>.
>



-- 
  Matthew Buckett, VLE Developer, IT Services, University of Oxford