Skip to content

Commit 507a7b1

Browse files
authored
fix: remove multiple dropdown option for documents (#844)
1 parent 57efab0 commit 507a7b1

File tree

1 file changed

+20
-15
lines changed

1 file changed

+20
-15
lines changed

ajax/field_specific_fields.php

Lines changed: 20 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -77,12 +77,13 @@ function ($itemtype) {
7777
}
7878
echo '</td>';
7979
} else {
80-
$dropdown_matches = [];
81-
$is_dropdown = $type == 'dropdown' || preg_match('/^dropdown-(?<class>.+)$/', $type, $dropdown_matches) === 1;
80+
$dropdown_matches = [];
81+
$is_dropdown = $type == 'dropdown' || preg_match('/^dropdown-(?<class>.+)$/', $type, $dropdown_matches) === 1;
82+
$is_dropdown_multi = ($is_dropdown && ($type != 'dropdown-Document'));
8283

8384
// Display "default value(s)" field
8485
echo '<td>';
85-
if ($is_dropdown) {
86+
if ($is_dropdown_multi) {
8687
echo __('Multiple dropdown', 'fields') . ' :';
8788
echo '<br />';
8889
}
@@ -94,21 +95,25 @@ function ($itemtype) {
9495

9596
echo '<td>';
9697
if ($is_dropdown) {
97-
$multiple = (bool) ($_POST['multiple'] ?? $field->fields['multiple']);
98+
if ($is_dropdown_multi) {
99+
$multiple = (bool) ($_POST['multiple'] ?? $field->fields['multiple']);
98100

99-
if ($field->isNewItem()) {
100-
Dropdown::showYesNo(
101-
'multiple',
102-
$multiple,
103-
-1,
104-
[
105-
'rand' => $rand,
106-
],
107-
);
101+
if ($field->isNewItem()) {
102+
Dropdown::showYesNo(
103+
'multiple',
104+
$multiple,
105+
-1,
106+
[
107+
'rand' => $rand,
108+
],
109+
);
110+
} else {
111+
echo Dropdown::getYesNo($multiple);
112+
}
113+
echo '<br />';
108114
} else {
109-
echo Dropdown::getYesNo($multiple);
115+
$multiple = false;
110116
}
111-
echo '<br />';
112117

113118
echo '<div style="line-height:var(--tblr-body-line-height);">';
114119
if ($field->isNewItem() && $type == 'dropdown') {

0 commit comments

Comments
 (0)