diff --git a/core/frontend/helpers/index.js b/core/frontend/helpers/index.js index 5213f0fe48..b484dcbec0 100644 --- a/core/frontend/helpers/index.js +++ b/core/frontend/helpers/index.js @@ -1,12 +1,14 @@ var _ = require('underscore'), moment = require('moment'), when = require('when'), - pagination = require('./paginate'), - navHelper = require('./navigation'), hbs = require('express-hbs'), + errors = require('../../shared/errorHandling'), coreHelpers; coreHelpers = function (ghost) { + var navHelper, + paginationHelper; + /** * [ description] * @todo ghost core helpers + a way for themes to register them @@ -119,10 +121,40 @@ coreHelpers = function (ghost) { } return ret; }); - // Just one async helper for now, but could be more in the future + + // ## Template driven helpers + // Template driven helpers require that their template is loaded before they can be registered. + + // ###Nav Helper + // `{{nav}}` + // Outputs a navigation menu built from items in config.js + navHelper = ghost.loadTemplate('nav').then(function (templateFn) { + ghost.registerThemeHelper('nav', function (options) { + if (!_.isObject(this.navItems) || _.isFunction(this.navItems)) { + errors.logAndThrowError('navItems data is not an object or is a function'); + return; + } + return new hbs.handlebars.SafeString(templateFn({links: this.navItems})); + }); + }); + + // ### Pagination Helper + // `{{paginate}}` + // Outputs previous and next buttons, along with info about the current page + paginationHelper = ghost.loadTemplate('pagination').then(function (templateFn) { + ghost.registerThemeHelper('paginate', function (options) { + if (!_.isObject(this.pagination) || _.isFunction(this.pagination)) { + errors.logAndThrowError('pagination data is not an object or is a function'); + return; + } + return new hbs.handlebars.SafeString(templateFn(this.pagination)); + }); + }); + + // Return once the template-driven helpers have loaded return when.join( - navHelper.registerWithGhost(ghost), - pagination.registerWithGhost(ghost) + navHelper, + paginationHelper ); }; diff --git a/core/frontend/helpers/navigation.js b/core/frontend/helpers/navigation.js deleted file mode 100644 index 0ed7a4a526..0000000000 --- a/core/frontend/helpers/navigation.js +++ /dev/null @@ -1,46 +0,0 @@ -var fs = require('fs'), - path = require('path'), - _ = require('underscore'), - handlebars = require('express-hbs').handlebars, - nodefn = require('when/node/function'), - - NavHelper; - -NavHelper = function (navTemplate) { - // Bind the context for our methods. - _.bindAll(this, 'compileTemplate', 'renderNavItems'); - - if (_.isFunction(navTemplate)) { - this.navTemplateFunc = navTemplate; - } else { - this.navTemplatePath = navTemplate; - } -}; - -NavHelper.prototype.compileTemplate = function (templatePath) { - var self = this; - - // Allow people to overwrite the navTemplatePath - templatePath = templatePath || this.navTemplatePath; - - return nodefn.call(fs.readFile, templatePath).then(function (navTemplateContents) { - // TODO: Can handlebars compile async? - self.navTemplateFunc = handlebars.compile(navTemplateContents.toString()); - }); -}; - -NavHelper.prototype.renderNavItems = function (navItems) { - return new handlebars.SafeString(this.navTemplateFunc({links: navItems})); -}; - -// A static helper method for registering with ghost -NavHelper.registerWithGhost = function (ghost) { - var templatePath = path.join(ghost.paths().frontendViews, 'nav.hbs'), - ghostNavHelper = new NavHelper(templatePath); - - return ghostNavHelper.compileTemplate().then(function () { - ghost.registerThemeHelper("nav", ghostNavHelper.renderNavItems); - }); -}; - -module.exports = NavHelper; \ No newline at end of file diff --git a/core/frontend/helpers/paginate.js b/core/frontend/helpers/paginate.js deleted file mode 100644 index 5ff4130df7..0000000000 --- a/core/frontend/helpers/paginate.js +++ /dev/null @@ -1,45 +0,0 @@ -var fs = require('fs'), - path = require('path'), - _ = require('underscore'), - handlebars = require('express-hbs').handlebars, - nodefn = require('when/node/function'), - - PaginationHelper; - -PaginationHelper = function (paginationTemplate) { - // Bind the context for our methods. - _.bindAll(this, 'compileTemplate', 'renderPagination'); - - if (_.isFunction(paginationTemplate)) { - this.paginationTemplateFunc = paginationTemplate; - } else { - this.paginationTemplatePath = paginationTemplate; - } -}; - -PaginationHelper.prototype.compileTemplate = function (templatePath) { - var self = this; - - // Allow people to overwrite the paginationTemplatePath - templatePath = templatePath || this.paginationTemplatePath; - - return nodefn.call(fs.readFile, templatePath).then(function (paginationContents) { - // TODO: Can handlebars compile async? - self.paginationTemplateFunc = handlebars.compile(paginationContents.toString()); - }); -}; - -PaginationHelper.prototype.renderPagination = function (context) { - return new handlebars.SafeString(this.paginationTemplateFunc(context)); -}; - -PaginationHelper.registerWithGhost = function (ghost) { - var templatePath = path.join(ghost.paths().frontendViews, 'pagination.hbs'), - paginationHelper = new PaginationHelper(templatePath); - - return paginationHelper.compileTemplate().then(function () { - ghost.registerThemeHelper("paginate", paginationHelper.renderPagination); - }); -}; - -module.exports = PaginationHelper; \ No newline at end of file diff --git a/core/ghost.js b/core/ghost.js index 59c999e1d7..0d0e0b4bf8 100644 --- a/core/ghost.js +++ b/core/ghost.js @@ -6,8 +6,10 @@ var config = require('./../config'), when = require('when'), express = require('express'), errors = require('../core/shared/errorHandling'), + fs = require('fs'), path = require('path'), hbs = require('express-hbs'), + nodefn = require('when/node/function'), _ = require('underscore'), Polyglot = require('node-polyglot'), @@ -167,6 +169,21 @@ Ghost.prototype.updateSettingsCache = function (settings) { } }; +// ## Template utils + +Ghost.prototype.compileTemplate = function (templatePath) { + return nodefn.call(fs.readFile, templatePath).then(function (templateContents) { + return hbs.handlebars.compile(templateContents.toString()); + }, errors.logAndThrowError); +}; + +Ghost.prototype.loadTemplate = function (name) { + // TODO: allow themes to override these templates + var templatePath = path.join(this.paths().frontendViews, name + '.hbs'); + + return this.compileTemplate(templatePath); +}; + /** * @param {string} name * @param {Function} fn diff --git a/core/test/ghost/fixtures/test.hbs b/core/test/ghost/fixtures/test.hbs new file mode 100644 index 0000000000..b8264bd9f3 --- /dev/null +++ b/core/test/ghost/fixtures/test.hbs @@ -0,0 +1 @@ +

HelloWorld

\ No newline at end of file diff --git a/core/test/ghost/frontend_helpers_index_spec.js b/core/test/ghost/frontend_helpers_index_spec.js new file mode 100644 index 0000000000..6aed2f5813 --- /dev/null +++ b/core/test/ghost/frontend_helpers_index_spec.js @@ -0,0 +1,55 @@ +/*globals describe, beforeEach, it*/ +var should = require('should'), + sinon = require('sinon'), + when = require('when'), + _ = require('underscore'), + handlebars = require('express-hbs').handlebars, + path = require('path'), + helpers = require('../../frontend/helpers'), + Ghost = require('../../ghost'); + +describe('Core Helpers', function () { + + var ghost; + + beforeEach(function () { + ghost = new Ghost(); + }); + + + describe('Navigation Helper', function () { + + it('can render nav items', function (done) { + var templateSpy = sinon.spy(function (data) { return "rendered " + data.links.length; }), + compileSpy = sinon.stub(ghost, 'compileTemplate').returns(when.resolve(templateSpy)), + fakeNavItems = [{ + title: 'test1', + url: '/test1' + }, { + title: 'test2', + url: '/test2' + }], + rendered; + + helpers.loadCoreHelpers(ghost).then(function () { + + should.exist(handlebars.helpers.nav); + + rendered = handlebars.helpers.nav.call({navItems: fakeNavItems}); + + // Returns a string returned from navTemplateFunc + should.exist(rendered); + rendered.string.should.equal("rendered 2"); + + compileSpy.called.should.equal(true); + templateSpy.called.should.equal(true); + templateSpy.calledWith({ links: fakeNavItems }).should.equal(true); + + + compileSpy.restore(); + + done(); + }, done); + }); + }); +}); \ No newline at end of file diff --git a/core/test/ghost/frontend_helpers_navigation_spec.js b/core/test/ghost/frontend_helpers_navigation_spec.js deleted file mode 100644 index 84aafad41e..0000000000 --- a/core/test/ghost/frontend_helpers_navigation_spec.js +++ /dev/null @@ -1,65 +0,0 @@ -/*globals describe, beforeEach, it*/ -var should = require('should'), - sinon = require('sinon'), - _ = require('underscore'), - path = require('path'), - NavHelper = require('../../frontend/helpers/navigation'); - -describe('Navigation Helper', function () { - var navTemplatePath = path.join(process.cwd(), 'core/frontend/views/nav.hbs'); - - should.exist(NavHelper, "Navigation helper exists"); - - it('can compile the nav template', function (done) { - var helper = new NavHelper(navTemplatePath); - - helper.compileTemplate().then(function () { - should.exist(helper.navTemplateFunc); - _.isFunction(helper.navTemplateFunc).should.equal(true); - - done(); - }, done); - }); - - it('can render nav items', function () { - var helper = new NavHelper(function (data) { return "rendered " + data.links.length; }), - templateSpy = sinon.spy(helper, 'navTemplateFunc'), - fakeNavItems = [{ - title: 'test1', - url: '/test1' - }, { - title: 'test2', - url: '/test2' - }], - rendered; - - rendered = helper.renderNavItems(fakeNavItems); - - // Returns a string returned from navTemplateFunc - should.exist(rendered); - rendered.string.should.equal("rendered 2"); - - templateSpy.calledWith({ links: fakeNavItems }).should.equal(true); - }); - - it('can register with ghost', function (done) { - var fakeGhost = { - paths: function () { - return { - frontendViews: path.join(process.cwd(), 'core/frontend/views/') - }; - }, - - registerThemeHelper: function () { - return; - } - }, - registerStub = sinon.stub(fakeGhost, 'registerThemeHelper'); - - NavHelper.registerWithGhost(fakeGhost).then(function () { - registerStub.called.should.equal(true); - - done(); - }, done); - }); -}); \ No newline at end of file diff --git a/core/test/ghost/ghost_spec.js b/core/test/ghost/ghost_spec.js index 109da82945..ba35e58110 100644 --- a/core/test/ghost/ghost_spec.js +++ b/core/test/ghost/ghost_spec.js @@ -2,9 +2,17 @@ var should = require('should'), when = require('when'), sinon = require('sinon'), + path = require('path'), + _ = require('underscore'), Ghost = require('../../ghost'); describe("Ghost API", function () { + var testTemplatePath = 'core/test/ghost/fixtures/', + ghost; + + beforeEach(function () { + ghost = new Ghost(); + }); it("is a singleton", function () { var logStub = sinon.stub(console, "log"), @@ -16,8 +24,7 @@ describe("Ghost API", function () { }); it("uses init() to initialize", function (done) { - var ghost = new Ghost(), - fakeDataProvider = { + var fakeDataProvider = { init: function () { return when.resolve(); } @@ -43,8 +50,7 @@ describe("Ghost API", function () { }); it("can register filters with specific priority", function () { - var ghost = new Ghost(), - filterName = 'test', + var filterName = 'test', filterPriority = 9, testFilterHandler = sinon.spy(); @@ -57,8 +63,7 @@ describe("Ghost API", function () { }); it("can register filters with default priority", function () { - var ghost = new Ghost(), - filterName = 'test', + var filterName = 'test', defaultPriority = 5, testFilterHandler = sinon.spy(); @@ -71,8 +76,7 @@ describe("Ghost API", function () { }); it("executes filters in priority order", function (done) { - var ghost = new Ghost(), - filterName = 'testpriority', + var filterName = 'testpriority', testFilterHandler1 = sinon.spy(), testFilterHandler2 = sinon.spy(), testFilterHandler3 = sinon.spy(); @@ -91,4 +95,44 @@ describe("Ghost API", function () { done(); }); }); + + it("can compile a template", function (done) { + var template = path.join(process.cwd(), testTemplatePath, 'test.hbs'); + + should.exist(ghost.compileTemplate, 'Template Compiler exists'); + + ghost.compileTemplate(template).then(function (templateFn) { + should.exist(templateFn); + _.isFunction(templateFn).should.equal(true); + + templateFn().should.equal('

HelloWorld

'); + done(); + }, done); + }); + + it("loads templates for helpers", function (done) { + var compileSpy = sinon.spy(ghost, 'compileTemplate'); + + should.exist(ghost.loadTemplate, 'load template function exists'); + + // In order for the test to work, need to replace the path to the template + ghost.paths = sinon.stub().returns({ + frontendViews: path.join(process.cwd(), testTemplatePath) + }); + + ghost.loadTemplate('test').then(function (templateFn) { + // test that compileTemplate was called with the expected path + compileSpy.calledOnce.should.equal(true); + compileSpy.calledWith(path.join(process.cwd(), testTemplatePath, 'test.hbs')).should.equal(true); + + should.exist(templateFn); + _.isFunction(templateFn).should.equal(true); + + templateFn().should.equal('

HelloWorld

'); + + compileSpy.restore(); + + done(); + }, done); + }); }); \ No newline at end of file