mirror of
https://github.com/TryGhost/Ghost.git
synced 2024-12-22 02:11:44 +03:00
22e13acd65
- All var declarations are now const or let as per ES6 - All comma-separated lists / chained declarations are now one declaration per line - This is for clarity/readability but also made running the var-to-const/let switch smoother - ESLint rules updated to match How this was done: - npm install -g jscodeshift - git clone https://github.com/cpojer/js-codemod.git - git clone git@github.com:TryGhost/Ghost.git shallow-ghost - cd shallow-ghost - jscodeshift -t ../js-codemod/transforms/unchain-variables.js . -v=2 - jscodeshift -t ../js-codemod/transforms/no-vars.js . -v=2 - yarn - yarn test - yarn lint / fix various lint errors (almost all indent) by opening files and saving in vscode - grunt test-regression - sorted!
91 lines
2.8 KiB
JavaScript
91 lines
2.8 KiB
JavaScript
const path = require('path');
|
|
const express = require('express');
|
|
const ampRouter = express.Router();
|
|
|
|
// Dirty requires
|
|
const common = require('../../../../server/lib/common');
|
|
|
|
const urlService = require('../../../services/url');
|
|
const helpers = require('../../../services/routing/helpers');
|
|
const templateName = 'amp';
|
|
|
|
function _renderer(req, res, next) {
|
|
res.routerOptions = {
|
|
type: 'custom',
|
|
templates: templateName,
|
|
defaultTemplate: path.resolve(__dirname, 'views', `${templateName}.hbs`)
|
|
};
|
|
|
|
// Renderer begin
|
|
// Format data
|
|
let data = req.body || {};
|
|
|
|
// CASE: we only support amp pages for posts that are not static pages
|
|
if (!data.post || data.post.page) {
|
|
return next(new common.errors.NotFoundError({message: common.i18n.t('errors.errors.pageNotFound')}));
|
|
}
|
|
|
|
// Render Call
|
|
return helpers.renderer(req, res, data);
|
|
}
|
|
|
|
// This here is a controller.
|
|
// In fact, this whole file is nothing more than a controller + renderer & doesn't need to be a router
|
|
function getPostData(req, res, next) {
|
|
req.body = req.body || {};
|
|
|
|
const urlWithoutSubdirectoryWithoutAmp = res.locals.relativeUrl.match(/(.*?\/)amp\/?$/)[1];
|
|
|
|
/**
|
|
* @NOTE
|
|
*
|
|
* We have to figure out the target permalink, otherwise it would be possible to serve a post
|
|
* which lives in two collections.
|
|
*
|
|
* @TODO:
|
|
*
|
|
* This is not optimal and caused by the fact how apps currently work. But apps weren't designed
|
|
* for dynamic routing.
|
|
*
|
|
* I think if the responsible, target router would first take care fetching/determining the post, the
|
|
* request could then be forwarded to this app. Then we don't have to:
|
|
*
|
|
* 1. care about fetching the post
|
|
* 2. care about if the post can be served
|
|
* 3. then this app would act like an extension
|
|
*
|
|
* The challenge is to design different types of apps e.g. extensions of routers, standalone pages etc.
|
|
*/
|
|
const permalinks = urlService.getPermalinkByUrl(urlWithoutSubdirectoryWithoutAmp, {withUrlOptions: true});
|
|
|
|
if (!permalinks) {
|
|
return next(new common.errors.NotFoundError({
|
|
message: common.i18n.t('errors.errors.pageNotFound')
|
|
}));
|
|
}
|
|
|
|
// @NOTE: amp is not supported for static pages
|
|
// @TODO: https://github.com/TryGhost/Ghost/issues/10548
|
|
helpers.entryLookup(urlWithoutSubdirectoryWithoutAmp, {permalinks, query: {controller: 'postsPublic', resource: 'posts'}}, res.locals)
|
|
.then((result) => {
|
|
if (result && result.entry) {
|
|
req.body.post = result.entry;
|
|
}
|
|
|
|
next();
|
|
})
|
|
.catch(next);
|
|
}
|
|
|
|
// AMP frontend route
|
|
ampRouter
|
|
.route('/')
|
|
.get(
|
|
getPostData,
|
|
_renderer
|
|
);
|
|
|
|
module.exports = ampRouter;
|
|
module.exports.renderer = _renderer;
|
|
module.exports.getPostData = getPostData;
|