Merge pull request #16075 from seamuslee001/sig_status_bounce

[REF] Convert more petition signature fatals to status bounce
civicrm · Dec 11, 2019 · aae460f · aae460f
2 parents ee3bab5 + 0082383
commit aae460f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/CRM/Campaign/Form/Petition/Signature.php b/CRM/Campaign/Form/Petition/Signature.php
@@ -160,15 +160,15 @@ public function preProcess() {
 
     //some sanity checks
     if (!$this->_surveyId) {
-      CRM_Core_Error::fatal('Petition id is not valid. (it needs a "sid" in the url).');
+      CRM_Core_Error::statusBounce('Petition id is not valid. (it needs a "sid" in the url).');
       return;
     }
     //check petition is valid and active
     $params['id'] = $this->_surveyId;
     $this->petition = [];
     CRM_Campaign_BAO_Survey::retrieve($params, $this->petition);
     if (empty($this->petition)) {
-      CRM_Core_Error::fatal('Petition doesn\'t exist.');
+      CRM_Core_Error::statusBounce('Petition doesn\'t exist.');
     }
     if ($this->petition['is_active'] == 0) {
       CRM_Core_Error::statusBounce('Petition is no longer active.');
@@ -197,7 +197,7 @@ public function preProcess() {
       $this->_contactProfileFields = CRM_Core_BAO_UFGroup::getFields($this->_contactProfileId, FALSE, CRM_Core_Action::ADD);
     }
     if (!isset($this->_contactProfileFields['email-Primary'])) {
-      CRM_Core_Error::fatal('The contact profile needs to contain the primary email address field');
+      CRM_Core_Error::statusBounce('The contact profile needs to contain the primary email address field');
     }
 
     $ufJoinParams['weight'] = 1;