fix: Removes redundant conditional check in house demolish gump (#2004)

modernuo · Dec 4, 2024 · 51d15d2 · 51d15d2
1 parent ba2a412
commit 51d15d2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/Projects/UOContent/Engines/Harvest/Core/HarvestSystem.cs b/Projects/UOContent/Engines/Harvest/Core/HarvestSystem.cs
@@ -212,8 +212,8 @@ public virtual void FinishHarvesting(Mobile from, Item tool, HarvestDefinition d
                         {
                             var bonusItem = Construct(bonus.Type, from);
 
-                            if (Give(from, bonusItem, true)
-                            ) // Bonuses always allow placing at feet, even if pack is full irregrdless of def
+                            // Bonuses always allow placing at feet, even if pack is full irregrdless of def
+                            if (Give(from, bonusItem, true))
                             {
                                 bonus.SendSuccessTo(from);
                             }

diff --git a/Projects/UOContent/Gumps/Houses/ConfirmDemolishHouseGump.cs b/Projects/UOContent/Gumps/Houses/ConfirmDemolishHouseGump.cs
@@ -124,7 +124,7 @@ public override void OnResponse(NetState state, in RelayInfo info)
 
             if (!from.BankBox.TryDropItem(from, deed, false))
             {
-                deed?.Delete();
+                deed.Delete();
                 from.SendLocalizedMessage(500390); // Your bank box is full.
                 return;
             }