azproduction / lmd

LMD - JavaScript Module-Assembler for building better web applications :warning: Project is no longer supported :warning:
http://azproduction.ru/lmd/
MIT License
449 stars 27 forks source link

слетела вёрстка нумерации строк в аналитике #179

Closed Kuznetsov-Ilia closed 10 years ago

Kuznetsov-Ilia commented 10 years ago

html bug зума нет, венда 7, хром последний, кастомных тем нет.

Kuznetsov-Ilia commented 10 years ago

Куда кстати рыть если возникают вот такие вкусняшки при хождении по модулям? TypeError: Cannot read property 'name' of undefined at __jade.unshift.lineno (eval at (/usr/local/lib/node_modules/lmd/node_modules/jade/lib/jade.js:160:8), :1340:18) at eval (eval at (/usr/local/lib/node_modules/lmd/node_modules/jade/lib/jade.js:160:8), :1521:4) at Object. (/usr/local/lib/node_modules/lmd/node_modules/jade/lib/jade.js:165:12) at ServerResponse.res._render (/usr/local/lib/node_modules/lmd/node_modules/express/lib/view.js:425:21) at ServerResponse.res.render (/usr/local/lib/node_modules/lmd/node_modules/express/lib/view.js:318:17) at /usr/local/lib/node_modules/lmd/stats_server/lib/admin.js:98:21 at /usr/local/lib/node_modules/lmd/stats_server/lib/admin.js:139:15 at fs.js:253:14 at Object.oncomplete (fs.js:94:15)

Менял www_root несколько раз причём основную массу модулей грузит успешно, буквально 5-7 модулей фейлятся с такой ошибкой...

azproduction commented 10 years ago

Привет. Это достаточно экспериментальный проект для lmd, который я не трогал год точно.

Верстку поправлю, можешь подробнее описать причину возникновения ошибки? Мб какой-то особый модуль?

Kuznetsov-Ilia commented 10 years ago

Привет! Охрененный модуль!) Видимо он не выполнил задачи которые ты перед ним ставил? с чем связано завершение его разработки? да в том то и дело что миксины где матчатся модули находятся глубже и он их хавает нормально root: '../src/components', modules: { '<%= dir[0] %>': '/view.js', '<%= dir[0] %>-model': '/model.js' }, а вот основные модули типа root: '../src' '<%= file %>': '*.js', хавать не хочет в придачу ещё 3 модуля есть которые фейлятся

все зафейленные модули обладают только 1-й характерной чертой - они не реквайрятся явно...

azproduction commented 10 years ago

По идее его нужно оторвать от LMD и продавать отдельно. Не развивается потому, что я его не использую, или использую другие инструменты вместо него.

они не реквайрятся явно

require(a + 'b.js') ?

Kuznetsov-Ilia commented 10 years ago

типа того, динамические имена(( я пока сам не понял что конкретно можно извлечь из статистики, но есть подозрение что оно может быть очень полезно при понимании того что в какой мере используется и что в какую сторону рефакторить. график с зависимостями require\require by вооще крутотень - очень быстро помогает понять что откуда растёт и выглядит нагляднее чев вывод в консоли от lmd info

azproduction commented 10 years ago

я пока сам не понял что конкретно можно извлечь из статистики

Я так же не понял и перестал использовать :) оно помогает понять что от чего зависит, а когда ты в проекте хорошо разбираешься то необходимость отпадает.

azproduction commented 10 years ago

Я так и не смог воспроизвести. Сделал проверку в потенциально проблемном месте. Попробуй поставить из git и проверить: npm i git://github.com/azproduction/lmd.git#f8395ed6baaa394fb1be236730af1ac227cf9677

Kuznetsov-Ilia commented 10 years ago

один модуль который фейлился вроде бы стал доступен остальные - нет TypeError: /Users/ikuznecov/projects/node_modules/lmd/stats_server/lib/../views/file.jade:141 139| .b-notes__section 140| span.b-label.b-label_size_normal.b-label_type_condition Conditions

141| 142| span.js-conditions.b-notes__tags 143| !{conditions.join(', ')} 144|

Cannot read property 'name' of undefined at __jade.unshift.lineno (eval at (/Users/ikuznecov/projects/node_modules/lmd/node_modules/jade/lib/jade.js:160:8), :1339:18) at eval (eval at (/Users/ikuznecov/projects/node_modules/lmd/node_modules/jade/lib/jade.js:160:8), :1520:4) at Object. (/Users/ikuznecov/projects/node_modules/lmd/node_modules/jade/lib/jade.js:165:12) at ServerResponse.res._render (/Users/ikuznecov/projects/node_modules/lmd/node_modules/express/lib/view.js:425:21) at ServerResponse.res.render (/Users/ikuznecov/projects/node_modules/lmd/node_modules/express/lib/view.js:318:17) at /Users/ikuznecov/projects/node_modules/lmd/stats_server/lib/admin.js:98:21 at /Users/ikuznecov/projects/node_modules/lmd/stats_server/lib/admin.js:139:15 at fs.js:266:14 at Object.oncomplete (fs.js:107:15)

Kuznetsov-Ilia commented 10 years ago

хотя не, вру. всё тоже самое, только сообщение об ошибке стало иным в принципе если ты планируешь выпиливать этот инструмент из lmd, то наверно не стоит париться - 95% юзкейсов он покрывает. Require Graph зависит от собранной статистики или он из конфига всё делает?

azproduction commented 10 years ago

Require Graph зависит от собранной статистики. Те считает все модули, которые когда-либо рекваились (не важно попали они в сборку или загрузились через .async).

azproduction commented 10 years ago

fixed in lmd@1.11.7