Skip to content

Conversation

@sksaju
Copy link
Contributor

@sksaju sksaju commented Jul 15, 2024

Description of the Change

Updated the enshrined/svg-sanitize package to 0.19.0 to fix the PHP 8.3 compatibility issue.

Closes #210

How to test the Change

Changelog Entry

Added - New feature
Changed - Existing functionality
Deprecated - Soon-to-be removed feature
Removed - Feature
Fixed - Bug fix
Security - Vulnerability

Credits

Props @username, @username2, ...

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

@sksaju sksaju requested review from dkotter and jeffpaul as code owners July 15, 2024 09:18
@github-actions github-actions bot added this to the 2.3.0 milestone Jul 15, 2024
@github-actions github-actions bot added the needs:code-review This requires code review. label Jul 15, 2024
@sksaju sksaju changed the title update enshrined/svg-sanitize to 0.19.0 Fix #210 | PHP 8.3 compatibility Jul 15, 2024
@sksaju sksaju requested a review from darylldoyle July 15, 2024 09:22
@jeffpaul jeffpaul requested review from a team and faisal-alvi and removed request for a team, darylldoyle, dkotter and jeffpaul August 14, 2024 01:51
Copy link
Contributor

@faisal-alvi faisal-alvi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the PR @sksaju. Could you please address the PHP compatibility and PHPCS pipeline issues?

image

@TylerB24890
Copy link
Contributor

PHP 7.4 Compatibility has been resolved.

@jeffpaul jeffpaul requested a review from faisal-alvi August 28, 2024 16:11
Copy link
Contributor

@faisal-alvi faisal-alvi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed that the get_magic_quotes_gpc function is not used in the library after version bump.

@faisal-alvi faisal-alvi merged commit 2ebe1a4 into develop Sep 2, 2024
@faisal-alvi faisal-alvi deleted the fix/210-php-8.3-compatibility branch September 2, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs:code-review This requires code review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PHP 8.3 compatibility

3 participants