notification email after media upload

For starters, do not ever edit WordPress core files. That’s the quickest way to bring your site down and/or introduce a security hole. You will effectively need to write a plugin to do this. Or you could look around and see if someone has already created one.

I searched a lot but could not find a suitable plugin

I just want to add something like this
<?php wp_mail( $to, $subject, $message, $headers, $attachments ); ?>
at the end of the upload function, i don’t think that this code will introduce a security hole…or not?

But i don’t find where to put this code…in my upload.php file

Thanks

i found this but doesn’t work
where i should put this code?

add_action('admin_init', 'redirect_after_media_save');
function redirect_after_media_save() {

    global $pagenow;

    // when visiting the upload screen, we save the number of attachments
    if ( $pagenow == 'media-new.php' ) {
        $attachments_before = array_sum((array)wp_count_attachments());
        update_option('count_attach_before', $attachments_before);
    }

    if ( $pagenow == 'upload.php' ) { // we are on media library page

        // get attachments count before and after upload
        $attachments_before = get_option('count_attach_before');
        $attachments_after = array_sum((array)wp_count_attachments());

        if (
            // there are new files uploaded
            ( wp_get_referer() == admin_url('media-new.php') && $attachments_after > $attachments_before )
            ||
            // or we have just edited media file
            isset($_GET['posted'])
        ) {
                // redirect to desired location
               wp_mail('[email protected]', 'subject', 'message', $headers, $attachments );
                exit;
        }
    }
}

[Moderator Note: Please post code & markup between backticks or use the code button. Your posted code may now have been permanently damaged by the forum’s parser.]

Listen to esmi.. she knows what she is talking about!

Instead of editing core files; you will want to create a plugin (or use your child themes functions.php file.

There is no action which can be fired during the ‘save media’ process. So, what we have to do is count total attachments BEFORE the upload process.. and compare it to total attachments AFTER the upload process. If the ‘before’ is less than the ‘after’.. we know an attachment was uploaded.

This Function (on pastebin.com) should work for you. Just place it in your child theme functions.php file (or make it a plugin).

Thank Josh

but doesn’t work in my functions.php file

Did you define your variables in the wp_mail() function?

For example, did you define the $to, $subject, $message, and $headers variables?

i wrote

wp_mail( ‘[email protected]’, $subject, $message, $headers, $attachments );

not enough?

Nope. How is WP going to know what is inside those other variables??

Let’s try this:

$to = '[email protected]';
$subject = 'New WP Attachment!';
$message = 'An attachment has been uploaded.';
$headers = 'From: My Name <[email protected]>' . "\r\n";

wp_mail( $to, $subject, $message, $headers );

yesss
super

I have to attend this forum to learn something more…

THANKS

Lol – My pleasure ??

(Please mark your thread as ‘resolved’. You can do this over on the right side of this page.)

second lesson…

to insert the logged in user’s email….what variable should i put here?

$headers = ‘From: My Name <[email protected]>’ . “\r\n”;

Well.. let’s try this:

$current_user = wp_get_current_user();  // Get current user object
$current_user_email = $current_user->user_email;  // Get current user email
$current_username = $current_user->user_login;  // Get current user login name

$to = 'redacted email';
$subject = 'New WP Attachment!';
$message = 'An attachment has been uploaded.';
$headers = 'From: '.$current_username.' <'.$current_user_email.'>' . "\r\n";

wp_mail( $to, $subject, $message, $headers );

Here, we are defining the username and the user email from the currently logged in user. Then, we are concatenating the variables to the $headers variable string.

Watch the syntax… but that should work for you.

Hi Josh
the script (also the old one with fixed email) works only sometimes

If i stay logged don’t come the notification email

I have to log out and re-log with other user.

Do you have some solotuons for that?

Tks