mirror of
https://github.com/TryGhost/Ghost.git
synced 2025-02-03 23:00:14 -05:00
0201c431d7
* 🎨 🔥 do not store settings in config and make settings cache easier available - remove remembering settings value in theme config - if we need a cache value, we are asking the settings cache directly - instead of settings.getSettingSync we use settings.cache.get - added TODO: - think about moving the settings cache out of api/settings - we could create a folder named cache cache/settings - this settings cache listens on model changes for settings - decoupling * 🔥 remove timezone from config - no need to store in overrides config and in defaults settings * 🎨 context object helper - replace config.get('theme') by settings cache * 🎨 replace config.get('theme') by settings.cache.get * 🎨 adapt tests * fixes from comments
88 lines
2.9 KiB
JavaScript
88 lines
2.9 KiB
JavaScript
var path = require('path'),
|
|
express = require('express'),
|
|
_ = require('lodash'),
|
|
ampRouter = express.Router(),
|
|
i18n = require('../../../i18n'),
|
|
|
|
// Dirty requires
|
|
errors = require('../../../errors'),
|
|
settingsCache = require('../../../api/settings').cache,
|
|
templates = require('../../../controllers/frontend/templates'),
|
|
postLookup = require('../../../controllers/frontend/post-lookup'),
|
|
setResponseContext = require('../../../controllers/frontend/context');
|
|
|
|
function controller(req, res, next) {
|
|
var defaultView = path.resolve(__dirname, 'views', 'amp.hbs'),
|
|
paths = templates.getActiveThemePaths(req.app.get('activeTheme')),
|
|
data = req.body || {};
|
|
|
|
if (res.error) {
|
|
data.error = res.error;
|
|
}
|
|
|
|
setResponseContext(req, res, data);
|
|
|
|
// we have to check the context. Our context must be ['post', 'amp'], otherwise we won't render the template
|
|
if (_.includes(res.locals.context, 'post') && _.includes(res.locals.context, 'amp')) {
|
|
if (paths.hasOwnProperty('amp.hbs')) {
|
|
return res.render('amp', data);
|
|
} else {
|
|
return res.render(defaultView, data);
|
|
}
|
|
} else {
|
|
return next();
|
|
}
|
|
}
|
|
|
|
function getPostData(req, res, next) {
|
|
req.body = req.body || {};
|
|
|
|
postLookup(res.locals.relativeUrl)
|
|
.then(function (result) {
|
|
if (result && result.post) {
|
|
req.body.post = result.post;
|
|
}
|
|
|
|
next();
|
|
})
|
|
.catch(function (err) {
|
|
next(err);
|
|
});
|
|
}
|
|
|
|
function checkIfAMPIsEnabled(req, res, next) {
|
|
var ampIsEnabled = settingsCache.get('amp');
|
|
|
|
if (ampIsEnabled) {
|
|
return next();
|
|
}
|
|
|
|
// CASE: we don't support amp pages for static pages
|
|
if (req.body.post && req.body.post.page) {
|
|
return next(new errors.NotFoundError({message: i18n.t('errors.errors.pageNotFound')}));
|
|
}
|
|
|
|
/**
|
|
* CASE: amp is disabled, we serve 404
|
|
*
|
|
* Alternatively we could redirect to the original post, as the user can enable/disable AMP every time.
|
|
*
|
|
* If we would call `next()`, express jumps to the frontend controller (server/controllers/frontend/index.js fn single)
|
|
* and tries to lookup the post (again) and checks whether the post url equals the requested url (post.url !== req.path).
|
|
* This check would fail if the blog is setup on a subdirectory.
|
|
*/
|
|
return next(new errors.NotFoundError({message: i18n.t('errors.errors.pageNotFound')}));
|
|
}
|
|
|
|
// AMP frontend route
|
|
ampRouter.route('/')
|
|
.get(
|
|
getPostData,
|
|
checkIfAMPIsEnabled,
|
|
controller
|
|
);
|
|
|
|
module.exports = ampRouter;
|
|
module.exports.controller = controller;
|
|
module.exports.getPostData = getPostData;
|
|
module.exports.checkIfAMPIsEnabled = checkIfAMPIsEnabled;
|