default for missing post_index parameter

Avoid getting NaN if post_index param is not set (NaN is passed through to redis, redis complains, and the user ends up at /404)
v1.18.x
Moritz Schmidt 11 years ago
parent f0d3b5dd00
commit d1273516b5

@ -72,7 +72,7 @@ topicsController.get = function(req, res, next) {
postIndex = Math.max((req.params.post_index || 1) - (settings.postsPerPage + 1), 0); postIndex = Math.max((req.params.post_index || 1) - (settings.postsPerPage + 1), 0);
} }
} else if (!req.query.page) { } else if (!req.query.page) {
var index = Math.max(parseInt(req.params.post_index, 10), 0); var index = Math.max(parseInt(req.params.post_index, 10), 0) || 0;
page = Math.ceil((index + 1) / settings.postsPerPage); page = Math.ceil((index + 1) / settings.postsPerPage);
} }

Loading…
Cancel
Save