Skip to content

Fix деления морфов#9111

Open
Infaselt wants to merge 28 commits intoss220-space:master220from
Infaselt:fix-morph-reproduce
Open

Fix деления морфов#9111
Infaselt wants to merge 28 commits intoss220-space:master220from
Infaselt:fix-morph-reproduce

Conversation

@Infaselt
Copy link
Copy Markdown
Contributor

@Infaselt Infaselt commented May 1, 2026

Что этот ПР делает

Исправлен баг, из-за которого новые морфы наследовали увеличенную стоимость деления от родителя. теперь при делении новому морфу выделяется собственный хэндлер, что сбрасывает цену его первого деления до базовых 150 еды
баг репорт: https://discord.com/channels/617003227182792704/1488943971550892113

Список изменений

🆑
bugfix: Исправлено наследование стоимости деления; новые морфы теперь начинают с базовой цены в 150 еды
/:cl:

@github-actions github-actions Bot added the Fix PR содержит исправления ошибок. label May 1, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

Аналитический отчёт от Большого Брата. Обработка запроса завершена. Данный пулл-реквест вносит изменения в механику размножения морфов, обеспечивая создание уникальных экземпляров обработчиков заклинаний для каждого существа и добавляя проверку состояния здоровья пользователя. В ходе ревью были выявлены критические замечания: необходимо устранить дублирование списания ресурсов (еды), так как это уже обрабатывается хэндлером заклинания, а также использовать revert_cast(user) при прерывании процесса из-за смерти, чтобы избежать потери ресурсов и некорректного запуска перезарядки. С технической точки зрения, работа требует доработок.

Comment thread code/game/gamemodes/miniantags/morph/spells/reproduce.dm Outdated
Comment thread code/game/gamemodes/miniantags/morph/spells/reproduce.dm
Comment thread code/game/gamemodes/miniantags/morph/morph.dm Outdated
Comment thread code/game/gamemodes/miniantags/morph/morph.dm Outdated
Comment thread code/game/gamemodes/miniantags/morph/morph.dm Outdated
Comment thread code/game/gamemodes/miniantags/morph/spells/reproduce.dm
@Infaselt Infaselt force-pushed the fix-morph-reproduce branch from 9e7833d to 92ebc40 Compare May 2, 2026 16:58
@littleboobs littleboobs added the Do Not Merge PR не должен быть замержен, поскольку кто-то чем-то недоволен. label May 2, 2026
Comment thread code/game/gamemodes/miniantags/morph/morph.dm Outdated
@Infaselt Infaselt requested a review from littleboobs May 3, 2026 00:03
Copy link
Copy Markdown
Collaborator

@littleboobs littleboobs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ты не понял. Тебе надо в /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 и оставить просто создания хендлера.

@Infaselt Infaselt requested a review from littleboobs May 5, 2026 18:05
Comment thread code/game/gamemodes/miniantags/morph/spells/reproduce.dm Outdated
Comment thread code/game/gamemodes/miniantags/morph/spells/morph_spell.dm
@littleboobs littleboobs removed the Do Not Merge PR не должен быть замержен, поскольку кто-то чем-то недоволен. label May 5, 2026
Copy link
Copy Markdown
Collaborator

@littleboobs littleboobs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

хотя, сук...

@littleboobs littleboobs added Do Not Merge PR не должен быть замержен, поскольку кто-то чем-то недоволен. and removed Do Not Merge PR не должен быть замержен, поскольку кто-то чем-то недоволен. labels May 5, 2026
Copy link
Copy Markdown
Collaborator

@littleboobs littleboobs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

если на тестах всё ок, то ок

Copy link
Copy Markdown
Collaborator

@Ksaikok46 Ksaikok46 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Выглядит нормально

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fix PR содержит исправления ошибок.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants