Skip to content
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

Sanitizer constructor documents array instead of spread #108

Merged
merged 1 commit into from
May 15, 2023

Conversation

dpi
Copy link
Contributor

@dpi dpi commented Mar 23, 2023

Fix errors surfaced by PHPStan...

Parameter #2 ...$items of class TYPO3\HtmlSanitizer\Sanitizer constructor expects array<TYPO3\HtmlSanitizer\Visitor\VisitorInterface>|TYPO3\HtmlSanitizer\Behavior, TYPO3\HtmlSanitizer\Visitor\CommonVisitor given.

@ohader
Copy link
Member

ohader commented Mar 23, 2023

Seems to be fine! Thx for the PR and the hint - introducing PHPstan to this library and the GitHub action makes sense!

@ohader ohader merged commit 476383a into TYPO3:main May 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants