| @@ -6,10 +6,10 @@ | ||
| 6 | 6 | use AcyMailing\Core\AcymObject; |
| 7 | 7 | |
| 8 | 8 | class FailActionType extends AcymObject |
| 9 | 9 | { |
| 10 | - public $values = []; | |
| 11 | - public $lists = []; | |
| 10 | + private array $values = []; | |
| 11 | + private array $lists = []; | |
| 12 | 12 | |
| 13 | 13 | public function __construct() |
| 14 | 14 | { |
| 15 | 15 | parent::__construct(); |
| @@ -32,9 +32,9 @@ | ||
| 32 | 32 | }'; |
| 33 | 33 | acym_addScript(true, $js); |
| 34 | 34 | } |
| 35 | 35 | |
| 36 | - public function display($num, $value) | |
| 36 | + public function display(string $num, string $value): string | |
| 37 | 37 | { |
| 38 | 38 | $js = 'jQuery(document).ready(function($){ updateSubAction("'.$num.'"); });'; |
| 39 | 39 | acym_addScript(true, $js); |
| 40 | 40 | |
| @@ -51,8 +51,9 @@ | ||
| 51 | 51 | 'text', |
| 52 | 52 | 'bounce_action_'.$num |
| 53 | 53 | ); |
| 54 | 54 | |
| 55 | + //We add a span with an ID around the dropdown as we can't hide/display the dropdown itself any more due to bootstrap integration | |
| 55 | 56 | $return .= '<span id="bounce_action_lists_'.$num.'" style="display:none">'; |
| 56 | 57 | |
| 57 | 58 | $return .= acym_select( |
| 58 | 59 | $this->lists, |
