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

WordPress Trac noreply at wordpress.org
Mon Jan 14 20:26:22 UTC 2019


#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                                       |
-------------------------------------------------+-------------------------

Comment (by calin):

 Hi @mikeschroder,

 I'm going to attempt another patch using the following approach: if the
 path is a stream wrapper, copy the file locally, apply changes and copy
 back. I think imagick does the same behinde the scenes, but the
 implementation seams flakey (see:
 https://github.com/mkoppanen/imagick/issues/225). This should also enable
 support for stream wrappers for older versions of imagick as well.

 Will upload the patch+tests in the following days.

 Btw. is there a deadline for including this in 5.1?

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


More information about the wp-trac mailing list