Laravel query works when given values but not variables

Question

OK, this is driving me insane. Can someone please explain why

Result::query()
    ->where('audit_id', $audit->id)
    ->where('result_type', 'App\Models\Touchpoint')
    ->whereIn('result_id', $tps)
    ->with('result')
    ->join('locations', function ($join) {
        $join->where('locations.id', $location->id);
    })
    ->join('location_weight', function ($join) {
        $join->on('results.result_id', '=', 'location_weight.weight_id')
            ->on('results.result_type', '=', 'location_weight.weight_type')
            ->on('locations.id', '=', 'location_weight.location_id');
    })
    ->get()

doesn't return results when $audit->id = 1 and $location->id = 1, yet

Result::query()
    ->where('audit_id', 1)
    ->where('result_type', 'App\Models\Touchpoint')
    ->whereIn('result_id', $tps)
    ->with('result')
    ->join('locations', function ($join) {
        $join->where('locations.id', 1);
    })
    ->join('location_weight', function ($join) {
        $join->on('results.result_id', '=', 'location_weight.weight_id')
            ->on('results.result_type', '=', 'location_weight.weight_type')
            ->on('locations.id', '=', 'location_weight.location_id');
    })
    ->get()

does return the correct results?

IGP · Accepted Answer · 2022-06-04 03:26:41Z

1

The first codeblock should throw an error. You're referencing a variable from the parent scope ($location) without passing it to the use language construct

// This
->join('locations', function ($join) {
    $join->where('locations.id', $location->id);
})

// should be
->join('locations', function ($join) use ($location) {
    $join->where('locations.id', $location->id);
})

https://www.php.net/manual/en/functions.anonymous.php (See Example #3)

answered Jun 4, 2022 at 3:26

IGP

16.4k5 gold badges35 silver badges55 bronze badges

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

Collectives™ on Stack Overflow

Laravel query works when given values but not variables

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related