separate form level part of renewal from bao part using an outer wrapper function

separate form level part of renewal from bao part using an outer wrapper function
further consolidation on membership creation

comment fixes

enotice fix

further refactoring to pass more params into smaller functions to make fixing membership line items doable
CRM-14197 make membership_payment create action take responsibility for setting entity_id for membership line items
CRm-14197 where a participant_payment contribution has a single line item point it towards the contribution
Merge pull request #3305 from eileenmcnaughton/CRM-14197-membership

CRM-14197 contribution form post process - set entity_id to membership id
Per discussions with @totten I am merging this as he felt that he could do relatively little QA on it & I could see that the constant rebasing of this work has been much riskier in terms of bugs than actual code changes & as I have already changed back-office flows we can't really go back. 

The heart of this is the changes so that line_items have membership_id as the entity_id for memberships. rather than contribution id. The PRs leading up to this have been about getting to the point where the code made enough sense to make this change.

The key flow that still needs testing is around single vs multiple membership on the contribution page