mirror of
https://github.com/TryGhost/Ghost.git
synced 2025-02-17 23:44:39 -05:00
4f9b72ff43
- This is a minor bugbare, but it will affect some configuration I'm about to do for c8 - I've been wanting to do it for ages, middleware is plural all on it's own so it's an odd affectation in our codebase - This also only exists in 2 places, everywhere else we use "middleware" - Sadly it did result in a lot of churn as I did a full find and replace, but consistency is king!
34 lines
1,012 B
JavaScript
34 lines
1,012 B
JavaScript
const tpl = require('@tryghost/tpl');
|
|
const errors = require('@tryghost/errors');
|
|
const urlUtils = require('../../../../shared/url-utils');
|
|
|
|
const messages = {
|
|
pageNotFound: 'Page not found.'
|
|
};
|
|
|
|
/**
|
|
* @description Middleware, which validates and interprets the page param e.g. /page/1
|
|
* @param {Object} req
|
|
* @param {Object} res
|
|
* @param {Function} next
|
|
* @param {Number} page
|
|
* @returns {*}
|
|
*/
|
|
module.exports = function handlePageParam(req, res, next, page) {
|
|
// routeKeywords.page: 'page'
|
|
const pageRegex = new RegExp('/page/(.*)?/');
|
|
|
|
page = parseInt(page, 10);
|
|
|
|
if (page === 1) {
|
|
// CASE: page 1 is an alias for the collection index, do a permanent 301 redirect
|
|
return urlUtils.redirect301(res, req.originalUrl.replace(pageRegex, '/'));
|
|
} else if (page < 1 || isNaN(page)) {
|
|
return next(new errors.NotFoundError({
|
|
message: tpl(messages.pageNotFound)
|
|
}));
|
|
} else {
|
|
req.params.page = page;
|
|
return next();
|
|
}
|
|
};
|