terribleplan/next.js
1
0
Fork 0
mirror of https://github.com/terribleplan/next.js.git synced 2024-01-19 02:48:18 +00:00
Code Issues Releases Wiki Activity

Remove unused properties (#5837)

Browse source
This commit is contained in:
Tim Neutkens 2018-12-07 00:57:39 +01:00 committed by GitHub
parent c03d25b97a
commit 8ca5749ff9
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
packages/next-server/server/render.js
View file

@ -16,8 +16,9 @@ export function renderToHTML (req, res, pathname, query, opts) {
return doRender(req, res, pathname, query, opts) return doRender(req, res, pathname, query, opts)
} }
export function renderErrorToHTML (err, req, res, pathname, query, opts = {}) { // _pathname is for backwards compatibility
return doRender(req, res, pathname, query, { ...opts, err, page: '/_error' }) export function renderErrorToHTML (err, req, res, _pathname, query, opts = {}) {
return doRender(req, res, '/_error', query, { ...opts, err })
} }
function getPageFiles (buildManifest, page) { function getPageFiles (buildManifest, page) {
@ -34,24 +35,20 @@ function getPageFiles (buildManifest, page) {
async function doRender (req, res, pathname, query, { async function doRender (req, res, pathname, query, {
err, err,
page,
buildId, buildId,
assetPrefix, assetPrefix,
runtimeConfig, runtimeConfig,
distDir, distDir,
dir,
dev = false, dev = false,
staticMarkup = false, staticMarkup = false,
nextExport nextExport
} = {}) { } = {}) {
page = page || pathname
const documentPath = join(distDir, SERVER_DIRECTORY, CLIENT_STATIC_FILES_PATH, buildId, 'pages', '_document') const documentPath = join(distDir, SERVER_DIRECTORY, CLIENT_STATIC_FILES_PATH, buildId, 'pages', '_document')
const appPath = join(distDir, SERVER_DIRECTORY, CLIENT_STATIC_FILES_PATH, buildId, 'pages', '_app') const appPath = join(distDir, SERVER_DIRECTORY, CLIENT_STATIC_FILES_PATH, buildId, 'pages', '_app')
let [buildManifest, reactLoadableManifest, Component, Document, App] = await Promise.all([ let [buildManifest, reactLoadableManifest, Component, Document, App] = await Promise.all([
require(join(distDir, BUILD_MANIFEST)), require(join(distDir, BUILD_MANIFEST)),
require(join(distDir, REACT_LOADABLE_MANIFEST)), require(join(distDir, REACT_LOADABLE_MANIFEST)),
requirePage(page, {distDir}), requirePage(pathname, {distDir}),
require(documentPath), require(documentPath),
require(appPath) require(appPath)
]) ])
@ -61,19 +58,19 @@ async function doRender (req, res, pathname, query, {
Component = Component.default || Component Component = Component.default || Component
if (typeof Component !== 'function') { if (typeof Component !== 'function') {
throw new Error(`The default export is not a React Component in page: "${page}"`) throw new Error(`The default export is not a React Component in page: "${pathname}"`)
} }
App = App.default || App App = App.default || App
Document = Document.default || Document Document = Document.default || Document
const asPath = req.url const asPath = req.url
const ctx = { err, req, res, pathname: page, query, asPath } const ctx = { err, req, res, pathname, query, asPath }
const router = new Router(page, query, asPath) const router = new Router(pathname, query, asPath)
const props = await loadGetInitialProps(App, {Component, router, ctx}) const props = await loadGetInitialProps(App, {Component, router, ctx})
const devFiles = buildManifest.devFiles const devFiles = buildManifest.devFiles
const files = [ const files = [
...new Set([ ...new Set([
...getPageFiles(buildManifest, page), ...getPageFiles(buildManifest, pathname),
...getPageFiles(buildManifest, '/_app'), ...getPageFiles(buildManifest, '/_app'),
...getPageFiles(buildManifest, '/_error') ...getPageFiles(buildManifest, '/_error')
]) ])
@ -136,7 +133,7 @@ async function doRender (req, res, pathname, query, {
const doc = <Document {...{ const doc = <Document {...{
__NEXT_DATA__: { __NEXT_DATA__: {
props, // The result of getInitialProps props, // The result of getInitialProps
page, // The rendered page page: pathname, // The rendered page
query, // querystring parsed / passed by the user query, // querystring parsed / passed by the user
buildId, // buildId is used to facilitate caching of page bundles, we send it to the client so that pageloader knows where to load bundles buildId, // buildId is used to facilitate caching of page bundles, we send it to the client so that pageloader knows where to load bundles
assetPrefix: assetPrefix === '' ? undefined : assetPrefix, // send assetPrefix to the client side when configured, otherwise don't sent in the resulting HTML assetPrefix: assetPrefix === '' ? undefined : assetPrefix, // send assetPrefix to the client side when configured, otherwise don't sent in the resulting HTML
@ -145,10 +142,7 @@ async function doRender (req, res, pathname, query, {
dynamicIds: dynamicImportsIds.length === 0 ? undefined : dynamicImportsIds, dynamicIds: dynamicImportsIds.length === 0 ? undefined : dynamicImportsIds,
err: (err) ? serializeError(dev, err) : undefined // Error if one happened, otherwise don't sent in the resulting HTML err: (err) ? serializeError(dev, err) : undefined // Error if one happened, otherwise don't sent in the resulting HTML
}, },
dev,
dir,
staticMarkup, staticMarkup,
buildManifest,
devFiles, devFiles,
files, files,
dynamicImports, dynamicImports,
@ -171,7 +165,6 @@ export function sendHTML (req, res, html, method, { dev, generateEtags }) {
if (dev) { if (dev) {
// In dev, we should not cache pages for any reason. // In dev, we should not cache pages for any reason.
// That's why we do this.
res.setHeader('Cache-Control', 'no-store, must-revalidate') res.setHeader('Cache-Control', 'no-store, must-revalidate')
} }