Fix not found

This commit is contained in:
René Pfeuffer
2018-06-25 13:27:24 +02:00
parent dda832775e
commit 0c88a03f27
4 changed files with 22 additions and 17 deletions

View File

@@ -44,9 +44,12 @@ public class GroupResource extends AbstractManagerResource<Group, GroupException
@ResponseCode(code = 500, condition = "internal server error")
})
public Response get(@Context Request request, @Context UriInfo uriInfo, @PathParam("id") String id) {
Group group = manager.get(id);
GroupDto groupDto = groupToGroupDtoMapper.map(group);
return Response.ok(groupDto).build();
Group group = manager.get(id);
if (group == null) {
return Response.status(Response.Status.NOT_FOUND).build();
}
GroupDto groupDto = groupToGroupDtoMapper.map(group);
return Response.ok(groupDto).build();
}
@Path("")

View File

@@ -46,9 +46,12 @@ public class UserResource extends AbstractManagerResource<User, UserException> {
@ResponseCode(code = 500, condition = "internal server error")
})
public Response get(@Context Request request, @Context UriInfo uriInfo, @PathParam("id") String id) {
User user = manager.get(id);
UserDto userDto = userToDtoMapper.map(user);
return Response.ok(userDto).build();
User user = manager.get(id);
if (user == null) {
return Response.status(Response.Status.NOT_FOUND).build();
}
UserDto userDto = userToDtoMapper.map(user);
return Response.ok(userDto).build();
}
@PUT