Cleanup FinancialAccount BAO to use writeRecord and deprecate add

[colemanw]

Overview

Cleanup the FinancialAccount BAO, consistent with other cleanups.

Technical Details

The thrust of this was to deprecate the add function in favor of writeRecord, but in the process I found and fixed several problems:

• Defaults for boolean fields were being set in the BAO::add function instead of declared in the schema.
• The default value for is_active was being set to 0. That's a bad default & we're trying to consistently set all is_active fields to default 1 across all entities.
• There was a mismatch in the is_deductible field default value. It was set to default 1 in the schema, but that was overridden by the BAO for no apparent reason, which forced it to default 0. I've changed it to 0 in the schema for consistency with the status-quo.
• Another inconsistency was that CRM_Core_PseudoConstant::flush() was being called after create/update but not after delete. I've fixed that via a post hook to handle all 3 operations.

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[eileenmcnaughton]

@MegaphoneJon did you see this?

[monishdeb]

• General standards
  • (r-explain) PASS
  • (r-user) PASS
  • (r-run) PASS (tested on local)
• Developer standards
  • (r-tech) PASS:
  • (r-code) PASS
  • (r-maint) PASS
  • (r-test) PASS (test build passed)