Skip to content

Commit

Permalink
Moved set context call into renderer
Browse files Browse the repository at this point in the history
  • Loading branch information
ErisDS committed Nov 10, 2017
1 parent 1e63e8b commit 042575a
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 22 deletions.
4 changes: 0 additions & 4 deletions core/server/apps/amp/lib/router.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ var path = require('path'),
// Dirty requires
errors = require('../../../errors'),
postLookup = require('../../../controllers/frontend/post-lookup'),
setResponseContext = require('../../../controllers/frontend/context'),
renderer = require('../../../controllers/frontend/renderer'),

templateName = 'amp';
Expand All @@ -29,9 +28,6 @@ function _renderer(req, res, next) {
return next(new errors.NotFoundError({message: i18n.t('errors.errors.pageNotFound')}));
}

// Context
setResponseContext(req, res, data);

// Render Call
return renderer(req, res, data);
}
Expand Down
4 changes: 0 additions & 4 deletions core/server/apps/private-blogging/lib/router.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ var path = require('path'),
express = require('express'),
middleware = require('./middleware'),
bodyParser = require('body-parser'),
setResponseContext = require('../../../controllers/frontend/context'),
renderer = require('../../../controllers/frontend/renderer'),
brute = require('../../../middleware/brute'),

Expand All @@ -27,9 +26,6 @@ function _renderer(req, res) {
data.error = res.error;
}

// Context
setResponseContext(req, res);

// Render Call
return renderer(req, res, data);
}
Expand Down
4 changes: 0 additions & 4 deletions core/server/apps/subscribers/lib/router.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ var path = require('path'),
errors = require('../../../errors'),
validator = require('../../../data/validation').validator,
postLookup = require('../../../controllers/frontend/post-lookup'),
setResponseContext = require('../../../controllers/frontend/context'),
renderer = require('../../../controllers/frontend/renderer'),

templateName = 'subscribe';
Expand All @@ -27,9 +26,6 @@ function _renderer(req, res) {
// Format data
var data = req.body;

// Context
setResponseContext(req, res);

// Render Call
return renderer(req, res, data);
}
Expand Down
6 changes: 1 addition & 5 deletions core/server/controllers/frontend/render-channel.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
var debug = require('ghost-ignition').debug('channels:render'),
formatResponse = require('./format-response'),
renderer = require('./renderer'),
setResponseContext = require('./context');
renderer = require('./renderer');

module.exports = function renderChannel(req, res) {
debug('renderChannel called');
Expand All @@ -11,9 +10,6 @@ module.exports = function renderChannel(req, res) {
// Do final data formatting and then render
var data = formatResponse.channel(result);

// Context
setResponseContext(req, res);

// Render Call
return renderer(req, res, data);
};
Expand Down
6 changes: 1 addition & 5 deletions core/server/controllers/frontend/render-entry.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
var debug = require('ghost-ignition').debug('channels:render-post'),
formatResponse = require('./format-response'),
renderer = require('./renderer'),
setResponseContext = require('./context');
renderer = require('./renderer');
/*
* Sets the response context around an entry (post or page)
* and renders it with the correct template.
Expand All @@ -15,9 +14,6 @@ module.exports = function renderEntry(req, res) {
// Format data 2 - 1 is in preview/entry
var data = formatResponse.entry(entry);

// Context
setResponseContext(req, res, data);

// Render Call
return renderer(req, res, data);
};
Expand Down
4 changes: 4 additions & 0 deletions core/server/controllers/frontend/renderer.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,11 @@
var debug = require('ghost-ignition').debug('renderer'),
setContext = require('./context'),
templates = require('./templates');

module.exports = function renderer(req, res, data) {
// Context
setContext(req, res, data);

// Template
templates.setTemplate(req, res, data);

Expand Down

0 comments on commit 042575a

Please sign in to comment.