From d1623673d82695163882f0cdfd789e44c9055711 Mon Sep 17 00:00:00 2001
From: Oliver Gierke <ogierke@gopivotal.com>
Date: Tue, 22 Jul 2014 06:33:18 +0200
Subject: [PATCH] Polishing of CustomerRepository in JPA Java 8 sample project.

---
 .../jpa/java8/CustomerRepository.java         | 20 ++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/jpa/java8/src/main/java/example/springdata/jpa/java8/CustomerRepository.java b/jpa/java8/src/main/java/example/springdata/jpa/java8/CustomerRepository.java
index d1d0c491..f924e9db 100644
--- a/jpa/java8/src/main/java/example/springdata/jpa/java8/CustomerRepository.java
+++ b/jpa/java8/src/main/java/example/springdata/jpa/java8/CustomerRepository.java
@@ -35,11 +35,29 @@ public interface CustomerRepository extends Repository<Customer, Long> {
 	 */
 	Optional<Customer> findOne(Long id);
 
+	/**
+	 * Saves the given {@link Customer}.
+	 * 
+	 * @param customer
+	 * @return
+	 */
 	<S extends Customer> S save(S customer);
 
+	/**
+	 * Sample method to derive a query from using JDK 8's {@link Optional} as return type.
+	 * 
+	 * @param lastname
+	 * @return
+	 */
 	Optional<Customer> findByLastname(String lastname);
 
+	/**
+	 * Sample default method to show JDK 8 feature support.
+	 * 
+	 * @param customer
+	 * @return
+	 */
 	default Optional<Customer> findByLastname(Customer customer) {
-		return findByLastname(customer.lastname);
+		return findByLastname(customer == null ? null : customer.lastname);
 	}
 }