From 5b29b57245b5c1f12a00d9f8600298c07632bce2 Mon Sep 17 00:00:00 2001 From: Josh Sherman Date: Thu, 4 Sep 2014 11:05:03 -0400 Subject: [PATCH] Added ability to override the arguments I needed to be able to use a different set of arguments. Updated to allow for the args to be passed in the same way the src and dest are. Could improve by comparing the two arrays instead of simply overriding them. --- tasks/wkhtmltopdf.js | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-) diff --git a/tasks/wkhtmltopdf.js b/tasks/wkhtmltopdf.js index 6f6dc05..8fb4def 100644 --- a/tasks/wkhtmltopdf.js +++ b/tasks/wkhtmltopdf.js @@ -51,15 +51,24 @@ module.exports = function(grunt) { "Converting " + src + " -> " + dest ); - // Launch PhantomJS. + // default args + var args = [ + '--dpi', '96', // workarround to wkhtmltopdf letter-spacing bug (see http://code.google.com/p/wkhtmltopdf/issues/detail?id=72) + '--print-media-type', // Use @print media type + ]; + + // overrides the args + if (file.args) { + args = file.args; + } + + // adds the src and dest + args = args.concat([src, dest]); + + // Launch wkhtmltopdf. helper.convert({ code: 90, - args: [ - '--dpi', '96', // workarround to wkhtmltopdf letter-spacing bug (see http://code.google.com/p/wkhtmltopdf/issues/detail?id=72) - '--print-media-type', // Use @print media type - src, - dest - ], + args: args, done: function(err) { if (err) { grunt.log('>>>', err);