Merge pull request #17666 from eileenmcnaughton/entity_log

Throw exception if id not passed into discard
civicrm · Jun 22, 2020 · f12d958 · f12d958
2 parents 6217c68 + 29d68d5
commit f12d958
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/CRM/Contact/BAO/Group.php b/CRM/Contact/BAO/Group.php
@@ -52,6 +52,9 @@ public static function retrieve(&$params, &$defaults) {
    * @param int $id Group id.
    */
   public static function discard($id) {
+    if (!$id || !is_numeric($id)) {
+      throw new CRM_Core_Exception('Invalid group request attempted');
+    }
     CRM_Utils_Hook::pre('delete', 'Group', $id, CRM_Core_DAO::$_nullArray);
 
     $transaction = new CRM_Core_Transaction();