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

WordPress Trac noreply at wordpress.org
Fri Sep 14 23:03:02 UTC 2018


#42663: Imagick support for stream wrappers
-------------------------------------------------+-------------------------
 Reporter:  calin                                |       Owner:  (none)
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  Awaiting
                                                 |  Review
Component:  Media                                |     Version:  trunk
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch 2nd-opinion reporter-      |     Focuses:
  feedback                                       |
-------------------------------------------------+-------------------------
Changes (by mikeschroder):

 * keywords:  has-patch 2nd-opinion => has-patch 2nd-opinion reporter-
               feedback


Comment:

 Hey @calin!
 Sorry for the wait on further comments, and thanks again for the patches.

 Yes, I've got a couple of questions:
 - Would it be possible to handle the writing part of this within
 `make_image()`, since this is much of the use-case for it? It's fine to
 extend `make_image()` for Imagick like GD does if that is necessary. If
 that's not preferred, any reasoning would be great.
 - Could you please walk me through the reasoning for `$imagick_filename`
 and its use?

 Thanks much!

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


More information about the wp-trac mailing list