Introduce PropertyBag to help standardise payment params
authorRich Lott / Artful Robot <forums@artfulrobot.uk>
Fri, 1 Nov 2019 10:59:02 +0000 (10:59 +0000)
committerRich Lott / Artful Robot <forums@artfulrobot.uk>
Mon, 25 Nov 2019 14:29:16 +0000 (14:29 +0000)
commit47c968543f12403209e2cdcebd48224540720be0
treef5e757334c28ebe89e0614305fa18ffa76f71cc2
parenta2366db066c413d41194dfcfb82801db6e7371cb
Introduce PropertyBag to help standardise payment params

WIP: proof of concept for payment property bag

WIP: keep civilint happy

WIP: keep civilint happy again

Add other getters, setters, tests

Add require() method and test

some wip code

linter fixes

remove :void from function declaration in Civi/Payment/PropertyBag

fix PropertyBagTest.php  tests

fix linter

Re-do payment code lost in rebase

linting fixes

More WIP on payment property bag

one day i'll run civilint before committing

WIP checkin

Correct test which referred to Credit card instead of Credit Card causing a fail with new property bag which is stricter

Payment PropertyBag: allow week as recurFrequencyUnit; accept ZLS for feeAmount for legacy sake

CRM_Core_Payment: remove stuff cut from release

remove test no longer needed

remove un-needed code because feature dropped before it was released

remove comment

Remove/disarm payment instrument setter

restore getPaymentInstrumentID behaviour

remove not required function

set x
CRM/Core/Payment.php
CRM/Core/Payment/Form.php
Civi/Payment/PropertyBag.php [new file with mode: 0644]
tests/phpunit/CRM/Contribute/Form/ContributionTest.php
tests/phpunit/Civi/Payment/PropertyBagTest.php [new file with mode: 0644]