[5.3] Model push() method not working if only related models contain changed data

[claasjansen]

The push() method on Illuminate\Database\Eloquent\Model will only update the model and all of it relations if the model itself has changed. As the save() method now returns false if the model has not changed since the last time it has been, saved the push() method will skip updating the relations.

[KKSzymanowski]

Yes, this is the case because push method calls $this->save() and it proceeds only if that didn't return false.

Also when you have something like Issue -> Comment -> Author and change only the Issue and Author models leaving Comment untouched, the push method will only update the Issue. Since Comment hasn't changed it will not even try to save it's related models.

I suppose you have to explicitly call save on related models until it's fixed.

[themsaid]

That behaviour was fixed in #15236, however it's not released yet.

[KKSzymanowski]

Glad to hear that.
How often do you release changes to Packagist?