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

[Hotfix] Хирургия части тела теперь уничтожаемы #73

Merged

Conversation

TokenStyle
Copy link

Описание PR

Части тела сейчас нельзя уничтожить, которые вывалились с тела. Это фиксит баг с вываливанием мозгов с головы и уничтожением частей тела.

Почему / Баланс

Мозг достать из головы не выйдет простыми способами (slash, blunt, heat уроном), нужна дробилка, мусоропереработчик, миксер и т.д.

Технические детали

Опечатку исправил, должно было в двух местах удаляться, но почему-то удаляется только со стороны тела.

Медиа

Критические изменения

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

🆑

  • fix: Теперь части тела могут быть уничтожены! Например уничтожить голову для получения мозга.

@TokenStyle
Copy link
Author

SERVER: 1,503s [ERRO] loc: "ru-RU"/null: "System.NullReferenceException: Object reference not set to an instance of an object.
at Robust.Shared.Localization.LocalizationManager.TryGetString(String messageId, ValueTuple`2 bundle, String& value) in /home/runner/work/space-station-14-next/space-station-14-next/RobustToolbox/Robust.Shared/Localization/LocalizationManager.cs:line 136"

Файлы локализации падают ru-RU/null.

@TokenStyle
Copy link
Author

Попробовать перезапустить можно CI/CD

@TokenStyle
Copy link
Author

Оно падает при генерации имён персонажей - возможно, что одно из имён где-то не проставлено, и сске пытается его проставить. Не находит. И падает с ошибкой (там спавнится dummy персонаж, а ему даётся случайное имя)

@AwareFoxy AwareFoxy merged commit 1d8ae69 into space-syndicate:master Nov 22, 2024
15 checks passed
desuwatch added a commit to desuwatch/space-station-14-next that referenced this pull request Jan 18, 2025
desuwatch added a commit to desuwatch/space-station-14-next that referenced this pull request Jan 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants