[PR #371] [MERGED] feat: Code update php8 im7 #436

New issue

Closed

opened 2026-02-25 22:35:27 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 22:35:27 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/flyimg/flyimg/pull/371
Author: @sadok-f
Created: 10/2/2023
Status: ✅ Merged
Merged: 10/2/2023
Merged by: @sadok-f

Base: main ← Head: code-update-php8-im7

📝 Commits (4)

86a1dc4 feat: update code to work on php8 and imagemagick 7
0d207d7 Edit docker base image
f69e1f6 Update unit test and fix duplicate content in Changelog file
0508b90 fixing linting errors

📊 Changes

25 files changed (+805 additions, -805 deletions)

View changed files

📝 CHANGELOG.md (+1 -2)
📝 Dockerfile (+2 -2)
📝 app.php (+1 -1)
📝 composer.json (+2 -3)
📝 composer.lock (+620 -735)
📝 phpunit.xml.dist (+17 -24)
📝 python/smartcrop.py (+2 -2)
📝 src/Core/Processor/FaceDetectProcessor.php (+1 -3)
📝 src/Core/Processor/Processor.php (+4 -4)
➕ src/Core/StorageProvider/FlysystemServiceProvider.php (+35 -0)
➕ src/Core/StorageProvider/LocalProviderTrait.php (+78 -0)
📝 src/Core/StorageProvider/LocalStorageProvider.php (+3 -4)
📝 src/Core/StorageProvider/S3StorageProvider.php (+0 -1)
📝 tests/Core/BaseTest.php (+16 -2)
📝 tests/Core/Controller/CoreControllerTest.php (+4 -3)
📝 tests/Core/Controller/DefaultControllerTest.php (+2 -2)
📝 tests/Core/Entity/ImageMetaInfoTest.php (+4 -4)
📝 tests/Core/Processor/ImageProcessorTest.php (+8 -8)
📝 tests/Core/Processor/ProcessorTest.php (+1 -1)
📝 tests/Core/Resolver/ControllerResolverTests.php (+3 -3)

...and 5 more files

📄 Description

a MR to update the code base to work with the latest PHP 8.2 and ImageMagick 7
This closes #360 and #359
This should release the next minor version of Flyimg 1.2.0

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/flyimg/flyimg/pull/371 **Author:** [@sadok-f](https://github.com/sadok-f) **Created:** 10/2/2023 **Status:** ✅ Merged **Merged:** 10/2/2023 **Merged by:** [@sadok-f](https://github.com/sadok-f) **Base:** `main` ← **Head:** `code-update-php8-im7` --- ### 📝 Commits (4) - [`86a1dc4`](https://github.com/flyimg/flyimg/commit/86a1dc4602ad000d6b78cf939477010ca92a2854) feat: update code to work on php8 and imagemagick 7 - [`0d207d7`](https://github.com/flyimg/flyimg/commit/0d207d77db6352efc3bba5bcbdbe9274f6c3e989) Edit docker base image - [`f69e1f6`](https://github.com/flyimg/flyimg/commit/f69e1f6b2c507cd960188dde18d59967f744bf29) Update unit test and fix duplicate content in Changelog file - [`0508b90`](https://github.com/flyimg/flyimg/commit/0508b90d7ab8959142696f61ca4a5fcc8880c4b5) fixing linting errors ### 📊 Changes **25 files changed** (+805 additions, -805 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+1 -2) 📝 `Dockerfile` (+2 -2) 📝 `app.php` (+1 -1) 📝 `composer.json` (+2 -3) 📝 `composer.lock` (+620 -735) 📝 `phpunit.xml.dist` (+17 -24) 📝 `python/smartcrop.py` (+2 -2) 📝 `src/Core/Processor/FaceDetectProcessor.php` (+1 -3) 📝 `src/Core/Processor/Processor.php` (+4 -4) ➕ `src/Core/StorageProvider/FlysystemServiceProvider.php` (+35 -0) ➕ `src/Core/StorageProvider/LocalProviderTrait.php` (+78 -0) 📝 `src/Core/StorageProvider/LocalStorageProvider.php` (+3 -4) 📝 `src/Core/StorageProvider/S3StorageProvider.php` (+0 -1) 📝 `tests/Core/BaseTest.php` (+16 -2) 📝 `tests/Core/Controller/CoreControllerTest.php` (+4 -3) 📝 `tests/Core/Controller/DefaultControllerTest.php` (+2 -2) 📝 `tests/Core/Entity/ImageMetaInfoTest.php` (+4 -4) 📝 `tests/Core/Processor/ImageProcessorTest.php` (+8 -8) 📝 `tests/Core/Processor/ProcessorTest.php` (+1 -1) 📝 `tests/Core/Resolver/ControllerResolverTests.php` (+3 -3) _...and 5 more files_ </details> ### 📄 Description a MR to update the code base to work with the latest PHP 8.2 and ImageMagick 7 This closes #360 and #359 This should release the next minor version of Flyimg 1.2.0 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>