removed additional hateoas wrapper to achieve consistency in endpoint's returns structure

krzysztofwedrowicz · krzysztofwedrowicz · commit c88c53839ffe · 2019-09-26T14:12:45.000+02:00
diff --git a/src/main/java/io/pillopl/library/lending/patronprofile/web/PatronProfileController.java b/src/main/java/io/pillopl/library/lending/patronprofile/web/PatronProfileController.java
@@ -53,8 +53,8 @@ class PatronProfileController {
     private final CancelingHold cancelingHold;
 
     @GetMapping("/profiles/{patronId}")
-    ResponseEntity<EntityModel<ProfileResource>> patronProfile(@PathVariable UUID patronId) {
-        return ok(new EntityModel<>(new ProfileResource(patronId)));
+    ResponseEntity<ProfileResource> patronProfile(@PathVariable UUID patronId) {
+        return ok(new ProfileResource(patronId));
     }
 
     @GetMapping("/profiles/{patronId}/holds/")

Original file line number	Diff line number	Diff line change
`@@ -53,8 +53,8 @@ class PatronProfileController {`
`53`	`53`	`private final CancelingHold cancelingHold;`
`54`	`54`
`55`	`55`	`@GetMapping("/profiles/{patronId}")`
`56`		`- ResponseEntity<EntityModel<ProfileResource>> patronProfile(@PathVariable UUID patronId) {`
`57`		`- return ok(new EntityModel<>(new ProfileResource(patronId)));`
	`56`	`+ ResponseEntity<ProfileResource> patronProfile(@PathVariable UUID patronId) {`
	`57`	`+ return ok(new ProfileResource(patronId));`
`58`	`58`	`}`
`59`	`59`
`60`	`60`	`@GetMapping("/profiles/{patronId}/holds/")`