Телеграмм чат группы laravel

В моем ПР Тейлор реальный кейс привел, который мой код не покрывал, но сегодня я доделал-таки его и код охеренно работает!
Но... Тут же сказали что он уже слил своё старое нерабочее дерьмо.

Я сунул в его код банальное создание записи в базу (https://github.com/laravel/framework/pull/31035#issuecomment-572181089) и тут же получил ошибку

Illuminate\Database\QueryException : SQLSTATE[HY000]: General error: 1 table test_model has no column named line_one (SQL: insert into "test_model" ("field_1", "field_2", "line_one", "line_two") values (foobar, 20, Address Line 1, Address Line 2))

GitHub

Custom Cast Types + Object / Value Object Casts by taylorotwell · Pull Request #31035 · laravel/framework

Description to come.

источник

21:05пожаловаться #12

ИЛ

Иван Лещенко in Laravel для начинающих

Andrey Helldar

@Adelf32, @ellrion
Да е**л я что-то новое больше предлагать: https://github.com/laravel/framework/pull/30958#issuecomment-572126512

GitHub

[6.x] Added make console command of custom Cast by andrey-helldar · Pull Request #30958 · laravel/framework

Recently, my #30931 PR was rejected and I understand the reason, since my proposal implies an increase in the amount of processed code. This is not entirely good.
In exchange, I suggest introducing...

Грустно, но всё же в защиту Тейлора тоже стоит сказать. Тут как минимум есть кэш кастов и нормальный каст примитивных типов

источник

21:05пожаловаться #13

A

Adel in Laravel для начинающих

с кастами проблема конечно была с производительностью...

источник

21:05пожаловаться #14

AH

Andrey Helldar in Laravel для начинающих

Adel

Тейлор конечно красавец))) взял, криво переделал то, что челы обговаривали неделю а ты все это делал, и за свою разработку выдал)))

Всмысле?
Я начал делать до того как он обсуждать начал потому как на работе реальный кейс по этой фиче выпал, а на нескольких проектах пакет внедрять ну его на фиг.
После Тейлор закрыл мой ПР и спустя некоторое время создал твит. Я видел что он его создал и забил на него. Предпочел свой код дописать. Дописал. Но...

источник

21:07пожаловаться #15

A

Adel in Laravel для начинающих

Andrey Helldar

Всмысле?
Я начал делать до того как он обсуждать начал потому как на работе реальный кейс по этой фиче выпал, а на нескольких проектах пакет внедрять ну его на фиг.
После Тейлор закрыл мой ПР и спустя некоторое время создал твит. Я видел что он его создал и забил на него. Предпочел свой код дописать. Дописал. Но...

ты точно понял, что я написал? :)

источник

21:07пожаловаться #16

AH

Andrey Helldar in Laravel для начинающих

Иван Лещенко

Грустно, но всё же в защиту Тейлора тоже стоит сказать. Тут как минимум есть кэш кастов и нормальный каст примитивных типов

Дык и у меня есть кэш кастов...

источник

21:07пожаловаться #17

AH

Andrey Helldar in Laravel для начинающих