- Issue created by @nubeli
- Status changed to Needs review
over 1 year ago 3:34am 19 July 2023 - 🇨🇦Canada nubeli
PR: https://github.com/colemanw/webform_civicrm/pull/894 - it check if handler is enabled in _fillCiviCRMData(). This at least seems to ensure that the webform_civicrm service isn't called.
- Status changed to Needs work
over 1 year ago 7:38am 23 July 2023 - 🇮🇳India jitendrapurohit
Hi @nubeli. I've reviewed & commented on the PR. Can you please have another look? Seems like there are more places that needs handling after civicrm handler is disabled.
- 🇨🇦Canada nubeli
Myself and another tester were unable to replicate the error you were getting with the PR @jitendrapurohit.
Please see https://github.com/colemanw/webform_civicrm/pull/894#issuecomment-212528....
To me it seems reasonable to expect the CiviCRM handler to not run at all if it is disabled. So please review again and hopefully can get something in to fix this issue.