save accepts Input::all() fixes "sometimes" validator

[tbergeron]

That validator only validates the field when it is specified in the form's data (when it's in the attributes). Ardent used the model's attributes instead of using the actual data coming from the form.

i.e. Saving a user without checking if his username is unique on every save, would simply never work.

public static $rules = array(
    'username'  => 'sometimes|required|unique:users'
);

Now we can do:

public function update($id)
    $user = User::findOrFail($id);

    // NOTE: No need to do ->fill() as ->save() now handles this part.
    if ($user->save(Input::all())) {
        // Validated!
    } else {
        return Redirect::back()->withInput()->withErrors($user->errors());
    }
}

save method now looks like this:

public function save(array $data = null,
                         array $rules = array(),
                         array $customMessages = array(),
                         array $options = array(),
                         Closure $beforeSave = null,
                         Closure $afterSave = null) {
// ...

Hope this helps somebody ;-)

[igorsantos07]

Sorry, I didn't understand the point on this PR. I haven't used Ardent for quite some time, but in my understanding, an AR model should always validate it's internal fields, unrelated to form data. If you want to validate what's on the form you pull it inside the model first, isn't it?

Furthermore, I think that although it makes sense to use the said $data argument as the first one, that's a huge breaking change in comparison with the current library usage. This would probably not be implemented alone, as I would need to release a new major tag only for this change, and people would need to update all their codebase just because of this.

[jayremias]

I'm having some trouble related with "sometimes". I'm trying to update a user, but without change the password field. So the Validator return an error, because the password field is already set on the model attributes but the password confirmation isn't. So I get an error. Maybe to get it working, unset unchanged attributes or validate attributes with changes. Any other options with the current library?