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

fix Evil Twin #1581

Closed
wants to merge 1 commit into from
Closed

Conversation

VigersRay
Copy link
Contributor

Описание PR

Заметил что нихуя не работает злой двойник, после входа на роль тебя бросает в госты, + цель на убийство не отображалась.
Почему бросало в госты? В душе не ебу, почему mind оставалось просле переноса на спавнере и после его удаления игрок обратно бросается в госты. Перекинул удаление перед переносом mind и сделал его без очереди.
Еще что забавно - если проверять спавн двойника в дебаг версии - такого бага не будет.

Медиа

Проверки

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

Изменения

@VigersRay VigersRay requested a review from Morb0 as a code owner November 22, 2023 21:01
@VigersRay VigersRay changed the title fix fix Evil Twin Nov 22, 2023
@github-actions github-actions bot added the Status: Needs Review This PR requires new reviews before it can be merged label Nov 22, 2023
@SonicHDC
Copy link
Collaborator

Лучше сразу кидать оффам, теперь двойник и у них.
P.S. Рад что ты живой и с нами (:

@VigersRay
Copy link
Contributor Author

Лучше сразу кидать оффам, теперь двойник и у них. P.S. Рад что ты живой и с нами (:

нее, я с оффами не дружу, раз у них то наверное он должен быть рабочий?.. Или нет?

@SonicHDC
Copy link
Collaborator

Лучше сразу кидать оффам, теперь двойник и у них. P.S. Рад что ты живой и с нами (:

нее, я с оффами не дружу, раз у них то наверное он должен быть рабочий?.. Или нет?

нет, у них тоже самое что и у нас

@lzk228
Copy link
Collaborator

lzk228 commented Nov 23, 2023

Лучше сразу кидать оффам, теперь двойник и у них. P.S. Рад что ты живой и с нами (:

чекни какие файлы изменены в этом пре

Comment on lines +68 to 70
Del(uid);
mind.CharacterName = MetaData(spawnedTwin.Value).EntityName;
_mindSystem.TransferTo(mindId, spawnedTwin);
Copy link
Member

Choose a reason for hiding this comment

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

Удаление должно быть последним шагом, используя QueueDel

Copy link
Contributor Author

@VigersRay VigersRay Nov 23, 2023

Choose a reason for hiding this comment

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

В таком случае при удалении спавнера игрок будет кинут в госты. Почему я в душе не чаю, очень странная штука, с аналогичным спавнером допустим беглеца у меня всё работает. Тут или оффы опять поломали что-то, или волшебство.
Почему-то даже после _mindSystem.TransferTo(mindId, spawnedTwin); внутри MindComponent спавнера остается записано mindId игрока.
Есть еще вариант не удалять спавнер а просто убрать у него компоненты призрак роли, чтобы новые игроки не заходили.
Впрочем я много вариантов пробывал и это пожалуй эдиный рабочий, удалить спавнер перед переносом сознания.
Кстати попытаться просто выкинуть создание через _mindSystem.TransferTo(mindId, null); тоже ничего не дает.

Copy link
Member

@Morb0 Morb0 Nov 23, 2023

Choose a reason for hiding this comment

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

В таком случае при удалении спавнера игрок будет кинут в госты.

Я не разбирался, но предположу, что возможно uid указывает на session.AttachedEntity и вызов QueueDel ставит удаление в очередь на следующий тик, в котором session.AttachedEntity уже будет тело двойника.

@SoulFN
Copy link
Contributor

SoulFN commented Apr 12, 2024

Если оффам надо, тогда может закроете ПР? Он все равно не обновляется давно

@github-actions github-actions bot added the Merge Conflict PR have unsolved merge conflicts label May 20, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Morb0 Morb0 closed this May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merge Conflict PR have unsolved merge conflicts Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants