Enhance Sanitizer Component and README Updates #4
Conversation
This file contains 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
- Remove debugging var_dump statement that was causing risky test warnings - Clean up TrimSanitizer implementation to follow best practices - Improve code documentation with proper English comments
…erage BREAKING CHANGES: - PhoneSanitizer now removes non-numeric by default, formatting is optional - AlphanumericSanitizer has improved special character handling - All sanitizers now have consistent default behaviors Features: - Add comprehensive test suites for all sanitizers - Add new configuration options for better flexibility - Implement trait-based code sharing between sanitizers Improvements: - Better handling of edge cases in all sanitizers - More consistent and predictable behavior - Enhanced code organization and documentation - Clear separation between sanitization and formatting - Better error handling and validation Sanitizers affected: - HtmlSpecialCharsSanitizer - NormalizeLineBreaksSanitizer - StripTagsSanitizer - TrimSanitizer - EmailSanitizer - NumericSanitizer - PhoneSanitizer - UrlSanitizer - AlphanumericSanitizer Tests: - Add comprehensive test cases - Improve test coverage to 100% - Add edge case testing - Better test organization and documentation
BREAKING CHANGE: HtmlPurifierSanitizer internal structure has been reorganized - Create new namespace KaririCode\Sanitizer\Processor\Domain\HtmlPurifier - Split functionality into specialized classes: - Config/Configuration: Manage allowed tags and attributes - Dom/DomHandler: Handle DOM operations - NodeSanitizer/NodeSanitizer: Process DOM nodes - Cleaner/InputCleaner: Pre-process input - Cleaner/OutputCleaner: Post-process output - AttributeCleaner/AttributeCleaner: Handle attribute sanitization
… behavior Following the refactoring of HtmlPurifierSanitizer, the tests have been updated to: - Add explicit allowedTags configuration in each test case - Verify content preservation when tags are removed - Test hierarchical structure maintenance - Ensure attribute handling works correctly for allowed/disallowed tags
- Create namespace KaririCode\Sanitizer\Processor\Security\Filename - Split FilenameSanitizer into specific classes: * ExtensionHandler: handling of extensions and validations * FilenameParser: parsing and splitting of filenames * BasenameSanitizer: sanitization of the base name * Configuration: configuration management * FilenameSanitizer: orchestration of the process
- Add `searchQuery` and `plainTextContent` fields to the `JobApplication` class - Implement getter/setter methods for the new fields - Update `displayResults` function to show the new fields - Add data examples to test the new sanitizers
- Revised and enhanced descriptions in both English and Portuguese README. - Expanded usage examples for better clarity. - Added more details about sanitizer configuration options. - Updated integration and registry explanation sections.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This pull request contains several improvements and updates to the KaririCode Sanitizer component. The main changes include:
Sanitizer Enhancements:
HtmlPurifierSanitizerinto modular components to improve maintainability and reusability.FilenameSanitizerto adhere to the Single Responsibility Principle.HtmlPurifierSanitizertests to align with the new modular structure.Application Improvements:
Sanitizercomponent.Documentation: