Как стать автором
Обновить

Комментарии 13

php artisan migrate
а не
php artisan migration

php artisan make:model ModelName -mrc
и будет вам и миграции и контроллер
Роуты жуть, почему не ресурсы?
И в целом материал какой-то устаревший, например:
<li><a href="{{URL::route('create_album_form')}}">Создание нового альбома</a></li>
сейчас просто
<li><a href="{{route('create_album_form')}}">Создание нового альбома</a></li>
php artisan make:model ModelName -mrc

Даже так:
php artisan make:model ModelName -a


А почему Bootstrap 3? Уже давно 4.3

Судя по примерам кода автор не до конца знает фреймворк, пишет какой то самопал.
Статья о том, как не нужно писать на Laravel, да и вообще на PHP

В миграции стоило бы добавить nullable
$table->text('description'); // описание нашего альбома

Потому что при создании альбома поле description не обязательное по валидатору и этот код вызовет ошибку при пустом значении description
$album = Album::create(array(
'name' => $request->get('name'),
'description' => $request->get('description'),
'cover_image' => $filename,
));


Про CodeStyle, похоже, никто не слышал
Про CRUD через php artisan make:resource и соответствующие роуты написали выше. И еще куча всего

Статья пойдет как для ну очень новичка, ИМХО

UPD
Зачем во вьюхах @include('includes.header')?
Можно же наследовать,
@extends('site.layouts.main')
@section('content')
@endsection
Данная статья действительно рассчитана на новичков, все верно

Тогда стоит сразу учить хорошему

Для новичков больше вреда чем полезности. Навалено в кучу ИМХО автора.
Начиная работу с Ларавелем перерыл кучу подобных мануалов. Пока тим лид не обьяснил на пальцах структуру где, что лежит и за что отвечает тот или иной кусок кода — в голове была жуткая каша.
Чутка добавлю.

В первой статье рассказывалось про кодогенераторы, но почему-то здесь контроллер написан вручную. Я понимаю, что новичкам не обязательно знать про ресурсы, но банальные конвенции именования методов лучше соблюдать, раз уж Вы учите их работе с фреймворком. Потом будет проще переучиться.

Удаление обязательно делать методом POST(лучше, конечно, через method_field('DELETE'), но списываем на новичков) и вкидывать csrf-token. Зачем-то же он нужен…

Конструкцию array(...) не видел уже миллион лет :)

Переменная $uploadSuccess присваивается, но нигде не используется. Там вообще лучше обернуть в try...catch, но, опять же, новичкам хватит и простой проверки if ($uploadSuccess) {...}.

Это вообще огонь :)
$album = Album::with('Photos')->find($id);
$albums = Album::with('Photos')->get();

Ну и про code style уже говорили. Вообще не понимаю людей, жадных на пробелы…
Нарисовать сову очень просто!

Оно самое. Практика без теории новичков не научит ничему

Собираюсь к вам на курс, а тут такое… Я хотел-бы научиться так не делать.
Статья никак не связана с программой курса. Она написана автором не имеющим отношения к процессу обучения.
Благодарю всех за комментарии. Все замечания переданы автору, для того, чтобы в последующих публикациях такого не повторялось.
php  artisan  make:controller  AlbumsController -r

Что бы такого ужаса не было в именах функций
Зарегистрируйтесь на Хабре, чтобы оставить комментарий