Logging Broken for Financial Records Following Upgrade

Following a clean upgrade DB Constraint violation issues were being reported on changes needing a new civicrm_financial_trxn record. The constraint was being thrown due to mandatory fields not having data set in the logging table.

My suspicion is this was caused by the differ script not checking if existing columns are no longer mandatory when it checked for differences in the schema. Pre-Upgrade version of CiviCRM was 4.1.2 and the trxn_id was required. In 4.3.4 its no longer required but the logging table wasn't altered to reflect this.

The Differ script should also be checking for Not Null constraints as part of its process in order to avoid such iussues.