Add context for Media

[frankiejarrett]

Media actions should be added as a core context of the Stream plugin.

Possible actions:

• Added
• Attached
• Unattached
• Updated
• Edited (images have an edit mode)
• Deleted

[shadyvb]

WIP 6bd7779

[shadyvb]

@fjarrett There is currently no way of detecting the 'attached' action, see: https://github.com/WordPress/WordPress/blob/0e959f06dce1040c88bf2aa729a947732197ff49/wp-admin/upload.php#L55
The only available way would be to track wp_redirect or _REQUEST on all requests, which sounds too much to me.

Also there is no such action as 'deattached', i'm unable to deattach an attachment from a post, neither from the post gallery dialog nor from the media library, also it doesn't have any trace in the referenced file.

[frankiejarrett]

@shadyvb Sad news! Well we will have to live without these for now. Closing the issue.

Are there hook placements that we can suggest for capturing these? If so, we should open a Trac ticket for it.

[shadyvb]

@fjarrett As for the 'attach' action, yes i think we can suggest a hook at https://github.com/WordPress/WordPress/blob/master/wp-admin/upload.php?source=c#L76 , something like do_action( 'media_attached', $attach, $parent_id ). It'll probably need to be called as well on the media upload dialog from post edit screen, probably here: https://github.com/WordPress/WordPress/blob/master/wp-admin/async-upload.php?source=c#L82

But for the unattach action, AFAIK it doesn't really exist at all!

What i would really want to push for is to add hooks for filesystem operations, like delete and edit.