-
Notifications
You must be signed in to change notification settings - Fork 1
File upload support with automatic handling #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Changes from 10 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
a1b5168
feat: Add file upload support with automatic $_FILES handling
koriym 9be3db8
feat: Add file upload demo with clean method parameter approach
koriym bcfc7e3
docs: Add file upload integration section to README
koriym a21f2be
docs: Add comprehensive framework comparison
koriym fd463af
feat: Add Koriym.FileUpload as suggested dependency
koriym 5c4f56b
docs: Add file upload integration design documentation
koriym 83b6fdf
refactor: Clean up code formatting and improve import statements in I…
koriym a05cf94
fix: Resolve all static analysis errors in file upload handling
koriym 2ceb721
feat: Enhance framework integration examples in documentation
koriym 45d2766
refactor: Remove unused parameter and improve method consistency
koriym 566aadc
feat: Add #[InputFile] attribute for file upload handling
koriym ecc28ee
feat: Implement hierarchical domain types for static analysis
koriym 4afd99d
feat: Achieve 100% code coverage with strict FileUpload validation
koriym 1ae1c14
refactor: Enhance domain types to eliminate generic array types
koriym 5d73646
fix: Add FileUpload classes to composer-require-checker whitelist
koriym e853a3a
test: Achieve 100% code coverage through public interface tests only
koriym 5de89cc
docs: Update documentation to reflect 100% coverage and InputFile att…
koriym 23a272e
docs: Update demo to use consistent method invocation pattern
koriym 70e3bad
docs: Add comprehensive framework integration guide
koriym 5729f5a
Update README.md
koriym 26f122f
docs: Fix bare URL formatting in README
koriym fa8f49c
docs: Reorder badge display and fix typo in README
koriym a1857be
feat: Implement allowedExtensions validation for file uploads
koriym e67c413
refactor: Extract FileUploadFactory following SRP and reduce complexity
koriym ef22cfe
feat: Achieve 100% code coverage and add comprehensive validations
koriym 52596f4
fix: Separate PHP 8.2+ specific tests to avoid syntax errors
koriym 1ba7131
revert: Remove PHP 8.2+ specific test due to compatibility issues
koriym 0ad20dd
fix: Add coverage ignore for PHP 8.2+ specific code
koriym File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.