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

(bug) Closed Fiscal Years can still be written to and posted to the GL #1647

Closed
jniles opened this issue May 12, 2017 · 1 comment · Fixed by #2254
Closed

(bug) Closed Fiscal Years can still be written to and posted to the GL #1647

jniles opened this issue May 12, 2017 · 1 comment · Fixed by #2254
Assignees
Labels

Comments

@jniles
Copy link
Collaborator

jniles commented May 12, 2017

A Fiscal Year that is closed can still have transactions written to it (in the Posting Journal) and subsequently posted to the General Ledger.

Steps to reproduce:

  1. Create a new Fiscal Year 2018
  2. Close the current Fiscal Year 2017
  3. Import balances into Fiscal Year 2018
  4. Make any caution payment in 2017 - note that it succeed without protest. (/cash)
  5. Navigate to the Posting Journal (/journal)
  6. Perform a Trial Balance on Fiscal Year 2017 - note that there are no errors thrown.
  7. Post to the GL.

Expected Result(s):

  1. At the Cash Payments page, an error should be thrown that says "Fiscal Year is closed and cannot receive any more transactions. Please confirm that your dates are correct or see the accountant."
  2. On the Trial Balance page, an error message should detail that all transactions within the Fiscal Year cannot be posted to the GL since the year is closed.

Actual Result(s):

  1. Application performs as if no action had be taken regarding Fiscal Years.

It seems to me that the following rules should be in place:

  1. The Fiscal Year cannot be closed if there are open transactions in the Posting Journal. This should be clearly stated by an error message.
  2. The Posting Journal should be able to be written to if there the Fiscal Year is closed. This prevents a user from accidentally typing a closed year into a date input. The error message should ask the user to confirm they have entered the correct date and then refer them to the accountant.
  3. The Trial Balance should date values that are outside of the current Fiscal Year and provide an error the the accountant. If the accountant wants to proceed, they must unlock the Fiscal Year first.
@jniles
Copy link
Collaborator Author

jniles commented Aug 11, 2017

I'll be working on this in my re-write of the Trial Balance checks.

@jniles jniles self-assigned this Aug 11, 2017
@jniles jniles removed their assignment Sep 26, 2017
@jniles jniles self-assigned this Oct 26, 2017
jniles added a commit to jniles/bhima that referenced this issue Oct 26, 2017
This commit fixes a bug in the Trial Balance where closed fiscal years
could still be posted to the GL.

Closes IMA-WorldHealth#1647.
bors bot added a commit that referenced this issue Oct 31, 2017
2254: fix(Trial Balance): disallow closed fiscal years r=mbayopanda a=jniles

This commit fixes a bug in the Trial Balance where closed fiscal years could still be posted to the GL.

Closes #1647.
@bors bors bot closed this as completed in #2254 Oct 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant