Merge pull request #2 from lordofthejars/npe

Adds logging
This commit is contained in:
Burr Sutter 2020-10-20 08:31:23 -04:00 committed by GitHub
commit a271315c53
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -84,12 +84,13 @@ class OwnerController {
@GetMapping("/owners") @GetMapping("/owners")
public String processFindForm(Owner owner, BindingResult result, Map<String, Object> model) { public String processFindForm(Owner owner, BindingResult result, Map<String, Object> model) {
System.out.println("***" + owner.getLastName());
// allow parameterless GET request for /owners to return all records // allow parameterless GET request for /owners to return all records
if (owner.getLastName() == null) { if (owner.getLastName() == null) {
owner.setLastName(""); // empty string signifies broadest possible search owner.setLastName(""); // empty string signifies broadest possible search
} }
System.out.println("Searching for " + owner.getLastName().trim());
// find owners by last name // find owners by last name
Collection<Owner> results = this.owners.findByLastName(owner.getLastName()); Collection<Owner> results = this.owners.findByLastName(owner.getLastName());
if (results.isEmpty()) { if (results.isEmpty()) {