Merge pull request #47 from murphybob/filename-template

Add support and test for output filename template.

Author: kevva

Gruntfile.js

@@ -45,6 +45,15 @@ module.exports = function (grunt) {
         src: ['*'],
         dest: 'test/tmp/expand'
       },
+      withFilenameProcessing: {
+        options: {
+          process: function(name, hash, extension) {
+            return name + '-processed-' + hash + '.' + extension;
+          }
+        },
+        src: ['test/fixtures/another.png'],
+        dest: 'test/tmp'
+      },
       withSummaryAttributeName: {
         options: {
           summary: 'foo'

tasks/filerev.js

@@ -46,7 +46,18 @@ module.exports = function (grunt) {
         var hash = crypto.createHash(options.algorithm).update(fs.readFileSync(file)).digest('hex');
         var suffix = hash.slice(0, options.length);
         var ext = path.extname(file);
-        var newName = [path.basename(file, ext), suffix, ext.slice(1)].join('.');
+        var newName;
+
+        if (typeof options.process === 'function') {
+          newName = options.process(path.basename(file, ext), suffix, ext.slice(1));
+        } else {
+          if (options.process) {
+            grunt.log.error('options.process must be a function; ignoring');
+          }
+
+          newName = [path.basename(file, ext), suffix, ext.slice(1)].join('.');
+        }
+
         var resultPath;
 
         if (move) {

test/test.js

@@ -7,7 +7,8 @@ var hashes = {
   'test/fixtures/cfgfile.png' : 'test/tmp/cfgfile.da63.png',
   'test/fixtures/math.js' : 'test/tmp/withSourceMaps/math.2f56179e.js',
   'test/fixtures/math.js.map' : 'test/tmp/withSourceMaps/math.2f56179e.js.map',
-  'test/fixtures/physics.js' : 'test/tmp/withSourceMaps/physics.14a0a482.js'
+  'test/fixtures/physics.js' : 'test/tmp/withSourceMaps/physics.14a0a482.js',
+  'test/fixtures/another.png' : 'test/tmp/another-processed-92279d3f.png'
 };
 
 it('should revision files based on content', function () {
@@ -52,3 +53,9 @@ it('should revision .js file ok without any .map', function () {
   assert(revisioned === original);
 });
 
+it('should allow a filename processing function', function () {
+  var file = 'test/fixtures/another.png';
+  var original = fs.statSync(file).size;
+  var revisioned = fs.statSync(hashes[file]).size;
+  assert(revisioned === original);
+});