[Plugin: Media Author] FEATURE REQUEST: Default to current author.

You got a point here.

The normal Edit Media screen gets the correct attachment_id as $_GET['attachment_id'] for the plugin.

The Add Media screen for a page or post only gets the current post_id of the page or post as $_GET['post_id'] and has no information about the actual attachment owner.

So either we could add a patch which sets the current post author as attachment author (as you propose, see patch #1) or the “Media Author” select should be removed (not included) in the Add Media screen popup (maybe better, see patch #2).

Locate this in file media_author.php:

/*
 * This is the usable method, as of WordPress 2.9.0
 */
function media_author_plugin_dropdown_2($args) {
	$author_id = get_post($_GET['attachment_id'])->post_author;
...

For patch #1 replace with this:

function media_author_plugin_dropdown_2($args) {
	if ( isset ( $_GET['attachment_id'] ) ) {
		$post_id = $_GET['attachment_id'];
	} else if ( isset ( $_GET['post_id'] ) ) {
		$post_id = $_GET['post_id'];
	} else {
		return $args;
	}
	$author_id = get_post( $post_id )->post_author;
...

For patch #2 replace with this:

function media_author_plugin_dropdown_2($args) {
	if ( isset ( $_GET['attachment_id'] ) ) {
		$post_id = $_GET['attachment_id'];
	} else {
		return $args;
	}
	$author_id = get_post( $post_id )->post_author;
...

Found a third patch, which is the best so far, it shows the correct attachment owners also in the Add Media screen.

Locate this in file media_author.php:

/*
 * This is the usable method, as of WordPress 2.9.0
 */
function media_author_plugin_dropdown_2($args) {
	$author_id = get_post($_GET['attachment_id'])->post_author;
...

For patch #3 replace with this, note the added “$post = false” arg:

function media_author_plugin_dropdown_2( $args, $post = false ) {
	if ( isset ( $_GET['attachment_id'] ) ) {
		$post_id = $_GET['attachment_id'];
	} else if ( $post ) {
		$post_id = $post->ID;
	} else {
		return $args;
	}
	$author_id = get_post( $post_id )->post_author;
...

Also locate this at the end of file media_author.php:

add_filter('attachment_fields_to_edit', 'media_author_plugin_dropdown_2', 5);

For patch #3 replace with this, note the added “, 2” at the end:

add_filter('attachment_fields_to_edit', 'media_author_plugin_dropdown_2', 5, 2);

@john: Feel free to add the change yourself. Won’t have time for that today.

I actually consider it as a bugfix as get_post($_GET['attachment_id'])->post_author returned an invalid value in Add Media screen before.

Hello John,

Before I begin, I would like to congratulate you for your work! Recently, I started using the WordPress 3.5 where I’ve added the Media Author 1.0.4. Unfortunately, I have problems in changing the albums’ authors. For example, if I have an album with 50 photos and I want to change the author, I can’t change it unless I take every single photo, it won’t let me to do it at once. And it’s taking me too much time. Could you please tell me if there is a much easier way to do these changes? I would much appreciated!

Thank you!

Onut