Remove ?last? instances of ->free() dev/core#562

[eileenmcnaughton]

Overview

These are almost the last places where ->free is to be found. The remaining ones are in the dao
and perhaps out of scope / need more careful investigation

Before

$dao->free() used

After

$dao->free() removed

Technical Details

This is a deprecated method (replaced by $dao->__deconstruct()) which in rare cases causes bugs - we've removed a tonne - I think this might be the last batch

Comments

@seamuslee001 @aydun

[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).

[aydun]

Not tested, but all looks good to me by code review.

[colemanw]

We're free()!

[eileenmcnaughton]

lol