Post version issues

We shall see. Thanks for digging those up. I honestly feel like I have run circles around this side of the internet for the past few weeks. It’s nice to see some fresh material.

I haven’t made any changes to the code and am still reading through the add_meta_box example, but stepping through the routine has revealed that the Auto Draft is saved as soon as I “Add New,” which I knew, but seeing that in the database tells me when I set the post_ID, I am not grabbing the correct DB entry and so it is creating a new post then updating the old Auto Draft with empty fields.

I hope I can determine a routine to create/update the correct post_ID in the DB.

Mark, I appreciate the help thus far, I just want to make it clear before I continue that I have probably spent 40+ hours on this. I have been through many versions of the code and have refactored for various reasons at different times. I really don’t like to ask for help until I have absolutely exhausted every method I can come up with or search for.

Early on, I was able to save the custom fields to one post by using add_action (‘save_post’, ‘save_details;) and could save the post_title by using the wp_insert_post($post) method to another. I found I could not add a wp_insert_post() call within the function because it would create a recursion and cause an infinite loop.

At one point, I could stop it from adding 600 DB entries by getting the post_id set correctly, but it would still infinitely loop on that one post.

That said…

Currently, the code will not save either the custom fields or the post_title. It is only causing the Auto Save to transition from auto-save to publish.

function save_details() {
		if ( !wp_verify_nonce( $_POST['myplugin_noncename'], plugin_basename(__FILE__) )) {
			return $post_id;
		}

		// verify if this is an auto save routine. If it is our form has not been submitted, so we dont want
		// to do anything
		if ( defined('DOING_AUTOSAVE') && DOING_AUTOSAVE )
		return $post_id;

		// Check permissions
		if ( 'page' == $_POST['post_type'] ) {
			if ( !current_user_can( 'edit_page', $post_id ) )
			return $post_id;
		} else {
			if ( !current_user_can( 'edit_post', $post_id ) )
			return $post_id;
		}

		global $post;

		$event_team = $_POST["event_team"];
		$event_opponent = $_POST["event_opponent"];
		$event_date = $_POST["event_date"];

		$my_post_title = ($event_team ." vs. " .$event_opponent ." - " .$event_date);
		$my_post_name = sanitize_title($event_team ."-vs-" .$event_opponent ."-" .$event_date);

		// Add the content of the form to $post as an array
		$post = array(
				'ID' => $post_ID, //Are you updating an existing post?
				'post_content' => 'This is a custom post of the sport_event type. If you can see this text, something is wrong', //The full text of the post.
				'post_name' => $my_post_name, // The name (slug) for your postet the status of the new post.
				'post_title' => $my_post_title,
				'post_type' => 'sport_event',
				'post_status' => 'future',
				'post_author' => 1
			);

		add_post_meta($post_id, "event_team", $_POST["event_team"]);
		add_post_meta($post_id, "event_opponent", $_POST["event_opponent"]);
		add_post_meta($post_id, "event_date", $_POST["event_date"]);
		add_post_meta($post_id, "event_time", $_POST["event_time"]);
		add_post_meta($post_id, "territory", $_POST["territory"]);
		add_post_meta($post_id, "region", $_POST["region"]);
		add_post_meta($post_id, "game_audio", $_POST["game_audio"]);
		add_post_meta($post_id, "local_score", $_POST["local_score"]);
		add_post_meta($post_id, "opponent_score", $_POST["opponent_score"]);

		return $post_id;
		return $post;
	}

	add_action ('save_post', 'save_details', 99, 2);

Disregard the code in the last post. I had dirty code on my clipboard. There are a lot of issues that anyone can find. I am still refactoring and will update later.

Alright, took your suggestion, completely replaced the function.php with just the code from the example, and played in the sandbox for a while.

I got a pretty good feel for what is going on and where.

With the following code, I can create a new post, publish, and update.

