Skip to content

chore: fix misleading Imagick error message#9928

Merged
michalsn merged 2 commits intocodeigniter4:developfrom
michalsn:chore/imagick
Feb 5, 2026
Merged

chore: fix misleading Imagick error message#9928
michalsn merged 2 commits intocodeigniter4:developfrom
michalsn:chore/imagick

Conversation

@michalsn
Copy link
Member

@michalsn michalsn commented Feb 5, 2026

Description
This PR fixes the exception message that is shared between image handlers.

Closes #9927

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value (without duplication)
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

Copy link
Contributor

@neznaika0 neznaika0 left a comment

Choose a reason for hiding this comment

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

Can we add a note?

https://github.com/michalsn/CodeIgniter4/blob/636d3e8f548850d4dd9645afdb086408dc0ed2c4/user_guide_src/source/libraries/images.rst?plain=1#L39

.. note:: The ImageMagick handler requires the imagick extension. When working, you should specify the full paths to the image file.

@michalsn
Copy link
Member Author

michalsn commented Feb 5, 2026

Can we add a note?

Done.

@michalsn michalsn merged commit 396d951 into codeigniter4:develop Feb 5, 2026
50 checks passed
@michalsn
Copy link
Member Author

michalsn commented Feb 5, 2026

Thank you for the review, everyone.

@michalsn michalsn deleted the chore/imagick branch February 6, 2026 06:50
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.

Bug: Imagick handler on Windows

5 participants