Level 2
@ismaile I added that to the model and got this:
$user->library()->beingProcessed();
BadMethodCallException with message 'Call to undefined method Illuminate/Database/Eloquent/Relations/BelongsToMany::beingProcessed()'
Jan 29, 2020
5
Level 2
Custom Eloquent query scope with pivot
I had a look at Eloquent query scopes, but could not find a way to do the following:
User model:
class User extends Model {
/**
* Returns the items in the user's library.
*
* @return \Illuminate\Database\Eloquent\Relations\BelongsToMany
*/
function library() {
return $this->belongsToMany(Item::class, UserItem::class, 'user_id', 'item_id')
->withPivot('status');
}
}
Desired behavior:
// Return the items in the library as normal
$user->library()
// Returns the same as:
// $user->library()->wherePivot('status', 'being_processed');
$user->library()->beingProcessed()
// This should attach an item, the same way as:
// $user->library()->attach(1, ['status' => 'being_processed']);
$user->library()->beingProcessed()->attach(1)
Can this be achieved using Eloquent query scopes, or maybe some other way?
Level 30
@musa11971 Sorry, I have been a bit quick to respond and didn't pay attention. I have removed the answer since it was not relevant. Local scopes are used to filter a model not a relation.
This should work:
public function libraryBeingProcessed()
{
return $this->library()->wherePivot('status', 'being_processed');
}
And you can use it like so:
$user->libraryBeingProcessed();
Level 33
Please try this following code in your model
public function scopeLibraryProcessed()
{
return $this->library()->wherePivot('status', 'being_processed');
}
and call the query scope with this way
$user = App\User::find(2)->LibraryProcessed()->get();
or
$user = App\User::LibraryProcessed()->get();
Level 2
@ismaile This works for retrieving the models. However, as I mentioned in the original post, I would really like this behavior as well:
// This should attach an item, the same way as:
// $user->library()->attach(1, ['status' => 'being_processed']);
$user->libraryBeingProcessed()->attach(1);
Current using $user->libraryBeingProcessed()->attach(1);, attaches it without the correct status.
Is there a way around this?
Level 30
I don't think this is possible. attach expects the id and the pivot table values.
You can embed your logic in a method instead:
public function addALibraryBeingProcessed($library_id)
{
$this->library()->attach($library_id, ['status' => 'being_processed']);
}
PS: On a side note, regarding the naming, I would change library to libraries for the methods library and libraryBeingProcessed since we are expecting many libraries per user.
1 like
Please or to participate in this conversation.