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

Return hijack objective #1494

Merged
merged 4 commits into from
Aug 4, 2024

Conversation

Kirus59
Copy link
Collaborator

@Kirus59 Kirus59 commented Jul 30, 2024

Описание PR

  • Добавить гиб от эвакуационного шаттла при его стыковке к станции (метод TryFTLDock сразу спавнит шаттл около соответствующих шлюзов. Из-за чего шаттл минует БСС прыжок, что в свою очередь не накладывает на него FTLComponent, от состояния которого у других шаттлов (при БСС прыжке например методом FTLToDock) и зависит гиб кукол при спавне на маппе. Поэтому я в метод TryFTLDock отдельно добавил вызов метода на гиб кукол).
  • Вернуть возможность емага консоли управления аварийным шаттлом.
  • Вернуть цель на угон эвакуационного шаттла трейторам.

Медиа
Демонстрация гиба и емага консоли эвака:

2024-07-31.19-25-36.mp4

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑 kirus59

  • tweak: Эвакуационный шаттл теперь гибает куклу при стыковке к станции.
  • tweak: Трейторам возвращена цель на угон эвакуационного шаттла.

@Kirus59 Kirus59 marked this pull request as ready for review July 31, 2024 17:32
@Kirus59
Copy link
Collaborator Author

Kirus59 commented Jul 31, 2024

На локалке почему то засчитывает цель даже если на шаттле есть другой член экипажа, кроме трейторов.
Хотя судя по методу IsShuttleHijacked в HijackShuttleConditionSystem цель не должна засчитываться, если на шаттле присутствует кто-то из обычных членов экипажа.
Не понимаю почему это происходит и как это исправить.

Оффы не меняли эту систему после удаления цели на хайджек, так что по идее должно работать как и раньше, мб проблема конкретно на локалке....

@Kirus59 Kirus59 changed the title [WiP] Return hijack objective Return hijack objective Jul 31, 2024
@DexlerXD
Copy link

@AliceValestray затестить бы...

@Kirus59
Copy link
Collaborator Author

Kirus59 commented Aug 1, 2024

@DexlerXD Нашли в чем проблема, в проверке на антагов на шаттле почему то ссылалось на самого хайджекера, а не на иных игроков на шаттле, из-за чего постоянно возвращалось true, независимо от того кто на шаттле.
Спасибо @ReeZer2 что помог найти в чем была проблема.

@DexlerXD
Copy link

DexlerXD commented Aug 4, 2024

@Kirus59 тесты нужны еще или мержить уже?

@Kirus59
Copy link
Collaborator Author

Kirus59 commented Aug 4, 2024

@Kirus59 тесты нужны еще или мержить уже?

Ща на локалке еще раз чекну, вроде после фикса все исправно работало.

@Kirus59
Copy link
Collaborator Author

Kirus59 commented Aug 4, 2024

@DexlerXD Да, можно мержить.
Цель не засчитывает если на эваке есть игроки, не являющиеся антагами (Однако если они будут в крите/мертвы/в наручниках - то цель засчитает).

@DexlerXD DexlerXD merged commit 80d70af into SerbiaStrong-220:master Aug 4, 2024
11 checks passed
@Kirus59 Kirus59 deleted the ReturnHijackObjective branch August 4, 2024 15:37
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