From 8e34abd3576774d7f81ed6cecc08d9ef6f209811 Mon Sep 17 00:00:00 2001 From: bill bittner Date: Mon, 27 Nov 2017 14:54:53 -0800 Subject: [PATCH] removed dead code serveOrShowAsset() --- helpers/serveHelpers.js | 47 +++++++++-------------------------------- 1 file changed, 10 insertions(+), 37 deletions(-) diff --git a/helpers/serveHelpers.js b/helpers/serveHelpers.js index d0facac5..2bccb894 100644 --- a/helpers/serveHelpers.js +++ b/helpers/serveHelpers.js @@ -1,42 +1,7 @@ const logger = require('winston'); -const SERVE = 'SERVE'; -const SHOW = 'SHOW'; -const SHOWLITE = 'SHOWLITE'; // const { postToStats, sendGoogleAnalytics } = require('../controllers/statsController.js'); -function createOpenGraphInfo ({ claimId, name, fileExt }) { - return { - embedUrl : `https://spee.ch/embed/${claimId}/${name}`, - showUrl : `https://spee.ch/${claimId}/${name}`, - source : `https://spee.ch/${claimId}/${name}.${fileExt}`, - directFileUrl: `https://spee.ch/${claimId}/${name}.${fileExt}`, - }; -} - module.exports = { - serveOrShowAsset (method, fileInfo, claimInfo, shortId, res) { - // add file extension to the file info - if (fileInfo.fileName) { - claimInfo['fileExt'] = fileInfo.fileName.substring(fileInfo.fileName.lastIndexOf('.') + 1); - } else { - claimInfo['fileExt'] = null; - } - // serve or show - switch (method) { - case SERVE: - module.exports.serveFile(fileInfo, claimInfo, shortId, res); - return fileInfo; - case SHOWLITE: - module.exports.showFileLite(fileInfo, claimInfo, shortId, res); - return fileInfo; - case SHOW: - module.exports.showFile(fileInfo, claimInfo, shortId, res); - return fileInfo; - default: - logger.error('I did not recognize that method'); - break; - } - }, serveFile ({ filePath }, { claimId, name, contentType }, res) { logger.verbose(`serving ${name}#${claimId}`); // set response options @@ -56,11 +21,19 @@ module.exports = { } }, showFile (claimInfo, shortId, res) { - const openGraphInfo = createOpenGraphInfo(claimInfo); + const openGraphInfo = module.exports.createOpenGraphInfo(claimInfo); res.status(200).render('show', { layout: 'show', claimInfo, shortId, openGraphInfo }); }, showFileLite (claimInfo, shortId, res) { - const openGraphInfo = createOpenGraphInfo(claimInfo); + const openGraphInfo = module.exports.createOpenGraphInfo(claimInfo); res.status(200).render('showLite', { layout: 'showlite', claimInfo, shortId, openGraphInfo }); }, + createOpenGraphInfo ({ claimId, name, fileExt }) { + return { + embedUrl : `https://spee.ch/embed/${claimId}/${name}`, + showUrl : `https://spee.ch/${claimId}/${name}`, + source : `https://spee.ch/${claimId}/${name}.${fileExt}`, + directFileUrl: `https://spee.ch/${claimId}/${name}.${fileExt}`, + }; + }, };