Skip to content
This repository

Дополнение к https://github.com/azproduction/lmd/issues/23 #28

Closed
myadzel opened this Issue June 18, 2012 · 1 comment

2 participants

Alexander Myadzel Mikhail Davydov
Alexander Myadzel

Если кто-то по чистой случайности (да и не только, вдруг где-то джоинится значение в рантайме) напишет в пути двойной разделитель, что в принципе будет корректно при общении с FS, то в данном случае получит в массиве пустые значения.

Лучше написать немного по-другому, да и вообще, вынести в "константу" --

var RE_DIRECTORY_SEPARATOR = /[\/]+/;

Например, в случае со значением пути в виде "path//to/config" будет все ок.

Mikhail Davydov
Owner

Спасибо за дополнение, я заметил твой коммент к коммиту - скоро будет большой релиз: code coverage+module stats+coverage/stats analiser+асинхронная загрузка плоских модулей+куча рефакторинга. В него войдет эта константа.

Mikhail Davydov azproduction closed this in b8018b2 June 22, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.