[wp-trac] [WordPress Trac] #42663: Imagick support for stream wrappers

WordPress Trac noreply at wordpress.org
Tue Oct 13 23:51:39 UTC 2020


#42663: Imagick support for stream wrappers
-------------------------------------------------+-------------------------
 Reporter:  calin                                |       Owner:
                                                 |  mikeschroder
     Type:  enhancement                          |      Status:  assigned
 Priority:  normal                               |   Milestone:  5.6
Component:  Media                                |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch 2nd-opinion has-unit-      |     Focuses:
  tests                                          |
-------------------------------------------------+-------------------------

Comment (by p00ya):

 I can spend some time this Friday updating the PR to shuffle more logic
 into `make_image`, assuming that's not too tight for beta cuts.

 I'll note that @joemcgill's messages didn't really justify the use of
 `make_image` for me.  There are some more points I could make in addition
 to the long post above (responding to the concerns about compatibility and
 separation of concerns), but I'm also happy to leave it at "respectfully
 disagree" and move the logic for Imagick in there anyway.  The maintainers
 are welcome to ping me on Slack to discuss further though.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/42663#comment:28>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list