Skip to content

Commit

Permalink
Merge pull request #64 from Icinga/php-81-support
Browse files Browse the repository at this point in the history
PHP 8.1 Support
  • Loading branch information
nilmerg authored Mar 24, 2022
2 parents ec2db19 + c44f252 commit afe05fe
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 10 deletions.
9 changes: 2 additions & 7 deletions .github/workflows/php.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ jobs:
strategy:
fail-fast: false
matrix:
php: ['7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1']
php: ['7.2', '7.3', '7.4', '8.0', '8.1']
os: ['ubuntu-latest']

steps:
Expand Down Expand Up @@ -51,13 +51,8 @@ jobs:
strategy:
fail-fast: false
matrix:
php: ['7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1']
php: ['7.2', '7.3', '7.4', '8.0', '8.1']
os: ['ubuntu-latest']
include:
- php: '7.0'
phpunit-version: 6.5
- php: '7.1'
phpunit-version: 7.5

steps:
- name: Checkout code base
Expand Down
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
}
},
"require": {
"php": ">=7.0",
"php": ">=7.2",
"ext-json": "*",
"ipl/html": ">=0.4.0",
"ipl/stdlib": ">=0.11.0",
Expand Down
2 changes: 1 addition & 1 deletion src/Control/SearchBar/Terms.php
Original file line number Diff line number Diff line change
Expand Up @@ -191,7 +191,7 @@ protected function assembleCondition(Filter\Condition $filter, BaseHtmlElement $

$this->assembleTerm($operatorData, $group);

if (! empty($value) || ctype_digit($value)) {
if (! empty($value) || ! is_string($value) || ctype_digit($value)) {
$valueData = [
'class' => 'value',
'type' => 'value',
Expand Down
2 changes: 1 addition & 1 deletion src/Widget/Tabs.php
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@ public function get($name)
*
* @return int
*/
public function count()
public function count(): int
{
return $this->tabs->count();
}
Expand Down

0 comments on commit afe05fe

Please sign in to comment.