Fix spring-projects#108 owner update

arey · arey · commit dd552f497084 · 2015-10-26T18:35:40.000+01:00
diff --git a/src/main/java/org/springframework/samples/petclinic/web/OwnerController.java b/src/main/java/org/springframework/samples/petclinic/web/OwnerController.java
@@ -111,10 +111,11 @@ public String initUpdateOwnerForm(@PathVariable("ownerId") int ownerId, Model mo
     }
 
     @RequestMapping(value = "/owners/{ownerId}/edit", method = RequestMethod.POST)
-    public String processUpdateOwnerForm(@Valid Owner owner, BindingResult result) {
+    public String processUpdateOwnerForm(@Valid Owner owner, @PathVariable("ownerId") int ownerId, BindingResult result) {
         if (result.hasErrors()) {
             return "owners/createOrUpdateOwnerForm";
         } else {
+            owner.setId(ownerId);
             this.clinicService.saveOwner(owner);
             return "redirect:/owners/{ownerId}";
         }

Original file line number	Diff line number	Diff line change
`@@ -111,10 +111,11 @@ public String initUpdateOwnerForm(@PathVariable("ownerId") int ownerId, Model mo`
`111`	`111`	`}`
`112`	`112`
`113`	`113`	`@RequestMapping(value = "/owners/{ownerId}/edit", method = RequestMethod.POST)`
`114`		`- public String processUpdateOwnerForm(@Valid Owner owner, BindingResult result) {`
	`114`	`+ public String processUpdateOwnerForm(@Valid Owner owner, @PathVariable("ownerId") int ownerId, BindingResult result) {`
`115`	`115`	`if (result.hasErrors()) {`
`116`	`116`	`return "owners/createOrUpdateOwnerForm";`
`117`	`117`	`} else {`
	`118`	`+ owner.setId(ownerId);`
`118`	`119`	`this.clinicService.saveOwner(owner);`
`119`	`120`	`return "redirect:/owners/{ownerId}";`
`120`	`121`	`}`