More logging

Author: CalvinAllen

src/main/java/org/springframework/samples/petclinic/owner/OwnerController.java

@@ -25,7 +25,6 @@
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Pageable;
-import org.springframework.samples.petclinic.clm.ClmController;
 import org.springframework.stereotype.Controller;
 import org.springframework.ui.Model;
 import org.springframework.validation.BindingResult;

src/main/java/org/springframework/samples/petclinic/vet/VetController.java

@@ -25,6 +25,7 @@
 import org.springframework.stereotype.Controller;
 import org.springframework.ui.Model;
 import org.springframework.web.bind.annotation.GetMapping;
+import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.RequestParam;
 import org.springframework.web.bind.annotation.ResponseBody;
 
@@ -57,6 +58,7 @@ public String showVetList(@RequestParam(defaultValue = "1") int page, Model mode
 			throw RandomExceptionGenerator.generateRandomException();
 		}
 		Page<Vet> paginated = findPaginated(page);
+		logger.info("Getting Vets for Page={}", page);
 		vets.getVetList().addAll(paginated.toList());
 		return addPaginationModel(page, paginated, model);
 
@@ -84,11 +86,21 @@ private Page<Vet> findPaginated(int page) {
 		Vets vets = new Vets();
 		Collection<Vet> vetList = this.vetRepository.findAll();
 		for (Vet vet : vetList) {
-			String speciality = vet.getSpecialties().get(0).getName();
-			logger.info("Vet {} has speciality {}", vet.getFirstName(), speciality);
+			logger.info("Vet='{}' has specialities='{}'", vet.getFirstName(), vet.getSpecialties());
 		}
 		vets.getVetList().addAll(vetList);
 		return vets;
 	}
 
+	@GetMapping("/vets/{lastName}")
+	public @ResponseBody Vets showResourcesVetList(@PathVariable(name = "lastName") String lastName) {
+		// This will throw when exercised from tester.sh as the Vet we send in has no specialties. Poor guy.
+		Vets vets = new Vets();
+		Collection<Vet> vetList = this.vetRepository.findByLastName(lastName);
+		for (Vet vet : vetList) {
+			logger.info("Vet Speciality='{}'", vet.getSpecialties().get(0).getName());
+		}
+		vets.getVetList().addAll(vetList);
+		return vets;
+	}
 }

src/main/java/org/springframework/samples/petclinic/vet/VetRepository.java

@@ -19,10 +19,13 @@
 import org.springframework.dao.DataAccessException;
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.Pageable;
+import org.springframework.data.jpa.repository.Query;
 import org.springframework.data.repository.Repository;
+import org.springframework.data.repository.query.Param;
 import org.springframework.transaction.annotation.Transactional;
 
 import java.util.Collection;
+import java.util.List;
 
 /**
  * Repository class for <code>Vet</code> domain objects All method names are compliant
@@ -55,4 +58,7 @@ public interface VetRepository extends Repository<Vet, Integer> {
 	@Cacheable("vets")
 	Page<Vet> findAll(Pageable pageable) throws DataAccessException;
 
+	@Query("SELECT DISTINCT vet FROM Vet vet WHERE vet.lastName LIKE :lastName% ")
+	@Transactional(readOnly = true)
+	List<Vet> findByLastName(@Param("lastName") String lastName);
 }

tester.sh

@@ -31,6 +31,8 @@ while true; do
     sleep 1
     curl -s -o /dev/null http://localhost:8081/vets
     sleep 1
+    curl -s -o /dev/null http://localhost:8081/vets/carter
+    sleep 1
     # curl -s -o /dev/null http://localhost:8081/oups
     # sleep 1
     curl -s -o /dev/null http://localhost:8081/clm/error