fix(Additional Salary): mapped amount gets overwritten by salary component amount (backport #2420)

[mergify]

Issue

Amount field in New Additional Salary Doctype gets overwritten by amount field in Salary Component after selecting salary component. This behaviour is ideal except for scenarios like creating a "deduction from salary" from "employee advance"

Before

While creating new additional salary

While creating deduction from salary

After

While creating new additional salary (same as before)

While creating deduction from salary: Deduction amount referencing the employee advance doctype isn't overweritten

Fix

Removed "fetch from" "Salary Component" Setting on amount field in Additional Salary Doctype
Added conditional fetching and setting based on referenced doctype

no-tests

---

This is an automatic backport of pull request #2420 done by Mergify.

[mergify]

Cherry-pick of a53a9a7 has failed:

On branch mergify/bp/version-15-hotfix/pr-2420
Your branch is up to date with 'origin/version-15-hotfix'.

You are currently cherry-picking commit a53a9a7a3.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   hrms/payroll/doctype/additional_salary/additional_salary.js

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   hrms/payroll/doctype/additional_salary/additional_salary.json

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

[frappe-pr-bot]

🎉 This PR is included in version 15.35.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