fixed JPA issue when updating existing owners/pets

src/main/java/org/springframework/samples/petclinic/repository/jpa/JpaOwnerRepositoryImpl.java

@@ -70,7 +70,12 @@ public class JpaOwnerRepositoryImpl implements OwnerRepository {
 
     @Override
     public void save(Owner owner) {
-        this.em.persist(owner);
+    	if (owner.getId() == null) {
+    		this.em.persist(owner);     		
+    	}
+    	else {
+    		this.em.merge(owner);    
+    	}
 
     }
 

src/main/java/org/springframework/samples/petclinic/repository/jpa/JpaPetRepositoryImpl.java

@@ -53,7 +53,12 @@ public class JpaPetRepositoryImpl implements PetRepository {
 
     @Override
     public void save(Pet pet) {
-        this.em.persist(pet);
+    	if (pet.getId() == null) {
+    		this.em.persist(pet);     		
+    	}
+    	else {
+    		this.em.merge(pet);    
+    	}
     }
 
 }

src/main/java/org/springframework/samples/petclinic/repository/jpa/JpaVisitRepositoryImpl.java

@@ -45,7 +45,12 @@ public class JpaVisitRepositoryImpl implements VisitRepository {
 
     @Override
     public void save(Visit visit) {
-        this.em.persist(visit);
+    	if (visit.getId() == null) {
+    		this.em.persist(visit);     		
+    	}
+    	else {
+    		this.em.merge(visit);    
+    	}
     }
 
 

src/test/java/org/springframework/samples/petclinic/service/AbstractClinicServiceTests.java

@@ -89,8 +89,8 @@ public abstract class AbstractClinicServiceTests {
         owner.setCity("Wollongong");
         owner.setTelephone("4444444444");
         this.clinicService.saveOwner(owner);
+        Assert.assertNotEquals("Owner Id should have been generated", owner.getId().longValue(), 0);
         owners = this.clinicService.findOwnerByLastName("Schultz");
-        assertNotNull("Owner Id should have been generated", owner.getId());
         assertEquals("Verifying number of owners after inserting a new one.", found + 1, owners.size());
     }