[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