[wp-trac] [WordPress Trac] #60798: Investigate potentially failing Imagick PDF alpha channel test
WordPress Trac
noreply at wordpress.org
Fri Jun 14 14:42:08 UTC 2024
#60798: Investigate potentially failing Imagick PDF alpha channel test
-------------------------------------------+---------------------
Reporter: swissspidy | Owner: antpb
Type: defect (bug) | Status: closed
Priority: normal | Milestone: 6.6
Component: Media | Version:
Severity: normal | Resolution: fixed
Keywords: dev-feedback has-patch commit | Focuses:
-------------------------------------------+---------------------
Changes (by antpb):
* status: assigned => closed
* resolution: => fixed
Comment:
In [changeset:"58415" 58415]:
{{{
#!CommitTicketReference repository="" revision="58415"
Media: Use `version_compare()` for Imagick version check when removing
alpha.
Previously `remove_pdf_alpha_channel()` used `Imagick::getVersion()` to
validate the environment is capable of handling alpha. This was the
incorrect function to use to check the module version as it will only
provide the ImageMagick API version. This patch adjusts to instead use
`phpversion()` as this is the correct method to get the Imagick version
needed to determine alpha compatibility. This fixes a number of host tests
that have been correctly failing on subsets of environments.
Serendipidously, sometimes the API version was high enough to avoid
shining light on this problem.
Props swissspidy, joedolson, antpb.
Fixes #60798.
}}}
--
Ticket URL: <https://core.trac.wordpress.org/ticket/60798#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list