Skip to content

Commit

Permalink
Pass value as variable, not property
Browse files Browse the repository at this point in the history
  • Loading branch information
eileenmcnaughton committed Dec 18, 2023
1 parent 435cbc4 commit 7d4bb5e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
3 changes: 1 addition & 2 deletions CRM/Contact/Form/Contact.php
Original file line number Diff line number Diff line change
Expand Up @@ -322,15 +322,14 @@ public function preProcess() {
$paramSubType = implode(',', $contactSubType);
}

$this->_getCachedTree = FALSE;
unset($customGroupCount[0]);
foreach ($customGroupCount as $groupID => $groupCount) {
if ($groupCount > 1) {
$this->set('groupID', $groupID);
//loop the group
for ($i = 1; $i <= $groupCount; $i++) {
CRM_Custom_Form_CustomData::preProcess($this, NULL, $contactSubType,
$i, $this->_contactType, $this->_contactId
$i, $this->_contactType, $this->_contactId, NULL, FALSE
);
CRM_Contact_Form_Edit_CustomData::buildQuickForm($this);
}
Expand Down
8 changes: 4 additions & 4 deletions CRM/Custom/Form/CustomData.php
Original file line number Diff line number Diff line change
Expand Up @@ -72,15 +72,16 @@ public static function addToForm(&$form, $entitySubType = NULL, $subName = NULL,
* participant data this could be a value representing role.
* @param null|string $subType
* @param null|int $groupCount
* @param string $type
* @param null $type
* @param null|int $entityID
* @param null $onlySubType
* @param bool $isLoadFromCache
*
* @throws \CRM_Core_Exception
*/
public static function preProcess(
&$form, $extendsEntityColumn = NULL, $subType = NULL,
$groupCount = NULL, $type = NULL, $entityID = NULL, $onlySubType = NULL
$groupCount = NULL, $type = NULL, $entityID = NULL, $onlySubType = NULL, $isLoadFromCache = TRUE
) {
if (!$type) {
CRM_Core_Error::deprecatedWarning('type should be passed in');
Expand Down Expand Up @@ -131,7 +132,6 @@ public static function preProcess(
}

$gid = (isset($form->_groupID)) ? $form->_groupID : NULL;
$getCachedTree = $form->_getCachedTree ?? TRUE;
if (!is_array($subType) && str_contains(($subType ?? ''), CRM_Core_DAO::VALUE_SEPARATOR)) {
CRM_Core_Error::deprecatedWarning('Using a CRM_Core_DAO::VALUE_SEPARATOR separated subType deprecated, use a comma-separated string instead.');
$subType = str_replace(CRM_Core_DAO::VALUE_SEPARATOR, ',', trim($subType, CRM_Core_DAO::VALUE_SEPARATOR));
Expand All @@ -154,7 +154,7 @@ public static function preProcess(
$gid,
$subType,
$extendsEntityColumn,
$getCachedTree,
$isLoadFromCache,
$onlySubType,
FALSE,
CRM_Core_Permission::EDIT,
Expand Down

0 comments on commit 7d4bb5e

Please sign in to comment.