function save_details($post_id) {
		if( 'POST' == $_SERVER['REQUEST_METHOD'] && !empty( $_POST['action'] ) && ($_POST['post_type'] == "sport_event")) {
			// verify if this is an auto save routine. If it is our form has not been submitted, so we dont want
			// to do anything

			// verify if this is an auto save routine. If it is our form has not been submitted, so we dont want
			// to do anything
			if ( defined('DOING_AUTOSAVE') && DOING_AUTOSAVE )
			return $post_id;

			$event_team = $_POST["event_team"];
			$event_opponent = $_POST["event_opponent"];
			$event_date = $_POST["event_date"];

			$my_post_title = ($event_team ." vs. " .$event_opponent ." - " .$event_date);
			$my_post_name = sanitize_title($event_team ."-vs-" .$event_opponent ."-" .$event_date);

			// Add the content of the form to $post as an array
			$post = array(
				'ID' => $post_id, //Are you updating an existing post?
				'post_content' => 'This is a custom post of the sport_event type. If you can see this text, something is wrong', //The full text of the post.
				'post_name' => $my_post_name, // The name (slug) for your post.
				'post_title' => $my_post_title,
				'post_type' => 'sport_event',
				'post_status' => 'future',
				'post_author' => 1
			);

			update_post_meta($post_id, "event_team", $_POST["event_team"]);
			update_post_meta($post_id, "event_opponent", $_POST["event_opponent"]);
			update_post_meta($post_id, "event_date", $_POST["event_date"]);
			update_post_meta($post_id, "event_time", $_POST["event_time"]);
			update_post_meta($post_id, "territory", $_POST["territory"]);
			update_post_meta($post_id, "region", $_POST["region"]);
			update_post_meta($post_id, "game_audio", $_POST["game_audio"]);
			update_post_meta($post_id, "local_score", $_POST["local_score"]);
			update_post_meta($post_id, "opponent_score", $_POST["opponent_score"]);

		}
		return $post;
	}
	add_action ('wp_insert_post','save_details');

…and get something similar to this:

Auto Draft Navasota Rattlers – 12 2010-08-28 05:00 pm Bryan – 10 away district

I have been at this point many times. What I am trying to do is take the team names and the date of the event, string them together, and save it as the ‘post_title’.

I have to use wp_insert_post instead of save_post because save_post does not properly update the custom fields. Because of this, I can’t call wp_insert_post within the function.

I guess that means my current question is how can I save the $post(array) to the database where I am already saving the custom fields?

On a side note, if I do add wp_insert_post($post) after I create the $post(array), it gets stuck in a loop, but everything in the array is saved just once and no custom fields are saved after.

The details of my project dictated that I would have to settle for half of the equation. The title will not update on the appropriate post and I end up with two posts, one with details and one with the proper title.

Since this is being used in house, I’m okay with it saying “Auto Draft.”

Currently the code has a lot of commented out sections so I can revisit this before next football season.

if( 'POST' == $_SERVER['REQUEST_METHOD'] && !empty( $_POST['action'] ) && ($_POST['post_type'] == "sport_event")) {

		function save_details($post_ID) {
			//if( 'POST' == $_SERVER['REQUEST_METHOD'] && !empty( $_POST['action'] ) && ($_POST['post_type'] == "sport_event")) {

				// verify if this is an auto save routine. If it is our form has not been submitted, so we dont want
				// to do anything
				if ( defined('DOING_AUTOSAVE') && DOING_AUTOSAVE )
				return $post_ID;

				update_post_meta($post_ID, "event_team", $_POST["event_team"]);
				update_post_meta($post_ID, "event_opponent", $_POST["event_opponent"]);
				update_post_meta($post_ID, "event_date", $_POST["event_date"]);
				update_post_meta($post_ID, "event_time", $_POST["event_time"]);
				update_post_meta($post_ID, "territory", $_POST["territory"]);
				update_post_meta($post_ID, "region", $_POST["region"]);
				update_post_meta($post_ID, "game_audio", $_POST["game_audio"]);
				update_post_meta($post_ID, "local_score", $_POST["local_score"]);
				update_post_meta($post_ID, "opponent_score", $_POST["opponent_score"]);
				update_post_meta($post_ID, "game_notes", $_POST["game_notes"]);

			}

		//}

		add_action ('wp_insert_post','save_details',99);

		/*$event_team = $_POST["event_team"];
		$event_opponent = $_POST["event_opponent"];
		$event_date = $_POST["event_date"];

		$my_post_title = ($event_team ." vs. " .$event_opponent ." - " .$event_date);
		$my_post_name = sanitize_title($event_team ."-vs-" .$event_opponent ."-" .$event_date);

		// Add the content of the form to $post as an array
		$post = array(
			'ID' => $post_ID, //Are you updating an existing post?
			'post_content' => 'This is a custom post of the sport_event type. If you can see this text, something is wrong', //The full text of the post.
			'post_name' => $my_post_name, // The name (slug) for your post.
			'post_title' => $my_post_title,
			'post_type' => 'sport_event',
			'post_status' => 'future',
			'post_author' => 1
		);

		wp_update_post($post);*/

	}

If you want to see what visitors will see, you can check it out at playbyreplay.com. I’m not linking it because I’m not one for shameless plugs.

Thanks for all your help Mark. I’m sure some other project I work on in the future will put the cherry on top of this one.