feat(profile): add user chapters tree #40
Conversation
grozwalker
force-pushed
the
profile-chapters
branch
2 times, most recently
from
38184df
to
17fe733
Compare
|
|
||
| public function getIsReadAttribute() | ||
| { | ||
| return $this->user->where('id', Auth::user()->id)->count() > 0; |
There was a problem hiding this comment.
Вспоминаем MVC и отношение понятие аутентификации по отношению к модели.
| use App\User; | ||
|
|
||
| class UserController extends Controller | ||
| { | ||
| public function show(string $name) | ||
| { | ||
| $user = User::where('name', $name)->firstOrFail(); | ||
|
|
||
| $allChapters = Chapter::with('user')->get(); |
There was a problem hiding this comment.
Такие штуки должны доставаться через связи
There was a problem hiding this comment.
так а это не доставание, а просто загрузка данных для жадной загрузки
https://laravel.com/docs/6.0/eloquent-relationships#eager-loading
There was a problem hiding this comment.
А ок, но возникает вопрос, откуда у chapter может быть понятие "пользователь"? Главы сами по себе не привязаны к пользователям напрямую.
| @@ -0,0 +1,3 @@ | |||
| .chapters-tree { | |||
There was a problem hiding this comment.
Кастомных стилей не надо, в бутстрапе все это есть
tests/Unit/Model/ChapterTest.php
Outdated
| use App\User; | ||
| use Tests\TestCase; | ||
|
|
||
| class ChapterTest extends TestCase |
There was a problem hiding this comment.
Вот смотри, куча кода ради одного примитивного метода. Правда метод все равно написан неправильно.
grozwalker
force-pushed
the
profile-chapters
branch
from
17fe733
to
dc6877b
Compare
grozwalker
force-pushed
the
profile-chapters
branch
from
dc6877b
to
9c5bd92
Compare
На странице профиля добавил вывод всех глав, если глава прочитана пользователем, то чекбокс нажат