diff --git a/hrms/payroll/doctype/retention_bonus/retention_bonus.py b/hrms/payroll/doctype/retention_bonus/retention_bonus.py
index 6f2418c7ca..ac23572ece 100644
--- a/hrms/payroll/doctype/retention_bonus/retention_bonus.py
+++ b/hrms/payroll/doctype/retention_bonus/retention_bonus.py
@@ -43,14 +43,14 @@ def on_submit(self):
 	def on_cancel(self):
 
 		additional_salary = self.get_additional_salary()
-		if self.additional_salary:
+		if additional_salary:
 			bonus_removed = (
-				frappe.db.get_value("Additional Salary", self.additional_salary, "amount") - self.bonus_amount
+				frappe.db.get_value("Additional Salary", additional_salary, "amount") - self.bonus_amount
 			)
 			if bonus_removed == 0:
-				frappe.get_doc("Additional Salary", self.additional_salary).cancel()
+				frappe.get_doc("Additional Salary", additional_salary).cancel()
 			else:
-				frappe.db.set_value("Additional Salary", self.additional_salary, "amount", bonus_removed)
+				frappe.db.set_value("Additional Salary", additional_salary, "amount", bonus_removed)
 
 			# self.db_set('additional_salary', '')