Method links does not exist in Laravel 5.6 app?

You haven't shown what happens when you post to url('search')

@Snapey when I clicked search button occured above error message.

my route for url('search ') is,

Route::get('search','SearchController@search');

and SearchController

if($request->has('search')){
            $vehicles = Vehicle::search($request->get('search'))->get()->sort(function ($a, $b) {
        if ($a->adtype !== $b->adtype) {
            return $b->adtype - $a->adtype;
        }
    
        return $a->adtype
            ? ($a->updated_at->gt($b->updated_at) ? -1 : 1)
            : ($a->created_at->gt($b->created_at) ? -1 : 1);
    });  
        }else{
            $vehicles = Vehicle::get();
        }
        return view('vehicles.index', compact('vehicles'));
    

@Snapey this is related to my old post https://laracasts.com/discuss/channels/laravel/how-can-i-put-paginate-method-in-my-controller-index-function in this case in searchController@search function I need what I need in the VehicleController index function. that mean after search attempt business rules should be satisfy.

on SearchController change $vehicles = Vehicle::get(); to $vehicles = Vehicle::paginate(5);

also

$vehicles = Vehicle::search($request->get('search'))->paginate(5)->sort(function ($a, $b) {

@andylord565 got same error

@msoft The problem is your SearchController code. You don't paginate() the $vehicles results there like you are in the VechicleController, but both controllers are sharing the same vehicles.index view that is expecting paginated results. So you need to send a paginated instance of $vehicles from both controllers to that view, otherwise links() won't exist in the non-paginated $vehicles instance.

@Cronix then how can do SearchController code as VehicleController index function??? I tried with many ways but got errors

Hard to say offhand. Show your Vehicles::search() method. Your first query (VehicleController) is straightforward using standard DB query methods. Your other one is more complex and using a custom method.

Simplify your query. Get it working without sort() and stuff. Once you get the pagination part working, then work on refining it how you need it.

@Cronix SearchController@search function is,

public function search(Request $request)
    {
        if($request->has('search')){
            $vehicles = Vehicle::search($request->get('search'))->get()->sort(function ($a, $b) {
        if ($a->adtype !== $b->adtype) {
            return $b->adtype - $a->adtype;
        }
    
        return $a->adtype
            ? ($a->updated_at->gt($b->updated_at) ? -1 : 1)
            : ($a->created_at->gt($b->created_at) ? -1 : 1);
    });  
        }else{
            $vehicles = Vehicle::get();
        }
        return view('vehicles.index', compact('vehicles'));
    }
    

You already showed that. I was talking about the Vehicle::search() method in the Vehicle model that you're using there

Vehicle::search($request->get('search'))...

You cannot paginate your search because you are using sort on eloquent collection.

Best you could do with the code you have now is sort the results on each page, which would not be any use.

I thought you marked your previous post as resolved?

@Cronix

use Searchable;
     protected $guarded = [];

    public function searchableAs()
    {
        return 'categoryname';
    }

@Snapey yes I have tried with following code,

if($request->has('search')){
            $vehicles = Vehicle::search($request->get('search'))
                        ->orderBy('adtype','DESC')

            ->latest('updated_at')
            ->paginate(10);
    });  //line 34
        }else{
            $vehicles = Vehicle::paginate(10);
        }
        return view('vehicles.index', compact('vehicles'));
    }

it occured

1/1) FatalErrorException

syntax error, unexpected ')'
in SearchController.php line 34

Line 34 should not exist

Indent your code carefully and this would be obvious.

I haven't used Scout, but it has a pagination method. Just try

Vehicle::search($request->get('search'))->paginate(10);

and if that works, try doing the sorting in the query itself instead of using sort() on the result. Not exactly sure what you're trying to do with your ordering there... Why not just order by updated_at? If it wasn't ever updated, created_at and updated_at would be equal.

maybe try

Vehicle::search($request->get('search'))->latest('updated_at')->paginate(10);

@Snapey now got

Method latest does not exist

error message

We don't see your code, so should we try guessing what you're doing wrong?

@Cronix in your answer this one is working,

Vehicle::search($request->get('search'))->paginate(10);

but search results did not display according to the updated_at values.

but when used

Vehicle::search($request->get('search'))->latest('updated_at')->paginate(10);

it is got

Method latest does not exist.

Maybe put latest() before search(). I can't advise you on using Scout (which changes things) - I don't use it.

if it is work latest method my problem may solve

if it is work latest method my problem may solve

Does that mean this is working now?

@Cronix working without latest method but without latest it is not satisfy my business logic here....

Just out of curiosity what happens you remove this line ->latest('updated_at')