Fix деления морфов#9111
Conversation
There was a problem hiding this comment.
Code Review
Аналитический отчёт от Большого Брата. Обработка запроса завершена. Данный пулл-реквест вносит изменения в механику размножения морфов, обеспечивая создание уникальных экземпляров обработчиков заклинаний для каждого существа и добавляя проверку состояния здоровья пользователя. В ходе ревью были выявлены критические замечания: необходимо устранить дублирование списания ресурсов (еды), так как это уже обрабатывается хэндлером заклинания, а также использовать revert_cast(user) при прерывании процесса из-за смерти, чтобы избежать потери ресурсов и некорректного запуска перезарядки. С технической точки зрения, работа требует доработок.
9e7833d to
92ebc40
Compare
…adise into fix-morph-reproduce
littleboobs
left a comment
There was a problem hiding this comment.
Ты не понял. Тебе надо в /datum/spell_handler/morph ищбавиться от var/hunger_cost и везде использовать для обращения к hunger_cost obj/effect/proc_holder/spell/morph_spell вместо obj/effect/proc_holder/spell.
А то что ты костылишь в enable_reproduce убрать.
По идее надо ещё убрать в /obj/effect/proc_holder/spell/morph_spell/create_new_handler() манипуляции с hunger_cost и оставить просто создания хендлера.
littleboobs
left a comment
There was a problem hiding this comment.
если на тестах всё ок, то ок
Что этот ПР делает
Исправлен баг, из-за которого новые морфы наследовали увеличенную стоимость деления от родителя. теперь при делении новому морфу выделяется собственный хэндлер, что сбрасывает цену его первого деления до базовых 150 еды
баг репорт: https://discord.com/channels/617003227182792704/1488943971550892113
Список изменений
🆑
bugfix: Исправлено наследование стоимости деления; новые морфы теперь начинают с базовой цены в 150 еды
/:cl: