Merge pull request #286 from /issues/285.requires.tadatuta

Support options.requires fallback for backward compatibility
enb · Mar 5, 2018 · 28103c3 · 28103c3
2 parents d1a0520 + 1552380
commit 28103c3
Show file tree

Hide file tree

Showing 11 changed files with 323 additions and 172 deletions.
diff --git a/api.ru.md b/api.ru.md
@@ -47,7 +47,9 @@
 
 #### requires
 
-Тип: `Object`. По умолчанию: `{}`.
+Устарело! Используйте `engineOptions.requires`.
+
+Тип: `Object`. По умолчанию: `undefined`.
 
 Задает имена или пути для подключения сторонних библиотек.
 
@@ -198,7 +200,9 @@ var BemhtmlTech = require('enb-bemxjst/techs/bemhtml'),
 
 #### requires
 
-Тип: `Object`. По умолчанию: `{}`.
+Устарело! Используйте `engineOptions.requires`.
+
+Тип: `Object`. По умолчанию: `undefined`.
 
 Задает имена или пути для подключения сторонних библиотек.
 

diff --git a/lib/bundle.js b/lib/bundle.js
@@ -11,13 +11,22 @@ var compileCommonJS = require('./compile-commonjs'),
  * @param {Object}   options                        Options.
  * @param {String}   opts.dirname                   Path to a directory with compiled file.
  * @param {String}   [options.exportName=BEMHTML]   Name for exports.
- * @param {Object}   [options.requires={}]          Names for dependencies.
+ * @param {Object}   [options.requires={}]          Deprecated. Fallback for backward
+ *                                                  compatibility, use options.engineOptions.requires
  * @returns {String}
  */
 exports.compile = function (code, options) {
     options || (options = {});
 
-    var requires = options.requires || {};
+    var requires;
+
+    if (options.requires) {
+        requires = options.requires;
+    } else if (options.engineOptions && options.engineOptions.requires) {
+        requires = options.engineOptions.requires;
+    } else {
+        requires = {};
+    }
 
     return compileCommonJS(requires, options.dirname)
         .then(function (commonJSModules) {