Moving Collection Count Check to isValid Step.

Would it make sense to remove the thrown DomainException when calling $form->setData() (here, here, and here) and add a validator that will check for this? Returning an accurate error message becomes very ugly when using multiple collections on one form.

From:

try{
    $form->setData($data);

    if($form->isValid()){
        ...
    }else{
        ...
    }
}catch(DomainException $e){
    //If I had multiple collections, I would need to check $data to see which one caused the exception.
    $errors = [
        'myCollection' => [
             'count' => 'Some error message'
        ]
    ];
}

To:

$form->setData($data);

if($form->isValid()){
    ...
}else{
    ...
}

[weierophinney]

Honestly, no idea. 😄

Can you provide a pull request that demonstrates how you'd like to consume this from a userland perspective? That may help us better understand where it's falling short, and potentially other directions for addressing it.

Thanks!

[weierophinney]

This repository has been closed and moved to laminas/laminas-form; a new issue has been opened at laminas/laminas-form#29.