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

Feature 19 #65

Merged
merged 11 commits into from
Apr 3, 2016
Merged

Feature 19 #65

merged 11 commits into from
Apr 3, 2016

Conversation

artesby
Copy link
Contributor

@artesby artesby commented Mar 25, 2016

Изменения по #19 задаче

@artesby
Copy link
Contributor Author

artesby commented Mar 25, 2016

я хотел сделать свой сервис провайдер, точнее я сделал его, и добавил в autoload, но почему-то он у меня не работал (даже в boot не заходил)
в апп провайдере все заработало

@malinink
Copy link
Owner

@artesby
Не используйте глобальный неймспейс, всегда классы прописывайте в use в шапке файла, так проще отследить зависимости. (тут и ниже)

В проверках соответственно если решите - используйте константы.
Мне также кажется что есть проблема в функции compose
надо как минимум проверять что пользователь есть, либо же, каким-то образом связать его с web middleware (тогда пользователь всегда точно будет)

@malinink
Copy link
Owner

@artesby

\Auth
\App\User

Уберите в use.

Используйте константы!

@artesby
Copy link
Contributor Author

artesby commented Mar 28, 2016

Да да, еще надо придумать что-то с авторизацией
у меня пока 2 мысли только: будет работать __construct дополненный auth как в контроллере либо для залогиненых юзеров отдельный layout делать

@malinink
Copy link
Owner

@artesby
Пока что нет необходимости делать отдельный layout на мой взгляд.
Но если вопрос его подключения Вам интересен, то почему би и не разобраться?

@artesby
Copy link
Contributor Author

artesby commented Apr 1, 2016

@malinink
что если просто проверку добавить такую
и правильно ли константы в шаблон вынес?

@malinink
Copy link
Owner

malinink commented Apr 2, 2016

@artesby
Попробуйте во View использовать напрямую константы, не передавая их из сompose. Должно работать (ну с префиксом класса конечно), поглядиет возможно есть другие, более элегантные решения.
Добавьте во все модицицированные файлы правильные хедеры.

@artesby
Copy link
Contributor Author

artesby commented Apr 2, 2016

@malinink
done

@malinink malinink merged commit b108e64 into malinink:dev Apr 3, 2016
@artesby artesby deleted the feature-19 branch April 19, 2016 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants