media_handle_upload() un required second parameter

[Tkama]

Why second parameter of media_handle_upload( $file_id, $post_id ) is required?

It's better to define it as 0 and make it optional. In some cases it's more convenient! And we lose nothing if do so...

[desrosj]

Hi @Tkama,

I don't know that there's any real benefit to making this change. Future function calls would no longer require the parameter, but requiring the parameter also requires the developer to make a conscious decision about where the image should be attached, if at all.

Marking as a close candidate, but also for a 2nd-opinion.

[antpb]

Moving this to 6.2 as this change improves the ease of using the function with little to no potential issues from setting default to 0.

This change should save a developer from having to read the docs to figure out why the function is failing.

[desrosj]

Removing the close keyword since @antpb has volunteered to own this one.

[costdev]

Make the $post_id parameter optional. Default 0.

[azaozz]

Replying to antpb:

This change should save a developer from having to read the docs...

Actually I tend to agree with @desrosj comment from 4 years ago:

requiring the parameter also requires the developer to make a conscious decision about where the image should be attached

WP adds context to images and other uploads by saving the post they were uploaded to. This context can be used for many things like permission settings, etc.

Imho would be nice to remind developers to add the post_ID for new uploads if possible. Perhaps this ticket can be repurposed to expand the docs for media_handle_upload() and explain in more details why it is good to add $post_id when possible.

[costdev]

This ticket was discussed during the bug scrub. As this ticket still needs some work and 6.2 Beta 1 is being released today, I'll move this to Future Release.

As the next step seems to be a docs-only change, if this is ready later in the cycle, feel free to pull it back into the milestone.

Additional props: @mukesh27