Merge pull request #27047 from eileenmcnaughton/is_required

Remove do-nothing code.
civicrm · Aug 11, 2023 · 50d6517 · 50d6517
2 parents f9c1e76 + 86707c3
commit 50d6517
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/CRM/Price/BAO/PriceSet.php b/CRM/Price/BAO/PriceSet.php
@@ -386,7 +386,7 @@ public static function getAssoc($withInactive = FALSE, $extendComponentName = FA
    * @param int $setID
    *   Price Set ID.
    * @param bool $required
-   *   Appears to have no effect based on reading the code.
+   *   Deprecated.
    * @param bool $doNotIncludeExpiredFields
    *   Should only fields where today's date falls within the valid range be returned?
    *
@@ -415,9 +415,6 @@ public static function getSetDetail($setID, $required = TRUE, $doNotIncludeExpir
       'visibility_id',
       'is_required',
     ];
-    if ($required == TRUE) {
-      $priceFields[] = 'is_required';
-    }
 
     // create select
     $select = 'SELECT ' . implode(',', $priceFields);