-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Don't use features only available since PHP 7.3 #126
Comments
yhabteab
changed the title
Don't use features only available sind PHP 7.3
Don't use features only available since PHP 7.3
Sep 25, 2023
Or we consider changing the minimum requirement. To my knowledge, no distribution we support still uses PHP 7.2 |
|
Well, seems we have to support 7.2 still. |
@lippserd FWIW:
|
@lippserd FWIW 2.0:
NotesRHEL
Otherwise, it's PHP 7.2/8. Amazon Linux 2
Otherwise, it's PHP 5.4. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I noticed in several places where
array_key_last()
is used, which is available since PHP 7.3.icinga-notifications-web/application/forms/EscalationRecipientForm.php
Line 227 in e6b64c5
icinga-notifications-web/application/forms/EscalationConditionForm.php
Line 260 in e6b64c5
icinga-notifications-web/library/Notifications/Widget/EventRuleConfig.php
Line 86 in e6b64c5
The text was updated successfully, but these errors were encountered: