Commit 711d1a81 authored by Martin Lowe's avatar Martin Lowe 🇨🇦
Browse files

Merge branch 'malowe/dev/fix-yarn' into 'dev'

Add patch for more logging and avoiding null returns

See merge request !445
parents dec703d1 d71153ad
Pipeline #1528 passed with stage
in 0 seconds
......@@ -623,7 +623,7 @@ public class OrganizationResource extends AbstractRESTResource {
// get unique person IDs to query for more details.
List<String> ids = people.entrySet().stream().flatMap(e -> e.getValue().stream()).distinct()
.collect(Collectors.toList());
LOGGER.debug("Found {} people with relations {} for organization {}", ids.size(), relations, organizationID);
LOGGER.info("Found {} people with relations {} for organization {}", ids.size(), relations, organizationID);
// create the params for cachekey
MultivaluedMap<String, String> params = new MultivaluedMapImpl<>();
params.put(DefaultUrlParameterNames.IDS.getName(), ids);
......@@ -648,7 +648,7 @@ public class OrganizationResource extends AbstractRESTResource {
return out;
});
if (rawPeopleData.isEmpty()) {
return null;
return Collections.emptyList();
}
LOGGER.debug("Mapped {} relations of {} for org {}", rawPeopleData.get().size(), ids.size(), organizationID);
return rawPeopleData.get().stream().map(p -> {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment