The refresh method in the Model should set wasRecentlyCreated to false

[leandrogehlen]

Laravel Version

12.1.1

PHP Version

8.3.17

Database Driver & Version

No response

Description

The refresh method in the Model does not set the wasRecentlyCreated property to false after it is called.

Steps To Reproduce

$product = new Product();
$product->name = 'Sample Product';
$product->price = 19.99;
$product->save();
$product->refresh();
dd($product->wasRecentlyCreated);  // It should be false, but it might still be true

[mohammadrasoulasghari]

The wasRecentlyCreated property is designed to indicate whether a model was created during the current request lifecycle. Its purpose is to track the model's creation status within the current request, not its data state.

When we create a record and then call refresh(), the property should remain true because:

1. The record was indeed created during the current request lifecycle
2. The refresh() method only reloads the model's data from the database
3. Refreshing the data doesn't change the fact that the model was newly created in this request

This is important for application logic that needs to know if a record was just created, even after refreshing to get the latest database values (like auto-incremented IDs or default values).

[leandrogehlen]

I understand your argument, but there seems to be a bit of confusion around the behavior of the wasRecentlyCreated property.

When you find an instance of the Product model like this:

$product = new Product();
$product->name = 'Sample Product';
$product->price = 19.99;
$product->save();
$product = Product::find($product->id);
dd($product->wasRecentlyCreated);  // this will be false

In this case request lifecycle is the same, and the model was reloaded from database.

Taking your argument into account, in my example, wasRecentlyCreated should be true

[ghabriel25]

@leandrogehlen

Please correct me if I'm wrong, but you're creating new object lifecycle so $wasRecentlyCreated will be false

[leandrogehlen]

It's relative because the behavior is the same and the record was loaded from the database the same way as with refresh().

From my point of view, it does not make sense that a record loaded from the database has a wasRecentlyCreated true value.

[devajmeireles]

From my point of view, it does not make sense that a record loaded from the database has a wasRecentlyCreated true value.

Makes perfect sense. IMHO, the correct thing is for fresh and refresh to reset the state of wasRecentlyCreated.

[macropay-solutions]

Also the refresh should use write pdo or at least take an option to use it. Because it does not, we are not using it.