-
Notifications
You must be signed in to change notification settings - Fork 3
We should not use bem-naming just for id getter #122
base: master
Are you sure you want to change the base?
Conversation
нужно тесты обновить, а по смыслу — like |
Кажется, что проще делать безусловный join через палки или какие-то символы, чтобы еще и разбирать можно было. Типа id: e => [
e.block,
e.elem || '',
e.mod ? e.mod.name : '',
(e.mod || e.mod.val !== true) && e.mod.val || ''
].join('−'),
parseId: s => {
const x = s.split(/−/g);
return new BemEntityName({
block: x[0],
elem: x[1] || null,
mod: x[2] && { name: x[2], val: x[3] || true }
});
}, |
я не очень понимаю зачем тебе parseId |
99a4e7f
to
c7646b8
Compare
Нравится мне иметь parseid как обратную операцию к id. |
Тесты поправил. Вернул старое поведение /ping |
@@ -1,6 +1,4 @@ | |||
import test from 'ava'; | |||
import sinon from 'sinon'; | |||
import proxyquire from 'proxyquire'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
fix: stop using bem-naming for generating id
cc @blond @tadatuta