Witam

Napotkałem się dzisiaj na kilka dziwnych problemów z powyższym komponentem.

Problemy:
1. Pokazuje błąd odbioru obrazka, a na serwer go wrzuca (pojawia się folder w docelowym folderze i jest przetwarzany dalej formularz).
2. Po walidacji formularza, jesli nie podam obrazka to zostaje wyświetlony ponownie formularz z wypełnionymi danymi (pole do pliku jest puste). Kiedy jednak zostanie wskazana ścieżka do obrazka (po wcześniejszym nie podaniu), który chcemy wrzucić to dostaję komunikat "Too many files, maximum '1' are allowed but '2' are given", chociaż jest tylko jeden wrzucany.

Jak rozwiązać powyższe problemy? Może robię coś w złej kolejności?

Fragment formularza:

[PHP] pobierz, plaintext 
$image = new Zend_Form_Element_File('image');
$image->setLabel('Dodaj zdjęcie główne: ');
$path = realpath(APPLICATION_PATH.'/../public/images');
$image->setDestination($path);
$image->setRequired(true);
 
$image->addValidator('count',false,1);
$image->addValidator('size',false,1024000);
$image->addValidator('extension',false,array('jpg'));
[PHP] pobierz, plaintext 

Kod z kontrolera odpowiedzialnego za obsługę formularza:

[PHP] pobierz, plaintext 
$form_offer = new Offer_Form_AddOffer();
$form_offer->getElement('image')->addFilter('Rename',APPLICATION_PATH.'/../public/images/'.$image_name);
 
if($this->_request->isPost() && $form_offer->isValid($_POST) && $form_offer->getValue('image') != null)
{
	if (!$form_offer->image->receive()) {
		$this->view->error =  "Error receiving the file";
	}
 
	//Pobranie danych za pomoca getValue i dalsze ich przetwarzanie
       // ale nie robię $form_offer->getValue('image')
 
	$form_offer->reset();
 
	$this->view->confirm = true;
}
if($form_offer->isErrors())
{
	$form_offer->populate($_POST);
}
[PHP] pobierz, plaintext 

==== EDYCJA ====
Problem rozwiązałem. Przyczyną tego było w złym miejscu postawienie warunku na odbiór pliku w kontrolerze. Tak wygląda poprawiony plik:

[PHP] pobierz, plaintext 
if($this->_request->isPost() && $form_offer->isValid($_POST) && $form_offer->getValue('image') != null)
{
	//Pobranie danych i operacje na nich
 
	//resetowanie formularza
	$form_offer->reset();
 
	$this->view->confirm = true;
}
else
{
	if ($this->_request->isPost() && !$form_offer->getElement('image')->receive() && $form_offer->getValue('image') != null) {
		$this->view->error =  "Error receiving the file";
	}
}
if($form_offer->isErrors())
{
	$form_offer->populate($_POST);
}
[PHP] pobierz, plaintext 

Jeśli mogę coś jeszcze zasugerować:

[PHP] pobierz, plaintext 
$form_offer->isValid($_POST);
[PHP] pobierz, plaintext 

- żeby pobrać dane nieprzefiltrowane (przez formularz, które pobieramy przez $form->getValue) używajmy requesta i metod getParams:

[PHP] pobierz, plaintext 
$form_offer->isValid($this->getRequest()->getParams());
[PHP] pobierz, plaintext 

Takie $_POST brzydko wygląda

A chodzi tylko o styl, czy jakiś bezpieczeństwo?

Bezpieczeństwo na pewno nie, bo dane z requesta nie są filtrowane, raczej o wygodę i dodatkowe możliwości, wszystko masz w jednym miejscu plus garść przydatnych metod, ma to znaczenie np. jeśli akcja ma obsługiwać żądania POST, GET, ajaxowe czy wykonywać pracę na rzecz innego kontrola. No i estetytka