mirror of
https://github.com/silverstripe/silverstripe-reports
synced 2024-06-17 18:21:38 +02:00
PATCH: ensure that composite and nested form fields can be included
This commit is contained in:
parent
0cb265434a
commit
096064486d
|
@ -307,11 +307,18 @@ class Report extends ViewableData
|
|||
|
||||
// Add search fields is available
|
||||
if ($this->hasMethod('parameterFields') && $parameterFields = $this->parameterFields()) {
|
||||
/** @var FormField $field */
|
||||
foreach ($parameterFields as $field) {
|
||||
// Namespace fields for easier handling in form submissions
|
||||
$field->setName(sprintf('filters[%s]', $field->getName()));
|
||||
$field->addExtraClass('no-change-track'); // ignore in changetracker
|
||||
|
||||
//standardise fields with filter[name] and .no-change-track
|
||||
$parameterFields->recursiveWalk(
|
||||
function(FormField $field) {
|
||||
if(strpos($field->getName(), 'filter[') !== 0) {
|
||||
$field->setName(sprintf('filters[%s]', $field->getName()));
|
||||
}
|
||||
$field->addExtraClass('no-change-track'); // ignore in changetracker
|
||||
}
|
||||
);
|
||||
// add fields
|
||||
foreach($parameterFields as $field) {
|
||||
$fields->push($field);
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue
Block a user