search

fuel / core

Fuel PHP Framework - The core of the Fuel v1 framework
http://fuelphp.com
813 stars 345 forks source link

Support multiple errors in the Upload class #241

Closed WanWizard closed 13 years ago

WanWizard commented 13 years ago

The current process() and save() methods of the Upload class stop at the first error detected, and only return that error (both code and message).

Especially for validation, all detected errors should be returned.

See http://fuelphp.com/forums/topics/view/2603

WanWizard commented 13 years ago

Moved to v1.1, as this will change the behaviour of the Upload class.

frankdejonge commented 13 years ago

@WanWizard: Hasn't this issue been worked out? Looking at the class it seems it does.

WanWizard commented 13 years ago

No, it hasn't. The point here was that an upload of a single file could cause more than one error.

Currently the file array contains only one error code and error message, this should be an array.