Take the 2-minute tour ×
WordPress Development Stack Exchange is a question and answer site for WordPress developers and administrators. It's 100% free, no registration required.

I have a custom post type with only 3 custom fields in it.

$post_types = get_post_meta($post->ID,'post_types',true);
$post_taxonomies = get_post_meta($post->ID,'post_taxonomies',true);
$post_terms = get_post_meta($post->ID,'post_terms',true);

I want to check if already exist post with this $post_types value, and if exist -> show message without publishing.

I created a metabox for this custom fields, which already have function for save_post action hook.

function sidebars_meta_save( $id )  {  
    if( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE ) return;  

    if( !isset( $_POST['sidebars_nonce'] ) || !wp_verify_nonce( $_POST['sidebars_nonce'], 'save_sidebars_meta' ) ) return;  
    if( !current_user_can( 'edit_post' ) ) return;  

    $allowed = array(  
        'p' => array()  
    );      

    if(isset( $_POST['post_terms'] )){
                    //check_exist_term - simple boolean function which check if 
                    //a post with this term already exist
        if(!check_exist_term($_POST['post_terms'])){
            if( isset( $_POST['post_types'] ) ){  
                update_post_meta( $id, 'post_types', wp_kses( $_POST['post_types'], $allowed ) ); 
            }
            if( isset( $_POST['post_taxonomies'] ) ) { 
                update_post_meta( $id, 'post_taxonomies', wp_kses( $_POST['post_taxonomies'], $allowed ) );
            }
                update_post_meta( $id, 'post_terms', wp_kses( $_POST['post_terms'], $allowed ) );
            }
        }else{
                        //if I will use wp_delete_post here, the message will not show
            //wp_delete_post($id);
            //wp_redirect('post-new.php?post_type=sidebars');
            //exit;
        }
    }
}  
add_action( 'save_post', 'sidebars_meta_save' );  

My redirect location function :

function sidebars_redirect_location($location,$post_id){
if( isset( $_POST['post_types'] ) ){  
    if(check_exist_term($_POST['post_terms'])){
        $status = get_post_status( $post_id );
        $location = add_query_arg('message', 21, $location);
                    //if I will use wp_delete_post here, I will be not able 
                    //to return to post edit page, because page will be already deleted
    }
}
return $location;
}
add_filter('redirect_post_location','sidebars_redirect_location',10,2);

My post updated messages function :

function sidebars_custom_messages($messages){
$messages['sidebars'][21] = 'Looks like you tried publishing a post with term which already exist!';
return $messages;
}    
 add_filter('post_updated_messages', 'sidebars_custom_messages');    

1) Where I should use wp_delete_post() If I want to first of all to show a message that this post already exist and just then delete this post and stay on this page? 2) How to show red message instead of yellow?

share|improve this question
add comment

1 Answer 1

up vote 5 down vote accepted

Checking if meta values exist

You can use get_posts() (or the WP_Query object) to query all posts which match the post meta you want to save. You will need to specify the post status (i.e. which statuses are you willing to ignore). Below is the untested code to this.

(For completeness I've left in arguments which could be omitted because they are given their default value). I also exlude the current post, in case the post is being updated - since otherwise that post will be returned and we'll think a post already exists with that meta data.

//Collect the terms
//I'm assuming that each of the values are strings. 
$meta_terms = $_POST['post_terms'];
$meta_taxs = $_POST['post_taxonomies'];
$meta_types = $_POST['post_types'];

//Query post type 'products' to check for posts whose meta values match the POSTed ones
$args = array(
  'post__not_in'=> array($id),
  'post_type' => 'product',
  'post_status' => array('publish','pending','draft','future','private'),
  'meta_query' => array(
      'relation' => 'AND',
      array(
        'key' => 'post_terms',
        'value' => $meta_terms,
        'compare' => '='
      ),
      array(
        'key' => 'post_taxonomies',
        'value' => $meta_taxs,
        'compare' => '='
      ),
      array(
        'key' => 'post_types',
        'value' => $meta_types
        'compare' => '='
      )
  )
);
$existingMeta = get_posts( $args );

if(empty($existingMeta)){
    //Go ahead and save meta data
}else{
    //Revert post back to draft status as shown in linked answer below.
}

Side remarks

The 'post_terms' etc seem quite generic. I would give them more descriptive and unique names - event more so if being used as key in the post meta table. Also, to avoid conflict you might want to send the data inside an array with a unique name: i.e.

<input type="text" name="myplugin[post_terms]" value=""/>

Then retrieve the data as $_POST['myplugin']['post_terms'] rather than $_POST['post_terms']. You can be reasonably sure that any data inside $_POST['myplugin'] has been over-written by another plug-in. Also you must use nonces to help verify source / intention.

Preventing the post from publishing

You can't prevent a post from being published (unless you use jQuery). However, immediately after a post is published, you can perform a check and, if desired, revert it back to draft status: See my solution to this related question.

Displaying a Custom message

In the linked solution, if a post had been reverted back to draft, I set the message variable to '10' so that the draft message is shown: 'Draft updated....':

add_filter('redirect_post_location','my_redirect_location',10,2);
function my_redirect_location($location,$post_id){
    //If post was published...
    if (isset($_POST['publish'])){
        //obtain current post status
        $status = get_post_status( $post_id );

        //The post was 'published', but if it is still a draft, display draft message (10).
        if($status=='draft')
            $location = add_query_arg('message', 10, $location)
    }

    return $location;
}

You can however specify your own message. The 10 refers to an array key where each value is a message. 0 is blank, with 1-9 being the default messages (post published, updated, etc.)

(This is untested) but you can add your own messages to this array:

add_filter('post_updated_messages', 'my_custom_messages');
function my_custom_messages($messages){
    //$message is an array of arrays. 
    //$message['post_type'] is an array of messages for post type 'post_type'
    //It is a non-assocative array. Value for 0 is blank. Keys 1-9 hold the default messages

    //Add a custom message to key 21 for post type 'my_post_type'
    $messages['my_post_type'][21] = 'Looks like you tried publishing, but something went wrong';

  return $messages
}

Then you can set the message to 21 rather than 10 as demonstrated above.

share|improve this answer
    
Hi Stephen! Thanks for quick response, I updated my question. For example user insert in custom fields values 1) Car 2) Bmw 3) X5 ...so I don't want to save this post if already created a post for 'Car','BMW','X5'. I would be very appreciated if you help me with solution ))) –  user13250 Mar 22 '12 at 11:27
    
See updated answer :D –  Stephen Harris Mar 22 '12 at 12:15
    
Is there a way to don't save a post? I mean when I'm clicking a publish button ....post will saved with status draft, but it will be in database..i'm trying to escape this –  user13250 Mar 23 '12 at 12:52
    
Yes you can use wp_delete_post - you'll then want to use wp_redirect to redirect the admin post list page. –  Stephen Harris Mar 23 '12 at 14:24
    
Almost got what I want. Just one question. At this moment I have a save function , redirect location function and post updated messages function. Where I should use wp_delete_post($id); function? If I use it in my save function -> I can't see the message, if I use it in redirect location function -> I see a white page with message 'You attempted to edit an item that doesn’t exist. Perhaps it was deleted?'. Look at updated question please! And also, I want to ask, how to make red message instead of yellow. –  user13250 Mar 23 '12 at 18:52
show 4 more comments

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

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