[PR #119] [MERGED] Refactor part 2: Move array params to separate classes, closes #118 #896

New issue

Closed

opened 2026-03-15 15:41:44 +03:00 by kerem · 0 comments

kerem commented

2026-03-15 15:41:44 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/flyimg/flyimg/pull/119
Author: @sadok-f
Created: 8/31/2017
Status: ✅ Merged
Merged: 8/31/2017
Merged by: @sadok-f

Base: master ← Head: refactor-part-2-more-classes

📝 Commits (3)

22a6169 Refactor part 2: Move array params to separate classes, closes #118
2d388c5 fix phpcs warning , closes #118
d289412 Fix codeclimate warnings, closes #118

📊 Changes

21 files changed (+337 additions, -176 deletions)

View changed files

📝 app.php (+6 -7)
📝 src/Core/Controller/CoreController.php (+3 -3)
➕ src/Core/Entity/AppParameters.php (+50 -0)
📝 src/Core/Entity/Image/InputImage.php (+26 -17)
📝 src/Core/Entity/Image/OutputImage.php (+20 -9)
➕ src/Core/Entity/OptionsBag.php (+93 -0)
📝 src/Core/Handler/ImageHandler.php (+20 -44)
📝 src/Core/Handler/SecurityHandler.php (+12 -11)
📝 src/Core/Processor/ExtractProcessor.php (+1 -1)
📝 src/Core/Processor/FaceDetectProcessor.php (+1 -1)
📝 src/Core/Processor/ImageProcessor.php (+2 -2)
📝 src/Core/Processor/Processor.php (+1 -1)
📝 src/Core/StorageProvider/S3StorageProvider.php (+1 -1)
📝 tests/Core/BaseTest.php (+1 -1)
➕ tests/Core/Entity/AppParametersTest.php (+19 -0)
📝 tests/Core/Entity/Image/InputImageTest.php (+5 -3)
📝 tests/Core/Entity/Image/OutputImageTest.php (+17 -17)
📝 tests/Core/Handler/ImageHandlerTest.php (+1 -20)
📝 tests/Core/Handler/SecurityHandlerTest.php (+46 -27)
📝 tests/Core/Processor/ImageProcessorTest.php (+1 -1)

...and 1 more files

📄 Description

No description provided

_{🔄 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/119 **Author:** [@sadok-f](https://github.com/sadok-f) **Created:** 8/31/2017 **Status:** ✅ Merged **Merged:** 8/31/2017 **Merged by:** [@sadok-f](https://github.com/sadok-f) **Base:** `master` ← **Head:** `refactor-part-2-more-classes` --- ### 📝 Commits (3) - [`22a6169`](https://github.com/flyimg/flyimg/commit/22a6169a4e295dc1ee75b438d96714c7bbac9e13) Refactor part 2: Move array params to separate classes, closes #118 - [`2d388c5`](https://github.com/flyimg/flyimg/commit/2d388c50a23f78b07a332f99862cf06af2e060d6) fix phpcs warning , closes #118 - [`d289412`](https://github.com/flyimg/flyimg/commit/d289412369b4b6ab8251fdd1a6e9a22afb7e1957) Fix codeclimate warnings, closes #118 ### 📊 Changes **21 files changed** (+337 additions, -176 deletions) <details> <summary>View changed files</summary> 📝 `app.php` (+6 -7) 📝 `src/Core/Controller/CoreController.php` (+3 -3) ➕ `src/Core/Entity/AppParameters.php` (+50 -0) 📝 `src/Core/Entity/Image/InputImage.php` (+26 -17) 📝 `src/Core/Entity/Image/OutputImage.php` (+20 -9) ➕ `src/Core/Entity/OptionsBag.php` (+93 -0) 📝 `src/Core/Handler/ImageHandler.php` (+20 -44) 📝 `src/Core/Handler/SecurityHandler.php` (+12 -11) 📝 `src/Core/Processor/ExtractProcessor.php` (+1 -1) 📝 `src/Core/Processor/FaceDetectProcessor.php` (+1 -1) 📝 `src/Core/Processor/ImageProcessor.php` (+2 -2) 📝 `src/Core/Processor/Processor.php` (+1 -1) 📝 `src/Core/StorageProvider/S3StorageProvider.php` (+1 -1) 📝 `tests/Core/BaseTest.php` (+1 -1) ➕ `tests/Core/Entity/AppParametersTest.php` (+19 -0) 📝 `tests/Core/Entity/Image/InputImageTest.php` (+5 -3) 📝 `tests/Core/Entity/Image/OutputImageTest.php` (+17 -17) 📝 `tests/Core/Handler/ImageHandlerTest.php` (+1 -20) 📝 `tests/Core/Handler/SecurityHandlerTest.php` (+46 -27) 📝 `tests/Core/Processor/ImageProcessorTest.php` (+1 -1) _...and 1 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>