Undefined variable. Laravel

Question

I am trying to filter my results. For this i have following query:

$destinations = $request['destinations'];
        if($request->has('destinations'))
        {
            $trips = Trip::where('status','=',1)->whereHas('destinations', function($q) {
            $q->where('destinationsid', '=', $destinations);
            })->get();
        }else{
             echo "No destination provided";
        }

But I'm getting undefined variable $destinations. If i replace $destinations with any id (6) it shows results. I have also tried to echo $destinations outside the query, it is returning array. What is wrong here? Can anyone help me?

Hiren Gohel · Accepted Answer · 2018-02-12 06:22:45Z

3

Try to add the use($destination) after function($q):

$destinations = $request['destinations'];

if($request->has('destinations'))
{
     $trips = Trip::where('status','=',1)->whereHas('destinations', function($q) use($destinations) {
        $q->where('destinationsid', '=', $destinations);
     })->get();
} else {
        echo "No destination provided";
}

edited Feb 12, 2018 at 6:22

Hiren Gohel

5,0496 gold badges31 silver badges50 bronze badges

answered Feb 12, 2018 at 5:35

user8265613

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

Comments

Alexey Mezenin · Accepted Answer · 2018-02-12 05:40:31Z

2

You need to add use() to the closure to pass variables. For example:

->whereHas('destinations', function($q) use($destinations) {

Or, you could just use the request() global helper:

->whereHas('destinations', function($q) {
    $q->where('destinationsid', request('destinations'));

From the docs:

Closures may also inherit variables from the parent scope. Any such variables must be passed to the use language construct.

edited Feb 12, 2018 at 5:40

answered Feb 12, 2018 at 5:34

Alexey Mezenin

164k27 gold badges309 silver badges287 bronze badges

Comments

Sohel0415 · Accepted Answer · 2018-02-12 05:34:24Z

1

You need to pass destinations variable in your where closure:

$destinations = $request['destinations'];
    if($request->has('destinations'))
    {
        $trips = Trip::where('status','=',1)->whereHas('destinations', function($q) use ($destinations){
        $q->where('destinationsid', '=', $destinations);
        })->get();
    }else{
         echo "No destination provided";
    }

answered Feb 12, 2018 at 5:34

Sohel0415

9,89124 silver badges32 bronze badges

Comments

Boghani Chirag · Accepted Answer · 2018-02-12 05:47:00Z

0

You need to use destinations variable in your where closure:

if($request->has('destinations'))
{

    $destinations = $request['destinations'];
    $trips = Trip::where('status','=',1)->whereHas('destinations', function($q) use ($destinations){
    $q->where('destinationsid', '=', $destinations);
    })->get();

answered Feb 12, 2018 at 5:47

Boghani Chirag

2251 silver badge8 bronze badges

Collectives™ on Stack Overflow

Undefined variable. Laravel

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related