stop breaking the game !!!!!!!!

Content.Shared/Weapons/Melee/Events/MeleeAttemptedEvent.cs

@@ -0,0 +1,37 @@
+namespace Content.Shared.Weapons.Melee.Events;
+
+/// <summary>
+/// Raised on a melee when someone is attempting to attack with it.
+/// Cancel this event to prevent it from attacking.
+/// </summary>
+[ByRefEvent]
+public record struct MeleeAttemptedEvent
+{
+    /// <summary>
+    /// The user attempting to attack with the weapon.
+    /// </summary>
+    public EntityUid User;
+
+    /// <summary>
+    /// The weapon being used.
+    /// </summary>
+    public EntityUid Used;
+
+    public bool Cancelled { get; private set; }
+
+    /// </summary>
+    /// Prevent the weapon from attacking
+    /// </summary>
+    public void Cancel()
+    {
+        Cancelled = true;
+    }
+
+    /// </summary>
+    /// Allow the weapon to attack again, only use if you know what you are doing
+    /// </summary>
+    public void Uncancel()
+    {
+        Cancelled = false;
+    }
+}

Content.Shared/Weapons/Melee/SharedMeleeWeaponSystem.cs

@@ -62,7 +62,7 @@ public override void Initialize()
         base.Initialize();
 
         SubscribeLocalEvent<MeleeWeaponComponent, HandSelectedEvent>(OnMeleeSelected);
-        SubscribeLocalEvent<MeleeWeaponComponent, ShotAttemptedEvent>(OnMeleeShotAttempted);
+        SubscribeLocalEvent<MeleeWeaponComponent, MeleeAttemptedEvent>(OnMeleeAttempted);
         SubscribeLocalEvent<MeleeWeaponComponent, GunShotEvent>(OnMeleeShot);
         SubscribeLocalEvent<BonusMeleeDamageComponent, GetMeleeDamageEvent>(OnGetBonusMeleeDamage);
         SubscribeLocalEvent<BonusMeleeDamageComponent, GetHeavyDamageModifierEvent>(OnGetBonusHeavyDamageModifier);
@@ -86,7 +86,7 @@ private void OnMapInit(EntityUid uid, MeleeWeaponComponent component, MapInitEve
 #endif
     }
 
-    private void OnMeleeShotAttempted(EntityUid uid, MeleeWeaponComponent comp, ref ShotAttemptedEvent args)
+    private void OnMeleeAttempted(EntityUid uid, MeleeWeaponComponent comp, ref MeleeAttemptedEvent args)
     {
         if (comp.NextAttack > Timing.CurTime)
             args.Cancel();

Content.Shared/Weapons/Ranged/Systems/SharedGunSystem.cs

@@ -76,7 +76,7 @@ public override void Initialize()
     {
         SubscribeAllEvent<RequestShootEvent>(OnShootRequest);
         SubscribeAllEvent<RequestStopShootEvent>(OnStopShootRequest);
-        SubscribeLocalEvent<GunComponent, MeleeHitEvent>(OnGunMelee);
+        //SubscribeLocalEvent<GunComponent, MeleeHitEvent>(OnGunMelee);
 
         // Ammo providers
         InitializeBallistic();
@@ -110,7 +110,8 @@ private void OnMapInit(Entity<GunComponent> gun, ref MapInitEvent args)
         RefreshModifiers((gun, gun));
     }
 
-    private void OnGunMelee(EntityUid uid, GunComponent component, MeleeHitEvent args)
+    // Sets the gun cooldown to the melee attack cooldown on attack. Removed since it felt very unintuitive.
+    /*private void OnGunMelee(EntityUid uid, GunComponent component, MeleeHitEvent args)
     {
         if (!TryComp<MeleeWeaponComponent>(uid, out var melee))
             return;
@@ -120,7 +121,7 @@ private void OnGunMelee(EntityUid uid, GunComponent component, MeleeHitEvent arg
             component.NextFire = melee.NextAttack;
             Dirty(component);
         }
-    }
+    }*/
 
     private void OnShootRequest(RequestShootEvent msg, EntitySessionEventArgs args)
     {