[PR #70] [MERGED] Optimization part 1 #868

New issue

Closed

opened 2026-03-15 15:40:06 +03:00 by kerem · 0 comments

kerem commented

2026-03-15 15:40:06 +03:00

Owner

📋 Pull Request Information

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

Base: master ← Head: optimization-part-1

📝 Commits (5)

7a80e19 optimization-part-1: Refactor current architectire, files structure and classes name
722c7ef Merge branch 'master' into optimization-part-1
18400dc Rename functions and edit long variable names
ef60a2a Rename class faceDetectProcessor
639027f Update benchmark.sh and benchmark result in Readme

📊 Changes

19 files changed (+501 additions, -454 deletions)

View changed files

➖ .github/ISSUE_TEMPLATE.md (+0 -2)
📝 README.md (+10 -10)
📝 app.php (+33 -14)
📝 benchmark.sh (+10 -3)
📝 src/Core/Controller/CoreController.php (+5 -5)
📝 src/Core/Controller/DefaultController.php (+2 -2)
📝 src/Core/Entity/Image.php (+0 -18)
➕ src/Core/Handler/ImageHandler.php (+175 -0)
➕ src/Core/Processor/FaceDetectProcessor.php (+67 -0)
📝 src/Core/Processor/ImageProcessor.php (+2 -149)
➕ src/Core/Processor/Processor.php (+65 -0)
➖ src/Core/Provider/StorageProvider.php (+0 -113)
📝 src/Core/Resolver/ControllerResolver.php (+2 -1)
➖ src/Core/Service/CoreManager.php (+0 -119)
➕ src/Core/StorageProvider/LocalStorageProvider.php (+50 -0)
➕ src/Core/StorageProvider/S3StorageProvider.php (+61 -0)
📝 tests/Core/BaseTest.php (+4 -4)
📝 tests/Core/Entity/ImageTest.php (+6 -5)
📝 tests/Core/Service/ImageProcessorTest.php (+9 -9)

📄 Description

#62

_{🔄 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/70 **Author:** [@sadok-f](https://github.com/sadok-f) **Created:** 6/30/2017 **Status:** ✅ Merged **Merged:** 6/30/2017 **Merged by:** [@sadok-f](https://github.com/sadok-f) **Base:** `master` ← **Head:** `optimization-part-1` --- ### 📝 Commits (5) - [`7a80e19`](https://github.com/flyimg/flyimg/commit/7a80e1934aa1a00d7a484fe96dcc345c33731013) optimization-part-1: Refactor current architectire, files structure and classes name - [`722c7ef`](https://github.com/flyimg/flyimg/commit/722c7ef7faf2affa13a242227d70f00a079d882b) Merge branch 'master' into optimization-part-1 - [`18400dc`](https://github.com/flyimg/flyimg/commit/18400dcff76dc8be7cce0099b668ec02ac79b3ef) Rename functions and edit long variable names - [`ef60a2a`](https://github.com/flyimg/flyimg/commit/ef60a2a6f16f11e52204cdb7d30c4e4e95c28afc) Rename class faceDetectProcessor - [`639027f`](https://github.com/flyimg/flyimg/commit/639027f13e72f7155a3f4ce1ec67ce028ff85d89) Update benchmark.sh and benchmark result in Readme ### 📊 Changes **19 files changed** (+501 additions, -454 deletions) <details> <summary>View changed files</summary> ➖ `.github/ISSUE_TEMPLATE.md` (+0 -2) 📝 `README.md` (+10 -10) 📝 `app.php` (+33 -14) 📝 `benchmark.sh` (+10 -3) 📝 `src/Core/Controller/CoreController.php` (+5 -5) 📝 `src/Core/Controller/DefaultController.php` (+2 -2) 📝 `src/Core/Entity/Image.php` (+0 -18) ➕ `src/Core/Handler/ImageHandler.php` (+175 -0) ➕ `src/Core/Processor/FaceDetectProcessor.php` (+67 -0) 📝 `src/Core/Processor/ImageProcessor.php` (+2 -149) ➕ `src/Core/Processor/Processor.php` (+65 -0) ➖ `src/Core/Provider/StorageProvider.php` (+0 -113) 📝 `src/Core/Resolver/ControllerResolver.php` (+2 -1) ➖ `src/Core/Service/CoreManager.php` (+0 -119) ➕ `src/Core/StorageProvider/LocalStorageProvider.php` (+50 -0) ➕ `src/Core/StorageProvider/S3StorageProvider.php` (+61 -0) 📝 `tests/Core/BaseTest.php` (+4 -4) 📝 `tests/Core/Entity/ImageTest.php` (+6 -5) 📝 `tests/Core/Service/ImageProcessorTest.php` (+9 -9) </details> ### 📄 Description #62 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>