Merge pull request #52 from Batch-697-Adam-Ranieri/khairston-patch-1

login function returns null if username cannot be found
Batch-697-Adam-Ranieri · Mar 5, 2020 · 034a04e · 034a04e
2 parents 0f57702 + 01c56f8
commit 034a04e
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/main/java/com/revature/services/EmployeeServiceImpl.java b/src/main/java/com/revature/services/EmployeeServiceImpl.java
@@ -88,11 +88,13 @@ public List<Employee> getActiveDrivers() {
 	@Override
 	public Employee loginEmployee(String username, String password) {
 		Employee e = er.findByUsername(username);
-		if (e.getPassword().equals(password)) {
+		if (e == null) {
+			return null;
+		}
+		else if (e.getPassword().equals(password)) {
 			return e;
 		}
-		else return null;
-
+		return null;
 //		StrongPasswordEncryptor passwordEncryptor = new StrongPasswordEncryptor();
 
 //		if (passwordEncryptor.checkPassword(password, e.getPassword())) {