Skip to content

Commit d09a64d

Browse files
authored
[5.4] PHP8.5 deprecated code (joomla#46137)
* PHP8.5 Deprecations * Update Select.php
1 parent 156bd61 commit d09a64d

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

libraries/src/HTML/Helpers/Select.php

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -488,46 +488,46 @@ public static function options($arr, $optKey = 'value', $optText = 'text', $sele
488488
$key = $options['option.key'] === null ? $elementKey : $element[$options['option.key']];
489489
$text = $element[$options['option.text']];
490490

491-
if (isset($element[$options['option.attr']])) {
491+
if ($options['option.attr'] !== null && isset($element[$options['option.attr']])) {
492492
$attr = $element[$options['option.attr']];
493493
}
494494

495-
if (isset($element[$options['option.id']])) {
495+
if ($options['option.id'] !== null && isset($element[$options['option.id']])) {
496496
$id = $element[$options['option.id']];
497497
}
498498

499-
if (isset($element[$options['option.label']])) {
499+
if ($options['option.label'] !== null && isset($element[$options['option.label']])) {
500500
$label = $element[$options['option.label']];
501501
}
502502

503-
if (isset($element[$options['option.disable']]) && $element[$options['option.disable']]) {
503+
if ($options['option.disable'] !== null && isset($element[$options['option.disable']]) && $element[$options['option.disable']]) {
504504
$extra .= ' disabled="disabled"';
505505
}
506506
} elseif (\is_object($element)) {
507507
$key = $options['option.key'] === null ? $elementKey : $element->{$options['option.key']};
508508
$text = $element->{$options['option.text']};
509509

510-
if (isset($element->{$options['option.attr']})) {
510+
if ($options['option.attr'] !== null && isset($element->{$options['option.attr']})) {
511511
$attr = $element->{$options['option.attr']};
512512
}
513513

514-
if (isset($element->{$options['option.id']})) {
514+
if ($options['option.id'] !== null && isset($element->{$options['option.id']})) {
515515
$id = $element->{$options['option.id']};
516516
}
517517

518-
if (isset($element->{$options['option.label']})) {
518+
if ($options['option.label'] !== null && isset($element->{$options['option.label']})) {
519519
$label = $element->{$options['option.label']};
520520
}
521521

522-
if (isset($element->{$options['option.disable']}) && $element->{$options['option.disable']}) {
522+
if ($options['option.disable'] !== null && isset($element->{$options['option.disable']}) && $element->{$options['option.disable']}) {
523523
$extra .= ' disabled="disabled"';
524524
}
525525

526-
if (isset($element->{$options['option.class']}) && $element->{$options['option.class']}) {
526+
if ($options['option.class'] !== null && isset($element->{$options['option.class']}) && $element->{$options['option.class']}) {
527527
$extra .= ' class="' . $element->{$options['option.class']} . '"';
528528
}
529529

530-
if (isset($element->{$options['option.onclick']}) && $element->{$options['option.onclick']}) {
530+
if ($options['option.onclick'] !== null && isset($element->{$options['option.onclick']}) && $element->{$options['option.onclick']}) {
531531
$extra .= ' onclick="' . $element->{$options['option.onclick']} . '"';
532532
}
533533
} else {

0 commit comments

Comments
 (0)