From dcc1c38fb729b00fdb15b79b38372f3e1e8bc951 Mon Sep 17 00:00:00 2001 From: sleepyyapril <123355664+sleepyyapril@users.noreply.github.com> Date: Sat, 23 Nov 2024 20:24:04 -0400 Subject: [PATCH 1/3] Engine Update v237.2.0 (#1270) # Description Updates to engine version v237.2.0. Fixes disposals systems breaking and not using the proper direction. --------- Co-authored-by: sleepyyapril --- .../Disposal/Unit/EntitySystems/DisposableSystem.cs | 9 +++++---- RobustToolbox | 2 +- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/Content.Server/Disposal/Unit/EntitySystems/DisposableSystem.cs b/Content.Server/Disposal/Unit/EntitySystems/DisposableSystem.cs index e7c83c8ac6..bcf240df53 100644 --- a/Content.Server/Disposal/Unit/EntitySystems/DisposableSystem.cs +++ b/Content.Server/Disposal/Unit/EntitySystems/DisposableSystem.cs @@ -137,12 +137,13 @@ public void ExitDisposals(EntityUid uid, DisposalHolderComponent? holder = null, else { _xformSystem.AttachToGridOrMap(entity, xform); + var direction = holder.CurrentDirection == Direction.Invalid ? holder.PreviousDirection : holder.CurrentDirection; - if (holder.PreviousDirection != Direction.Invalid && _xformQuery.TryGetComponent(xform.ParentUid, out var parentXform)) + if (direction != Direction.Invalid && _xformQuery.TryGetComponent(gridUid, out var gridXform)) { - var direction = holder.PreviousDirection.ToAngle(); - direction += _xformSystem.GetWorldRotation(parentXform); - _throwing.TryThrow(entity, direction.ToWorldVec() * 3f, 10f); + var directionAngle = direction.ToAngle(); + directionAngle += _xformSystem.GetWorldRotation(gridXform); + _throwing.TryThrow(entity, directionAngle.ToWorldVec() * 3f, 10f); } } } diff --git a/RobustToolbox b/RobustToolbox index 32bca7cfd4..92b0e7f1a8 160000 --- a/RobustToolbox +++ b/RobustToolbox @@ -1 +1 @@ -Subproject commit 32bca7cfd417edcad9a60c2b1703eba8675f56af +Subproject commit 92b0e7f1a853979a1361ed24d2fb5ffc11f43f66 From 613e7937392d5d4fd9bf906f571adc237c706367 Mon Sep 17 00:00:00 2001 From: Skubman Date: Mon, 25 Nov 2024 02:44:47 +0800 Subject: [PATCH 2/3] [QoL] Show Oni Damage Bonus In Damage Examine (#1268) # Description Examining an item's damage values as an Oni now calculates your bonus damage, and it also works with all of the Oni combat traits. ## Technical details `OniSystem`, instead of adding melee damage through `MeleeHitEvent`, now adds damage through `GetMeleeDamageEvent`, the same event that examining melee weapon damage raises through `GetDamage`. ## Media **Normal damage values** **Oni damage values** # Changelog :cl: Skubman - tweak: As an Oni, examining the damage values of weapons now takes into account the melee damage bonus from your species or trait. --- .../Nyanotrasen/Abilities/Oni/OniSystem.cs | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/Content.Server/Nyanotrasen/Abilities/Oni/OniSystem.cs b/Content.Server/Nyanotrasen/Abilities/Oni/OniSystem.cs index 4fc078e85b..a7e2295b75 100644 --- a/Content.Server/Nyanotrasen/Abilities/Oni/OniSystem.cs +++ b/Content.Server/Nyanotrasen/Abilities/Oni/OniSystem.cs @@ -3,6 +3,7 @@ using Content.Shared.Tools.Components; using Content.Shared.Damage.Events; using Content.Shared.Nyanotrasen.Abilities.Oni; +using Content.Shared.Weapons.Melee; using Content.Shared.Weapons.Melee.Events; using Content.Shared.Weapons.Ranged.Components; using Robust.Shared.Containers; @@ -19,8 +20,7 @@ public override void Initialize() base.Initialize(); SubscribeLocalEvent(OnEntInserted); SubscribeLocalEvent(OnEntRemoved); - SubscribeLocalEvent(OnOniMeleeHit); - SubscribeLocalEvent(OnHeldMeleeHit); + SubscribeLocalEvent(OnGetMeleeDamage); SubscribeLocalEvent(OnStamHit); } @@ -55,17 +55,12 @@ private void OnEntRemoved(EntityUid uid, OniComponent component, EntRemovedFromC RemComp(args.Entity); } - private void OnOniMeleeHit(EntityUid uid, OniComponent component, MeleeHitEvent args) + private void OnGetMeleeDamage(EntityUid uid, MeleeWeaponComponent component, ref GetMeleeDamageEvent args) { - args.ModifiersList.Add(component.MeleeModifiers); - } - - private void OnHeldMeleeHit(EntityUid uid, HeldByOniComponent component, MeleeHitEvent args) - { - if (!TryComp(component.Holder, out var oni)) + if (!TryComp(args.User, out var oni)) return; - args.ModifiersList.Add(oni.MeleeModifiers); + args.Modifiers.Add(oni.MeleeModifiers); } private void OnStamHit(EntityUid uid, HeldByOniComponent component, TakeStaminaDamageEvent args) From 5fe4b1e94b987f787807cc10df06f667b87d124e Mon Sep 17 00:00:00 2001 From: SimpleStation Changelogs Date: Sun, 24 Nov 2024 18:45:12 +0000 Subject: [PATCH 3/3] Automatic Changelog Update (#1268) --- Resources/Changelog/Changelog.yml | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/Resources/Changelog/Changelog.yml b/Resources/Changelog/Changelog.yml index ec25696311..ddc14396ae 100644 --- a/Resources/Changelog/Changelog.yml +++ b/Resources/Changelog/Changelog.yml @@ -8129,3 +8129,12 @@ Entries: id: 6539 time: '2024-11-22T21:54:10.0000000+00:00' url: https://github.com/Simple-Station/Einstein-Engines/pull/1001 +- author: Skubman + changes: + - type: Tweak + message: >- + As an Oni, examining the damage values of weapons now takes into account + the melee damage bonus from your species or trait. + id: 6540 + time: '2024-11-24T18:44:48.0000000+00:00' + url: https://github.com/Simple-Station/Einstein-Engines/pull/1268