В версии 0.4.0 стал неверно собираться клиентский bh с использованием bh-client-module

[1999]

В версии 0.4.0 поломалась сборка с использованием технологии bh-client-module и dependencies.
В версии 0.2.3 собранный файл выглядел примерно так:

modules.define('bh', ["module1"], function(provide, module1) {
  var BH = ...
  var bh = new BH();
    bh.setOptions({
        jsAttrName: 'data-bem',
        jsAttrScheme: 'json'
    });
    bh.lib.module1 = module1;

  bh.match(...);
  bh.match(...);
  bh.match(...);

  provide(bh);
});

В версии 0.4.0 так:

(function() {
  var BH = ...
  var bh = new BH();
    bh.setOptions({
        jsAttrName: 'data-bem',
        jsAttrScheme: 'json'
    });
    bh.lib.module1 = module1;

  bh.match(...);
  bh.match(...);
  bh.match(...);

  modules.define('bh', ["module1"], function(provide, module1) {
    provide(bh);
  });
})();

Это приводит к тому, что может быть ошибка module1 is not defined, поскольку он не загружен на момент выполнения кода bh.lib.module1 = module1. Этот пулреквест восстанавливает правильное поведение из 0.2.3

[blond]

Почему mimic зависит от bh, а не от реальных зависимостей?

[1999]

Ну, насколько я понял, mimic - это что-то, имитирующее bh (к слову зачем тогда делается modules.define('bh')?). И проще сразу объявить его зависимость от bh, разве нет?

[blond]

к слову зачем тогда делается modules.define('bh')

При имитации должна оставаться возможность использовать модуль как bh в том числе.

И проще сразу объявить его зависимость от bh, разве нет?

Если bh модуль в коде после modules.define будет использовать зависимости, то код в mimic сломается.

[1999]

Если bh модуль в коде после modules.define будет использовать зависимости, то код в mimic сломается.

Поясни? Вот у меня есть сейчас такое:

modules.define('bh', ['i18n'], function (provide, i18n) {
  bh.lib.i18n = i18n;
  provide(bh);
});

modules.define('mimic_bh', ['bh'], function (provide, bh) {
  provide(bh);
});

modules.require('mimic_bh', function (mimic) {
  console.log(mimic.lib.i18n); // ok
});

[arikon]

@andrewblond Андрей, а тестов на эту технологию нет? Мы со временем тоже хотим перейти на bh-шаблоны на клиенте. Не хочется, чтобы оно внезапно взрывалось ;)

[blond]

@arikon, тестов в enb-bh пока нету совсем =(

[blond]

@1999, кажется, что правильнее откатиться к варианту из v0.2.3 и просто добавлять в конце:

modules.define('mimic_bh', ['bh'], function (provide, bh) {
  provide(bh);
});

[1999]

Так ведь в этом и фикс. Сейчас так собирается.

[blond]

@1999, сейчас это оборачивается в анонимную функцию, а можно весь код bh вставлять в modules.define('mimic_bh', ...` как это делалось раньше.

[1999]

@andrewblond поправил

[blond]

@1999, спасибо большое! ;)

[blond]

Изменения вошли в v0.4.1.