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

Контроллер для главной страницы #39

Closed
Ananaskelly opened this issue Mar 8, 2016 · 11 comments
Closed

Контроллер для главной страницы #39

Ananaskelly opened this issue Mar 8, 2016 · 11 comments

Comments

@Ananaskelly
Copy link
Contributor

Родительская задача #29
Исполнитель @

Описание
Создать контроллер для главной странички, где будет осуществляться поиск подходящей игры, или создание новой игры и редирект на view c игрой, если таковая найдена, создание необходимых моделей - Games, BordInfo, TurnInfo, UserGame (?)

Теги: controller

@malinink
Copy link
Owner

malinink commented Mar 9, 2016

Предлагаю функционал поиска игры вынести на отдельный метод контроллера.
А из index добавить ссылку на него, только назвать роуты надо правильно, кто будет делать - подумайте, не делайте роуты наобум. Или же перед реализацией давайте тут обсудим.

@malinink
Copy link
Owner

malinink commented Mar 9, 2016

@Ananaskelly
Да, надо также написать логику поиска игры(хотябы абстрактно), иначе такая задачу будет заведомо обречена на провал.

@Ananaskelly
Copy link
Contributor Author

но есть вопрос: у нас должен быть допустим метод index тут, который, например, будет возвращать view с выбором параметров игры и кнопочкой, чтобы начать этот поиск?

@malinink
Copy link
Owner

@Ananaskelly
Думаю что index метод должен возвращать отображение игры, и должен быть ещё какой-нибудь метод который будет возвращать уже непосредственно форму для поиска игры.

@Ananaskelly
Copy link
Contributor Author

Метод create для этих целей подойдет?
ещё у меня возникло два вопроса, когда я пыталась сделать view с формой
(не уверена, что их надо задавать тут, но)

  1. Была ошибка, что класс Form не определен, я нашла в документации к формам, что надо прописать aliases в config/app.php, это помогло. Это правильно?
  2. потом была какая-то странная штука - "maximum function nesting level of '100' reached. aborting". Я нашла такое решение , но как-то это странно

@malinink
Copy link
Owner

@Ananaskelly
Create всёже создавать, Search искать

  1. Мы аналогично делали в нашем первом проекте, еще и библиотеку надо поключить, смотрите тут и тут
  2. Эта ошибка означает что произошел рекурсивный вызов функций глубиной в 100, если таковое не задумывалось то где-то ошибка логики. Править явно некорректно, надо разбираться в чем именно проблема.

Если она останется пишите будем думать.

P.S. Поглядел что библиотеку то я подключил, а про фасады забыл, будете делать что-то с формами - добавьте.

@artesby
Copy link
Contributor

artesby commented Mar 24, 2016

@malinink
у меня тоже эта рекурсия появлялась вчера при попытке залогиниться\зарегистрироваться, после того как я рефрешил миграции, сидя на сайте из под своего аккаунта
фиксил точно так же
после откатов всех изменений и смены ветки на дев ошибка не пропадала
но потом как-то сама по себе проходила
и после каждого рефреша появлялась

@malinink
Copy link
Owner

@artesby @Ananaskelly
Очень странно, у меня ошибки нет, проверьте ещё раз на всякий случай на последней dev версии.

Кстати да, пришлось таки добавить в последнюю миграцию nullable на тип игры по умолчанию иначе пользователь то не создается.

@Ananaskelly
Copy link
Contributor Author

У меня проблема не исчезла, но она исчезает, если прописать рут вне Route::group()

@malinink
Copy link
Owner

@Ananaskelly
Так он и так вне или Вы о чем?

@Ananaskelly
Copy link
Contributor Author

проблема была с новым рутом, который я прописала, чтобы проверить контроллер и вью с формой для игры.

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

No branches or pull requests

3 participants