Laravel query variable as string

Question

What i am doing here might be stupid but i am newbie to laravel, I am applying search filters for database using a query. Query is being built using javascript and being passed to laravel function. E.g.

window.open("/filterResults?types="+types);
//types contains a string like "international#global#europe#  etc

Now in Controller i am getting that string types and exploding it to make a query like this

    $tquery = "Where('type',".$typesArray[0].")";
    for($offset=1; $offset < count($typesArray); $offset++) {
        $tquery .= "->orWhere('type', ".$typesArray[$offset].")";
    }

and executing query like this

$firms = Firm::$tquery->get();

it gives me error

Access to undeclared static property: App\Firm::$tquery

How can i do what i want to.

PS. types are checkboxes some of them can be checked or all.

Alexey Mezenin · Accepted Answer · 2016-10-22 08:18:01Z

4

It's better to use whereIn() method:

$firms = Firm::whereIn('type', $typesArray)->get();

But if you need to use your way, you can do something like this:

$firm = new App\Firm;

foreach ($typesArray as $type) {
    $firm = $firm->orWhere('type', $type);
}

$firms = $firm->get();

answered Oct 22, 2016 at 8:18

Alexey Mezenin

164k27 gold badges309 silver badges287 bronze badges

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

1 Comment

Noman Ali Over a year ago

Awesome whereIn Rocks, :) Thanks Man.

Mahfuzul Alam · Accepted Answer · 2016-10-22 08:24:32Z

1

Did you consider query builder ?

$firmsQuery = (new Firm)->newQuery();

if($request->has('types')) {
    $typesArray =  $request->input('types');

    foreach ($typesArray as $type) {
        $firmsQuery->where(function($query) use ($type) {
            $query->orWhere('type', $type);
        });
    }

}

$firms = $firmsQuery->get();

edited Oct 22, 2016 at 8:24

answered Oct 22, 2016 at 8:12

Mahfuzul Alam

3,17716 silver badges15 bronze badges

4 Comments

Noman Ali Over a year ago

Non-static method Illuminate\Database\Eloquent\Model::newQuery() should not be called statically, assuming $this from incompatible context

Mahfuzul Alam Over a year ago

Whoops! Check updated answer please. There was an unintentional mistake. It should be $firmsQuery = (new Firm)->newQuery();

Noman Ali Over a year ago

Thankyou for your attention and time, but now the error is Undefined variable: type

Mahfuzul Alam Over a year ago

Fixed. It supposed to be $typesArray in foreach. foreach ($typesArray as $type)

aleksejjj · Accepted Answer · 2016-10-22 08:22:48Z

0

Actually you are using bad style but you can take a look to DB Raw Expressions

answered Oct 22, 2016 at 8:22

aleksejjj

1,83512 silver badges21 bronze badges

Collectives™ on Stack Overflow

Laravel query variable as string

3 Answers 3

1 Comment

4 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

4 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related