Laravel Validation not returning error

Question

I am new in Laravel.

This is my laravel controller:

public function store()
    {
        $validator = Validator::make($data = Input::all(), City::$rules);
        if ($validator->fails())
        {
            return Redirect::back()->withErrors($validator)->withInput();

        }
        $image_temp=Input::file('image');
        $name = Input::file('image')->getClientOriginalName();
        $data['image']='';
        if(Image::make($image_temp->getRealPath())->save('public/up/city/'.$name)){
            $data['image']='up/city/'.$name;
        }
        City::create($data);
        return Redirect::route('admin.cities.index');
    }

and This is my model:

class City extends \Eloquent {
   protected $primaryKey='city_id';

    // Add your validation rules here
    public static $rules = [
         'title'     => 'required',
         'image'     => 'mimes:jpeg',
         'parent_id' => 'required',
         'name'      => 'required',
         'english_name'=>'unique:cities,english_name|required'

    ];
    // Don't forget to fill this array
    protected $fillable = ['name', 'parent_id', 'english_name','population','phone_prefix','image'];
}

And I have a form I use {{ $errors->first('inputName','<p class="error">:message</p>') }} bellow my form inputs, when I send form without filling inputs I get error under each form input. But when I fill out all form inputs and then submit the Laarvel validation return fail (I mean mass assignment not working and not registering, and redirects back to create page.) what is the problem?

Looks to me like you forgot title in the $fillable array. — lukasgeiter
– lukasgeiter, Commented Jan 15, 2015 at 18:31
City::$rules looks wonky to me... Should it be City::rules or City->rules? — Tim Lewis
– Tim Lewis, Commented Jan 15, 2015 at 18:32
@TimLewis It is the right syntax for accessing static properties. — lukasgeiter
– lukasgeiter, Commented Jan 15, 2015 at 18:34
@Asker Haha no, your syntax better than a lot I see on SO. The syntax on accessing static variables, as per my first comment. — Tim Lewis
– Tim Lewis, Commented Jan 15, 2015 at 18:53

lukasgeiter · Accepted Answer · 2015-01-15 18:58:29Z

1

Almost always the reason for a mass assignment error is a missing attribute in the $fillable array. In your case it is title.

protected $fillable = ['title', 'name', 'parent_id', 'english_name','population','phone_prefix','image'];
                          ^

Edit

Apparently the problem was actually that the title in the $rules array, which should have been name...

answered Jan 15, 2015 at 18:58

lukasgeiter

154k29 gold badges349 silver badges280 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

user4434835 Over a year ago

Oh you are wrong here, the problem is from my $rules here. I mean the title in $rules array should be name. And anyway your answer is true because both of them should be same.Thanks

user4434835 Over a year ago

Accepted + up vote + A big thanks + Give me your Email :) (Kidding)

user4434835 Over a year ago

can you please answer to this stackoverflow.com/questions/27985327/…

Collectives™ on Stack Overflow

Laravel Validation not returning error

1 Answer 1

Edit

3 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Edit

3 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related