Enforce conditions only for draft posts using WyPiekacz, ignore pending and published posts

Question

I use WyPiekacz to set various conditions for authors when creating posts, for example checking if the post has a featured image or how many tags that are being used. It works fine.

However, these conditions are checked every time the post is submitted for review, updated or published. I want WyPiekacz to only check the conditions if the post is in Draft status. How can I do that?

The problem that I am experiencing currently is that for instance an editor changes the amount of tags and updates the pending post which then fails the tag rules set in WyPiekacz and thus the post gets changed back to draft status and the editor cannot access it again. Editors are only allowed to edit pending posts.

Answer 1

The check is invoked after a call to get_option( 'wypiekacz_allow_skip_rules' ).

When you filter pre_option_wypiekacz_allow_skip_rules and returns something different than FALSE, it should stop the check early. Not tested.

add_filter(
    'pre_option_wypiekacz_allow_skip_rules',
    'wpse_100503_wypiekacz_for_drafts_only'
);

function wpse_100503_wypiekacz_for_drafts_only( $bool )
{
    if ( 'draft' === get_post_status( $GLOBALS['post'] ) )
        return $bool;

    return 0;
}

To prevent the post status reset, something like this could work:

add_filter(
    'wypiekacz_check_post',
    'wpse_100503_wypiekacz_prevent_draft_reset',
    10,
    4
);

function wpse_100503_wypiekacz_prevent_draft_reset( $errors, $text, $title, $post_data )
{
    if ( empty ( $errors ) )
        return $errors;

    if ( 'pending' === $post_data->post_status )
        remove_filter( 'query', array( $GLOBALS['wp_wypiekacz'], 'kill_sql_query' ) );
}

But, honestly, the plugin code is a mess.