Answers.Body: <p>This is a core bug and have reported issue at <a href="https://lab.civicrm.org/dev/financial/issues/28" rel="nofollow noreferrer">https://lab.civicrm.org/dev/financial/issues/28</a>  and also submitted fix at <a href="https://github.com/civicrm/civicrm-core/pull/12500" rel="nofollow noreferrer">https://github.com/civicrm/civicrm-core/pull/12500</a> </p>
  
  <p>Can you please try after applying the patch from <a href="https://patch-diff.githubusercontent.com/raw/civicrm/civicrm-core/pull/12500.patch" rel="nofollow noreferrer">here</a>?</p>
  
  <p>If the patch works without any error than please comment on the <a href="https://github.com/civicrm/civicrm-core/pull/12500" rel="nofollow noreferrer">PR</a> as it would help for PR to get merge quickly into core.</p>
  
  <p>HTH</p>
  
  <p>Pradeep</p>
  
Answers.ContentLicense: CC BY-SA 4.0
Answers.CreationDate: 2018-07-18T05:57:30.103Z
Answers.Text: patch is merged for 5.5 release
Answers.UserDisplayName: user4278
Answers.ContentLicense: CC BY-SA 4.0
Answers.CreationDate: 2018-07-18T17:56:12.180Z
Answers.Text: Pradeep, thanks so much. the patch works great. I have commented with thank you on PR
Answers.UserDisplayName: LaurynnLowe
Answers.ContentLicense: CC BY-SA 4.0
Answers.CreationDate: 2018-07-18T17:56:42.517Z
Answers.Text: So can you mark this as answer for your question?
Answers.UserDisplayName: Pradeep Nayak
Answers.ContentLicense: CC BY-SA 4.0
Answers.CreationDate: 2018-07-17T20:46:17.470Z
Answers.LastActivityDate: 2018-07-17T20:46:17.470Z
Answers.UserDisplayName: Pradeep Nayak
Body: <p>This seems like a core issue, possible with 4.7, but I cannot find under <a href="https://lab.civicrm.org/groups/dev/-/issues" rel="nofollow noreferrer">https://lab.civicrm.org/groups/dev/-/issues</a></p>
  
  <p>Once a Financial Account has been marked as deductible, it cannot be changed back to non-deductible.</p>
  
  <p>To reproduce:
  Edit a non deductible Financial Account
  Check box for "deductible?"
  Save
  On the Financial Accounts screen, the account will now show as Deductible
  Edit the same Financial Account (which is now deductible)
  Uncheck box for "deductible?"
  Save
  On the Financial Accounts screen, the account will not have changed</p>
  
Comments.ContentLicense: CC BY-SA 4.0
Comments.CreationDate: 2018-07-17T19:50:56.503Z
Comments.Text: Do you mean Financial Type?
Comments.UserDisplayName: KarinG - Semper IT
Comments.ContentLicense: CC BY-SA 4.0
Comments.CreationDate: 2018-07-18T17:54:00.140Z
Comments.Text: no I mean Financial Account
Comments.UserDisplayName: LaurynnLowe
Tags: <financial-accounts>
Title: Financial Account cannot be change to non deductible
ThreadText: # Financial Account cannot be change to non deductible
  
  (Asked by: username_0 on 2018-07-17)
  
  This seems like a core issue, possible with 4.7, but I cannot find under <https://lab.civicrm.org/groups/dev/-/issues>
  
  Once a Financial Account has been marked as deductible, it cannot be changed back to non-deductible.
  
  To reproduce:
  Edit a non deductible Financial Account
  Check box for "deductible?"
  Save
  On the Financial Accounts screen, the account will now show as Deductible
  Edit the same Financial Account (which is now deductible)
  Uncheck box for "deductible?"
  Save
  On the Financial Accounts screen, the account will not have changed
  
  **Tags:** financial-accounts
  
  > **Comments:**
  > **username_1**: Do you mean Financial Type?  
  > **username_0**: no I mean Financial Account  
  
  ---
  
  ## 1 answers
  
  ### Answer by username_2 on 2018-07-17. Score: 2
  
  This is a core bug and have reported issue at <https://lab.civicrm.org/dev/financial/issues/28> and also submitted fix at <https://github.com/civicrm/civicrm-core/pull/12500>
  
  Can you please try after applying the patch from [here](https://patch-diff.githubusercontent.com/raw/civicrm/civicrm-core/pull/12500.patch)?
  
  If the patch works without any error than please comment on the [PR](https://github.com/civicrm/civicrm-core/pull/12500) as it would help for PR to get merge quickly into core.
  
  HTH
  
  username_2
  
  > **Comments:**
  > **username_3**: patch is merged for 5.5 release  
  > **username_0**: username_2, thanks so much. the patch works great. I have commented with thank you on PR  
  > **username_2**: So can you mark this as answer for your question?  
  
  ---
  
  