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

Remove all uses of FILTER_SANITIZE_STRING, since it's deprecated in newer PHP versions. #529

Merged
merged 2 commits into from
Jan 27, 2024

Conversation

jonathanstegall
Copy link
Member

What does this Pull Request do?

Replaces FILTER_SANITIZE_STRING with either FILTER_SANITIZE_SPECIAL_CHARS or FILTER_UNSAFE_RAW, depending on what the rest of the code is doing.

How do I test this Pull Request?

  • save plugin settings (especially fieldmaps)
  • save some data in push and pull operations

@jonathanstegall jonathanstegall added chore Pull request that does maintenance tasks patch pull request that requires a patch release, ex v2.1.2. This is the default for new releases. labels Jan 27, 2024
@jonathanstegall jonathanstegall added this to the v2.2.9 milestone Jan 27, 2024
@jonathanstegall jonathanstegall merged commit 3cfa276 into master Jan 27, 2024
3 checks passed
@jonathanstegall jonathanstegall deleted the php-8.2-replace-sanitize-string branch January 27, 2024 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Pull request that does maintenance tasks patch pull request that requires a patch release, ex v2.1.2. This is the default for new releases.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant