Skip to content

[FIX] Dock pod#2647

Merged
Darkiich merged 2 commits intoAdventureTimeSS14:masterfrom
CrimeMoot:fix_dock_ddd
Mar 21, 2026
Merged

[FIX] Dock pod#2647
Darkiich merged 2 commits intoAdventureTimeSS14:masterfrom
CrimeMoot:fix_dock_ddd

Conversation

@CrimeMoot
Copy link
Contributor

@CrimeMoot CrimeMoot commented Mar 20, 2026

Описание PR

Исправил спасательные капсулы - теперь они открыты после инициализации карты. А до этого они не открывались, т.к при инициализации в дверях нет энергии. Добавил систему что шлюз запоминает что он вообще-то подключен к порту. И при подаче света, открыть шлюз. Работает со всеми шаттлами которые стыкуются к порту - если тот обесточен, то при подаче света, он автоматически откроется.

Техническая информация

  • Изменения были протестированы на локальном сервере, и всё работает отлично.
  • PR закончен и требует просмотра изменений.

Чейнджлог

🆑 CrimeMoot

  • fix: Спасательные капсулы теперь имеют сразу открытые стыковые шлюзы, ранее они были закрыты.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 20, 2026

Пошаговое описание

В системе шлюзов добавлена логика для обработки восстановления электропитания. При включении питания система проверяет, находится ли объект в состоянии стыковки и закрыта ли дверь. Если оба условия верны, выполняются операции открытия двери и установки болтов. Эта логика срабатывает перед существующим вызовом автоматического закрытия. Добавлен импорт необходимого пространства имён для работы с компонентом стыковки.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed Заголовок '[FIX] Dock pod' кратко относится к основному назначению PR - исправлению проблемы с дверьми стыковочных капсул, хотя и использует сокращённую формулировку.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed Описание PR связано с внесённо изменениями и объясняет цель исправления проблемы с закрытыми шлюзами спасательных капсул.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@Content.Server/Doors/Systems/AirlockSystem.cs`:
- Around line 67-71: The code calls DoorSystem.TryOpen(uid, door) but ignores
its boolean result; change the logic so you only call
DoorSystem.SetBoltsDown((uid, doorBolt), true) when TryOpen returns true and a
DoorBoltComponent exists: check the return value of DoorSystem.TryOpen(uid,
door) first, then if it returned true and TryComp<DoorBoltComponent>(uid, out
var doorBolt) succeeds, call DoorSystem.SetBoltsDown((uid, doorBolt), true) (use
the existing uid, door, and doorBolt symbols).

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 986e685e-4074-4653-89ef-6acbd6867192

📥 Commits

Reviewing files that changed from the base of the PR and between 55e3e56 and 8191491.

📒 Files selected for processing (1)
  • Content.Server/Doors/Systems/AirlockSystem.cs

@Darkiich Darkiich merged commit 7c8f8af into AdventureTimeSS14:master Mar 21, 2026
13 checks passed
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