Views refractoring

package.json

@@ -17,7 +17,6 @@
     "blueimp-file-upload": "^9.10.1",
     "body-parser": "^1.12.4",
     "bootstrap": "^3.3.5",
-    "compression": "^1.5.0",
     "config": "^1.14.0",
     "connect-livereload": "^0.5.3",
     "debug": "^2.2.0",

routes/api/v1/index.js

@@ -1,10 +1,12 @@
 ;(function () {
   'use strict'
 
-  var api = {}
+  var express = require('express')
-  api.videos = require('./videos')
+  var router = express.Router()
-  api.remoteVideos = require('./remoteVideos')
-  api.pods = require('./pods')
 
-  module.exports = api
+  router.use('/videos', require('./videos'))
+  router.use('/remotevideos', require('./remoteVideos'))
+  router.use('/pods', require('./pods'))
+
+  module.exports = router
 })()

routes/index.js

@@ -1,24 +1,10 @@
 ;(function () {
   'use strict'
 
-  function getPartial (req, res) {
+  var routes = {
-    var directory = req.params.directory
+    api: require('./api/' + global.API_VERSION),
-    var name = req.params.name
+    views: require('./views')
-
-    res.render('partials/' + directory + '/' + name)
   }
 
-  function getIndex (req, res) {
+  module.exports = routes
-    res.render('index')
-  }
-
-  var express = require('express')
-  var middleware = require('../middlewares')
-
-  var router = express.Router()
-
-  router.get('/partials/:directory/:name', middleware.cache(), getPartial)
-  router.get(/^\/(index)?$/, middleware.cache(), getIndex)
-
-  module.exports = router
 })()

routes/views.js

@@ -0,0 +1,24 @@
+;(function () {
+  'use strict'
+
+  function getPartial (req, res) {
+    var directory = req.params.directory
+    var name = req.params.name
+
+    res.render('partials/' + directory + '/' + name)
+  }
+
+  function getIndex (req, res) {
+    res.render('index')
+  }
+
+  var express = require('express')
+  var middleware = require('../middlewares')
+
+  var router = express.Router()
+
+  router.get('/partials/:directory/:name', middleware.cache(), getPartial)
+  router.get(/^\/(index)?$/, middleware.cache(), getIndex)
+
+  module.exports = router
+})()

server.js

@@ -32,12 +32,10 @@
   var config = require('config')
   var logger = require('./src/logger')
   var routes = require('./routes')
-  var api = require('./routes/api/' + global.API_VERSION)
   var utils = require('./src/utils')
   var videos = require('./src/videos')
   var webtorrent = require('./src/webTorrentNode')
 
-  var compression
   var port = config.get('listen.port')
   var uploads = config.get('storage.uploads')
 
@@ -52,51 +50,25 @@
 
   // ----------- Views, routes and static files -----------
 
-  if (process.env.NODE_ENV === 'production') {
+  // Livereload
-    // logger.log('Production : static files in dist/\n')
+  app.use(require('connect-livereload')({
+    port: 35729
+  }))
 
-    // GZip compression
+  require('segfault-handler').registerHandler()
-    compression = require('compression')
-    app.use(compression())
 
-    // A month
+  app.use(express.static(path.join(__dirname, '/public'), { maxAge: 0 }))
-    var maxAge = 86400000 * 30
 
-    // TODO
+  // Jade template from ./views directory
-    app.get(/^\/(index|(partials\/[a-z\/]+))?$/, function (req, res, next) {
+  app.set('views', path.join(__dirname, '/views'))
-      if (req.url === '/') {
+  app.set('view engine', 'jade')
-        req.url = '/index'
-      }
 
-      req.url += '.html'
+  // API
-      next()
-    })
-
-    app.use(express.static(path.join(__dirname, '/dist/public'), { maxAge: maxAge }))
-    app.use(express.static(path.join(__dirname, '/dist/views'), { maxAge: maxAge }))
-  } else {
-    // Livereload
-    app.use(require('connect-livereload')({
-      port: 35729
-    }))
-
-    require('segfault-handler').registerHandler()
-
-    app.use(express.static(path.join(__dirname, '/public'), { maxAge: 0 }))
-
-    // Jade template from ./views directory
-    app.set('views', path.join(__dirname, '/views'))
-    app.set('view engine', 'jade')
-
-    // Views routes
-    app.use('/', routes)
-  }
-
-  // ----------- Routes -----------
   var api_route = '/api/' + global.API_VERSION
-  app.use(api_route + '/videos', api.videos)
+  app.use(api_route, routes.api)
-  app.use(api_route + '/remotevideos', api.remoteVideos)
+
-  app.use(api_route + '/pods', api.pods)
+  // Views routes
+  app.use('/', routes.views)
 
   // ----------- Tracker -----------