Multiple Where Clauses from GET Variables

public function search(Request $request)
{
    $dresses = Dresses::query();

    if($request->has('price'))
    {
        $dresses->where('price', $request->get('price'));
    }

    $dresses->paginate(10);

    return view('dresses.index', compact('dresses'));
}

First of all you don't need all those wheres, these are going to be "ands", also its handled same way as in basic PHP. Refer back to your basic PHP skills and you will figure it out.

Unfortunately, this is doing nothing for me. It is just returning some sort of object that cannot be converted to a string. It does enter into the conditional, but I am missing something somewhere that I do not know what it is. I am getting pretty frustrated and disappointed about what I could be doing wrong. Again, any help would be truly appreciated.

public function index(Requests\VestidosSearchRequest $request)
    {
        $search = $request->all();
        if(!empty($search)){
            $vestidos = Vestido::query();
            if($request->has('price'))
            {
                //dd($search['price']);
                $vestidos->where('price', '<', $request->get('price'));
            }
            $vestidos->with('images')->with('user')->paginate(6);
            return view('vestidos/index', compact('vestidos', 'search'));
        }
        $vestidos = Vestido::with('images')->with('user')->paginate(6);
        return view('vestidos/index', compact('vestidos', 'search'));
    }

Try this

public function index(Requests\VestidosSearchRequest $request)
{

    $search = $request->all();
    
    $vestidos = Vestido::with(['images', 'user']);

    if($search->has('price'))
    {
        $vestidos->where('price', '<', $search->get('price'));
    }

    $vestidos->paginate(6);

    return view('vestidos/index', compact('vestidos', 'search'));
}

@RachidLaasri Simple little mistakes always. Had to set the variable equal to the modified object. If you edit your answer to show it, I will definitely select as answered.

public function index(Requests\VestidosSearchRequest $request)
    {
        $search = $request->all();

        if(!empty($search)){

            $vestidos = Vestido::with(['images', 'user']);
            if($request->has('price'))
            {
                $vestidos = $vestidos->where('price', '<', $request->get('price'));
            }

            if($request->has('length'))
            {
                $vestidos = $vestidos->where('length', $request->get('length'));
            }

            $vestidos = $vestidos->paginate(6);
            return view('vestidos/index', compact('vestidos', 'search'));
        }
        $vestidos = Vestido::with('images')->with('user')->paginate(6);
        return view('vestidos/index', compact('vestidos', 'search'));
    }

Is it working now? if yes, you may need to do some refactoring, because there is a lot of repetition on your code.

@RachidLaasri for sure I will refactor, I just wanted to get it working. Thank you so much for you help and responsiveness.

Good, You are welcome.

@deevoweb For simple filtering, I’ve added a scope to my model that I can pass an array of parameters to, and the scope will add the relevant clauses. An example:

use Illuminate\Http\Request;
use Illuminate\Database\Eloquent\Model;

class Dress extends Model
{
    public function scopeFilterByRequest($query, Request $request)
    {
        if ($request->has('color')) {
            $query->where('color', '=', $request->get('color'));
        }

        if ($request->has('size')) {
            $query->where('size', '=', $request->get('size'));
        }

        // And so on

        return $query;
    }
}

Usage:

public function index(Request $request)
{
    $dresses = Dress::filterByRequest($request)->paginate();

    return view('dresses.index', compact('dresses'));
}

Hope that helps.

@martinbean I like that. I will give it a go.

@martinbean This was so beautiful it almost made me cry. The only thing I did different was built up the query by adding $query = $query->something and just went through all of my GET variables that were passed through. Works like a charm and really cleaned up my controller.