From d1ba93d61a3cbfe9fdf0a33ab33ccf875679af9b Mon Sep 17 00:00:00 2001 From: Rick Barrette Date: Sat, 28 Feb 2026 07:41:05 -0500 Subject: [PATCH] Refactor persist method in EmployeeSyncService to use 'local' variable for clarity --- app/services/employee_sync_service.rb | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/app/services/employee_sync_service.rb b/app/services/employee_sync_service.rb index eb11d0b..9782b77 100644 --- a/app/services/employee_sync_service.rb +++ b/app/services/employee_sync_service.rb @@ -68,18 +68,18 @@ class EmployeeSyncService # Create or update a local Employee record based on the QBO remote data def persist(remote) - employee = Employee.find_or_initialize_by(id: remote.id) + local = Employee.find_or_initialize_by(id: remote.id) if remote.active? - employee.name = remote.display_name + local.name = remote.display_name - if employee.changed? - employee.save + if local.changed? + local.save log "Updated employee #{remote.id}" end else - if employee.persisted? - employee.destroy + if local.persisted? + local.destroy log "Deleted employee #{remote.id}" end end