[PR #51] [MERGED] Refactor part 1 #249

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

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

Base: master ← Head: refactor-part-1

📝 Commits (3)

97de83c First commit in refactoring task
28e12be Fix phpcs Warning
d4fa3a4 Refactoring main logic, add CoreManager that handle the image process action

📊 Changes

11 files changed (+252 additions, -156 deletions)

View changed files

📝 app.php (+41 -19)
📝 config/parameters.yml (+2 -2)
📝 src/Core/Controller/CoreController.php (+4 -3)
📝 src/Core/Controller/DefaultController.php (+7 -7)
📝 src/Core/Entity/Image.php (+17 -50)
➕ src/Core/Service/CoreManager.php (+92 -0)
📝 src/Core/Service/ImageProcessor.php (+19 -44)
📝 src/Core/Tests/BaseTest.php (+9 -1)
📝 src/Core/Tests/ImageProcessorTest.php (+9 -27)
📝 src/Core/Tests/ImageTest.php (+3 -3)
➕ src/Core/Traits/ParserTrait.php (+49 -0)

📄 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/51 **Author:** [@sadok-f](https://github.com/sadok-f) **Created:** 3/7/2017 **Status:** ✅ Merged **Merged:** 3/7/2017 **Merged by:** [@sadok-f](https://github.com/sadok-f) **Base:** `master` ← **Head:** `refactor-part-1` --- ### 📝 Commits (3) - [`97de83c`](https://github.com/flyimg/flyimg/commit/97de83cb73a2f794ca3ea21053ee76dbf9601f08) First commit in refactoring task - [`28e12be`](https://github.com/flyimg/flyimg/commit/28e12beaea0b44e22a8b83d254abd6eeea47c15a) Fix phpcs Warning - [`d4fa3a4`](https://github.com/flyimg/flyimg/commit/d4fa3a4c238b05b03db83cd96bb6992a5b9bbae3) Refactoring main logic, add CoreManager that handle the image process action ### 📊 Changes **11 files changed** (+252 additions, -156 deletions) <details> <summary>View changed files</summary> 📝 `app.php` (+41 -19) 📝 `config/parameters.yml` (+2 -2) 📝 `src/Core/Controller/CoreController.php` (+4 -3) 📝 `src/Core/Controller/DefaultController.php` (+7 -7) 📝 `src/Core/Entity/Image.php` (+17 -50) ➕ `src/Core/Service/CoreManager.php` (+92 -0) 📝 `src/Core/Service/ImageProcessor.php` (+19 -44) 📝 `src/Core/Tests/BaseTest.php` (+9 -1) 📝 `src/Core/Tests/ImageProcessorTest.php` (+9 -27) 📝 `src/Core/Tests/ImageTest.php` (+3 -3) ➕ `src/Core/Traits/ParserTrait.php` (+49 -0) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>