tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-06-28 23:49:26 +02:00
Code Issues Projects Releases Wiki Activity

Merge pull request #8381 from open-sausages/pulls/4.2/sort-out-your-field

Browse Source 
BUG Changes being detected on TreeMulti as values not sorted
This commit is contained in:
Andre Kiste 2018-09-21 16:38:56 +12:00 committed by GitHub
commit e8fd49b394
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/Forms/TreeMultiselectField.php
View File

@ -103,7 +103,12 @@ class TreeMultiselectField extends TreeDropdownField
// cannot rely on $this->value as this could be a many-many relationship
$value = array_column($values, 'id');
$data['value'] = ($value) ? $value : 'unchanged';
if ($value) {
sort($value);
$data['value'] = $value;
} else {
$data['value'] = 'unchanged';
}
return $data;
}
@ -182,6 +187,7 @@ class TreeMultiselectField extends TreeDropdownField
}
$title = implode(", ", $titleArray);
sort($idArray);
$value = implode(",", $idArray);
} else {
$title = $emptyTitle;