Wo ist der richtige Weg?

Question

$provider ist optional, ist es richtig, whereHas in diesem Szenario zu verwenden, auch wenn ich bereits with() Eager Loading verwendet habe?

public function findByStatus($status, $provider = null) 
{ 
    $result = $this->hosts->with('logins')->where('status', $status); 

    if ($provider) { 
     $result->whereHas('logins', function ($query) use ($provider) { 
      $query->where('provider', $provider); 
     }); 
    } 

    return $result->get(); 
} 

Answer 1

Sie können es in der Art, wie Sie verwenden. Aber ich würde es so machen.

$result = $this->hosts->whereHas('logins', function ($query) use ($status, $provider) { 
    $query->where('status', $status); 
    if ($provider) { 
     $query->where('provider', $provider); 
    } 
    return $query; 
})->get(); 

Da sowohl die Lagen über logins Sie gruppieren können sie in einer Abfrage.