Pass variable to laravel method from within the same model

Question

I am working on some multi tenancy updates to a Laravel app but hitting an issue when trying to pass a specific team ID into a method on a model from within another method.

Example:

In Controller:

$waitTime = $booking->estimatedWaitTime($teamId);

In Booking model:

public function queueLength()
{
    $booking = $this->todaysBookings;

    foreach ($bookings as $booking) {
        // Calculate the length of all bookings
    }
}

public function todaysBookings()
{
    return $this->hasMany('App\UserBooking')->whereHas('booking', function ($q) {
        $q->where('team_id', 2);
    });
}

This correctly returns the bookings and allows me to loop through them in the queueLength method. However, I want to be able to pass the team_id into the todaysBooking method.

When instead calling todaysBookings as a method:

$booking = $this->todaysBookings();

It doesn't return anything for me to loop through.

Any ideas how to achieve what I want to do here?

Maraboc · Accepted Answer · 2017-11-22 16:09:54Z

1

You can pass the id as parameter and then get the results :

public function todaysBookings($team_id)
{
    return $this->hasMany('App\UserBooking')->whereHas('booking', function ($q) use($team_id) {
        $q->where('team_id', $team_id);
    });
}

In the call you can do as follow :

$some_teame_id = 2;
$booking = $this->todaysBookings($some_teame_id)->get();

Why ?

Because the relationships in laravel serve as powerful query builders (documentation) :

Eloquent relationships are defined as methods on your Eloquent model classes. Since, like Eloquent models themselves, relationships also serve as powerful query builders, defining relationships as methods provides powerful method chaining and querying capabilities.

Relationship Methods Vs. Dynamic Properties :

The call of the relationship as Methods is needed when you want to add some more conditions and filters like this :

$booking->todaysBookings()->where('active', 1)->get(); //just an example :)

And if you do not need to add additional constraints to an Eloquent relationship query, you may simply access the relationship as if it were a property, Laravel will add the get() for you :)

$booking->todaysBookings

edited Nov 22, 2017 at 16:09

answered Nov 22, 2017 at 15:34

Maraboc

11.1k3 gold badges43 silver badges51 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Lovelock Over a year ago

Interesting, when not calling it as a method what is happening differently? Is that 'executing' it so you don't need to call get on it? (Yours works btw, thanks!)

Maraboc Over a year ago

I will assume you did it twice ;)

Collectives™ on Stack Overflow

Pass variable to laravel method from within the same model

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related