Merge pull request #76 from nokazn/fix/multiple-include

Fix: multiple `<include>` tag with global locals evaluate locals uncorrectry

Author: Scrum

lib/index.js

@@ -20,7 +20,7 @@ module.exports = (options = {}) => {
       let content;
       let subtree;
       let source;
-      const posthtmlExpressionsOptions = options.posthtmlExpressionsOptions || {locals: false};
+      let posthtmlExpressionsOptions = options.posthtmlExpressionsOptions || {locals: false};
       if (options.delimiters) {
         posthtmlExpressionsOptions.delimiters = options.delimiters;
       }
@@ -32,7 +32,10 @@ module.exports = (options = {}) => {
         try {
           const localsRaw = node.attrs.locals || (node.content ? node.content.join().replace(/\n/g, '') : false);
           const localsJson = JSON.parse(localsRaw);
-          posthtmlExpressionsOptions.locals = posthtmlExpressionsOptions.locals ? Object.assign(localsJson, posthtmlExpressionsOptions.locals) : localsJson;
+          posthtmlExpressionsOptions = {
+            ...posthtmlExpressionsOptions,
+            locals: posthtmlExpressionsOptions.locals ? Object.assign(localsJson, posthtmlExpressionsOptions.locals) : localsJson
+          };
         } catch {}
 
         if (posthtmlExpressionsOptions.locals) {

test/expected/multiple-include.html

@@ -0,0 +1,14 @@
+<div>local1</div>
+<div>global</div>
+
+<div>local2</div>
+<div>global</div>
+
+<div>local3</div>
+<div>global</div>
+
+<div>local4</div>
+<div>global</div>
+
+<div>local5</div>
+<div>global</div>

test/fixtures/multiple-include.html

@@ -0,0 +1,5 @@
+<include src="./test/fixtures/includes/6.html">{ "localVariable": "local1" }</include>
+<include src="./test/fixtures/includes/6.html">{ "localVariable": "local2" }</include>
+<include src="./test/fixtures/includes/6.html">{ "localVariable": "local3" }</include>
+<include src="./test/fixtures/includes/6.html">{ "localVariable": "local4" }</include>
+<include src="./test/fixtures/includes/6.html">{ "localVariable": "local5" }</include>

test/test.js

@@ -1,5 +1,5 @@
 const test = require('ava')
-const plugin = require('../lib')
+const plugin = require('../lib/index.js')
 const posthtml = require('posthtml')
 
 const path = require('path')
@@ -60,6 +60,13 @@ test('Should merge global locals', t => {
     }})
 })
 
+test('Should merge global variables and evaluate each local variable', t => {
+  return process(t, 'multiple-include', {
+    posthtmlExpressionsOptions: {
+      locals: {globalVariable: 'global'}
+    }})
+})
+
 test('addDependency message', t => {
   const includePath = require('path').resolve('./test/fixtures/blocks/button/button.html')