Skip to content

Commit

Permalink
Merge pull request #22214 from yashodha/dev-2979
Browse files Browse the repository at this point in the history
(dev/core#2979) remove the limit of 15 max values for multiple values…
  • Loading branch information
demeritcowboy authored Dec 9, 2021
2 parents 59b516c + 2359255 commit c46601a
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions CRM/Report/Utils/Get.php
Original file line number Diff line number Diff line change
Expand Up @@ -170,10 +170,12 @@ public static function intParam($fieldName, &$field, &$defaults) {
// send the type as string so that multiple values can also be retrieved from url.
// for e.g url like - "memtype_in=in&memtype_value=1,2,3"
$value = self::getTypedValue("{$fieldName}_value", CRM_Utils_Type::T_STRING);
if (!preg_match('/^(\d+)(,\d+){0,14}$/', $value)) {
// extra check. Also put a limit of 15 max values.

//change the max value to 20, ideally remove condition
if (!preg_match('/^(\d+)(,\d+){0,20}$/', $value)) {
$value = NULL;
}

if ($value !== NULL) {
$defaults["{$fieldName}_value"] = explode(",", $value);
$defaults["{$fieldName}_op"] = $fieldOP;
Expand Down

0 comments on commit c46601a

Please sign in to comment.