Resource minification/aggregation css

lib/server.js

 'use strict';
 
 var express = require('express'),
     server = express(),
     fs = require('fs'),
     path = require('path'),
     config = require('../config').config.server,
     debugMode = (String(process.argv[2]).toLowerCase() === 'debug'),
     public_dir = config.public_dir,
     Templates = require('./templates.js'),
     view = require('./view.js');
 
 
+/**
+ * It finds a file under the given path. The first match wins.
+ * @param {string} path starting point.
+ * @param {string} fileName name of the file to be found.
+ * @return {string} the file path.
+ */
+function findFile(path, fileName) {
+  var file, files = fs.readdirSync(path);
+  for (var i = 0; i < files.length; i = i + 1) {
+    if (files[i] === fileName) {
+      return path + '/' + files[i];
+    }
+    file = fs.statSync(path + '/' + files[i]);
+    if (file.isDirectory()) {
+      file = findFile(path + '/' + files[i], fileName);
+      if (file) {
+        return file;
+      }
+    }
+  }
+  return null;
+}
+
 server.configure(function() {
   server.set('views', __dirname + './lib/views/');
   server.set('view engine', 'handlebars');
   server.set('view options', {layout: false});
   server.engine('handlebars', view.handlebars);
 
   server.use(express.logger('dev'));
   // 'static' is a reserved word so dot notation is not used to
   // avoid annoying the linter.
   server.use('/juju-ui', express['static'](public_dir));
@@ skipping 20 matching lines @@
   res.json({
     uptime: process.uptime(),
     memory: process.memoryUsage()
   });
 });
 
 server.get('/assets/all-third.js', function(req, res) {
   res.sendfile('app/assets/javascripts/generated/all-third.js');
 });
 
+server.get('/assets/stylesheets/:file', function(req, res) {

[frankban, 2012/11/13 17:27:44]

We discussed this on IRC: adding a 'juju-ui' prefix here and before (in
"/assets/all-third.js") allows us to easily integrate the gui in other projects.
As you said, express should still work after this change.

[thiago, 2012/11/13 18:19:17]

Done.

+  var file, fileName = req.params.file;
+  if (path.extname(fileName).toLowerCase() === '.css') {
+    res.sendfile('app/assets/stylesheets/' + fileName);
+  } else {
+    // We've merged all the yui and third party files into a single css file.
+    // YUI expects to load its images from the same path as its css files, so
+    // we need to mock this position. When the system tries to load a resource
+    // from "app/assets/stylesheets/", it tries to find this resource under the
+    // "./app/assets" directory. The first match wins. Another solution would
+    // be to manually copy all the images we need to the
+    // "./app/assets/stylesheets" directory.

[matthew.scott, 2012/11/13 17:08:24]

I think this is a good way to do it, so long as the overhead isn't too great (I
expect not)

[thiago, 2012/11/13 18:19:17]

This is lightning fast.

+    // This is an example of image...
+    // ./yui/datatable-sort/assets/skins/sam/sort-arrow-sprite-ie.png
+    file = findFile('./app/assets', fileName);
+    res.sendfile(file);
+  }
+});
+
 server.get('/assets/modules.js', function(req, res) {
   if (debugMode) {
     res.sendfile('app/assets/javascripts/generated/all-app-debug.js');
   } else {
     res.sendfile('app/assets/javascripts/generated/all-app.js');
   }
 });
 
 server.get('/assets/yui.js', function(req, res) {
   if (debugMode) {
     res.sendfile('app/assets/javascripts/yui/yui/yui-debug.js');
   } else {
     res.sendfile('app/assets/javascripts/generated/all-yui.js');
   }
 });
 
 server.get('*', function(req, res) {
   res.sendfile('app/index.html');
 });
 
 exports.server = server;