AM
#44 — это баг?
Если да, я бы попробовал пофиксить вечером...
Если да, я бы попробовал пофиксить вечером...
Проверил. Да, баг.
Size: a a a
2020 September 04
СП
ок, попробую исправить
AM
ок, попробую исправить
Кстати, что думаешь на тему аргументов сейчас? Логичные? Все нужны?
СП
В toarray?
AM
СП
Логично. $recursive - не знаю, нужен ли на пратике?
T
please remember that I use ArrayHelper in db, active-record :)
AM
Логично. $recursive - не знаю, нужен ли на пратике?
Не очень нужно его отключать. Разве что если не надо от юзеру давать вглубь уходить, но это не задача array helper.
СП
Я пофиксил баг, добавил в PR45. Но ArrayableTrait явно требует рефакторинга.
Очень не нравится, что в пакете есть две функции с разным поведением, но одним названием:
Может быть ту, что в ArrayHelper переименовать в какой-нибудь маппер. И добавить ещё функцию как раз с названием
Очень не нравится, что в пакете есть две функции с разным поведением, но одним названием:
ArrayableTrait::toArray()ArrayHelper::toArray()Может быть ту, что в ArrayHelper переименовать в какой-нибудь маппер. И добавить ещё функцию как раз с названием
toArray, которая будет работать по логике из ArrayableTrait?T
yes type-coverage psalm
AM
Я пофиксил баг, добавил в PR45. Но ArrayableTrait явно требует рефакторинга.
Очень не нравится, что в пакете есть две функции с разным поведением, но одним названием:
Может быть ту, что в ArrayHelper переименовать в какой-нибудь маппер. И добавить ещё функцию как раз с названием
Очень не нравится, что в пакете есть две функции с разным поведением, но одним названием:
ArrayableTrait::toArray()ArrayHelper::toArray()Может быть ту, что в ArrayHelper переименовать в какой-нибудь маппер. И добавить ещё функцию как раз с названием
toArray, которая будет работать по логике из ArrayableTrait?Так это две разные штуки.
AM
TradersVE
yes type-coverage psalm
Why removing Mutation testing?
AM
Ah, I see it. Moved.
T
mutation test is in all implementations individually
СП
Так это две разные штуки.
Я понимаю, но называются одинаково.
Логично подумать, что если
ArrayableTrait::toArray() - подразумевается, что объект можно преобразовать в массив.Логично подумать, что если
ArrayHelper::toArray() принимает объект, то она работает также. Но на деле нет. Поэтому и предлагаю переименовать.AM
TradersVE
mutation test is in all implementations individually
✅
AM
Я понимаю, но называются одинаково.
Логично подумать, что если
ArrayableTrait::toArray() - подразумевается, что объект можно преобразовать в массив.Логично подумать, что если
ArrayHelper::toArray() принимает объект, то она работает также. Но на деле нет. Поэтому и предлагаю переименовать.Во что?