framework: Validation error message is empty on array of file

Hi, after doing some testing, I think this is a bug.

I have this html

<input type="file" name="images[]">
<input type="file" name="images[]">
.
.
<input type="file" name="images[]">

Form Request’s rules

'images.*' => 'mimes:jpeg',
'name' => 'required',

When the file type is not jpeg, it redirects back to the input page. So the validation works, but there’s no error message. When I checked the error message bag, it is empty.

Illuminate\Support\ViewErrorBag Object ( [bags:protected] => Array ( ) )

I’m using Laravel Framework version 5.2.12

About this issue

Original URL
State: closed
Created 8 years ago
Comments: 38 (1 by maintainers)

Most upvoted comments

Potrential fix: Try replacing the withInputmethod in the Illuminate\Http\RedirectResponse class with the one below. I only added a return statement inside the conditional that checks if $value is an array. This should prevent it from executing the following line and adding the array that contains the UploadedFile object to the session which causes the eventual serialization error.

    /**
     * Flash an array of input to the session.
     *
     * @param  array  $input
     * @return $this
     */
    public function withInput(array $input = null)
    {
        $input = $input ?: $this->request->input();

        $this->session->flashInput($data = array_filter($input, $callback = function (&$value) use (&$callback) {
            if (is_array($value)) {
                $value = array_filter($value, $callback);
                return false;
            }

            return ! $value instanceof UploadedFile;
        }));

        return $this;
    }

Update I just realized adding that return statement breaks other arrays in the form, for example, an array of text inputs. The old data for those inputs wont be added to the session. We’d need to add a bit more logic to the withInput statement or figure something else out.

kkiernan on Feb 5, 2016