A few bug fixes

Ugh, already up to 0.2.3 :'( Fix config parsing Fixes #26
Polymer · Apr 1, 2014 · e86d845 · e86d845
1 parent f21125e
commit e86d845
Show file tree

Hide file tree

Showing 7 changed files with 35 additions and 15 deletions.
diff --git a/lib/optparser.js b/lib/optparser.js
@@ -12,7 +12,7 @@ var DEFAULT = 'vulcanized.html';
 
 // validate options with boolean return
 function processOptions(options, callback) {
-  excludes = {
+  var excludes = {
     imports: [ABS_URL],
     scripts: [ABS_URL],
     styles: [ABS_URL]
@@ -37,7 +37,7 @@ function processOptions(options, callback) {
       return callback('Malformed config JSON!');
     }
     if (config.excludes) {
-      var e = options.excludes;
+      var e = config.excludes;
       try {
         if (e.imports) {
           e.imports.forEach(function(r) {
@@ -59,6 +59,7 @@ function processOptions(options, callback) {
       }
     }
   }
+  options.excludes = excludes;
 
   if (!options.output) {
     options.output = path.resolve(path.dirname(options.input), DEFAULT);

diff --git a/lib/pathresolver.js b/lib/pathresolver.js
@@ -6,12 +6,7 @@
 
 var path = require('path');
 var constants = require('./constants.js');
-
-// directly update the textnode child of <style>
-// equivalent to <style>.textContent
-function setTextContent(node, text) {
-  node[0].children[0].data = text;
-}
+var setTextContent = require('./utils.js').setTextContent;
 
 function resolvePaths($, input, output) {
   var assetPath = path.relative(output, input);

diff --git a/lib/utils.js b/lib/utils.js
@@ -0,0 +1,24 @@
+/*
+ * Copyright 2013 The Polymer Authors. All rights reserved.
+ * Use of this source code is governed by a BSD-style
+ * license that can be found in the LICENSE file.
+ */
+
+module.exports = {
+  // directly update the textnode child of <style>
+  // equivalent to <style>.textContent
+  setTextContent: function(node, text) {
+    var child = node[0].children[0];
+    if (child) {
+      child.data = text;
+    } else {
+      node[0].children[0] = {
+        data: text,
+        type: 'text',
+        next: null,
+        prev: null,
+        parent: node[0]
+      };
+    }
+  }
+};
diff --git a/lib/vulcan.js b/lib/vulcan.js
@@ -15,6 +15,7 @@ var url = require('url');
 var constants = require('./constants.js');
 var optparser = require('./optparser.js');
 var pathresolver = require('./pathresolver');
+var utils = require('./utils');
 
 var read = {};
 var options = {};
@@ -37,21 +38,19 @@ function exclude(regexes, href) {
 }
 
 function excludeImport(href) {
-  return exclude(excludes.imports, href);
+  return exclude(options.excludes.imports, href);
 }
 
 function excludeScript(href) {
-  return exclude(excludes.scripts, href);
+  return exclude(options.excludes.scripts, href);
 }
 
 function excludeStyle(href) {
-  return exclude(excludes.styles, href);
+  return exclude(options.excludes.styles, href);
 }
 
-// directly update the textnode child of <style>
-// equivalent to <style>.textContent
 function setTextContent(node, text) {
-  node[0].children[0].data = text;
+  utils.setTextContent(node, text);
 }
 
 function readFile(file) {

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "vulcanize",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "Process Web Components into one output file",
   "main": "lib/vulcan.js",
   "bin": {

diff --git a/test/empty.css b/test/empty.css
diff --git a/test/index.html b/test/index.html
@@ -6,6 +6,7 @@
   <script src="bower_components/platform/platform.js"></script>
   <link rel="import" href="absolutes.html">
   <link rel="import" href="import-test.html" />
+  <link rel="stylesheet" href="empty.css">
 </head>
 <body>
   <x-import>Hello Import!</x-import>