Skip wrapping v4 api requests in a transaction
authorColeman Watts <coleman@civicrm.org>
Fri, 24 May 2019 18:27:52 +0000 (14:27 -0400)
committerColeman Watts <coleman@civicrm.org>
Mon, 27 May 2019 16:33:18 +0000 (12:33 -0400)
commit3635b82351c60aee01ee4ad2f9523a6b61f15acb
tree22aee7e12ba566b5ad5031815ebce9c16b773a5d
parent6233ba43e2b5b5abce319ba5bdc3f0cd3ba61e1d
Skip wrapping v4 api requests in a transaction

Transactions aren't a bad thing and we probably want to revisit this but
for now I'm disabling it becuase it's broken, as seen in api_v3_ActivityTest.
I think the problem is with the onApiException event not being dispatched
because api4 doesn't go through Kernel->runSafe, so transactions aren't
being rolled back when they should.
Civi/API/Subscriber/TransactionSubscriber.php