Merge bf83ecf into 298a0aa

lib/renderer.js

@@ -1,17 +1,70 @@
 'use strict';
 
 
-const handlebars = require('handlebars');
+const Handlebars = require('handlebars');
+const path = require('path');
+const fs = require('fs');
 
-// const fs = require('fs');
+let isPartialsRegistered = false;
+const PARTIAL_EXTNAME = '.hbs';
+
+function render(data, locals) {
+  return doRenderInternal(data, locals);
+}
 
 function doRenderInternal(data, locals) {
-  let template = handlebars.compile(data.text);
+  const template = Handlebars.compile(data.text);
+  // Actually, data.path always be effective in Hexo
+  if (!isPartialsRegistered && data.path) {
+    const basedir = data.rootPath || path.dirname(data.path);
+    registerPartials(basedir);
+  }
+  if (data.path) {
+    isPartialsRegistered = true;
+  }
   return template(Object.assign({filename: data.path}, locals));
 }
 
-function render(data, locals) {
-  return doRenderInternal(data, locals);
+function registerPartials(basedir) {
+  const files = fs.readdirSync(basedir);
+  const partialDirs = [];
+  files.forEach(file => {
+    const filePath = path.join(basedir, file);
+    const stat = fs.statSync(filePath);
+    // partial dir must start with '_'
+    if (stat.isDirectory() && file.startsWith('_')) {
+      partialDirs.push(filePath);
+    }
+  });
+  const partialFiles = [];
+  partialDirs.forEach(d => {
+    partialFiles.push(...listPartialFiles(d));
+  });
+  partialFiles.forEach(file => {
+    const start = basedir.length + 1;
+    const length = file.length - start - PARTIAL_EXTNAME.length;
+    const name = file.substr(start, length);
+    const data = fs.readFileSync(file, {encoding: 'utf-8'});
+    Handlebars.registerPartial(name, data);
+  });
 }
 
+
+function listPartialFiles(dir, list = []) {
+  const files = fs.readdirSync(dir);
+  files.forEach(file => {
+    const filePath = path.join(dir, file);
+    if (fs.statSync(filePath).isFile()){
+      // partial file must have '.hbs' extname.
+      if (path.extname(filePath) === PARTIAL_EXTNAME) {
+        list.push(filePath);
+      }
+    } else {
+      listPartialFiles(filePath, list);
+    }
+  });
+  return list;
+}
+
+
 module.exports = render;

package.json

@@ -1,6 +1,6 @@
 {
-  "name": "hexo-renderer-handlebars13",
-  "version": "0.0.1",
+  "name": "hexo-renderer-hbars",
+  "version": "0.0.3",
   "description": "Handlebars renderer plugin for hexo",
   "main": "index.js",
   "scripts": {

test/_partial/the-partial.hbs

@@ -0,0 +1 @@
+handlebars

test/index.js

@@ -13,7 +13,6 @@ describe('Handlebars renderer tests', function() {
 
   it('escaped variable test', function() {
     const source = 'Hello, {{name}}!';
-
     const result = render({text: source}, {
       name: 'handlebars'
     });
@@ -123,11 +122,12 @@ describe('Handlebars renderer tests', function() {
   //   }).should.eql('Hello world!');
   // });
   //
-  // it('partials test', function () {
-  //   var body = [
-  //     'Hello {{> _partial/test-partial }}!'
-  //   ].join('\n')
-  //
-  //   r({text: body, path: 'test/test.mustache'}).should.eql('Hello world!')
-  // })
+  it('partials test', function() {
+    const source = 'Hello, {{> _partial/the-partial }}!';
+    // const source = 'Hello, {{name}}!';
+    const result = render({text: source, path: 'test/main', rootPath: 'test'}, {
+      name: 'handlebars'
+    });
+    assert.equal(result, 'Hello, handlebars!');
+  });
 });

test/main.hbs

@@ -0,0 +1 @@
+Hello, {{> 'test/the-partial.hbs' }}!