From 2253e4ead7502933acc34f7baba3a95a1f66c02e Mon Sep 17 00:00:00 2001
From: hacksalot <hacksalot@indevious.com>
Date: Tue, 19 Jan 2016 16:01:34 -0500
Subject: [PATCH] Fix theme counts.

The N in "Applying theme FOOBAR (N formats)" should reflect the count of
explicit + freebie output formats.
---
 src/cli/msg.yml    |  2 +-
 src/cli/out.js     |  2 +-
 src/verbs/build.js | 32 +++++++++++++++++++++++---------
 3 files changed, 25 insertions(+), 11 deletions(-)

diff --git a/src/cli/msg.yml b/src/cli/msg.yml
index 8d35cff..f0b84ad 100644
--- a/src/cli/msg.yml
+++ b/src/cli/msg.yml
@@ -13,7 +13,7 @@ events:
     msg:
       - Merging **%s**
       - " onto **%s**"
-  afterMerge:
+  applyTheme:
     msg: Applying **%s** theme (%s format%s)
   afterBuild:
     msg:
diff --git a/src/cli/out.js b/src/cli/out.js
index 4adccb4..8300a20 100644
--- a/src/cli/out.js
+++ b/src/cli/out.js
@@ -108,7 +108,7 @@ Output routines for HackMyResume.
 
         case HME.applyTheme:
           var numFormats = Object.keys( this.theme.formats ).length;
-          L( M2C(this.msgs.afterMerge.msg, 'green'),
+          L( M2C(this.msgs.applyTheme.msg, 'green'),
             this.theme.name.toUpperCase(),
             numFormats, ( numFormats === 1 ? '' : 's') );
           break;
diff --git a/src/verbs/build.js b/src/verbs/build.js
index fbbba2a..bd55cb4 100644
--- a/src/verbs/build.js
+++ b/src/verbs/build.js
@@ -90,7 +90,6 @@ Implementation of the 'build' verb for HackMyResume.
     // Merge input resumes, yielding a single source resume.
     var rez;
     if( sheets.length > 1 ) {
-
       var isFRESH = !sheets[0].basics;
       var mixed = _.any( sheets, function(s) { return isFRESH ? s.basics : !s.basics; });
       this.stat( HMEVENT.beforeMerge, { f: _.clone(sheetObjects), mixed: mixed });
@@ -106,9 +105,6 @@ Implementation of the 'build' verb for HackMyResume.
       rez = sheets[0];
     }
 
-    this.stat( HMEVENT.applyTheme, { r: rez });
-
-
     // Convert the merged source resume to the theme's format, if necessary
     var orgFormat = rez.basics ? 'JRS' : 'FRESH';
     var toFormat = theme.render ? 'JRS' : 'FRESH';
@@ -118,6 +114,11 @@ Implementation of the 'build' verb for HackMyResume.
       this.stat( HMEVENT.afterInlineConvert, { file: sheetObjects[0].file, fmt: toFormat });
     }
 
+    // Add freebie formats to the theme
+    addFreebieFormats( theme  );
+    this.stat( HMEVENT.applyTheme, { r: rez });
+
+
     // Load the resume into a FRESHResume or JRSResume object
     _rezObj = new (RTYPES[ toFormat ])().parseJSON( rez );
 
@@ -257,13 +258,15 @@ Implementation of the 'build' verb for HackMyResume.
 
 
   /**
-  Expand output files. For example, "foo.all" should be expanded to
-  ["foo.html", "foo.doc", "foo.pdf", "etc"].
-  @param dst An array of output files as specified by the user.
+  Reinforce the chosen theme with "freebie" formats provided by HackMyResume.
+  A "freebie" format is an output format such as JSON, YML, or PNG that can be
+  generated directly from the resume model or from one of the theme's declared
+  output formats. For example, the PNG format can be generated for any theme
+  that declares an HTML format; the theme doesn't have to provide an explicit
+  PNG template.
   @param theTheme A FRESHTheme or JRSTheme object.
   */
-  function expand( dst, theTheme ) {
-
+  function addFreebieFormats( theTheme ) {
     // Add freebie formats (JSON, YAML, PNG) every theme gets...
     // Add HTML-driven PNG only if the theme has an HTML format.
     theTheme.formats.json = theTheme.formats.json || {
@@ -280,6 +283,17 @@ Implementation of the 'build' verb for HackMyResume.
         ext: 'yml', path: null, data: null
       };
     }
+  }
+
+
+
+  /**
+  Expand output files. For example, "foo.all" should be expanded to
+  ["foo.html", "foo.doc", "foo.pdf", "etc"].
+  @param dst An array of output files as specified by the user.
+  @param theTheme A FRESHTheme or JRSTheme object.
+  */
+  function expand( dst, theTheme ) {
 
     // Set up the destination collection. It's either the array of files passed
     // by the user or 'out/resume.all' if no targets were specified.