[Fix/Tweak] Bodybags with Laying system #983
Conversation
ОбзорWalkthroughИзменения затрагивают систему хранения сущностей в морге и крематории. В коде системы Changes
Poem
Tip CodeRabbit's docstrings feature is now available as part of our Early Access Program! Simply use the command Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
Content.Shared/Morgue/EntityStorageLayingDownOverrideSystem.cs (1)
26-26: Улучшить читаемость условияТекущее условие с двойным отрицанием может быть сложным для понимания. Предлагаю переписать его более явно.
-if (HasComp<BodyComponent>(ent) && (!_standing.IsDown(ent) || !_mobState.IsDead(ent))) +if (HasComp<BodyComponent>(ent) && (IsStanding(ent) || IsAlive(ent)))Где
IsStandingиIsAlive- новые вспомогательные методы:private bool IsStanding(EntityUid entity) => !_standing.IsDown(entity); private bool IsAlive(EntityUid entity) => !_mobState.IsDead(entity);
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
Content.Shared/Morgue/EntityStorageLayingDownOverrideSystem.cs(3 hunks)Resources/Prototypes/Entities/Structures/Storage/morgue.yml(0 hunks)
💤 Files with no reviewable changes (1)
- Resources/Prototypes/Entities/Structures/Storage/morgue.yml
🔇 Additional comments (2)
Content.Shared/Morgue/EntityStorageLayingDownOverrideSystem.cs (2)
2-3: LGTM: Добавление зависимости MobStateSystem
Добавление системы MobStateSystem и соответствующих using-директив логично дополняет функциональность проверки состояния мобов.
Also applies to: 13-13
24-28: Проверить обработку ошибок
Необходимо убедиться в корректной обработке невалидных сущностей при проверке состояний.
✅ Verification successful
Проверки на невалидные сущности реализованы корректно
Анализ кода показывает, что необходимые проверки уже реализованы:
IsDown()в StandingStateSystem содержит проверку черезResolve(), которая защищает от невалидных сущностейIsDead()в MobStateSystem также включает проверку через_mobStateQuery.Resolve()HasComp<BodyComponent>()безопасно проверяет наличие компонента
Дополнительные проверки на невалидность сущностей в данном контексте не требуются, так как базовые системы уже обеспечивают необходимую защиту.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Поиск проверок на null/невалидные сущности в связанных системах
# Проверяем реализацию IsDown в StandingStateSystem
ast-grep --pattern 'class StandingStateSystem {
$$$
IsDown($_) {
$$$
}
$$$
}'
# Проверяем реализацию IsDead в MobStateSystem
ast-grep --pattern 'class MobStateSystem {
$$$
IsDead($_) {
$$$
}
$$$
}'
Length of output: 178
Script:
#!/bin/bash
# Поиск файлов с реализацией систем
fd "StandingStateSystem|MobStateSystem" --type f
# Поиск реализации IsDown
rg "IsDown.*\(" -A 5
# Поиск реализации IsDead
rg "IsDead.*\(" -A 5
# Поиск использования EntityUid.Invalid или подобных проверок
rg "EntityUid\.Invalid|!EntityManager\.Valid|TryComp|HasComp" "Content.Shared/Morgue/EntityStorageLayingDownOverrideSystem.cs"
Length of output: 41883
Описание PR
Thx to Remuchi
Игроки более не могут находиться в мешке для трупов Не Мёртвыми.
В крематорий и морг можно "лечь" снова стоя.
Тип PR
Изменения
🆑 Warete
Summary by CodeRabbit
Новые функции
EntityStorageLayingDownOverrideдля объектов Морга и Крематория, улучшая управление содержимым.Исправления ошибок