CIVIBLD-343 CiviCRM CRM_Financial_BAO_Order/getPriceSetID sometimes r…

…eturns NULL and causes CiviCRM to crash
civicrm · Apr 30, 2024 · e24238d · e24238d
1 parent 959b33d
commit e24238d
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/CRM/Financial/BAO/Order.php b/CRM/Financial/BAO/Order.php
@@ -471,6 +471,13 @@ public function getPriceSetID(): int {
         $this->setPriceSetIDFromSelectedField($fieldID);
       }
     }
+
+    if (!$this->priceSetID) {
+      $error_msg = 'Failure: Unable to look up Price Set ID for Price Options: ' . json_encode($this->getPriceOptions());
+      Civi::log()->debug($error_msg);
+      throw new CRM_Core_Exception($error_msg);
+    }
+
     return $this->priceSetID;
   }