Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Statistical Components for Salary Structure #8429

Merged
merged 3 commits into from
Apr 13, 2017

Conversation

ckosiegbu
Copy link
Contributor

@ckosiegbu ckosiegbu commented Apr 12, 2017

Today, any component included in a Salary Structure will contribute to the calculation of Gross Pay and Total Deductions. To make the calculation of components easier, we have introduced the ability to flag components as Statistical. This behaves something like a throwaway variable that can hold the results of calculations but does not contribute to the final calculation of the Gross Pay and Total Deductions. It allows for incremental calculation of Components and reuse of values across many steps.

image
Field "Statistical Component" is added. Selecting this option allows the Salary Detail to be used in calculations without appearing in the Salary Slip and without adding to the sum total or earnings or deductions (something like a transient variable)

image
Earnings section of Salary Structure show items that are statistical

image
Deductions section of Salary Structure show items that are statistical

image
Create Salary Slip

image
Salary Slip created with only non-statistical Components in Earnings and Deductions. Statistical Components do not get included in the final Salary Slip

Regards,
Chude

…nt is statistical. This allows components to be used in calculations without being added/deducted from earnings deductions.
@rmehta
Copy link
Member

rmehta commented Apr 13, 2017

@ckosiegbu tabs please 😅

@ckosiegbu
Copy link
Contributor Author

@rmehta Oops! Sorry. Converted to tabs now. I'll try to remember next time :-)

@nabinhait nabinhait merged commit f6149f1 into frappe:develop Apr 13, 2017
@sumitdas4u
Copy link

but it should reflect in the Accural Journal Entry at least

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants