[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