[ZF][ZendFramework] Pole select waliduje się niepoprawnie Opcje

[Vill]

Mam pole select, które zawiera wartość nullową i kilku klientów do wyboru. Walidacja przechodzi poprawnie tylko w przypadku wyboru wartości null, w przypadku wyboru jakiegoś klienta to nie przechodzi. Nie mam ustawionych na tym polu żadnych walidatorów ani filtrów.

Polę buduję w ten sposób:

Tworzę pole w klasie dziedziczącej po Zend_Form budującej formularz

[PHP] pobierz, plaintext 
$client_login = $this->createElement('select', 'client', array('label' => 'Wybierz klienta:'));
[PHP] pobierz, plaintext 

Potem uzupełniam w odpowiednim kontrolerze pobierającym wcześniej dane z bazy (formularz w zmiennej $this->createForm)

[PHP] pobierz, plaintext 
$stmt = $this->db->query($select);
$result = $stmt->fetchAll();
 
$clientSelect = $this->createForm->getElement('client');
 
$clientSelect->addMultiOption(null, 'Brak klienta');
 
foreach($result as $c) {
     $clientSelect->addMultiOption($c['ClientLogin'], $c['ClientLogin']);
}
[PHP] pobierz, plaintext 

No i przekazuję formularz do widoku.

Gdzie może leżeć błąd? Dlaczego walidacja nie przepuszcza wszystkich wartości?

Ten post edytował Vill 14.12.2014, 14:44:24

[Turson]

Zobacz jak wygląda rzeczywiście select poprzez "Zbadaj element"

[Vill]

[HTML] pobierz, plaintext 
<select name="client" id="client">
    <option value="">Brak klienta</option>
    <option value="Mort">Mort</option>
    <option value="Julian">Julian</option>
</select>
[HTML] pobierz, plaintext 

Wartości są przekazywane postem poprawnie.

[Lysiur]

Wyglada na to, że po wysłaniu formularza, nie jest on prawidłowo zbudowany. Fragment z foreachem się nie wykonuje. Pokaż może jak wygląda fragment controllera gdzie budujesz formularz i validujesz requesta

Ten post edytował Lysiur 14.12.2014, 16:05:03

[Vill]

Klasa z formularzem

[PHP] pobierz, plaintext 
class Application_Form_CreateEvent extends Zend_Form {
 
    public function init() {
 
        $this->setMethod('post');
 
        $this->setAttrib('id', 'create-event-form');
        $this->setAttrib('class', 'form');
 
        //elementy
        $client_login = $this->createElement('select', 'client', array('label' => 'Wybierz klienta:'));
        $title = $this->createElement('text', 'title', array('label' => 'Tytuł:'));
        $desc = $this->createElement('textarea', 'desc', array('label' => 'Opis:'));
        $date = $this->createElement('text', 'date', array('label' => 'Data:'));
        $time = $this->createElement('text', 'time', array('label' => 'Godzina:'));
 
        $login_input = $this->createElement('hidden', 'login_input');
        $submit = $this->createElement('submit', 'submit', array('value' => 'Wyślij'));
 
        //walidatory i filtry
        $title->addValidator('stringLength', false, array(2, 255));
         ....
 
       //złożenie formularza
        $this->addElement($client_login)
             ->addElement($title)
             ->addElement($date)
             ->addElement($time)
             ->addElement($desc)
             ->addElement($login_input)
             ->addElement($submit)
             ->addElement('hash', 'csrf', array('ignore' => true));
       }
}
[PHP] pobierz, plaintext 

Akcja kontrolera uzupełniająca ten formularz o dodatkowe dane:

[PHP] pobierz, plaintext 
 
public function createAction() {
 
       //ACL    
        $hasAccess = $this->_helper->aclHelper($this->_request->getControllerName(), $this->getRequest()->getActionName());
 
        if(!$hasAccess['access']){ 
            $this->view->message = $this->_helper->messageHelper('FORBIDDEN','RED');
            $this->view->redirectPath = '/trainer/login';
            return;
        }
 
        //login trenera 
        $login = $hasAccess['login'];
 
        $this->createForm->setAction('/individevent/createform');
 
        //uzupełnienie formularza o login trenera
        $loginInput = $this->createForm->getElement('login_input');
        $loginInput->setValue($login);
 
        //uzupełnienie pola select o klientów             
        $select = $this->db->select()
                    ->from('trainer_client', 'ClientLogin')
                    ->where('TrainerLogin = ?', $login);
 
        $stmt = $this->db->query($select);
        $result = $stmt->fetchAll();
 
        $clientSelect = $this->createForm->getElement('client');
 
        $clientSelect->addMultiOption(null, 'Brak klienta');
        foreach($result as $c) {
            $clientSelect->addMultiOption($c['ClientLogin'], $c['ClientLogin']);
        }
 
        //przekazanie formularza do widoku
        $this->view->createForm = $this->createForm;
}
[PHP] pobierz, plaintext 

