Что не так с моей формой?

Question

У меня возникла проблема, когда я пытаюсь сохранить информацию о форме в базе данных. Моя форма кажется недействительной даже после ручной установки идентификатора Театра для каждого театра в выбранной сети. Вот соответствующая часть моего модуля actions.class.php:

Вот executeCreate():

public function executeCreate(sfWebRequest $request) { 
    $this->form = $this->configuration->getForm(); 
    $this->showing = $this->form->getObject(); 
    $this->processCreateForm($request, $this->form); 
    $this->setTemplate('new'); 
} 

и теперь processCreateForm():

protected function processCreateForm(sfWebRequest $request, sfForm $form) { 
    $form->bind($request->getParameter($form->getName()), $request->getFiles($form->getName())); 

    $form_name = $form->getName(); 
    $parameters = $request->getParameter($form_name); 
    $network_id = $parameters['network_id']; 

    $theaters_list = Doctrine_Query::create() 
      [...] 
      ->execute(); 

    foreach ($theaters_list as $theater) { 
     $form->getObject()->setTheaterId($theater->theater_id); 
     $form->bind($request->getParameter($form->getName()), $request->getFiles($form->getName())); 

     if ($form->isValid()) { 
      $showing = $form->save(); 
     } else { 
      foreach ($form->getErrorSchema()->getErrors() as $key => $error) { 
       echo '<p>' . $key . ': ' . $error . '</p>'; 
      } 
     } 
    } 
    $this->getUser()->setFlash('update_success', true); 
    $this->setTemplate('new'); 
} 

Вот выход:

Спасибо за вашу помощь

Answer 1

Есть две странные вещи, происходящие здесь, которые я думаю, сломать свой код.

1. Вы запустите метод bind() дважды, который может быть сброс ваших значений на объекте.

2. Я не думаю, что метод getObject() возвращает объект по ссылке.

Итак, когда вы запустите:

$form->getObject()->setX($val); 
    $form->save(); 

затем обновить поле объекта, возвращенного формой, но затем сохранить исходный объект, который по-прежнему привязан к форме.

Попробуйте делать что-то вроде этого:

$myObject = $form->updateObject()->getObject(); 
    $myObject->setX($value); 
    $myObject->save(); 

updateObject() важно, если вы используете форму для редактирования существующего объекта, а не создавать новый. Без этого вы получите старые значения объекта.

Если вы хотите запустить его в цикле, вы можете выполнить только настройку и сохранить часть. Таким образом, вы бы что-то подобное в вашем processCreateForm:

protected function processCreateForm(sfWebRequest $request, sfForm $form) 
{ 
    $form->bind($request->getParameter($form->getName()), $request->getFiles($form->getName())); 

    if ($form->isValid()) { //You can check the validity of your form at this point. 

     //Create $theatersList 
     ... 

     $myObject = $form->updateObject(); 

     foreach ($theatersList as $theater) { 
      $myObject->setTheaterId($theater->theater_id); 
      $showing = $myObject->save(); 

      //Do something with $showing 

     } 
    } else { 
     //Print the errors. 
    } 
} 

Используя этот код вы можете уничтожить виджет theatre_id в вашей форме, как это не должно быть установлено пользователем, и не должен быть частью формы Проверка.

EDIT

Некоторые изменения в коде:

protected function processCreateForm(sfWebRequest $request, sfForm $form) 
{ 
    $form->bind($request->getParameter($form->getName()), $request->getFiles($form->getName())); 

    if ($form->isValid()) { //You can check the validity of your form at this point. 

     //Create $theatersList 
     ... 

     $myObject = $form->updateObject(); 
     $myObjectVars = $myObject->toArray(); 

     foreach ($theatersList as $theater) { 

      $myNewObject = new SomeClass(); 
      $myNewObject->fromArray($myObjectVars); 
      $myNewObject->setTheaterId($theater->theater_id); 
      $showing = $myNewObject->save(); 

      //Do something with $showing 

      $myNewObject->free(); 
      unset($myNewObject); 
     } 
    } else { 
     //Print the errors. 
    } 
}