Skip to content

Respect function signatures from PHP 8.1#655

Merged
come-nc merged 6 commits intomasterfrom
fix/support-php-8.1
Feb 14, 2022
Merged

Respect function signatures from PHP 8.1#655
come-nc merged 6 commits intomasterfrom
fix/support-php-8.1

Conversation

@come-nc
Copy link
Contributor

@come-nc come-nc commented Feb 8, 2022

Should also fix #82

Signed-off-by: Côme Chilliet come.chilliet@nextcloud.com

Currently logreader UI do not load at all for me on Nextcloud 24, with either PHP 8 or PHP 8.1, with no error in the log.

@come-nc come-nc self-assigned this Feb 8, 2022
@come-nc come-nc force-pushed the fix/support-php-8.1 branch from 92df073 to 1f09992 Compare February 10, 2022 15:17
@CarlSchwan
Copy link
Member

/rebase

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@CarlSchwan
Copy link
Member

can you also add php 8.1 support in the CI?

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc merged commit 3b41377 into master Feb 14, 2022
@delete-merged-branch delete-merged-branch bot deleted the fix/support-php-8.1 branch February 14, 2022 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error PHP stripos() expects parameter 1 to be string

2 participants