Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

app.set('view options', {layout: 'application_layout'}) #4

Closed
kratkar opened this issue Mar 3, 2011 · 9 comments
Closed

app.set('view options', {layout: 'application_layout'}) #4

kratkar opened this issue Mar 3, 2011 · 9 comments

Comments

@kratkar
Copy link

kratkar commented Mar 3, 2011

Надо добавить app.set('view options', {layout: 'application_layout'}) по умолчанию. иначе EBADF, Bad file descriptor '/var/www/testjs/app/views/layout.ejs'

@anatoliychakkaev
Copy link
Collaborator

В версии 0.0.5 я переместил лэйауты в app/views/layouts (чтоб как в рельсах). Сейчас контроллер опеределяет откуда брать лэйаут, пока что либо берется application_layout, либо то что указано в контроллере. Так что необходимости в этой строчке в app.js нет. Полностью "рельсовую" логику пока не реплицировал.

@kratkar
Copy link
Author

kratkar commented Mar 3, 2011

Я вроде недавно скачал с github у меня как раз выдаёт ошибку EBADF, Bad file descriptor '/var/www/testjs/app/views/layout.ejs'

@anatoliychakkaev
Copy link
Collaborator

Понятно, а как выглядит вызов в контроллере? Сейчас идет тотальный rewrite, кое-что переписывается на новый лад и перестает работать на старом коде. Скоро будет стабильный релиз 0.1.0 и уже с ним можно будет идти хоть в продакшн, хоть куда.

@kratkar
Copy link
Author

kratkar commented Mar 3, 2011

В контроллере render({text: 'index', title: 'index user'})
Да еще вопрос. Как правильнее в express-on-railway, перенести helper-s c /usr/local/lib/node/.npm/express-on-railway/active/package/lib/helpers.js во все views?

@anatoliychakkaev
Copy link
Collaborator

Не понял вопроса. Если хэлперы не доступны во view, тогда это баг, потому что в `lib/controller.js' line 164:

locals: safe_merge(params, path_to, require('./helpers')),

Так что хэлперы должны быть. Кстати, сейчас в ejs вышел фикс который я реквестировал давеча, так что хэлпер form_for будет переписан скоро. Первый параметр не надо будет больше передавать.

@anatoliychakkaev
Copy link
Collaborator

Собственно фикс уже в мастере.

@kratkar
Copy link
Author

kratkar commented Mar 3, 2011

Хэлперы работают. Моя ошибка)) Я для главной не определил роут - там get / expressa. А как для главной определить? map.get('/', 'user#index') не работает.

@anatoliychakkaev
Copy link
Collaborator

должно работать, если удалить из app.js тот роут который там прописан

@kratkar
Copy link
Author

kratkar commented Mar 3, 2011

Ага после удаления заработало. Надо бы при railway init закомментировать)) app.get("/"..

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants