Fix weather blocking in some instances (#14561)

Content.Client/Weather/WeatherOverlay.cs

@@ -95,6 +95,7 @@ public sealed class WeatherOverlay : Overlay
         {
             var bodyQuery = _entManager.GetEntityQuery<PhysicsComponent>();
             var xformQuery = _entManager.GetEntityQuery<TransformComponent>();
+            var weatherIgnoreQuery = _entManager.GetEntityQuery<IgnoreWeatherComponent>();
 
             foreach (var grid in _mapManager.FindGridsIntersecting(mapId, worldAABB))
             {
@@ -105,7 +106,7 @@ public sealed class WeatherOverlay : Overlay
                 foreach (var tile in grid.GetTilesIntersecting(worldAABB))
                 {
                     // Ignored tiles for stencil
-                    if (_weather.CanWeatherAffect(grid, tile, bodyQuery))
+                    if (_weather.CanWeatherAffect(grid, tile, weatherIgnoreQuery, bodyQuery))
                     {
                         continue;
                     }

Content.Client/Weather/WeatherSystem.cs

@@ -82,6 +82,7 @@ public sealed class WeatherSystem : SharedWeatherSystem
             // If we don't have a nearest node don't play any sound.
             EntityCoordinates? nearestNode = null;
             var bodyQuery = GetEntityQuery<PhysicsComponent>();
+            var weatherIgnoreQuery = GetEntityQuery<IgnoreWeatherComponent>();
             var visited = new HashSet<Vector2i>();
 
             while (frontier.TryDequeue(out var node))
@@ -89,7 +90,7 @@ public sealed class WeatherSystem : SharedWeatherSystem
                 if (!visited.Add(node.GridIndices))
                     continue;
 
-                if (!CanWeatherAffect(grid, node, bodyQuery))
+                if (!CanWeatherAffect(grid, node, weatherIgnoreQuery, bodyQuery))
                 {
                     // Add neighbors
                     // TODO: Ideally we pick some deterministically random direction and use that

Content.Shared/Weather/IgnoreWeatherComponent.cs

@@ -0,0 +1,12 @@
+using Robust.Shared.GameStates;
+
+namespace Content.Shared.Weather;
+
+/// <summary>
+/// This entity will be ignored for considering weather on a tile
+/// </summary>
+[RegisterComponent, NetworkedComponent]
+public sealed class IgnoreWeatherComponent : Component
+{
+
+}

Content.Shared/Weather/SharedWeatherSystem.cs

@@ -36,7 +36,11 @@ public abstract class SharedWeatherSystem : EntitySystem
         }
     }
 
-    public bool CanWeatherAffect(MapGridComponent grid, TileRef tileRef, EntityQuery<PhysicsComponent> bodyQuery)
+    public bool CanWeatherAffect(
+        MapGridComponent grid,
+        TileRef tileRef,
+        EntityQuery<IgnoreWeatherComponent> weatherIgnoreQuery,
+        EntityQuery<PhysicsComponent> bodyQuery)
     {
         if (tileRef.Tile.IsEmpty)
             return true;
@@ -50,7 +54,10 @@ public abstract class SharedWeatherSystem : EntitySystem
 
         while (anchoredEnts.MoveNext(out var ent))
         {
-            if (bodyQuery.TryGetComponent(ent, out var body) && body.CanCollide)
+            if (!weatherIgnoreQuery.HasComponent(ent.Value) &&
+                bodyQuery.TryGetComponent(ent, out var body) &&
+                body.Hard &&
+                body.CanCollide)
             {
                 return false;
             }

Resources/Prototypes/Entities/Objects/Decoration/flora.yml

@@ -37,6 +37,7 @@
   description: Yep, it's a tree.
   abstract: true
   components:
+  - type: IgnoreWeather
   - type: SpriteFade
   - type: Clickable
   - type: Sprite