[Resolved] cred_form_validate not returning error

[Author]

Posts

[coc-admin]

I am trying to: Add a custom validation for my "serial number" field. This is displayed to the user as Serial Number on the form, but is actually stored as the Post Title. I am validating that the post is not a duplicate serial/title. When stepping thru the code, it sets the error message correctly. But the message does not show, and the post continues to be added (creating a duplicate). My code is below:

add_filter('cred_form_validate','validate_dup_serial_nbr',10,2);
function validate_dup_serial_nbr($error_fields, $form_data)
{
//field data are field values and errors
list($fields,$errors)=$error_fields;
//uncomment this if you want to print the field values
//print_r($fields);
//validate if specific form
if ($form_data['id'] == 1190 || $form_data['id'] == 1191) // Container Add or Edit forms
{
$serialExists = get_posts([
'post_type' => 'container',
'name' => $fields['post-title'],
'numberposts' => 1
]);

//check my_field value
if ($serialExists)
{
//set error message for my_field
$errors['post-title']='This Serial Number is already on file';
}
}
//return result
return array($fields,$errors);

Link to a page where the issue can be seen:

I expected to see: The message "This Serial Number is already on file"

Instead, I got: The post continued to add.This Serial Number is already on file

[Christian Cox]

I think you have the post title field label wrong, it should have an underscore not a hyphen. Like this:

$errors['post_title']='This Serial Number is already on file';

Let me know if that does not resolve the issue.

[coc-admin]

That was it. Besides the underscore-vs-hyphen, I also was not qualifying enough to get the value. I had just ['post_title'] instead of ['post_title']['value'].

Thanks!