Zend Framework

Automatically set form encoding on upload

Details

  • Type: Improvement Improvement
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.7.2
  • Fix Version/s: 1.8.0
  • Component/s: Zend_Form
  • Labels:
    None

Description

When using the file element, the encoding of the form should automatically be set to 'multipart' to prevent unintentionally broken form uploads.

Activity

Hide
Matthew Weier O'Phinney added a comment -

This will be difficult to accomplish, as the form object is not injected into individual elements. Additionally, we will not allow any code in Zend_Form that is specific to a given type of element.

Show
Matthew Weier O'Phinney added a comment - This will be difficult to accomplish, as the form object is not injected into individual elements. Additionally, we will not allow any code in Zend_Form that is specific to a given type of element.
Hide
Thomas Weidner added a comment -

Matthew, I know this.
This is just a reminder for me to see if there is a way.

You know that I will first ask you when there is a solution which is not strictly part of the file element before implementation.

Show
Thomas Weidner added a comment - Matthew, I know this. This is just a reminder for me to see if there is a way. You know that I will first ask you when there is a solution which is not strictly part of the file element before implementation.
Hide
Thomas Weidner added a comment -

Under code review since 29.Jan.2009

Show
Thomas Weidner added a comment - Under code review since 29.Jan.2009
Hide
Matthew Weier O'Phinney added a comment -

Thomas – I have a patch in the incubator that accomplishes this; it was an issue of order-of-operations. Tests for ZF-5370 now all pass; feel free to apply to trunk.

Show
Matthew Weier O'Phinney added a comment - Thomas – I have a patch in the incubator that accomplishes this; it was an issue of order-of-operations. Tests for ZF-5370 now all pass; feel free to apply to trunk.
Hide
Thomas Weidner added a comment -

Integrated with r13988

Show
Thomas Weidner added a comment - Integrated with r13988

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: