1

I made this action hook which will send an email to the users every time a new post in the custom post type "events" is made.

function email_members($post_id) {
global $wpdb;

$usersarray = $wpdb->get_col("SELECT user_email FROM wp_users");
$users = implode(",", $usersarray);

if( ( $_POST['post_status'] == 'publish' ) && ( $_POST['original_post_status'] != 'publish' ) ) {
    $subject = 'Event Reminder';

    $post_url = get_permalink($post_id);
    $post_title = get_the_title($post_id);
    $event_date = get_post_meta( $post_id, 'ch_event_date', true );

    $message = "APAC Ministries Event Reminder for: ".$post_title."\n\n";
    $message .= "Date: ".$event_date;

    wp_mail($users, $subject, $message );
  }
}

add_action('publish_th_events', 'email_members');

However, the $event_date is always empty. I tried running this code in a different file and it works:

global $post;
$test = get_post_meta($post->ID, 'ch_event_date', true);
echo $test;

I have also tried using global $post; and using $post->ID as the ID. I have also tried getting the terms, and it displays fine. I'm just having trouble displaying the meta.

Any alternative solutions are welcome. Thanks!

8
  • This may sound stupid, but did you try adding echo $event_date? Sep 27, 2013 at 2:44
  • Why do I have to echo it if I'm appending its value to $message? Also, I tried adding an if-else statement which displays the word "Empty" if $event_date is empty. And it shows "Empty" T_T Sep 27, 2013 at 2:50
  • Because php can be screwy every now and again. Its all I could think of off the top of my head, only because get_the_... statements only return the value. Something isn't adding up...when I get by my computer I'll try and help some more. Sep 27, 2013 at 3:02
  • Show your do_action('publish_th_events', ....
    – brasofilo
    Sep 27, 2013 at 5:01
  • I suspect that 'publish_th_events' is passing a post object, not a post ID. Sep 27, 2013 at 5:06

2 Answers 2

1

get_post_meta as seen here is nothing more than a wrapper for get_metadata. Now when you say the meta you're getting back, if its not null but rather an empty string, it means you're hitting the end of get_metadata as seen here. With true defined as the third variable of your get_post_meta call, you should be seeing an empty string if this is the case. If you change it to false, you should see an empty array.

Now to the issue of why you're at the bottom of get_metadata.

Based on what I can see, the only way to get there is that you don't actually have a post_meta_field called "ch_event_date" defined for that post_object.

Perhaps, try doing this right above your call to get "ch_event_date" as a test: update_post_meta($post_id,'ch_event_date', 'Chris figured it out');

Then dump that get_post_meta call.

0

Okay, so I just tried it out on a site I'm developing and using something similar to below, I was able to do what you're having trouble doing, using a custom field.

$message = print "APAC Ministries Event Reminder for: " . $post_title . "\n\n";
$message .= print "Date: " . $event_date;

The only reasoning that makes any sense to me is using the get_the_... is only returning the values for future use, not for displaying them. In the Codex it appears that the function automatically prints or echoes the $users and $subject

If you look closely at your test where it displays the date properly, you echo it. I hope this helps!

2
  • I added "print" but it produces an error: Warning: Cannot modify header information - headers already sent by (output started at /home/wishbone/public_html/demo/apacministries/wp-content/themes/apacministries/functions.php:416) in /home/wishbone/public_html/demo/apacministries/wp-admin/post.php on line 222 and Warning: Cannot modify header information - headers already sent by (output started at /home/wishbone/public_html/demo/apacministries/wp-content/themes/apacministries/functions.php:416) in /home/wishbone/public_html/demo/apacministries/wp-includes/pluggable.php on line 875 Sep 27, 2013 at 5:26
  • Check out this thread or change it to echo them? This is an odd case... Sep 27, 2013 at 6:41

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.