Mongoose Pagination is not working properly. It works when I refresh the page twice

Question

I am using mongoose pagination v5.0.3, node 9.3.0. I have this code for my blogs pagination:

 router.get('/pages/:page', function(req, res){
     var page = req.params.page > 0? req.params.page : 0;

     Blog.paginate({}, { page: page, limit: 2 })
       .then(function(data){
           res.render('blog/index', {blog: data.docs});
       }).catch(function(err){
           res.send(err)
     });
 });

This code works fine when the page loads, but when I refresh the page it loads 'not found page' app.get("*", function(req, res) { res.render("pages/404page"); });

Are you sure that app.get('*') is getting hit every second refresh? Put a console.log there. — itsundefined, Mar 01 '18 at 16:46
yes. it is happening every second refresh. any ideas what might it be? — Alisher Musurmonv, Mar 01 '18 at 18:04
@thomann061 what code do you need? if you want you can take a look https://github.com/Alisher778/vector-node repo — Alisher Musurmonv, Mar 03 '18 at 16:12
I figured it is because `express-params` package. `router.param('page', /[0-9]/g)` — Alisher Musurmonv, Mar 03 '18 at 17:04

score 1 · Accepted Answer · answered Mar 03 '18 at 18:07

It actually isn't that pagination function. It is this regex statement above:

router.param('page', /[0-9]/g)

Remove the 'g' and it will work.

Explanation:

You're using a g (global) RegExp. In JavaScript, global regexen have state: you call them (with exec, test etc.) the first time, you get the first match in a given string. Call them again and you get the next match, and so on until you get no match and it resets to the start of the next string.

Source

Btw, your regex only checks digits from 0-9, you might want that to take into digits higher than 9, if you are expecting more than 10 blog posts.

yes you are right. I just wanted to use numbers. Than you for the explanation — Alisher Musurmonv, Mar 05 '18 at 18:33

score 0 · Answer 2 · answered Mar 01 '18 at 17:00

0

You can do same using find skip and limit.

router.get('/pages/:page', function (req, res) {
    var page = req.params.page > 0 ? req.params.page : 0;
    var limit = 2;
    Blog.find({})
        .limit(limit)
        .skip(limit * page)
        .exec()
        .then(function (data) {
            res.render('blog/index', {
                blog: data.docs
            });
        }).catch(function (err) {
            res.send(err)
        });
});

answered Mar 01 '18 at 17:00

Rahul Sharma

9,534
1
15
37

I tried this method as well. But no luck. it is very interesting. why it is happening 1st time it loads correctly, second time it loads 404 page – Alisher Musurmonv Mar 01 '18 at 18:07
so if second request comes page value is 404page? – Rahul Sharma Mar 01 '18 at 18:10
yes. I mean if it is '/blog/pages/2' and if I refresh the page with the same url, it say the page not found – Alisher Musurmonv Mar 01 '18 at 18:13
Can you console.log inside the app.all('*') to see what req.url is? – itsundefined Mar 01 '18 at 18:17

Mongoose Pagination is not working properly. It works when I refresh the page twice

2 Answers2