Size: a a a

2020 December 30

AN

Alexander Nazarov in symfony
Вы полностью правы когда говорите что коды могут как сто сложно генерится и являться логикой приложения, также как и изменение баланса. Но судя по обсуждению ваша логика вполне себе уживается в коде миграции. Это не поход в какой то сервис, это чистая функция. Вот допустим вам надо было бы в миграции, открытые пароли, заменить на md5 хеши. Вы бы замену паролей на md5 вынесли в отдельную команду?
источник

ВМ

Виктор Монастырев... in symfony
готовы ли вы к тому, что будет временной зазор между исполнение команды и изменением структуры кода проекта?
источник

KN

Konstantin Nosov in symfony
Виктор Монастырев
Вопрос:
Миграция без команды ( не пишет хэш )
Приложение Ваше сможет работать с пустым полем?
да, но для части пользователей работа будет некорректной (они получат неуникальный реф код, в котором будет null для всей группы старых пользователей)
источник

KN

Konstantin Nosov in symfony
Виктор Монастырев
готовы ли вы к тому, что будет временной зазор между исполнение команды и изменением структуры кода проекта?
да, это не есть проблемой, временная задержка это ок
источник

DS

Dima Sikorskiy in symfony
Konstantin Nosov
да, но для части пользователей работа будет некорректной (они получат неуникальный реф код, в котором будет null для всей группы старых пользователей)
это уже ошибка  проверки  кода.  это должен был предусмотреть разработчик.
источник

Kd

Konstantin dmz9 in symfony
Konstantin Nosov
да, это не есть проблемой, временная задержка это ок
ко времени когда захочется пооткатывать миграции - код системы может быть другой.
и метода user->doSomething или какой то связи может и не быть
источник

Kd

Konstantin dmz9 in symfony
отсюда запрет на использование любой изменяемой по времени хрени в миграциях
источник

ВМ

Виктор Монастырев... in symfony
Konstantin Nosov
да, это не есть проблемой, временная задержка это ок
ну если не кретично, задержка, то можно и командой, если же изменения кода не допускают задержки ( сразу нужно поле с хэшом ) то только в миграции
источник

AN

Alexander Nazarov in symfony
У них нет отката миграций. Только вперед
источник

Kd

Konstantin dmz9 in symfony
Alexander Nazarov
У них нет отката миграций. Только вперед
это неважно
источник

Kd

Konstantin dmz9 in symfony
твой код уйдет вперед, ты его спулишь.
начнешь делать миграции а потом в миграции будет не найден метод, который порефакторен промежуточно и забыт, ну или связь
источник

KN

Konstantin Nosov in symfony
Konstantin dmz9
это неважно
собственно мы выяснили что код миграций не должен зависеть от кода приложения, он должен быть изолирован
источник

AN

Alexander Nazarov in symfony
тоже ответили, что нет такого кейса. Они раскатывают все с дампа
источник

Kd

Konstantin dmz9 in symfony
ну да, должен, круто набивать шишки самому
источник

DS

Dima Sikorskiy in symfony
Alexander Nazarov
Вы полностью правы когда говорите что коды могут как сто сложно генерится и являться логикой приложения, также как и изменение баланса. Но судя по обсуждению ваша логика вполне себе уживается в коде миграции. Это не поход в какой то сервис, это чистая функция. Вот допустим вам надо было бы в миграции, открытые пароли, заменить на md5 хеши. Вы бы замену паролей на md5 вынесли в отдельную команду?
да я бы вынес это отдельно.  задача может  быть простой, а может быть и сложной.   но  сложность задачи не должна влиять на местоопределение  кода
источник

AN

Alexander Nazarov in symfony
Dima Sikorskiy
да я бы вынес это отдельно.  задача может  быть простой, а может быть и сложной.   но  сложность задачи не должна влиять на местоопределение  кода
Вы не понимаете сложная задача или нет в случае замены пароля на хеш?
источник

KN

Konstantin Nosov in symfony
Dima Sikorskiy
да я бы вынес это отдельно.  задача может  быть простой, а может быть и сложной.   но  сложность задачи не должна влиять на местоопределение  кода
ну как бы тут просто переходит перекладывание с больной головы на здоровую. Мы говорим всем что вместо запуска миграций который происходит атвоматически каждому человеку нужно еще сходить в CLI и запустить команду
источник

ВМ

Виктор Монастырев... in symfony
Konstantin Nosov
ну как бы тут просто переходит перекладывание с больной головы на здоровую. Мы говорим всем что вместо запуска миграций который происходит атвоматически каждому человеку нужно еще сходить в CLI и запустить команду
вот кстати аргумент
источник

ВМ

Виктор Монастырев... in symfony
Это же по сути изменение порядка деплоя, то есть тот кто будет делать деплой должен знать что по окончанию основной стадии надо зайти еще и выполнить команду
источник

DS

Dima Sikorskiy in symfony
Alexander Nazarov
Вы не понимаете сложная задача или нет в случае замены пароля на хеш?
генерация хеша с помощью  ключа (просто для примера).   где нужно считать файл.   почему  тогда,  когда я буду считывать файловую систему  у меня код должен быть в одном месте,  а когда просто  sql функция в другом месте?
источник