Zend Form Edit and Zend_Validate_Db_NoRecordExists
Asked Answered
B

7

10

I am slowly building up my Zend skills by building some utility websites for my own use. I have been using Zend Forms and Form validation and so far have been happy that I have been understanding the Zend way of doing things. However I am a bit confused with how to use Zend_Validate_Db_NoRecordExists() in the context of an edit form and a field that maps to database column that has to be unique.

For example using this simple table

TABLE Test
(
  ID INT AUTO_INCREMENT,
  Data INT UNIQUE
);

If I was simply adding a new row to the Table Test, I could add a validator to the Zend Form element for the Data field as such:

$data = new Zend_Form_Element_Text('Data');
$data->addValidator( new Zend_Validate_Db_NoRecordExists('Test', 'Data') )

At form validation this validator will check that the contents of the Data element does not already exist in the table. Thus the insert into Test can go ahead without violating the Data fields UNIQUE qualifier.

However the situation is different when editing an existing row of the Test table. In that case the validator needs to check that the element value meets one of two mutually exclusive conditions conditions:

  1. The user has changed the element value, and the new value does not currently exist in the table.

  2. The user has Not changed the element value. Thus the value does currently exist in the table (and this is OK).

The Zend Validation Docs talk about adding a parameter to the NoRecordExists() validator for the purpose of excluding records from the validation process. The idea being to "validate the table looking for any matching rows, but ignore any hits where the a field has this specific value". Such a use case is what is needed for the validating the element when editing a table. The pseudo code to do this in 1.9 is like so (actually I got this from the 1.9 source code - I think the current docs may be wrong):

$data = new Zend_Form_Element_Text('Data');
$data->addValidator( new Zend_Validate_Db_NoRecordExists('Test', 'Data',
                     array ('field'=>'Data', 'Value'=> $Value) );

The problem is that the value that is to be excluded ($Value) is bound to the validator at the time it is instantiated (also when the form is instantiated). But when the form is editing a record, that value needs to be bound to the contents of the $data field when the form was initially populated with data - IE the Data value initially read from the Test table row. But in typical Zend patterns a form is instantiated and populated in two separate steps which precludes binding the exclude value to the desired element value.

The following Zend psuedo code marks where I would like the binding of $Value to the NoRecordExists() validator to occur (and note that this is a common Zend controller pattern):

$form = new Form() 
if (is Post) {
    $formData = GetPostData()
    if ($form->isValid($formData)) {
        Update Table with $formData
        Redirect out of here
    } else {
        $form->populate($formData)
    }
} else {
    $RowData = Get Data from Table
    $form->populate($RowData)     <=== This is where I want ('value' => $Value) bound
}

I could sub-class Zend_Form and override the populate() method to do a one-shot insertion of the NoRecordExists() validator on initial form population, but that seems like a huge hack to me. So I wanted to know what other people think and is there some pattern already written down that solves this problem?

Edit 2009-02-04

I've been thinking that the only decent solution to this problem is to write a custom validator and forget about the Zend version. My form has the record ID as hidden field, so that given the table and column names I could craft some SQL to test for uniqueness and exclude the row with an ID of such an such. Of course this started me thinking about how I would be tying the form to the dB layer that the Model is supposed to hide!

Butane answered 1/2, 2010 at 22:51 Comment(0)
B
1

After reviewing the overwhelming response I've decided that I'm going with a custom validator

Butane answered 9/2, 2010 at 13:52 Comment(0)
F
6

This is how it's done:

  1. I your FORM, you add this validator (for example email field):

 

$email->addValidator('Db_NoRecordExists', true, array('table' => 'user', 'field' => 'email'));
  1. Don't add custom error message for this since after that it didn't work for me, e.g.:

 

$email->getValidator('Db_NoRecordExists')->setMessage('This email is already registered.');

 

  1. In your Controller add this:

 

/* Don't check for Db_NoRecordExists if editing the same field */

    $form->getElement('email')
             ->addValidator('Db_NoRecordExists',
                                 false,
                                 array('table' => 'user',
                                       'field' => 'email',
                                       'exclude' => array ('field' => 'id', 'value' => $this->request->get('id'))));

And after this you do verifications, e.g.:

    if ($this->getRequest()->isPost())
            {
                if($form->isValid($this->getRequest()->getPost()))
                {

    ....

That's it!

Fluent answered 3/3, 2011 at 10:54 Comment(0)
M
4

This will also work :

$this->addElement('text', 'email', array(
        'label'      => 'Your email address:',
        'required'   => true,
        'filters'    => array('StringTrim'),
        'validators' => array(
            'EmailAddress',
             array('Db_NoRecordExists', true, array(
                    'table' => 'guestbook', 
                    'field' => 'email',
                    'messages' => array(
                        'recordFound' => 'Email already taken'
                    )
                )
            )
        )
    ));
Morey answered 20/8, 2012 at 14:21 Comment(0)
B
1

After reviewing the overwhelming response I've decided that I'm going with a custom validator

Butane answered 9/2, 2010 at 13:52 Comment(0)
A
1

Look at this one: Answer raised by me and well-solved by Dickie

Artair answered 21/3, 2011 at 15:25 Comment(0)
C
1
private $_id;

public function setId($id=null)
{
    $this->_id=$id;    
}

public function init()
{
    .....
    if(isset($this->_id)){
        $email->addValidator('Db_NoRecordExists', false, array('table' => 'user', 'field' => 'email','exclude' => array ('field' => 'id', 'value' => $this->_id) )); 
        $email->getValidator('Db_NoRecordExists')->setMessage('This email is already registered.');      
    }

Now u can use:

$form = new Form_Test(array('id'=>$id));
Coreligionist answered 6/5, 2011 at 22:32 Comment(0)
T
0

You could just call $form->getElement('input')->removeValidator('Zend_Validator_Db_NoRecordExists'); instead of supplying the exclusion.

Toleration answered 2/2, 2010 at 14:42 Comment(4)
@The Guy - That would seem to defeat the purpose of ensuring that the data is unique prior to doing the dB update during the edit cycle. I feel it also has the effect of polluting the controller with something that belongs in the form.Butane
Yeah, I agree it is a bit hackish. I am actually not very happy with the way Zend_Form is working right now. Albeit I still use an old ZF 1.6 style implementation :PToleration
Guy - I only started with 1.9 in December, so I am still learning things - but things like this make me go "hmmmm?"Butane
Watch out: removing the validation is a potential hazard to your db table. It means that you are able to update a record with the same value as another record which was exactly what you want to prevent.Scathe
C
0

I have just tried this example for email address uniqueness and it works perfectly with below stuffs :

1] In my form:

// Add an email element
    $this->addElement('text', 'email', array(
        'label'      => 'Email :',
        'required'   => true,
        'filters'    => array('StringTrim'),
        'validators' => array(
            'EmailAddress',
        )
    ));

Here's something special that I needed to add for unique email address to work:

    $email = new Zend_Form_Element_Text('email');
    $email->addValidator('Db_NoRecordExists', true, array('table' => 'guestbook', 'field' => 'email'));

2] In my controller:

$form->getElement('email')
     ->addValidator('Db_NoRecordExists',
                         false,
                         array('table' => 'guestbook',
                               'field' => 'email',
                               'exclude' => array ('field' => 'id', 'value' => $request->get('id'))));

if ($this->getRequest()->isPost()) {
        if ($form->isValid($request->getPost())) {

Hope it helps you people !

Thanks

Comose answered 16/8, 2012 at 8:23 Comment(0)

© 2022 - 2024 — McMap. All rights reserved.