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

Add missing WebP Lexicon entry #16688

Merged
merged 1 commit into from
Jan 31, 2025
Merged

Add missing WebP Lexicon entry #16688

merged 1 commit into from
Jan 31, 2025

Conversation

smg6511
Copy link
Collaborator

@smg6511 smg6511 commented Jan 23, 2025

What does it do?

Adds new entry to source Lexicon file.

Why is it needed?

Prevent debug notices of missing array key.

How to test

Before applying this PR...

  1. Set logging level to Debug
  2. Visit the Media Browser. You should see multiple debug messages stating: Language string not found: "WebP"

After applying this PR...

  1. Revisit the Media Browser. You should no longer see the referenced debug messages

Related issue(s)/PR(s)

Resolves #16622

Add WebP entry to prevent errors
@smg6511 smg6511 changed the title Add WebP Lexicon entry Add missing WebP Lexicon entry Jan 23, 2025
@smg6511 smg6511 added the pr/review-needed Pull request requires review and testing. label Jan 23, 2025
@opengeek opengeek added this to the v2.9.0 milestone Jan 31, 2025
@opengeek opengeek merged commit bb8aaf6 into modxcms:2.x Jan 31, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/review-needed Pull request requires review and testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants