CRM-14197 pass line items & isTest into renewMembership (derive at form level

towards CRM-14197 pass lineitems & isTest into renewMembership

remove minor code duplication, ensure  set to form value

minor cleanups
param fix & clarify object to pass to updateRecurmembership fn

declare membershipContributionID & use that

remove  param

don't check for empty on  - would have had an error

remove unrequired param

set membershipContribution directly as required
refactor processing of secondary transaction into separate function
evaluate form contribution TypeID on form function
use try catch rather than passing errors as a reference
Merge pull request #3317 from eileenmcnaughton/CRM-14197-postprocesfn

CRM-14197 more refactoring of postProcess function