metarhia/metavm

Remove null scenario and return empty file Error instead

Closed this issue · 0 comments

Тут читается src из файла и если !src возвращается null. Это срабатывает только на пустом файле.

if (!src) return null;

Тут в impress читаются изменения файлов в lib и domain и в логи прилетает ошибка
https://github.com/metarhia/impress/blob/fd1d8b320838ae86e020b0010f145e109314ffdf/lib/modules.js#L69

00:09:44  W1   error   TypeError: Cannot read property 'exports' of null
  Modules.change (/node_modules/impress/lib/modules.js:65:28)

Пустой файл подразумевается как ошибка, поэтому на мой взгляд логично выдавать на этот случай ошибку, что файл пуст, а не ошибку, которая отсылает к библиотекам метрахии. Такое возникает только при работе с прикладным кодом.
При работе с api (interfaces в импрессе) ошибка возникает только если добавить пустой файл в папку api. Если положить в папку внутри папки api ошибка не выводится. Видимо загрузка вложенных папок и файлов в них в api отличается от domain и lib.