티스토리 수익 글 보기

티스토리 수익 글 보기

Remove string type hint from webp_uploads_sanitize_image_format() to prevent possible fatal error by westonruter · Pull Request #1410 · WordPress/performance · GitHub
Skip to content

Conversation

@westonruter
Copy link
Member

Fixes #1409

Sanitization functions by definition should not have type hints since they are all about doing such type checking at runtime!

@westonruter westonruter added [Type] Bug An existing feature is broken [Plugin] Modern Image Formats Issues for the Modern Image Formats plugin (formerly WebP Uploads) labels Jul 29, 2024
@westonruter westonruter added this to the webp-uploads n.e.x.t milestone Jul 29, 2024
@github-actions
Copy link

github-actions bot commented Jul 29, 2024

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you’re merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: westonruter <westonruter@git.wordpress.org>
Co-authored-by: swissspidy <swissspidy@git.wordpress.org>
Co-authored-by: adamsilverstein <adamsilverstein@git.wordpress.org>
Co-authored-by: adam-ainsworth <adamainsworth@git.wordpress.org>

To understand the WordPress project’s expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@westonruter
Copy link
Member Author

Build for testing: webp-uploads.zip

Copy link
Member

@adamsilverstein adamsilverstein left a comment

Choose a reason for hiding this comment

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

Nifty!

@westonruter westonruter merged commit fb754ed into trunk Jul 30, 2024
@westonruter westonruter deleted the fix/webp-uploads-sanitize-image-format-function branch July 30, 2024 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Plugin] Modern Image Formats Issues for the Modern Image Formats plugin (formerly WebP Uploads) [Type] Bug An existing feature is broken

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Saving Media settings page causes 500 under certain conditions

4 participants