[wp-trac] [WordPress Trac] #38959: Allow sideloading images without assigning to a post.

WordPress Trac noreply at wordpress.org
Thu Apr 11 13:56:14 UTC 2019


#38959: Allow sideloading images without assigning to a post.
--------------------------------------------------+------------------------
 Reporter:  dotancohen                            |       Owner:  (none)
     Type:  enhancement                           |      Status:  closed
 Priority:  normal                                |   Milestone:
Component:  Media                                 |     Version:
 Severity:  normal                                |  Resolution:  duplicate
 Keywords:  has-patch needs-testing dev-feedback  |     Focuses:
--------------------------------------------------+------------------------
Changes (by antpb):

 * status:  new => closed
 * resolution:   => duplicate
 * milestone:  Awaiting Review =>


Old description:

> Currently one cannot sideload an image without assigning it to a post.
> After careful review I see that this feature can be enabled in the
> current WordPress by simply providing a default Post ID of 0.
>
> The attached post enables this feature.
>
> In case one wonders what the use case is, consider that Wordpress 4.x is
> no longer simply a blogging platform but now a full featured CMS with an
> expressive REST API that allows for very rapid development.

New description:

 Currently one cannot sideload an image without assigning it to a post.
 After careful review I see that this feature can be enabled in the current
 WordPress by simply providing a default Post ID of 0.

 The attached post enables this feature.

 In case one wonders what the use case is, consider that WordPress 4.x is
 no longer simply a blogging platform but now a full featured CMS with an
 expressive REST API that allows for very rapid development.

--

Comment:

 Hi @dotancohen , thank you so much for contributing this patch! The
 solution is very solid. This ticket was discussed in the most recent Media
 meeting and we agreed that this should move forward into the next release.
 The only tweak we'd suggest is instead of setting the post_id value to `0`
 it should maybe be set to `null`. In the process of discussing this
 ticket, we identified another trac ticket that almost identically solves
 this the way that you approached it, but with the null value instead of 0.
 #43579

 We are going to close this ticket in favor of #43579 as it already has the
 change to `null` but I will make sure you both get props as you both have
 provided an excellent contribution. Thanks so much! if you have any other
 input, please feel free to jump into #43579 where we will be moving it
 forward.

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


More information about the wp-trac mailing list