Code style (game + scripts only):

"if(" --> "if ("

--HG--
branch : trunk

1 files changed, 114 insertions, 114 deletions

diff --git a/src/game/ItemHandler.cpp b/src/game/ItemHandler.cpp
index 8d7b06227c3..d323f84d016 100644
--- a/src/game/ItemHandler.cpp
+++ b/src/game/ItemHandler.cpp
@@ -41,19 +41,19 @@ void WorldSession::HandleSplitItemOpcode( WorldPacket & recv_data )
     uint16 src = ( (srcbag << 8) | srcslot );
     uint16 dst = ( (dstbag << 8) | dstslot );
 
-    if(src==dst)
+    if (src==dst)
         return;
 
     if (count==0)
         return;                                             //check count - if zero it's fake packet
 
-    if(!_player->IsValidPos(srcbag,srcslot))
+    if (!_player->IsValidPos(srcbag,srcslot))
     {
         _player->SendEquipError( EQUIP_ERR_ITEM_NOT_FOUND, NULL, NULL );
         return;
     }
 
-    if(!_player->IsValidPos(dstbag,dstslot))
+    if (!_player->IsValidPos(dstbag,dstslot))
     {
         _player->SendEquipError( EQUIP_ERR_ITEM_DOESNT_GO_TO_SLOT, NULL, NULL );
         return;
@@ -71,16 +71,16 @@ void WorldSession::HandleSwapInvItemOpcode( WorldPacket & recv_data )
     //sLog.outDebug("STORAGE: receive srcslot = %u, dstslot = %u", srcslot, dstslot);
 
     // prevent attempt swap same item to current position generated by client at special checting sequence
-    if(srcslot==dstslot)
+    if (srcslot==dstslot)
         return;
 
-    if(!_player->IsValidPos(INVENTORY_SLOT_BAG_0,srcslot))
+    if (!_player->IsValidPos(INVENTORY_SLOT_BAG_0,srcslot))
     {
         _player->SendEquipError( EQUIP_ERR_ITEM_NOT_FOUND, NULL, NULL );
         return;
     }
 
-    if(!_player->IsValidPos(INVENTORY_SLOT_BAG_0,dstslot))
+    if (!_player->IsValidPos(INVENTORY_SLOT_BAG_0,dstslot))
     {
         _player->SendEquipError( EQUIP_ERR_ITEM_DOESNT_GO_TO_SLOT, NULL, NULL );
         return;
@@ -99,13 +99,13 @@ void WorldSession::HandleAutoEquipItemSlotOpcode( WorldPacket & recv_data )
     recv_data >> itemguid >> dstslot;
 
     // cheating attempt, client should never send opcode in that case
-    if(!Player::IsEquipmentPos(INVENTORY_SLOT_BAG_0, dstslot))
+    if (!Player::IsEquipmentPos(INVENTORY_SLOT_BAG_0, dstslot))
         return;
 
     Item* item = _player->GetItemByGuid(itemguid);
     uint16 dstpos = dstslot | (INVENTORY_SLOT_BAG_0 << 8);
 
-    if(!item || item->GetPos() == dstpos)
+    if (!item || item->GetPos() == dstpos)
         return;
 
     _player->SwapItem(item->GetPos(), dstpos);
@@ -123,16 +123,16 @@ void WorldSession::HandleSwapItem( WorldPacket & recv_data )
     uint16 dst = ( (dstbag << 8) | dstslot );
 
     // prevent attempt swap same item to current position generated by client at special checting sequence
-    if(src==dst)
+    if (src==dst)
         return;
 
-    if(!_player->IsValidPos(srcbag,srcslot))
+    if (!_player->IsValidPos(srcbag,srcslot))
     {
         _player->SendEquipError( EQUIP_ERR_ITEM_NOT_FOUND, NULL, NULL );
         return;
     }
 
-    if(!_player->IsValidPos(dstbag,dstslot))
+    if (!_player->IsValidPos(dstbag,dstslot))
     {
         _player->SendEquipError( EQUIP_ERR_ITEM_DOESNT_GO_TO_SLOT, NULL, NULL );
         return;
@@ -150,10 +150,10 @@ void WorldSession::HandleAutoEquipItemOpcode( WorldPacket & recv_data )
     //sLog.outDebug("STORAGE: receive srcbag = %u, srcslot = %u", srcbag, srcslot);
 
     Item *pSrcItem  = _player->GetItemByPos( srcbag, srcslot );
-    if( !pSrcItem )
+    if ( !pSrcItem )
         return;                                             // only at cheat
 
-    if(pSrcItem->m_lootGenerated)                           // prevent swap looting item
+    if (pSrcItem->m_lootGenerated)                           // prevent swap looting item
     {
         //best error message found for attempting to swap while looting
         _player->SendEquipError( EQUIP_ERR_CANT_DO_RIGHT_NOW, pSrcItem, NULL );
@@ -162,18 +162,18 @@ void WorldSession::HandleAutoEquipItemOpcode( WorldPacket & recv_data )
 
     uint16 dest;
     uint8 msg = _player->CanEquipItem( NULL_SLOT, dest, pSrcItem, !pSrcItem->IsBag() );
-    if( msg != EQUIP_ERR_OK )
+    if ( msg != EQUIP_ERR_OK )
     {
         _player->SendEquipError( msg, pSrcItem, NULL );
         return;
     }
 
     uint16 src = pSrcItem->GetPos();
-    if(dest==src)                                           // prevent equip in same slot, only at cheat
+    if (dest==src)                                           // prevent equip in same slot, only at cheat
         return;
 
     Item *pDstItem = _player->GetItemByPos( dest );
-    if( !pDstItem )                                         // empty slot, simple case
+    if ( !pDstItem )                                         // empty slot, simple case
     {
         _player->RemoveItem( srcbag, srcslot, true );
         _player->EquipItem( dest, pSrcItem, true );
@@ -185,7 +185,7 @@ void WorldSession::HandleAutoEquipItemOpcode( WorldPacket & recv_data )
         uint8 dstslot = pDstItem->GetSlot();
 
         msg = _player->CanUnequipItem( dest, !pSrcItem->IsBag() );
-        if( msg != EQUIP_ERR_OK )
+        if ( msg != EQUIP_ERR_OK )
         {
             _player->SendEquipError( msg, pDstItem, NULL );
             return;
@@ -194,30 +194,30 @@ void WorldSession::HandleAutoEquipItemOpcode( WorldPacket & recv_data )
         // check dest->src move possibility
         ItemPosCountVec sSrc;
         uint16 eSrc = 0;
-        if( _player->IsInventoryPos( src ) )
+        if ( _player->IsInventoryPos( src ) )
         {
             msg = _player->CanStoreItem( srcbag, srcslot, sSrc, pDstItem, true );
-            if( msg != EQUIP_ERR_OK )
+            if ( msg != EQUIP_ERR_OK )
                 msg = _player->CanStoreItem( srcbag, NULL_SLOT, sSrc, pDstItem, true );
-            if( msg != EQUIP_ERR_OK )
+            if ( msg != EQUIP_ERR_OK )
                 msg = _player->CanStoreItem( NULL_BAG, NULL_SLOT, sSrc, pDstItem, true );
         }
-        else if( _player->IsBankPos( src ) )
+        else if ( _player->IsBankPos( src ) )
         {
             msg = _player->CanBankItem( srcbag, srcslot, sSrc, pDstItem, true );
-            if( msg != EQUIP_ERR_OK )
+            if ( msg != EQUIP_ERR_OK )
                 msg = _player->CanBankItem( srcbag, NULL_SLOT, sSrc, pDstItem, true );
-            if( msg != EQUIP_ERR_OK )
+            if ( msg != EQUIP_ERR_OK )
                 msg = _player->CanBankItem( NULL_BAG, NULL_SLOT, sSrc, pDstItem, true );
         }
-        else if( _player->IsEquipmentPos( src ) )
+        else if ( _player->IsEquipmentPos( src ) )
         {
             msg = _player->CanEquipItem( srcslot, eSrc, pDstItem, true);
-            if( msg == EQUIP_ERR_OK )
+            if ( msg == EQUIP_ERR_OK )
                 msg = _player->CanUnequipItem( eSrc, true);
         }
 
-        if( msg != EQUIP_ERR_OK )
+        if ( msg != EQUIP_ERR_OK )
         {
             _player->SendEquipError( msg, pDstItem, pSrcItem );
             return;
@@ -231,11 +231,11 @@ void WorldSession::HandleAutoEquipItemOpcode( WorldPacket & recv_data )
         _player->EquipItem(dest, pSrcItem, true);
 
         // add to src
-        if( _player->IsInventoryPos( src ) )
+        if ( _player->IsInventoryPos( src ) )
             _player->StoreItem(sSrc, pDstItem, true);
-        else if( _player->IsBankPos( src ) )
+        else if ( _player->IsBankPos( src ) )
             _player->BankItem(sSrc, pDstItem, true);
-        else if( _player->IsEquipmentPos( src ) )
+        else if ( _player->IsEquipmentPos( src ) )
             _player->EquipItem(eSrc, pDstItem, true);
 
         _player->AutoUnequipOffhandIfNeed();
@@ -253,10 +253,10 @@ void WorldSession::HandleDestroyItemOpcode( WorldPacket & recv_data )
     uint16 pos = (bag << 8) | slot;
 
     // prevent drop unequipable items (in combat, for example) and non-empty bags
-    if(_player->IsEquipmentPos(pos) || _player->IsBagPos(pos))
+    if (_player->IsEquipmentPos(pos) || _player->IsBagPos(pos))
     {
         uint8 msg = _player->CanUnequipItem( pos, false );
-        if( msg != EQUIP_ERR_OK )
+        if ( msg != EQUIP_ERR_OK )
         {
             _player->SendEquipError( msg, _player->GetItemByPos(pos), NULL );
             return;
@@ -264,13 +264,13 @@ void WorldSession::HandleDestroyItemOpcode( WorldPacket & recv_data )
     }
 
     Item *pItem  = _player->GetItemByPos( bag, slot );
-    if(!pItem)
+    if (!pItem)
     {
         _player->SendEquipError( EQUIP_ERR_ITEM_NOT_FOUND, NULL, NULL );
         return;
     }
 
-    if(count)
+    if (count)
     {
         uint32 i_count = count;
         _player->DestroyItemCount( pItem, i_count, true );
@@ -289,7 +289,7 @@ void WorldSession::HandleItemQuerySingleOpcode( WorldPacket & recv_data )
     sLog.outDetail("STORAGE: Item Query = %u", item);
 
     ItemPrototype const *pProto = objmgr.GetItemPrototype( item );
-    if( pProto )
+    if ( pProto )
     {
         std::string Name        = pProto->Name1;
         std::string Description = pProto->Description;
@@ -370,7 +370,7 @@ void WorldSession::HandleItemQuerySingleOpcode( WorldPacket & recv_data )
             // send DBC data for cooldowns in same way as it used in Spell::SendSpellCooldown
             // use `item_template` or if not set then only use spell cooldowns
             SpellEntry const* spell = sSpellStore.LookupEntry(pProto->Spells[s].SpellId);
-            if(spell)
+            if (spell)
             {
                 bool db_data = pProto->Spells[s].SpellCooldown >= 0 || pProto->Spells[s].SpellCategoryCooldown >= 0;
 
@@ -378,7 +378,7 @@ void WorldSession::HandleItemQuerySingleOpcode( WorldPacket & recv_data )
                 data << pProto->Spells[s].SpellTrigger;
                 data << uint32(-abs(pProto->Spells[s].SpellCharges));
 
-                if(db_data)
+                if (db_data)
                 {
                     data << uint32(pProto->Spells[s].SpellCooldown);
                     data << uint32(pProto->Spells[s].SpellCategory);
@@ -452,12 +452,12 @@ void WorldSession::HandleReadItem( WorldPacket & recv_data )
     //sLog.outDetail("STORAGE: Read bag = %u, slot = %u", bag, slot);
     Item *pItem = _player->GetItemByPos( bag, slot );
 
-    if( pItem && pItem->GetProto()->PageText )
+    if ( pItem && pItem->GetProto()->PageText )
     {
         WorldPacket data;
 
         uint8 msg = _player->CanUseItem( pItem );
-        if( msg == EQUIP_ERR_OK )
+        if ( msg == EQUIP_ERR_OK )
         {
             data.Initialize (SMSG_READ_ITEM_OK, 8);
             sLog.outDetail("STORAGE: Item page sent");
@@ -496,7 +496,7 @@ void WorldSession::HandleSellItemOpcode( WorldPacket & recv_data )
 
     recv_data >> vendorguid >> itemguid >> count;
 
-    if(!itemguid)
+    if (!itemguid)
         return;
 
     Creature *pCreature = GetPlayer()->GetNPCIfCanInteractWith(vendorguid,UNIT_NPC_FLAG_VENDOR);
@@ -508,42 +508,42 @@ void WorldSession::HandleSellItemOpcode( WorldPacket & recv_data )
     }
 
     // remove fake death
-    if(GetPlayer()->hasUnitState(UNIT_STAT_DIED))
+    if (GetPlayer()->hasUnitState(UNIT_STAT_DIED))
         GetPlayer()->RemoveAurasByType(SPELL_AURA_FEIGN_DEATH);
 
     Item *pItem = _player->GetItemByGuid( itemguid );
-    if( pItem )
+    if ( pItem )
     {
         // prevent sell not owner item
-        if(_player->GetGUID()!=pItem->GetOwnerGUID())
+        if (_player->GetGUID()!=pItem->GetOwnerGUID())
         {
             _player->SendSellError( SELL_ERR_CANT_SELL_ITEM, pCreature, itemguid, 0);
             return;
         }
 
         // prevent sell non empty bag by drag-and-drop at vendor's item list
-        if(pItem->IsBag() && !((Bag*)pItem)->IsEmpty())
+        if (pItem->IsBag() && !((Bag*)pItem)->IsEmpty())
         {
             _player->SendSellError( SELL_ERR_CANT_SELL_ITEM, pCreature, itemguid, 0);
             return;
         }
 
         // prevent sell currently looted item
-        if(_player->GetLootGUID()==pItem->GetGUID())
+        if (_player->GetLootGUID()==pItem->GetGUID())
         {
             _player->SendSellError( SELL_ERR_CANT_SELL_ITEM, pCreature, itemguid, 0);
             return;
         }
 
         // special case at auto sell (sell all)
-        if(count==0)
+        if (count==0)
         {
             count = pItem->GetCount();
         }
         else
         {
             // prevent sell more items that exist in stack (possible only not from client)
-            if(count > pItem->GetCount())
+            if (count > pItem->GetCount())
             {
                 _player->SendSellError( SELL_ERR_CANT_SELL_ITEM, pCreature, itemguid, 0);
                 return;
@@ -551,11 +551,11 @@ void WorldSession::HandleSellItemOpcode( WorldPacket & recv_data )
         }
 
         ItemPrototype const *pProto = pItem->GetProto();
-        if( pProto )
+        if ( pProto )
         {
-            if( pProto->SellPrice > 0 )
+            if ( pProto->SellPrice > 0 )
             {
-                if(count < pItem->GetCount())               // need split items
+                if (count < pItem->GetCount())               // need split items
                 {
                     Item *pNewItem = pItem->CloneItem( count, _player );
                     if (!pNewItem)
@@ -567,12 +567,12 @@ void WorldSession::HandleSellItemOpcode( WorldPacket & recv_data )
 
                     pItem->SetCount( pItem->GetCount() - count );
                     _player->ItemRemovedQuestCheck( pItem->GetEntry(), count );
-                    if( _player->IsInWorld() )
+                    if ( _player->IsInWorld() )
                         pItem->SendUpdateToPlayer( _player );
                     pItem->SetState(ITEM_CHANGED, _player);
 
                     _player->AddItemToBuyBackSlot( pNewItem );
-                    if( _player->IsInWorld() )
+                    if ( _player->IsInWorld() )
                         pNewItem->SendUpdateToPlayer( _player );
                 }
                 else
@@ -611,14 +611,14 @@ void WorldSession::HandleBuybackItem(WorldPacket & recv_data)
     }
 
     // remove fake death
-    if(GetPlayer()->hasUnitState(UNIT_STAT_DIED))
+    if (GetPlayer()->hasUnitState(UNIT_STAT_DIED))
         GetPlayer()->RemoveAurasByType(SPELL_AURA_FEIGN_DEATH);
 
     Item *pItem = _player->GetItemFromBuyBackSlot( slot );
-    if( pItem )
+    if ( pItem )
     {
         uint32 price = _player->GetUInt32Value( PLAYER_FIELD_BUYBACK_PRICE_1 + slot - BUYBACK_SLOT_START );
-        if( _player->GetMoney() < price )
+        if ( _player->GetMoney() < price )
         {
             _player->SendBuyError( BUY_ERR_NOT_ENOUGHT_MONEY, pCreature, pItem->GetEntry(), 0);
             return;
@@ -626,7 +626,7 @@ void WorldSession::HandleBuybackItem(WorldPacket & recv_data)
 
         ItemPosCountVec dest;
         uint8 msg = _player->CanStoreItem( NULL_BAG, NULL_SLOT, dest, pItem, false );
-        if( msg == EQUIP_ERR_OK )
+        if ( msg == EQUIP_ERR_OK )
         {
             _player->ModifyMoney( -(int32)price );
             _player->RemoveItemFromBuyBackSlot( slot, false );
@@ -695,7 +695,7 @@ void WorldSession::HandleListInventoryOpcode( WorldPacket & recv_data )
 
     recv_data >> guid;
 
-    if(!GetPlayer()->isAlive())
+    if (!GetPlayer()->isAlive())
         return;
 
     sLog.outDebug(  "WORLD: Recvd CMSG_LIST_INVENTORY" );
@@ -716,14 +716,14 @@ void WorldSession::SendListInventory( uint64 vendorguid )
     }
 
     // remove fake death
-    if(GetPlayer()->hasUnitState(UNIT_STAT_DIED))
+    if (GetPlayer()->hasUnitState(UNIT_STAT_DIED))
         GetPlayer()->RemoveAurasByType(SPELL_AURA_FEIGN_DEATH);
 
     // Stop the npc if moving
     pCreature->StopMoving();
 
     VendorItemData const* vItems = pCreature->GetVendorItems();
-    if(!vItems)
+    if (!vItems)
     {
         _player->SendSellError( SELL_ERR_CANT_FIND_VENDOR, NULL, 0, 0);
         return;
@@ -740,11 +740,11 @@ void WorldSession::SendListInventory( uint64 vendorguid )
 
     for (int i = 0; i < numitems; ++i )
     {
-        if(VendorItem const* crItem = vItems->GetItem(i))
+        if (VendorItem const* crItem = vItems->GetItem(i))
         {
-            if(ItemPrototype const *pProto = objmgr.GetItemPrototype(crItem->item))
+            if (ItemPrototype const *pProto = objmgr.GetItemPrototype(crItem->item))
             {
-                if((pProto->AllowableClass & _player->getClassMask()) == 0 && pProto->Bonding == BIND_WHEN_PICKED_UP && !_player->isGameMaster())
+                if ((pProto->AllowableClass & _player->getClassMask()) == 0 && pProto->Bonding == BIND_WHEN_PICKED_UP && !_player->isGameMaster())
                     continue;
                 // Only display items in vendor lists for the team the
                 // player is on. If GM on, display all items.
@@ -786,10 +786,10 @@ void WorldSession::HandleAutoStoreBagItemOpcode( WorldPacket & recv_data )
     //sLog.outDebug("STORAGE: receive srcbag = %u, srcslot = %u, dstbag = %u", srcbag, srcslot, dstbag);
 
     Item *pItem = _player->GetItemByPos( srcbag, srcslot );
-    if( !pItem )
+    if ( !pItem )
         return;
 
-    if(!_player->IsValidPos(dstbag,NULL_SLOT))
+    if (!_player->IsValidPos(dstbag,NULL_SLOT))
     {
         _player->SendEquipError( EQUIP_ERR_ITEM_DOESNT_GO_TO_SLOT, NULL, NULL );
         return;
@@ -798,10 +798,10 @@ void WorldSession::HandleAutoStoreBagItemOpcode( WorldPacket & recv_data )
     uint16 src = pItem->GetPos();
 
     // check unequip potability for equipped items and bank bags
-    if(_player->IsEquipmentPos ( src ) || _player->IsBagPos ( src ))
+    if (_player->IsEquipmentPos ( src ) || _player->IsBagPos ( src ))
     {
         uint8 msg = _player->CanUnequipItem( src, !_player->IsBagPos ( src ));
-        if(msg != EQUIP_ERR_OK)
+        if (msg != EQUIP_ERR_OK)
         {
             _player->SendEquipError( msg, pItem, NULL );
             return;
@@ -810,14 +810,14 @@ void WorldSession::HandleAutoStoreBagItemOpcode( WorldPacket & recv_data )
 
     ItemPosCountVec dest;
     uint8 msg = _player->CanStoreItem( dstbag, NULL_SLOT, dest, pItem, false );
-    if( msg != EQUIP_ERR_OK )
+    if ( msg != EQUIP_ERR_OK )
     {
         _player->SendEquipError( msg, pItem, NULL );
         return;
     }
 
     // no-op: placed in same slot
-    if(dest.size()==1 && dest[0].pos==src)
+    if (dest.size()==1 && dest[0].pos==src)
     {
         // just remove grey item state
         _player->SendEquipError( EQUIP_ERR_NONE, pItem, NULL );
@@ -838,7 +838,7 @@ void WorldSession::HandleBuyBankSlotOpcode(WorldPacket& recvPacket)
     // cheating protection
     /* not critical if "cheated", and check skip allow by slots in bank windows open by .bank command.
     Creature *pCreature = GetPlayer()->GetNPCIfCanInteractWith(guid, UNIT_NPC_FLAG_BANKER);
-    if(!pCreature)
+    if (!pCreature)
     {
         sLog.outDebug( "WORLD: HandleBuyBankSlotOpcode - Unit (GUID: %u) not found or you can't interact with him.", uint32(GUID_LOPART(guid)) );
         return;
@@ -854,7 +854,7 @@ void WorldSession::HandleBuyBankSlotOpcode(WorldPacket& recvPacket)
 
     BankBagSlotPricesEntry const* slotEntry = sBankBagSlotPricesStore.LookupEntry(slot);
 
-    if(!slotEntry)
+    if (!slotEntry)
         return;
 
     uint32 price = slotEntry->price;
@@ -877,12 +877,12 @@ void WorldSession::HandleAutoBankItemOpcode(WorldPacket& recvPacket)
     sLog.outDebug("STORAGE: receive srcbag = %u, srcslot = %u", srcbag, srcslot);
 
     Item *pItem = _player->GetItemByPos( srcbag, srcslot );
-    if( !pItem )
+    if ( !pItem )
         return;
 
     ItemPosCountVec dest;
     uint8 msg = _player->CanBankItem( NULL_BAG, NULL_SLOT, dest, pItem, false );
-    if( msg != EQUIP_ERR_OK )
+    if ( msg != EQUIP_ERR_OK )
     {
         _player->SendEquipError( msg, pItem, NULL );
         return;
@@ -901,14 +901,14 @@ void WorldSession::HandleAutoStoreBankItemOpcode(WorldPacket& recvPacket)
     sLog.outDebug("STORAGE: receive srcbag = %u, srcslot = %u", srcbag, srcslot);
 
     Item *pItem = _player->GetItemByPos( srcbag, srcslot );
-    if( !pItem )
+    if ( !pItem )
         return;
 
-    if(_player->IsBankPos(srcbag, srcslot))                 // moving from bank to inventory
+    if (_player->IsBankPos(srcbag, srcslot))                 // moving from bank to inventory
     {
         ItemPosCountVec dest;
         uint8 msg = _player->CanStoreItem( NULL_BAG, NULL_SLOT, dest, pItem, false );
-        if( msg != EQUIP_ERR_OK )
+        if ( msg != EQUIP_ERR_OK )
         {
             _player->SendEquipError( msg, pItem, NULL );
             return;
@@ -921,7 +921,7 @@ void WorldSession::HandleAutoStoreBankItemOpcode(WorldPacket& recvPacket)
     {
         ItemPosCountVec dest;
         uint8 msg = _player->CanBankItem( NULL_BAG, NULL_SLOT, dest, pItem, false );
-        if( msg != EQUIP_ERR_OK )
+        if ( msg != EQUIP_ERR_OK )
         {
             _player->SendEquipError( msg, pItem, NULL );
             return;
@@ -934,7 +934,7 @@ void WorldSession::HandleAutoStoreBankItemOpcode(WorldPacket& recvPacket)
 
 void WorldSession::HandleSetAmmoOpcode(WorldPacket & recv_data)
 {
-    if(!GetPlayer()->isAlive())
+    if (!GetPlayer()->isAlive())
     {
         GetPlayer()->SendEquipError( EQUIP_ERR_YOU_ARE_DEAD, NULL, NULL );
         return;
@@ -945,7 +945,7 @@ void WorldSession::HandleSetAmmoOpcode(WorldPacket & recv_data)
 
     recv_data >> item;
 
-    if(!item)
+    if (!item)
         GetPlayer()->RemoveAmmo();
     else
         GetPlayer()->SetAmmo(item);
@@ -981,7 +981,7 @@ void WorldSession::HandleItemNameQueryOpcode(WorldPacket & recv_data)
 
     sLog.outDebug("WORLD: CMSG_ITEM_NAME_QUERY %u", itemid);
     ItemPrototype const *pProto = objmgr.GetItemPrototype( itemid );
-    if( pProto )
+    if ( pProto )
     {
         std::string Name;
         Name = pProto->Name1;
@@ -1009,7 +1009,7 @@ void WorldSession::HandleItemNameQueryOpcode(WorldPacket & recv_data)
 /*    else
     {
         // listed in dbc or not expected to exist unknown item
-        if(sItemStore.LookupEntry(itemid))
+        if (sItemStore.LookupEntry(itemid))
             sLog.outErrorDb("WORLD: CMSG_ITEM_NAME_QUERY for item %u failed (item listed in Item.dbc but not exist in DB)", itemid);
         else
             sLog.outError("WORLD: CMSG_ITEM_NAME_QUERY for item %u failed (unknown item, not listed in Item.dbc)", itemid);
@@ -1029,13 +1029,13 @@ void WorldSession::HandleWrapItemOpcode(WorldPacket& recv_data)
     sLog.outDebug("WRAP: receive gift_bag = %u, gift_slot = %u, item_bag = %u, item_slot = %u", gift_bag, gift_slot, item_bag, item_slot);
 
     Item *gift = _player->GetItemByPos( gift_bag, gift_slot );
-    if(!gift)
+    if (!gift)
     {
         _player->SendEquipError( EQUIP_ERR_ITEM_NOT_FOUND, gift, NULL );
         return;
     }
 
-    if(!gift->HasFlag(ITEM_FIELD_FLAGS, ITEM_FLAGS_WRAPPER))// cheating: non-wrapper wrapper
+    if (!gift->HasFlag(ITEM_FIELD_FLAGS, ITEM_FLAGS_WRAPPER))// cheating: non-wrapper wrapper
     {
         _player->SendEquipError( EQUIP_ERR_ITEM_NOT_FOUND, gift, NULL );
         return;
@@ -1043,50 +1043,50 @@ void WorldSession::HandleWrapItemOpcode(WorldPacket& recv_data)
 
     Item *item = _player->GetItemByPos( item_bag, item_slot );
 
-    if( !item )
+    if ( !item )
     {
         _player->SendEquipError( EQUIP_ERR_ITEM_NOT_FOUND, item, NULL );
         return;
     }
 
-    if(item==gift)                                          // not possable with pacjket from real client
+    if (item==gift)                                          // not possable with pacjket from real client
     {
         _player->SendEquipError( EQUIP_ERR_WRAPPED_CANT_BE_WRAPPED, item, NULL );
         return;
     }
 
-    if(item->IsEquipped())
+    if (item->IsEquipped())
     {
         _player->SendEquipError( EQUIP_ERR_EQUIPPED_CANT_BE_WRAPPED, item, NULL );
         return;
     }
 
-    if(item->GetUInt64Value(ITEM_FIELD_GIFTCREATOR))        // HasFlag(ITEM_FIELD_FLAGS, ITEM_FLAGS_WRAPPED);
+    if (item->GetUInt64Value(ITEM_FIELD_GIFTCREATOR))        // HasFlag(ITEM_FIELD_FLAGS, ITEM_FLAGS_WRAPPED);
     {
         _player->SendEquipError( EQUIP_ERR_WRAPPED_CANT_BE_WRAPPED, item, NULL );
         return;
     }
 
-    if(item->IsBag())
+    if (item->IsBag())
     {
         _player->SendEquipError( EQUIP_ERR_BAGS_CANT_BE_WRAPPED, item, NULL );
         return;
     }
 
-    if(item->IsSoulBound())
+    if (item->IsSoulBound())
     {
         _player->SendEquipError( EQUIP_ERR_BOUND_CANT_BE_WRAPPED, item, NULL );
         return;
     }
 
-    if(item->GetMaxStackCount() != 1)
+    if (item->GetMaxStackCount() != 1)
     {
         _player->SendEquipError( EQUIP_ERR_STACKABLE_CANT_BE_WRAPPED, item, NULL );
         return;
     }
 
     // maybe not correct check  (it is better than nothing)
-    if(item->GetProto()->MaxCount>0)
+    if (item->GetProto()->MaxCount>0)
     {
         _player->SendEquipError( EQUIP_ERR_UNIQUE_CANT_BE_WRAPPED, item, NULL );
         return;
@@ -1109,7 +1109,7 @@ void WorldSession::HandleWrapItemOpcode(WorldPacket& recv_data)
     item->SetUInt32Value(ITEM_FIELD_FLAGS, ITEM_FLAGS_WRAPPED);
     item->SetState(ITEM_CHANGED, _player);
 
-    if(item->GetState()==ITEM_NEW)                          // save new item, to have alway for `character_gifts` record in `item_instance`
+    if (item->GetState()==ITEM_NEW)                          // save new item, to have alway for `character_gifts` record in `item_instance`
     {
         // after save it will be impossible to remove the item from the queue
         item->RemoveFromUpdateQueueOf(_player);
@@ -1129,7 +1129,7 @@ void WorldSession::HandleSocketOpcode(WorldPacket& recv_data)
     uint64 gem_guids[MAX_GEM_SOCKETS];
 
     recv_data >> item_guid;
-    if(!item_guid)
+    if (!item_guid)
         return;
 
     for (int i = 0; i < MAX_GEM_SOCKETS; ++i)
@@ -1141,11 +1141,11 @@ void WorldSession::HandleSocketOpcode(WorldPacket& recv_data)
         return;
 
     Item *itemTarget = _player->GetItemByGuid(item_guid);
-    if(!itemTarget)                                         //missing item to socket
+    if (!itemTarget)                                         //missing item to socket
         return;
 
     ItemPrototype const* itemProto = itemTarget->GetProto();
-    if(!itemProto)
+    if (!itemProto)
         return;
 
     //this slot is excepted when applying / removing meta gem bonus
@@ -1168,11 +1168,11 @@ void WorldSession::HandleSocketOpcode(WorldPacket& recv_data)
         if (!itemProto->Socket[i].Color)
         {
             // no prismatic socket
-            if(!itemTarget->GetEnchantmentId(PRISMATIC_ENCHANTMENT_SLOT))
+            if (!itemTarget->GetEnchantmentId(PRISMATIC_ENCHANTMENT_SLOT))
                 return;
 
             // not first not-colored (not normaly used) socket
-            if(i!=0 && !itemProto->Socket[i-1].Color && (i+1 >= MAX_GEM_SOCKETS || itemProto->Socket[i+1].Color))
+            if (i!=0 && !itemProto->Socket[i-1].Color && (i+1 >= MAX_GEM_SOCKETS || itemProto->Socket[i+1].Color))
                 return;
 
             // ok, this is first not colored socket for item with prismatic socket
@@ -1198,7 +1198,7 @@ void WorldSession::HandleSocketOpcode(WorldPacket& recv_data)
     // check unique-equipped conditions
     for (int i = 0; i < MAX_GEM_SOCKETS; ++i)
     {
-        if(!Gems[i])
+        if (!Gems[i])
             continue;
 
         // continue check for case when attempt add 2 similar unique equipped gems in one item.
@@ -1209,7 +1209,7 @@ void WorldSession::HandleSocketOpcode(WorldPacket& recv_data)
         {
             for (int j = 0; j < MAX_GEM_SOCKETS; ++j)
             {
-                if(i==j)                                    // skip self
+                if (i==j)                                    // skip self
                     continue;
 
                 if (Gems[j])
@@ -1220,9 +1220,9 @@ void WorldSession::HandleSocketOpcode(WorldPacket& recv_data)
                         return;
                     }
                 }
-                else if(OldEnchants[j])
+                else if (OldEnchants[j])
                 {
-                    if(SpellItemEnchantmentEntry const* enchantEntry = sSpellItemEnchantmentStore.LookupEntry(OldEnchants[j]))
+                    if (SpellItemEnchantmentEntry const* enchantEntry = sSpellItemEnchantmentStore.LookupEntry(OldEnchants[j]))
                     {
                         if (iGemProto->ItemId == enchantEntry->GemID)
                         {
@@ -1239,7 +1239,7 @@ void WorldSession::HandleSocketOpcode(WorldPacket& recv_data)
         int32 limit_newcount = 0;
         if (iGemProto->ItemLimitCategory)
         {
-            if(ItemLimitCategoryEntry const* limitEntry = sItemLimitCategoryStore.LookupEntry(iGemProto->ItemLimitCategory))
+            if (ItemLimitCategoryEntry const* limitEntry = sItemLimitCategoryStore.LookupEntry(iGemProto->ItemLimitCategory))
             {
                 for (int j = 0; j < MAX_GEM_SOCKETS; ++j)
                 {
@@ -1248,8 +1248,8 @@ void WorldSession::HandleSocketOpcode(WorldPacket& recv_data)
                         // destroyed gem
                         if (OldEnchants[j])
                         {
-                            if(SpellItemEnchantmentEntry const* enchantEntry = sSpellItemEnchantmentStore.LookupEntry(OldEnchants[j]))
-                                if(ItemPrototype const* jProto = ObjectMgr::GetItemPrototype(enchantEntry->GemID))
+                            if (SpellItemEnchantmentEntry const* enchantEntry = sSpellItemEnchantmentStore.LookupEntry(OldEnchants[j]))
+                                if (ItemPrototype const* jProto = ObjectMgr::GetItemPrototype(enchantEntry->GemID))
                                     if (iGemProto->ItemLimitCategory == jProto->ItemLimitCategory)
                                         --limit_newcount;
                         }
@@ -1259,16 +1259,16 @@ void WorldSession::HandleSocketOpcode(WorldPacket& recv_data)
                             ++limit_newcount;
                     }
                     // existed gem
-                    else if(OldEnchants[j])
+                    else if (OldEnchants[j])
                     {
-                        if(SpellItemEnchantmentEntry const* enchantEntry = sSpellItemEnchantmentStore.LookupEntry(OldEnchants[j]))
-                            if(ItemPrototype const* jProto = ObjectMgr::GetItemPrototype(enchantEntry->GemID))
+                        if (SpellItemEnchantmentEntry const* enchantEntry = sSpellItemEnchantmentStore.LookupEntry(OldEnchants[j]))
+                            if (ItemPrototype const* jProto = ObjectMgr::GetItemPrototype(enchantEntry->GemID))
                                 if (iGemProto->ItemLimitCategory == jProto->ItemLimitCategory)
                                     ++limit_newcount;
                     }
                 }
 
-                if(limit_newcount > 0 && uint32(limit_newcount) > limitEntry->maxCount)
+                if (limit_newcount > 0 && uint32(limit_newcount) > limitEntry->maxCount)
                 {
                     _player->SendEquipError( EQUIP_ERR_ITEM_UNIQUE_EQUIPPABLE_SOCKETED, itemTarget, NULL );
                     return;
@@ -1277,9 +1277,9 @@ void WorldSession::HandleSocketOpcode(WorldPacket& recv_data)
         }
 
         // for equipped item check all equipment for duplicate equipped gems
-        if(itemTarget->IsEquipped())
+        if (itemTarget->IsEquipped())
         {
-            if(uint8 res = _player->CanEquipUniqueItem(Gems[i],slot,limit_newcount >= 0 ? limit_newcount : 0))
+            if (uint8 res = _player->CanEquipUniqueItem(Gems[i],slot,limit_newcount >= 0 ? limit_newcount : 0))
             {
                 _player->SendEquipError( res, itemTarget, NULL );
                 return;
@@ -1298,10 +1298,10 @@ void WorldSession::HandleSocketOpcode(WorldPacket& recv_data)
 
     for (int i = 0; i < MAX_GEM_SOCKETS; ++i)
     {
-        if(GemEnchants[i])
+        if (GemEnchants[i])
         {
             itemTarget->SetEnchantment(EnchantmentSlot(SOCK_ENCHANTMENT_SLOT+i), GemEnchants[i],0,0);
-            if(Item* guidItem = _player->GetItemByGuid(gem_guids[i]))
+            if (Item* guidItem = _player->GetItemByGuid(gem_guids[i]))
                 _player->DestroyItem(guidItem->GetBagSlot(), guidItem->GetSlot(), true );
         }
     }
@@ -1310,7 +1310,7 @@ void WorldSession::HandleSocketOpcode(WorldPacket& recv_data)
         _player->ApplyEnchantment(itemTarget,EnchantmentSlot(enchant_slot),true);
 
     bool SocketBonusToBeActivated = itemTarget->GemsFitSockets();//current socketbonus state
-    if(SocketBonusActivated ^ SocketBonusToBeActivated)     //if there was a change...
+    if (SocketBonusActivated ^ SocketBonusToBeActivated)     //if there was a change...
     {
         _player->ApplyEnchantment(itemTarget,BONUS_ENCHANTMENT_SLOT,false);
         itemTarget->SetEnchantment(BONUS_ENCHANTMENT_SLOT, (SocketBonusToBeActivated ? itemTarget->GetProto()->socketBonus : 0), 0, 0);
@@ -1330,15 +1330,15 @@ void WorldSession::HandleCancelTempEnchantmentOpcode(WorldPacket& recv_data)
     recv_data >> eslot;
 
     // apply only to equipped item
-    if(!Player::IsEquipmentPos(INVENTORY_SLOT_BAG_0,eslot))
+    if (!Player::IsEquipmentPos(INVENTORY_SLOT_BAG_0,eslot))
         return;
 
     Item* item = GetPlayer()->GetItemByPos(INVENTORY_SLOT_BAG_0, eslot);
 
-    if(!item)
+    if (!item)
         return;
 
-    if(!item->GetEnchantmentId(TEMP_ENCHANTMENT_SLOT))
+    if (!item->GetEnchantmentId(TEMP_ENCHANTMENT_SLOT))
         return;
 
     GetPlayer()->ApplyEnchantment(item,TEMP_ENCHANTMENT_SLOT,false);