Front End Delete and WordPress database error

Hi Long,

I have found the issue, I added the following filter a few days ago; which I can confirm that once removed fixed the database error.

add_filter( 'pre_delete_post', function( $null, $post, $force_delete ) {
    if( $force_delete == false ) {
        return wp_trash_post( $post->ID );
    }
    return $null;
}, 10, 3 );

I added the above code as all my custom post types were being force deleted every time (when set to false) https://support.metabox.io/topic/deleting-custom-post_type-with-frontend-submission-form-permanently-deletes-post/

Any ideas why?

Thanks

Hi

Ok so i have made 2 screen recordings.

I have removed the post_id from the shortcode and tested again with the 'pre_delete_post' filter active, you can see in the video that I receive the PHP database error in the logs.
https://drive.google.com/file/d/1a_p4exzFVSQlibYuhVhRZaYwKDlyjmwf/view?usp=sharing

I have also made another screen recording with the 'pre_delete_post' filter removed (commented out). I do not receive any PHP database errors.
https://drive.google.com/file/d/1RNFq7B-q62oa1HPBpLB3kCk2TAeskAjm/view?usp=sharing

Thanks

Hi Long,

sure, here is my code. Do you need anything else?

add_filter( 'rwmb_meta_boxes', 'dc_product_register_meta_boxes' );

function dc_product_register_meta_boxes( $meta_boxes ) {
    
    global $wpdb;

    $meta_boxes[] = [
        'title'   => 'Products',
        'post_types' => 'product',
        'storage_type' => 'custom_table', 
        'table'        => $wpdb->prefix . 'dc_products', 
        'id'      => 'dc-product',
        'context' => 'normal',
        'fields'  => [
            [
                'type'  => 'select',
                'name'  => 'Type',
                'id'    => 'type',
                'options' => array(
                    'Gift' => 'Gift',
                    'Ask' => 'Ask',
                ),
                'multiple' => false,
                'placeholder' => "",
                'select_all_none' => false,
                'required' => true,
            ],
            [
                'type'  => 'text',
                'name'  => 'Title', 
                'id'    => 'post_title',
                'class' => 'dc-post-title',
                'limit' => 50, 
                'required' => true,
            ],
            [
                'type'  => 'textarea',
                'name'  => 'Description',
                'id'    => 'description', 
                'required' => true,
            ],
            [
                'type'      => 'dc_flatpickr_calendar',
                'name'      => 'End Date (optional)', 
                'id'        => 'end_date', 
                'attributes' => [ //my custom attributes
                    'altInput'    => true,
                    'altFormat'   => 'd-m-Y H:i',
                    'inline'          => false,
                    'allowInput'  => false,
                    'disableMobile' => false,
                    'minDate'         => 'today',
                    'enableTime'  => true,
                    'time_24hr'   => false,
                    'enableSeconds' => false,
                    'defaultHour'     => 0,
                ],
            ],
            [
                'type'  => 'select',
                'name'  => 'Condition',
                'id'    => 'condition', 
                'options' => array(
                    'used' => 'Used',
                    'like new' => 'Like New',
                    'new' => 'New',
                ),
                'multiple' => false,
                'placeholder' => "",
                'select_all_none' => false,
                'required' => true,
            ],
            [
                'type'  => 'select',
                'name'  => 'Location',
                'id'    => 'location', 
                'options' => array(
                    'Hucknall' => 'Hucknall',
                ),
                'multiple' => false,
                'placeholder' => "",
                'select_all_none' => false,
                'required' => true,
            ],
            [
                'type'    => 'dc_image_to_base64', 
                'name'    => 'Images', 
                'id'      => 'images', 
                'attributes' => [ 
                    'max'  => 6, //max of 10 files
                    'buttonlabel'  => 'upload',
                    'maxWidth'  => 300, //px
                    'maxHeight' => 300, //px
                    'description' => '(drag to reorder)', 
                    //'maxFileSize'  => 8000000', //default wp upload size
                ],
                'saveToCustomFolder' => [ 
                    'enabled'  => true, //enable folder save (default false)
                    'folder'  => 'products', //foldername no slashes
                    'deleteOnUpdate'  => true, //force deletetion
                ],
                'required' => true,
            ],
        ],
    ];

    return $meta_boxes;
}

Hi

Ok i have removed the post_title field and can confirm that there is no error upon deletion.

So I have to create a duplicate database column to fix this issue? I just assumed that the post_title was automatically being populated in the post table.

also as a note I have added a check in the following filter for is_admin() which has fixed the bulk delete error message issue for now.

add_filter( 'pre_delete_post', function( $null, $post, $force_delete ) {
    if( $force_delete == false ) {
        if ( ! is_admin() ) {
            return wp_trash_post( $post->ID );
        }
    }
    return $null;
}, 10, 3 );