Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PRIME] Неразрушимый крематорий и телепортация при гибе шаттлом в космосе #1497

Merged
merged 11 commits into from
Sep 11, 2024

Conversation

ThaumicNik
Copy link

@ThaumicNik ThaumicNik commented Sep 11, 2024

Что этот PR делает

Крематорий теперь не разрушается от взрывов. Его можно починить используя новый igniter и сварочный аппарат.
Все шаттлы теперь вместо гиба телепортируют попавших под себя мобов и мехов в случайную клетку вокруг. Работает только в космосе, на земле по-прежнему гибает.

Почему это хорошо для игры

Так хотел Муня (Клип делать лень, сорри)

Тестирование

Тестировал на локалочке, с разными мехами, куклами и шаттлами.

Changelog

🆑
tweak: [PRIME only] Шаттлы теперь телепортируют вместо гиба при столкновении, но только в космосе. На земле всё по-старому.
fix: [PRIME only] Крематорий теперь невозможно полностью уничтожить при помощи взрыва
/:cl:

@github-actions github-actions bot added the 🔧 Фикс Переписываем ошибку так, чтобы она проявлялась в других обстоятельствах label Sep 11, 2024
Comment on lines 648 to 650
// SS220 PRIME EDIT - START
continue // It's required to avoid qdeling of mech in case of space turf. Non space turf are handled by get_out_and_die() proc
// SS220 PRIME EDIT - END
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
// SS220 PRIME EDIT - START
continue // It's required to avoid qdeling of mech in case of space turf. Non space turf are handled by get_out_and_die() proc
// SS220 PRIME EDIT - END
continue // SS220 EDIT: It's required to avoid qdeling of mech in case of space turf. Non space turf are handled by get_out_and_die() proc

@@ -0,0 +1,39 @@
#define SHUTTLE_ROADKILL_TELEPORTATION_RANGE 24

/obj/docking_port/mobile/proc/roadkill_living(var/mob/living/L) // Used in /obj/docking_port/mobile/proc/roadkill for modular behavior
Copy link
Collaborator

@AyIong AyIong Sep 11, 2024

Choose a reason for hiding this comment

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

Suggested change
/obj/docking_port/mobile/proc/roadkill_living(var/mob/living/L) // Used in /obj/docking_port/mobile/proc/roadkill for modular behavior
/obj/docking_port/mobile/proc/roadkill_living(mob/living/player) // Used in /obj/docking_port/mobile/proc/roadkill for modular behavior

Буковы не экономим

L.stop_pulling()
if(isspaceturf(get_turf(L)))
L.visible_message(
"<span class='warning'>[L] иcчезает в спышке блю-спейс излучения в тот момент, когда шаттл материализуется в нашем пространстве!</span>",
Copy link
Collaborator

Choose a reason for hiding this comment

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

  1. Есть дефайны на спаны
  2. блю-спейс -> блюспейс

Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

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

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

@ThaumicNik
Copy link
Author

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

Я не совсем разобрался как это у нас работает. Думал что просто ПР стянуть только на прайм ветку. В разработке уже пояснили. Я вынесу в отдельный модуль для изменения в код коде и сделаю оверрайд в модуле прайма.

@dj-34
Copy link
Collaborator

dj-34 commented Sep 11, 2024

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

Я не совсем разобрался как это у нас работает. Думал что просто ПР стянуть только на прайм ветку. В разработке уже пояснили. Я вынесу в отдельный модуль для изменения в код коде и сделаю оверрайд в модуле прайма.

у нас нет прайм ветки и не используется

@m-dzianishchyts
Copy link
Collaborator

Надо еще в чейнджлог начирикать, что это только на прайм

Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts left a comment

Choose a reason for hiding this comment

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

Повторяются строчки, давай их макросом типа

#define MESSAGE(input) "blablbalba [input] blala"

modular_ss220/prime_only/code/shuttle_gib.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/shuttle_gib.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/shuttle_gib.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/shuttle_gib.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/shuttle_gib.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/shuttle_gib.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/shuttle_gib.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/shuttle_gib.dm Outdated Show resolved Hide resolved
@ThaumicNik
Copy link
Author

@dj-34 @AyIong Вроде всё привёл в нормальный вид. Посмотрите, пожалуйста.

@AyIong AyIong added the 🕛 Ждёт мержа Совсем скоро это будет залито label Sep 11, 2024
@dj-34 dj-34 merged commit 329f02d into ss220club:master Sep 11, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🕛 Ждёт мержа Совсем скоро это будет залито 📜 CL валиден 🔧 Фикс Переписываем ошибку так, чтобы она проявлялась в других обстоятельствах
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants