Fixed bug, that prevented all related history entries for an element

As the query param had the same name for each iteration, it got overwritten every time
This commit is contained in:
Jan Böhmer 2023-07-19 00:01:42 +02:00
parent 55aabddd41
commit e2dbd3d873

View file

@ -293,9 +293,13 @@ class LogDataTable implements DataTableTypeInterface
$target_type = LogTargetType::fromElementClass($element); $target_type = LogTargetType::fromElementClass($element);
$target_id = $element->getID(); $target_id = $element->getID();
$builder->orWhere('log.target_type = :filter_target_type AND log.target_id = :filter_target_id'); //We have to create unique parameter names for each element
$builder->setParameter('filter_target_type', $target_type); $target_type_var = 'filter_target_type_' . uniqid('', false);
$builder->setParameter('filter_target_id', $target_id); $target_id_var = 'filter_target_id_' . uniqid('', false);
$builder->orWhere("log.target_type = :$target_type_var AND log.target_id = :$target_id_var");
$builder->setParameter($target_type_var, $target_type);
$builder->setParameter($target_id_var, $target_id);
} }
} }
} }