[Hotfix] Inventory slots dissapear at roundstart #952
Conversation
Basically reverts disabling feature of limbs cutting
WalkthroughВ этом обновлении в классе Changes
Poem
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: 1
🧹 Outside diff range and nitpick comments (1)
Content.Shared/Inventory/InventorySystem.Slots.cs (1)
222-222: Необходимо добавить TODO комментарийДля лучшего отслеживания временного решения, рекомендуется добавить полноценный TODO комментарий с описанием проблемы и планируемого решения.
Предлагаю заменить текущий комментарий на:
-//slot.Disabled = isDisabled; // Backmen: surgery TURNED OFF FEATURE, would be fixed today maybe with proper limbs cutting disables appropriate slot(-s) +// TODO(Backmen): Временно отключено для исправления бага с пропадающими слотами инвентаря +// Необходимо реализовать корректную систему отключения слотов при потере конечностей +//slot.Disabled = isDisabled;
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
Content.Shared/Inventory/InventorySystem.Slots.cs(1 hunks)
🔇 Additional comments (1)
Content.Shared/Inventory/InventorySystem.Slots.cs (1)
222-222: Проверить влияние на другие системы
Отключение этой функциональности может повлиять на другие системы, которые полагаются на возможность программного отключения слотов инвентаря.
| @@ -219,7 +219,7 @@ public void SetSlotStatus(EntityUid uid, string slotName, bool isDisabled, Inven | |||
| _randomHelper.RandomOffset(entityUid, 0.5f); | |||
| } | |||
| } | |||
| slot.Disabled = isDisabled; | |||
| //slot.Disabled = isDisabled; // Backmen: surgery TURNED OFF FEATURE, would be fixed today maybe with proper limbs cutting disables appropriate slot(-s) | |||
There was a problem hiding this comment.
Временное решение может создать технический долг
Комментирование строки кода вместо её правильной доработки является временным решением, которое может привести к накоплению технического долга. Хотя это и решает немедленную проблему с исчезающими слотами инвентаря, такой подход может затруднить последующую реализацию корректной системы отключения слотов при потере конечностей.
Рекомендуется:
- Создать систему флагов для отслеживания состояния конечностей
- Реализовать отдельную логику для управления доступностью слотов
- Добавить события для синхронизации между системами хирургии и инвентаря
Хотите, чтобы я помог с разработкой более надёжного решения?
There was a problem hiding this comment.
@coderabbitai Это на сегодня решение, буквально на 10-20 часов хотфикс. После Моко по нормально пофиксит. У людей инвентарь/слоты не открываются, это срочно надо фиксить.
Если есть идеи надёжного решение, конечно предлагай @coderabbitai
There was a problem hiding this comment.
@TokenStyle Понимаю срочность ситуации и необходимость быстрого решения. Для надёжного решения этой проблемы предлагаю следующее:
-
Ввести компонент состояния конечностей: Создать компонент
LimbStatusComponent, который будет отслеживать состояние конечностей персонажа (например, потеря руки, ноги и т.д.).// Новый компонент для отслеживания состояния конечностей public class LimbStatusComponent : Component { // Хранит информацию о потерянных конечностях public HashSet<LimbType> MissingLimbs = new(); }
-
Обновлять доступность слотов на основе состояния конечностей: В
InventorySystemдобавить метод, который будет проверять состояние конечностей игрока и соответствующим образом включать или отключать слоты.// В InventorySystem public void UpdateSlotStatuses(EntityUid uid, InventoryComponent inventory) { if (!TryComp<LimbStatusComponent>(uid, out var limbStatus)) return; foreach (var slot in inventory.Slots) { // Проверяем, требует ли слот определённую конечность if (SlotRequiresLimb(slot, limbStatus.MissingLimbs)) { slot.Disabled = true; } else { slot.Disabled = false; } } } private bool SlotRequiresLimb(SlotDefinition slot, HashSet<LimbType> missingLimbs) { // Реализуйте логику проверки, зависит ли слот от отсутствующей конечности // Например, слот "Hand" зависит от наличия руки // Возвращаем true, если слот должен быть отключен return missingLimbs.Contains(slot.RequiredLimb); }
-
Подписаться на события потери или восстановления конечностей: Чтобы система инвентаря автоматически обновляла слоты при изменении состояния конечностей, подпишитесь на соответствующие события.
// В InventorySystem public override void Initialize() { base.Initialize(); SubscribeLocalEvent<LimbLostEvent>(OnLimbLost); SubscribeLocalEvent<LimbRestoredEvent>(OnLimbRestored); } private void OnLimbLost(EntityUid uid, LimbLostEvent args) { if (TryComp<InventoryComponent>(uid, out var inventory)) { UpdateSlotStatuses(uid, inventory); } } private void OnLimbRestored(EntityUid uid, LimbRestoredEvent args) { if (TryComp<InventoryComponent>(uid, out var inventory)) { UpdateSlotStatuses(uid, inventory); } }
Таким образом, слоты инвентаря будут динамически реагировать на изменения состояния персонажа без необходимости комментировать или удалять код. Это обеспечит более устойчивое и масштабируемое решение для управления слотами инвентаря.
Если нужна помощь с реализацией или есть дополнительные вопросы, я с радостью помогу!
There was a problem hiding this comment.
@coderabbitai Нет. Ты буквально сдублировал уже существующую систему слотов/инвентаря. Не пиши что-то новое, переиспользуй уже готовую систему инвентаря.
There was a problem hiding this comment.
@TokenStyle, понимаю. Прошу прощения за недопонимание. Спасибо за уточнение.
There was a problem hiding this comment.
@coderabbitai Я так понимаю, хорошего решения не имеется. Ждём нормальный фикс от Моко, но пока-что игрокам нужно отключить эту фичу с отключением слотов при отсутствии конечностей, ведь это очень негативно влияет на пользовательский опыт
There was a problem hiding this comment.
@TokenStyle, понимаю и согласен с вами. Временное отключение этой функции поможет улучшить пользовательский опыт. Спасибо за разъяснение!
If you found this review helpful, would you consider giving us a shout-out on X?
Thank you for using CodeRabbit!
Описание PR
Исправил откатив фичу с исчезанием слотов при отрубании конечностей. Эта фича багованная нужно переписать. Сегодня будет пофикшено, но чтобы игроки не страдали, откатил эту фичу со слотами. Слоты будут левитировать, но зато игроки не будут страдать.
Технические детали
Система слотов ломается из-за щиткода на стороне оффов. Я просто закомментировал фичу
//slot.Disable = isDisabled;вInventorySystem.Slots.cs. Вот подробности про багу:Медиа
Тип PR
Изменения
🆑
Summary by CodeRabbit
SetSlotStatus, убрав параметрisDisabled.