Refactoring - Githubissues

enb / enb-bem-i18n

BEM internationalization for ENB

Other

8 stars 7 forks source link

Refactoring #65

Closed blond closed 9 years ago

blond commented 9 years ago

Changed compile API
Moved tanker to lib dir
Renamed i18n-js tech to i18n
Added read function to keysets module

blond commented 9 years ago

/сс @tadatuta @tormozz48 @SwinX

SwinX commented 9 years ago

_readKeysetsFile по 3м файлам размазана - может её в 1 место куда-то можно положить?

blond commented 9 years ago

_readKeysetsFile по 3м файлам размазана - может её в 1 место куда-то можно положить?

Я не вижу способа, кроме как сделать абстрактный класс с таким методом и наследоваться от него. Но это больше похоже на хак, который усложнит логику.

SwinX commented 9 years ago

Я не вижу способа, кроме как сделать абстрактный класс с таким методом и наследоваться от него. Но это больше похоже на хак, который усложнит логику.

Сходу подумал на то, что можно логику унести в отдельный хелпер, но он получается как функция с 4мя параметрами, что тоже не очень красиво выглядит. Наверное действительно лучше не трогать.