Skip to content
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

Fix/images php 8.0 #29479

Merged
merged 3 commits into from
Nov 2, 2021
Merged

Fix/images php 8.0 #29479

merged 3 commits into from
Nov 2, 2021

Conversation

come-nc
Copy link
Contributor

@come-nc come-nc commented Oct 28, 2021

Subset of #29432 designed to fix #29466

Fix image handling for PHP 8.0.

This makes sure using resource or GdImage (PHP>=8) behaves the same.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
This reverts commit d690f90.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc added the 3. to review Waiting for reviews label Oct 28, 2021
@come-nc come-nc added this to the Nextcloud 23 milestone Oct 28, 2021
@come-nc come-nc self-assigned this Oct 28, 2021
@come-nc come-nc requested review from a team, ArtificialOwl, juliushaertl and CarlSchwan and removed request for a team October 28, 2021 15:52
@skjnldsv skjnldsv mentioned this pull request Nov 1, 2021
19 tasks
@skjnldsv skjnldsv added the bug label Nov 1, 2021
@skjnldsv
Copy link
Member

skjnldsv commented Nov 1, 2021

/backport to stable22

@skjnldsv
Copy link
Member

skjnldsv commented Nov 1, 2021

/backport to stable21

@skjnldsv
Copy link
Member

skjnldsv commented Nov 1, 2021

8.0 is stable21 and above only
#22767

@come-nc come-nc merged commit f90176e into master Nov 2, 2021
@come-nc come-nc deleted the fix/images-php-8.0 branch November 2, 2021 09:57
@come-nc come-nc added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Nov 2, 2021
@blizzz blizzz mentioned this pull request Nov 3, 2021
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OC_Image.php needs fixes for PHP 8 object support
3 participants