Fixed issue with employees put route, all routes have been confirmed to work as expected

655ff5ca · Ben Anderson · 4bd0d682 · 655ff5ca
Commit 655ff5ca authored Mar 31, 2021 by Ben Anderson
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

EmployeeService.java ...va/com/nisum/employeeservice/service/EmployeeService.java +2 -2

No files found.
--- a/src/main/java/com/nisum/employeeservice/service/EmployeeService.java
+++ b/src/main/java/com/nisum/employeeservice/service/EmployeeService.java
@@ -34,11 +34,11 @@ public class EmployeeService {
      return null;
    }

-    if (!updatedEmployee.getFirstName().equals(staleEmployee.getFirstName())) {
+    if (updatedEmployee.getFirstName() != null && !updatedEmployee.getFirstName().equals(staleEmployee.getFirstName())) {
      staleEmployee.setFirstName(updatedEmployee.getFirstName());
    }

-    if (!updatedEmployee.getLastName().equals(staleEmployee.getFirstName())) {
+    if (updatedEmployee.getLastName() != null &&!updatedEmployee.getLastName().equals(staleEmployee.getFirstName())) {
      staleEmployee.setLastName(updatedEmployee.getLastName());
    }