[PR #89] [MERGED] Improve current architecture #64 #275

New issue

Closed

opened 2026-02-25 22:34:56 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 22:34:56 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/flyimg/flyimg/pull/89
Author: @sadok-f
Created: 7/6/2017
Status: ✅ Merged
Merged: 7/6/2017
Merged by: @sadok-f

Base: master ← Head: oop-refactoring

📝 Commits (2)

8482ef2 Improve current architecture #64
55e121e CodeClimate warining: remove unused variable

📊 Changes

14 files changed (+555 additions, -478 deletions)

View changed files

📝 app.php (+2 -2)
📝 src/Core/Controller/CoreController.php (+16 -15)
➖ src/Core/Entity/Image.php (+0 -321)
➕ src/Core/Entity/InputImage.php (+126 -0)
➕ src/Core/Entity/OutputImage.php (+241 -0)
📝 src/Core/Handler/ImageHandler.php (+40 -37)
📝 src/Core/Processor/FaceDetectProcessor.php (+11 -11)
📝 src/Core/Processor/ImageProcessor.php (+39 -39)
📝 src/Core/Processor/Processor.php (+12 -7)
📝 tests/Core/BaseTest.php (+2 -2)
➕ tests/Core/Entity/InputImageTest.php (+24 -0)
📝 tests/Core/Entity/OutputImageTest.php (+20 -22)
📝 tests/Core/Handler/ImageHandlerTest.php (+18 -18)
📝 tests/Core/Processor/FaceDetectProcessorTest.php (+4 -4)

📄 Description

Separating the Image Entity class into 2 classes: InputImage, OutputImage
Improve folders and classes naming.
#64

_{🔄 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/89 **Author:** [@sadok-f](https://github.com/sadok-f) **Created:** 7/6/2017 **Status:** ✅ Merged **Merged:** 7/6/2017 **Merged by:** [@sadok-f](https://github.com/sadok-f) **Base:** `master` ← **Head:** `oop-refactoring` --- ### 📝 Commits (2) - [`8482ef2`](https://github.com/flyimg/flyimg/commit/8482ef2984370770c0876312ab371859bd3f5a01) Improve current architecture #64 - [`55e121e`](https://github.com/flyimg/flyimg/commit/55e121e95c22aa17faf1391e9d542760b7c9535f) CodeClimate warining: remove unused variable ### 📊 Changes **14 files changed** (+555 additions, -478 deletions) <details> <summary>View changed files</summary> 📝 `app.php` (+2 -2) 📝 `src/Core/Controller/CoreController.php` (+16 -15) ➖ `src/Core/Entity/Image.php` (+0 -321) ➕ `src/Core/Entity/InputImage.php` (+126 -0) ➕ `src/Core/Entity/OutputImage.php` (+241 -0) 📝 `src/Core/Handler/ImageHandler.php` (+40 -37) 📝 `src/Core/Processor/FaceDetectProcessor.php` (+11 -11) 📝 `src/Core/Processor/ImageProcessor.php` (+39 -39) 📝 `src/Core/Processor/Processor.php` (+12 -7) 📝 `tests/Core/BaseTest.php` (+2 -2) ➕ `tests/Core/Entity/InputImageTest.php` (+24 -0) 📝 `tests/Core/Entity/OutputImageTest.php` (+20 -22) 📝 `tests/Core/Handler/ImageHandlerTest.php` (+18 -18) 📝 `tests/Core/Processor/FaceDetectProcessorTest.php` (+4 -4) </details> ### 📄 Description Separating the Image Entity class into 2 classes: InputImage, OutputImage Improve folders and classes naming. #64 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>