Previous, Next Post

Your links are wrong.

Try

{{ URL::to( 'blog/' . $previous->id ) }}

and

{{ URL::to( 'blog/' . $next->id ) }}

Then, in your controller, you do

$next = Post::where('id', '>', $post->id)->first();

and you get the previous post in a similar way.

By the way, you should also check that there is a next or previous record. You might be viewing the first or last one.

You should name your route and use the route('route.name', [$previous]) helper to generate urls.

yes. As @pmaill is saying, something like this would be better:

Route::get('blog/{id}',[
    'as' => 'blog.show',
    'uses' => 'BlogController@art'
]);

Then, in your view, you build your links like this:

<a href="{{ route( 'blog.show', $previous->id ) }}">Previous</a>

@Prullenbak quick tip, when you want to generate a route with the id of an object, you can pass the object only : route('show.blog', [$previous]).

@pmall Does that also work if you don't make use of route-model binding? If that's the case: nice! Thanks for the tip :)

Of course, route model binding is unrelated to url generation. When the url generator see a model as parameter it automagically takes its id. There is a method you can put in the model to return anything else than its id in this case (a slug for example) but I dont remember its name.

@Prullenbak can you tell me how check that is first or last post ? and fix this?

Well, if it's the last post, then

$next = Post::where('id', '>', $post->id)->first();

would be Null. So in your view, you could do

@if($next)
<a href="{{ route( 'blog.show', $next->id ) }}">Next</a>
@endif

Or, as @pmall pointed out:

@if($next)
<a href="{{ route( 'blog.show', $next ) }}">Next</a>
@endif

that's a little bit cleaner.

thanks a lot, i appreciate this !

That was very helpful guys :)

Hi, it works however I have couple of questions. what if we:

• use UUID for id (we cannot compare id for greater or less than)
• want to reduce amount of DB queries.

Try

$previous = Post::where('id', '<', $post->id)->orderBy('id', 'desc')->first();
$next = Post::where('id', '>', $post->id)->first();

try this one :

public function single($slug){

    $post = Post::where('slug',$slug)->first();

    $previous_id = Post::where('id','<',$post->id)->max('id');
    $next_id = Post::where('id','>',$post->id)->min('id');

    $next = Post::find($next_id);
    $previous = Post::find($previous_id);


     return view('front-end.single',compact('post',$post))
         ->with('previous',$previous)
         ->with('next',$next);
}

I've used this https://usingphp.com/post/get-next-and-previous-post-link-in-laravel It works too