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

No vocal emotes when muzzled #1101

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

SkaldetSkaeg
Copy link

@SkaldetSkaeg SkaldetSkaeg commented Jun 8, 2024

Описание PR

Увидел на стриме у Никиты, что игроки спокойно кричат эмоционируют, с кляпом во рту, а так же спящие.
Это явно баг. Чтобы сделать систему со звуками приглушенными была мысль, но идей особо нету, как это сделать адекватно, так как сама VocalSystem отсосная.
fixes #1181

Медиа

Проверки

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

Изменения
🆑

  • fix: персонажи в кляпах больше не воспроизводят эмоции со звуком.
  • fix: спящие персонажи больше не управляют своими эмоциями, а воспроизводят рандомные из пулла тихих.

@SkaldetSkaeg
Copy link
Author

SkaldetSkaeg commented Jun 8, 2024

Нужно будет согласование/возможно какие-то идеи от других.

@Kit0vras
Copy link

Kit0vras commented Jun 8, 2024

а во сне? я мог плакать, свистеть и тд во сне

@SkaldetSkaeg
Copy link
Author

а во сне? я мог плакать, свистеть и тд во сне

эт, вроде фиксили уже с ноктюрином, но посмотрю.

@SkaldetSkaeg
Copy link
Author

а во сне? я мог плакать, свистеть и тд во сне

эт, вроде фиксили уже с ноктюрином, но посмотрю.

бля, охуенно, там чисто на кнопку ограничения



// SS220 no_vocal_with_muzzle begin
if (_entities.TryGetComponent<ReplacementAccentComponent>(uid, out var accentComp) && accentComp.Accent == "mumble")
Copy link
Member

Choose a reason for hiding this comment

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

хардкод убери, сделай через константую переменную лучше где-то в классе

Copy link
Author

Choose a reason for hiding this comment

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

хардкод убери, сделай через константую переменную лучше где-то в классе

там ещё при сне это работает, я думаю на уровне Чата перенести

[RegisterComponent, NetworkedComponent()]
[Access(typeof(SharedMuzzleSystem))]
/// <summary>
/// Added to entities when they have to block entityes vocal emotions
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
/// Added to entities when they have to block entityes vocal emotions
/// Added to entities that are blocking other entity's vocal emotions

Content.Server/SS220/Muzzle/MuzzleSystem.cs Outdated Show resolved Hide resolved
SubscribeLocalEvent<MuzzleComponent, ClothingGotUnequippedEvent>(OnGotUnequipped);
}

private void OnGotUnequipped(EntityUid uid, MuzzleComponent component, ref ClothingGotUnequippedEvent args)
Copy link
Member

Choose a reason for hiding this comment

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

по новой форме обработки ивентов оформь
т. е.

private void Func(Entity<Comp> ent, ref EventClass args) {}


namespace Content.Shared.SS220.Muzzle;

[RegisterComponent, NetworkedComponent()]
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
[RegisterComponent, NetworkedComponent()]
[RegisterComponent, NetworkedComponent]


namespace Content.Shared.SS220.Muzzle;

[RegisterComponent, NetworkedComponent()]
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
[RegisterComponent, NetworkedComponent()]
[RegisterComponent, NetworkedComponent]

Comment on lines 64 to 65
[Dependency] private readonly IEntityManager _entities = default!;// SS220 No vocal emotes when muzzled
[Dependency] private readonly IPrototypeManager _proto = default!;// SS220 No vocal emotes when muzzled
Copy link
Member

Choose a reason for hiding this comment

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

на это пометки можно не ставить

@SkaldetSkaeg
Copy link
Author

@DexlerXD ну, крч, я пофиксил всё, кроме переноса в Server. При попытке переноса туда у меня почему-то ломаются голопроекторы атмосские.

@DexlerXD
Copy link
Member

@DexlerXD ну, крч, я пофиксил всё, кроме переноса в Server. При попытке переноса туда у меня почему-то ломаются голопроекторы атмосские.

я посмотрю сегодня

@Alwayswannahunt
Copy link

@SkaldetSkaeg добавь в описание PR следующую строчку - fixes #1181 чтобы github сам закрыл issue при мердже PR.

Copy link

github-actions bot commented Jul 2, 2024

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

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.

Крик во сне
4 participants