projects / civicrm-core.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: d37776a) | patch
Improve financial trxn spec to require required fields
authorEileen McNaughton <emcnaughton@wikimedia.org>
Wed, 19 Jan 2022 04:20:39 +0000 (17:20 +1300)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Thu, 14 Apr 2022 01:23:31 +0000 (13:23 +1200)
commit8b3f4fafbdfeb97824046778737a60fa202d626d
treedea4f582d65718a16407dacf11d55ac6e28d48fdtree | snapshot (zip tar.gz)
parentd37776a0cd166b1123ae8b549261eb93794396bccommit | diff
Improve financial trxn spec to require required fields

It turns out the apiv4 conformance test only passes because it
is bypassing the BAO create method - which requires more parameters

Not this will fail :-( because it messes with the test expectations
- I don't know how to create complex valid test data for it though....

I guess we could create a contribution & pass it as entity_id
CRM/Contribute/BAO/Contribution.php diff | blob | blame | history
CRM/Core/BAO/FinancialTrxn.php diff | blob | blame | history
Civi/Api4/Service/Spec/Provider/FinancialTrxnCreationSpecProvider.php [new file with mode: 0644] blob
tests/phpunit/api/v3/ContributionTest.php diff | blob | blame | history