[5.3] Allow models omitted by global scopes to be refreshed

[deefour]

RE: #15269

[reinink]

This has the negative side affect of actually loading models that are soft deleted.

<?php

namespace App;

use Illuminate\Database\Eloquent\SoftDeletes;

class Thing extends Model
{
    use SoftDeletes;
}

$thing = Thing::find(1);
$thing->delete();

$thing->fresh(); // returns object
$thing = Thing::find(1); // returns null

I sort of think fresh() should replicate the behaviour of find(), which it now does not.

[deefour]

@reinink What you're calling a negative side effect is the single intended effect of the PR.

You're holding onto an existing model instance in $thing. The fact that it's been soft-deleted doesn't matter. You know it exists in the database. You're past the find().

fresh() is asking for the very latest attributes and requested relations for an object you already know about. The request for this info shouldn't come up empty because global scopes (soft-delete filter or otherwise) are preventing the record we already know exists from being found.

[reinink]

Okay, fair enough, but that does make this a breaking change.