mjml-parser-xml
Advanced tools
Comparing version 4.0.0-alpha.4 to 4.0.0-beta.1
@@ -92,2 +92,3 @@ 'use strict'; | ||
var options = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {}; | ||
var includedIn = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : []; | ||
var _options$addEmptyAttr = options.addEmptyAttributes, | ||
@@ -120,3 +121,3 @@ addEmptyAttributes = _options$addEmptyAttr === undefined ? true : _options$addEmptyAttr, | ||
var cur = null; | ||
var inInclude = false; | ||
var inInclude = !!includedIn.length; | ||
@@ -127,4 +128,8 @@ var findTag = function findTag(tagName, tree) { | ||
var lineIndexes = indexesForNewLine(safeXml); | ||
var handleInclude = function handleInclude(file, line) { | ||
var partialPath = _path2.default.resolve(cwd, file); | ||
if ((0, _find2.default)(cur.includedIn, { file: partialPath })) throw new Error('Circular inclusion detected on file : ' + partialPath); | ||
var content = void 0; | ||
@@ -155,3 +160,7 @@ | ||
filePath: partialPath | ||
})); | ||
}), [].concat((0, _toConsumableArray3.default)(cur.includedIn), [{ | ||
file: cur.absoluteFilePath, | ||
line: line | ||
}])); | ||
var bindToTree = function bindToTree(children) { | ||
@@ -203,3 +212,2 @@ var tree = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : cur; | ||
inInclude = true; | ||
return handleInclude(decodeURIComponent(attrs.path), line); | ||
@@ -221,2 +229,3 @@ } | ||
line: line, | ||
includedIn: includedIn, | ||
parent: cur, | ||
@@ -239,3 +248,2 @@ tagName: name, | ||
inInclude = false; | ||
return; | ||
} | ||
@@ -242,0 +250,0 @@ |
{ | ||
"name": "mjml-parser-xml", | ||
"description": "mjml-parser-xml", | ||
"version": "4.0.0-alpha.4", | ||
"version": "4.0.0-beta.1", | ||
"main": "lib/index.js", | ||
@@ -6,0 +6,0 @@ "repository": { |
License Policy Violation
LicenseThis package is not allowed per your license policy. Review the package's license to ensure compliance.
Found 1 instance in 1 package
License Policy Violation
LicenseThis package is not allowed per your license policy. Review the package's license to ensure compliance.
Found 1 instance in 1 package
New author
Supply chain riskA new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.
Found 1 instance in 1 package
16108
322
0