Core/Items: Corrected logic check when placing item on auction

Closes #1017

2 files changed, 3 insertions, 3 deletions

diff --git a/src/server/game/Entities/Item/Item.cpp b/src/server/game/Entities/Item/Item.cpp
index 69ed9f625ff..c483f72a58d 100755
--- a/src/server/game/Entities/Item/Item.cpp
+++ b/src/server/game/Entities/Item/Item.cpp
@@ -285,8 +285,8 @@ bool Item::Create(uint32 guidlow, uint32 itemid, Player const* owner)
 // Returns false if Item is not a bag OR it is an empty bag.
 bool Item::IsNotEmptyBag() const
 {
-    if (const Bag* pBag = ToBag())
-        return !pBag->IsEmpty();
+    if (Bag const* bag = ToBag())
+        return !bag->IsEmpty();
     return false;
 }
 
diff --git a/src/server/game/Server/Protocol/Handlers/AuctionHouseHandler.cpp b/src/server/game/Server/Protocol/Handlers/AuctionHouseHandler.cpp
index 454ecdeddf4..099309b9d12 100755
--- a/src/server/game/Server/Protocol/Handlers/AuctionHouseHandler.cpp
+++ b/src/server/game/Server/Protocol/Handlers/AuctionHouseHandler.cpp
@@ -188,7 +188,7 @@ void WorldSession::HandleAuctionSellItem(WorldPacket & recv_data)
         return;
     }
 
-    if (!it->IsNotEmptyBag())
+    if (it->IsNotEmptyBag())
     {
         SendAuctionCommandResult(0, AUCTION_SELL_ITEM, AUCTION_INTERNAL_ERROR);
         return;