[5.4] Add Model::refresh()

[themsaid]

This method eliminates the need for:

$user = $user->fresh()

You can do this instead:

$user->refresh()

[mattstauffer]

Woo!

[calebporzio]

Wondering if there is a way to not have to call ->load(... every time I want to refresh(). Like ->fresh() has optional "with" param... possibly same deal here?

[themsaid]

refresh() will reload all relationships currently loaded automatically.

[calebporzio]

got it, cool!

[morloderex]

@themsaid what is the different between fresh and refresh here?

couldn't refresh just call fresh?

[decadence]

@morloderex fresh loads relations based on $with parameter, refresh reloads all relations based on current model loaded relations.

[lucasmichot]

No param

[JosephSilber]

Should this be findOrFail? What should happen if the model has been deleted in the meantime?

[themsaid]

If the model was deleted exists would return false and the method will just bail.

[ghost]

Where are unit tests?

[JosephSilber]

I meant if it was deleted somewhere else, not in this request.

[themsaid]

@JosephSilber good point, switched to using findOrFail

[projct1]

@themsaid Why refresh return a void, not the model itself? Its inconvenient for use in chains:

return view('admin.selections.list-item')->with('selection', $selection->refresh());

[themsaid]

@rorc you can use fresh() for that.

[projct1]

@themsaid But, if i want reload only current relations?

[themsaid]

@rorc just call refresh() and return the original value of yours, thats' the point :)

[projct1]

@themsaid Sigh, except 1 line i have to use 2 lines? Are you kidding? 😒

$selection->refresh();

return view('admin.selections.list-item')->with('selection', $selection);

[themsaid]

@rorc haha .. well I think I have an idea

[projct1]

@themsaid Why not just return the refreshed model? 😉

[Vasiliy-Bondarenko]

plus 1 vote for
return $this;
instead of void.

Very useful for chaining in tests.

        $comment->refresh(); // redundant line here!
        $this->assertCount(1, $comment->replies);