Fix enotices on paypal express (dev/financial/issues/14)
authoreileen <emcnaughton@wikimedia.org>
Mon, 7 May 2018 04:06:12 +0000 (16:06 +1200)
committereileen <emcnaughton@wikimedia.org>
Sun, 27 May 2018 22:01:56 +0000 (10:01 +1200)
commit819c0bd6785d90fb6adfe3df10c6d7c8233b4eb6
treed91658d216355046e406ae6084300701182ced24
parentccc7f59c0e73902497856e95903cab85e6a8d66b
Fix enotices on paypal express (dev/financial/issues/14)

For background - Paypal Express was not originally supported for recurring. It kind of snuck
in the back door since it was actually not possible to block in when configuring paypal pro
and people kept adding little fixes since they assumed it was broken rather than deliberately
missing functionality. Somewhere along the line this function was probably copied & extracted
from the main function - but it still contained a bunch of lines not
required for paypal express. I took a look at what data is actually available in this function
and removed reference to unavailable fields.

This is from https://developer.paypal.com/docs/classic/api/merchant/CreateRecurringPaymentsProfile_API_Operation_NVP/ and from testing and from https://lab.civicrm.org/dev/financial/issues/14

The one are where I differed from logic from @carbar1103
@carbar1103
  is the trxn_id - I retained the profile id which I think is consistent with use for other processsors, but needs testing
CRM/Core/Payment/PayPalImpl.php