1
0
mirror of https://github.com/JuanCanham/HackMyResume.git synced 2024-11-22 00:10:15 +00:00

Fix theme counts.

The N in "Applying theme FOOBAR (N formats)" should reflect the count of
explicit + freebie output formats.
This commit is contained in:
hacksalot 2016-01-19 16:01:34 -05:00
parent 2f628f8564
commit 2253e4ead7
3 changed files with 25 additions and 11 deletions

View File

@ -13,7 +13,7 @@ events:
msg: msg:
- Merging **%s** - Merging **%s**
- " onto **%s**" - " onto **%s**"
afterMerge: applyTheme:
msg: Applying **%s** theme (%s format%s) msg: Applying **%s** theme (%s format%s)
afterBuild: afterBuild:
msg: msg:

View File

@ -108,7 +108,7 @@ Output routines for HackMyResume.
case HME.applyTheme: case HME.applyTheme:
var numFormats = Object.keys( this.theme.formats ).length; 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(), this.theme.name.toUpperCase(),
numFormats, ( numFormats === 1 ? '' : 's') ); numFormats, ( numFormats === 1 ? '' : 's') );
break; break;

View File

@ -90,7 +90,6 @@ Implementation of the 'build' verb for HackMyResume.
// Merge input resumes, yielding a single source resume. // Merge input resumes, yielding a single source resume.
var rez; var rez;
if( sheets.length > 1 ) { if( sheets.length > 1 ) {
var isFRESH = !sheets[0].basics; var isFRESH = !sheets[0].basics;
var mixed = _.any( sheets, function(s) { return isFRESH ? s.basics : !s.basics; }); var mixed = _.any( sheets, function(s) { return isFRESH ? s.basics : !s.basics; });
this.stat( HMEVENT.beforeMerge, { f: _.clone(sheetObjects), mixed: mixed }); this.stat( HMEVENT.beforeMerge, { f: _.clone(sheetObjects), mixed: mixed });
@ -106,9 +105,6 @@ Implementation of the 'build' verb for HackMyResume.
rez = sheets[0]; rez = sheets[0];
} }
this.stat( HMEVENT.applyTheme, { r: rez });
// Convert the merged source resume to the theme's format, if necessary // Convert the merged source resume to the theme's format, if necessary
var orgFormat = rez.basics ? 'JRS' : 'FRESH'; var orgFormat = rez.basics ? 'JRS' : 'FRESH';
var toFormat = theme.render ? '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 }); 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 // Load the resume into a FRESHResume or JRSResume object
_rezObj = new (RTYPES[ toFormat ])().parseJSON( rez ); _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 Reinforce the chosen theme with "freebie" formats provided by HackMyResume.
["foo.html", "foo.doc", "foo.pdf", "etc"]. A "freebie" format is an output format such as JSON, YML, or PNG that can be
@param dst An array of output files as specified by the user. 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. @param theTheme A FRESHTheme or JRSTheme object.
*/ */
function expand( dst, theTheme ) { function addFreebieFormats( theTheme ) {
// Add freebie formats (JSON, YAML, PNG) every theme gets... // Add freebie formats (JSON, YAML, PNG) every theme gets...
// Add HTML-driven PNG only if the theme has an HTML format. // Add HTML-driven PNG only if the theme has an HTML format.
theTheme.formats.json = theTheme.formats.json || { theTheme.formats.json = theTheme.formats.json || {
@ -280,6 +283,17 @@ Implementation of the 'build' verb for HackMyResume.
ext: 'yml', path: null, data: null 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 // 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. // by the user or 'out/resume.all' if no targets were specified.