Akcja kontrolera przetwarzająca dane z formularza:

[PHP] pobierz, plaintext 
public function createformAction() {
 
        //sprawdzenie, czy nastąpiło przesłanie danych
        if(!$this->_request->isPost()) {
            $this->view->message = $this->_helper->messageHelper('POSTFAIL','RED');
            $this->view->redirectPath = '/';
            return;
        }
 
        //walidacja   
        if (!$this->createForm->isValid($this->_request->getPost())) {
            echo 'client: '.$this->_request->getPost('client').'</br>'; //wypisuje się prawidłowo wybrany klient
            $this->view->message = $this->_helper->messageHelper('VALIDATIONFAIL','RED');
            $this->view->redirectPath = '/';
            return;
        }
 
        //przedworzenie danych
        //.......
 
}
[PHP] pobierz, plaintext 

Ten post edytował Vill 14.12.2014, 16:34:32

[Lysiur]

Okej, co prawda nie jak wiem co dokładnie siedzi w zmiennej $this->createForm, ale zakładam, że w controllerze ( w init) tworzysz formularz i go wkładasz do $this->createForm.

Twój problem z formularzem na tym, że w metodzie createAction() uzupełniasz opcje związane nowymi pozycjami (select). Natomiast po wysłaniu requesta, do metody createformAction(), formularz jest czysty (to nie ten sam obiekt co zbudowany w createAction()) i nie posiada dodatkowych pozycji w selekcie. W akcji przetwarzającej ten formularz także musisz go uzupełnić (przed walidacją) o możliwe do wyboru pozycjie inaczej formularz będzie krzyczał, że nie ma opcji.

Zrób sobie mniejwięcej coś takiego:

[PHP] pobierz, plaintext 
//Formularz
class Application_Form_CreateEvent extends Zend_Form {
    public function init() {
        //......
    }
 
    public function setClientsOption(array $clients) {        
        $this->getElement('client')->setMultioptions($clients);
    }
 
}
 
//Controller
class SomeController {
 
    private function getForm() {
 
        $clients = array(); //Lista klientów do wyświetlenia
 
        $form = new Application_Form_CreateEvent();
        $form->setClientsOption($clients);
        return $form;
    }
 
    public function createAction() {
        $form = $this->getForm();
        //....
        $this->view->form = $form;
    }
 
    public function createformAction() {
        $form = $this->getForm();
 
        if($form->isValid()) {
            //...
        }
 
    }
 
}
[PHP] pobierz, plaintext 

Generalnie chodzi o to, byś formularz (już ze wszsytkimi opcjami) pobierał w jednolity sposób do akcji wyświetlania i zapisu.

[Vill]

Faktycznie masz rację. Ale to dziwne, ponieważ wszystkie wartości, które wpisuję w ukryte inputy są widoczne w każdej akcji (jeśli ją podepnę do action formularza). Testowałam też na polach 'text' i także ich wartości są widoczne w każdej akcji. Tylko select coś się resetuje. Ale pokombinuję jeszcze i może coś mi się uda z tym zrobić.

Dzięki!

[Lysiur]

Dzieje się tak ponieważ 'wartości' są przekazywane do formularza z requesta, poprzez metodę $form->isValid(requrest), natomiast same opcje poszczególnych pól należą do obiektu Form i nie są one przekazywane z requestem.

[Vill]

No tak, wiem, tylko myślałam że na wartości walidowane po submicie nie wpływa struktura pola, że sama wartość wybranej opcji jest brana pod uwagę podczas walidacji. A wartości przychodzą dobre.
Zrobiłam teraz tak, że waliduje to pole osobno klasycznym czysto phpowym sposobem, potem je usuwam i na formularzu z usuniętym polem wywołuje isValid() żeby walidowała się cała reszta. Może i trochę nachalne rozwiązanie ale szybkie i niezbyt kłopotliwe. Jak zostanie czasu to wykombinuję coś ładniejszego .

Ten post edytował Vill 15.12.2014, 20:03:29