Skip to content
This repository has been archived by the owner on Aug 1, 2019. It is now read-only.

Bug in interpreting nested scheme #21

Open
qfox opened this issue May 18, 2017 · 0 comments
Open

Bug in interpreting nested scheme #21

qfox opened this issue May 18, 2017 · 0 comments
Labels

Comments

@qfox
Copy link

qfox commented May 18, 2017

Atm layer used as an absolute path, it's incorrect.

Layer is an abstract identifier and does not know anything about paths on fs.
In the result fs-scheme can return just relative path to level directory.
So these lines https://github.com/bem-sdk/bem-fs-scheme/blob/master/lib/schemes/nested.js#L50-L52 should looks like:

var folder = path.join(layer + '.blocks', entity.block,
    entity.elem ? (elemDelim + entity.elem) : '',
    entity.mod ? (modDelim + entity.mod.name) : '');

Because layer should contains just 'common' or 'desktop' strings.

p.s. Guess there is the same issue in flat scheme.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant