Merge from trunk - no conflicts
Added code so that the mailcode combo on the gift/GL transactions tab will allow an empty selection which will put NULL in the database.
Some detailed fixes to the new way that the code is encapsulated.  So now I have checked posting and importing and they seem to work well.  One of the significant issues with these was that the Year Ending combo box became blanked after a import/post.  The code has now been considerably simplified.
Uncrustify
I noticed that changing the mailing code setup did not affect the content of the ComboBox on the Gift Transactions tab (until the server was restarted).  Christian then realised that the Mailing Code setup screen should have been derived from windowMaintainCacheable table rather than windowMaintainTable!  This changes the yaml file to be correct.
The OpenGiftDestination form only occurs when the recipient key is a family.  But the message box asking if you wnat to set up a gift destiantion is displayed whatever partner class the recipient is.  So this is confusing - you get askked if you would like to do something and you say yes and then nothing happens.

This checkin adds a test to the already long list of checks whether to display the message box that the recipient is a family.  (I wonder though if other partner classes should be included - need to ask)
merge from trunk - no conflicts
Uncrustify
merge from lp:~alan-paterson/openpetraorg/2014_10_07_back_to_bugs