Zend Framework

Zend_Form_Element_File has mixed error messages

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.7 Preview Release
  • Fix Version/s: 1.7.0
  • Labels:
    None

Description

When I have this form:

$form = new Zend_Form();
        $form->setName('upload_form');
        $form->setMethod(Zend_Form::METHOD_POST);
        $form->setEnctype('multipart/form-data');

        $element = new Zend_Form_Element_File('upload_file');
        $element->setLabel('upload_file:')->setRequired(true)
            ->setDestination('C:\Temp')
            ->addFilter(new Zend_Filter_File_Rename('XXX', 'XXX'))
            ;
        $form->addElement($element);

        $element2 = new Zend_Form_Element_File('upload_file2');
        $element2->setLabel('upload_file2:')
            ->setDestination('C:\Temp')
            ->addFilter(new Zend_Filter_File_Rename('XXX', 'XXX'))
            ;
        $form->addElement($element2);
        
        $form->addElement(new Zend_Form_Element_Submit(array(
            'name' => 'Upload',
        )));

        $this->view->form = $form;
        
        if ($this->getRequest()->isPost()) {
            if (!$form->isValid($_POST)) {
                echo "Error";
            }
        }

And I submit it without choosing files, it should show the error message "The file 'upload_file' was not uploaded", but it shows "The file 'upload_file2' was not uploaded" by the first element.

Issue Links

Activity

Hide
Thomas Weidner added a comment -

Duplication of ZF-4411

Show
Thomas Weidner added a comment - Duplication of ZF-4411
Hide
Thomas Weidner added a comment -

Simply said:
You can't set one part of your upload required and the other not.
Eighter the uploaded files ARE required or they are not required.
And this is the reason why your above example does not work as expected.

Show
Thomas Weidner added a comment - Simply said: You can't set one part of your upload required and the other not. Eighter the uploaded files ARE required or they are not required. And this is the reason why your above example does not work as expected.
Hide
Thomas Weidner added a comment -

New feature for individual options added.
Issue fixed with r12122

Show
Thomas Weidner added a comment - New feature for individual options added. Issue fixed with r12122
Hide
Wil Sinclair added a comment -

Changing issues in preparation for the 1.7.0 release.

Show
Wil Sinclair added a comment - Changing issues in preparation for the 1.7.0 release.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: