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

Даров. @Helldar мб хоть ты пр в Лару закинешь? Я сегодня писал

00:52пожаловаться #1

MP

Переслано от Maksim (Ellrion) Pla...

господа, есть желающие внести в свое партфолио хотфикс в лару?)

00:52пожаловаться #2

MP

Contribute to laravel/framework development by creating an account on GitHub.

Переслано от Maksim (Ellrion) Pla...

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

вот тут:
https://github.com/laravel/framework/blob/7.x/src/Illuminate/Database/Eloquent/Relations/MorphTo.php#L199

выстaвляется $model->getKey() а должно то которое я в связи указал сначало провериться, как в предке этого же класса

https://github.com/laravel/framework/blob/7.x/src/Illuminate/Database/Eloquent/Relations/BelongsTo.php#L204

тоесть $model->getAttribute($this->ownerKey)

GitHub

laravel/framework

Алихан Закиров... in Laravel для начинающих

00:52пожаловаться #3

АЗ

Contribute to laravel/framework development by creating an account on GitHub.

Переслано от Maksim (Ellrion) Platonov

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

вот тут:
https://github.com/laravel/framework/blob/7.x/src/Illuminate/Database/Eloquent/Relations/MorphTo.php#L199

выстaвляется $model->getKey() а должно то которое я в связи указал сначало провериться, как в предке этого же класса

https://github.com/laravel/framework/blob/7.x/src/Illuminate/Database/Eloquent/Relations/BelongsTo.php#L204

тоесть $model->getAttribute($this->ownerKey)

GitHub

laravel/framework

Очень интересно
Но так впадлу

00:53пожаловаться #4

MP

Да вот довелось в Лару вернуться и сразу багу поймал)

Алихан Закиров... in Laravel для начинающих

00:54пожаловаться #5

АЗ

Я сегодня затарился пивом на случай изоляции, и решил изолироваться, а параллельно и продезинфицироваться уже сегодня

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

00:54пожаловаться #6

AH

Даров. @Helldar мб хоть ты пр в Лару закинешь? Я сегодня писал

Звучит интересно, только после моего срача с Тейлором по поводу кастомных кастов и того, что он своровал мою идею, всё меньше хочется вкладывать в проект :(

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

00:55пожаловаться #7

AH

Да вот довелось в Лару вернуться и сразу багу поймал)

Случается)
Периодически тоже баги да недоработки ловлю по паре штук за месяц точно.

00:56пожаловаться #8

MP

Andrey Helldar

Звучит интересно, только после моего срача с Тейлором по поводу кастомных кастов и того, что он своровал мою идею, всё меньше хочется вкладывать в проект :(

Ты кстати пакет не сделал такой? А то они в ларе только с седьмой а мне в 6 надо

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

00:57пожаловаться #9

AH

Ты кстати пакет не сделал такой? А то они в ларе только с седьмой а мне в 6 надо

С кастами? Не, не делал. У спати, вроде, было нечто похожее но с дикими фриками под капотом.

00:58пожаловаться #10

MP