[patch] Mysterious Egg (item 39878) - Ru-MaNGOS

KiriX · 26.06.2010, 15:51

Итем спустя 7 дней пропадает, но новый не появляется.
Должно работать так. Покупаем Таинственное яйцо, через 7 дней оно превращается в Треснутое яйцо у которого есть интересный лут.
Патч реализует работу данного итема.

PHP код:


			
diff --git a/src/game/Item.cpp b/src/game/Item.cpp 

index e94bdf3..b5a7655 100644 

--- a/src/game/Item.cpp 

+++ b/src/game/Item.cpp 

@@ -282,6 +282,19 @@ void Item::UpdateDuration(Player* owner, uint32 diff) 

  

     if (GetUInt32Value(ITEM_FIELD_DURATION)<=diff) 

     { 

+        if (GetEntry() == 39878) 

+        { 

+            owner->DestroyItem(GetBagSlot(), GetSlot(), true); 

+            uint32 itemId = 39883; 

+            uint8 count = 1; 

+            ItemPosCountVec dest; 

+            if(owner->CanStoreNewItem(NULL_BAG, NULL_SLOT, dest, itemId, count) == EQUIP_ERR_OK) 

+            { 

+                Item* item = owner->StoreNewItem(dest, itemId, true, Item::GenerateItemRandomPropertyId(itemId)); 

+                owner->SendNewItem(item, count, true, false); 

+            } 

+            return; 

+        } 

         owner->DestroyItem(GetBagSlot(), GetSlot(), true); 

         return; 

     }

Спасибо LordJZ за подсказку по коду!

**LordJZ** · 26.06.2010, 16:20

Может лучше так

PHP код:


			
diff) 

  

     if (GetUInt32Value(ITEM_FIELD_DURATION)<=diff) 

     { 

         owner->DestroyItem(GetBagSlot(), GetSlot(), true); 

+        if (GetEntry() == 39878) 

+        { 

+            uint32 itemId = 39883; 

+            uint8 count = 1; 

+            ItemPosCountVec dest; 

+            if(owner->CanStoreNewItem(NULL_BAG, NULL_SLOT, dest, itemId, count) == EQUIP_ERR_OK) 

+            { 

+                Item* item = owner->StoreNewItem(dest, itemId, true, Item::GenerateItemRandomPropertyId(itemId)); 

+                owner->SendNewItem(item, count, true, false); 

+            } 

+        } 

         return; 

     }

Или может стоит создать для этого отдельную таблицу?

KiriX · 26.06.2010, 16:24

Цитата:

Сообщение от LordJZ

Может лучше так

PHP код:

***

Или может стоит создать для этого отдельную таблицу?

А разве после дестроя мы не потеряем GetEntry()? Я исходил из этого когда писал. Таблица? А много ли таких вещей?

**LordJZ** · 26.06.2010, 16:43

Может и потеряем, не знаю.

Ну item entry все-равно не примут в код...

**NeatElves** · 26.06.2010, 17:04

Вот с таблицей..)
http://getmangos.com/community/showt...ration-expires

Viste · 26.06.2010, 20:45

таких вещей всего две, думаю таблица незачем

xex · 28.06.2010, 04:38

Мой старый код. Работает с полгода, всё нормально.

PHP код:


			
    if (GetUInt32Value(ITEM_FIELD_DURATION) <= diff)

    {

        uint8 bagSlot = GetBagSlot();

        uint8 slot = GetSlot();

        uint32 ItemId = GetProto()->ItemId;



        owner->DestroyItem(bagSlot, slot, true);



        // Mysterious Egg (39878) & Disgusting Jar (44717)

        if (ItemId == 39878 || ItemId == 44717)

        {

            // Cracked Egg (39883) & Ripe Disgusting Jar (44718)

            uint32 newItemId = (ItemId == 39878) ? 39883 : 44718;



            // first try store new item instead of deleted

            ItemPosCountVec dest;

            uint8 msg = owner->CanStoreNewItem(bagSlot, slot, dest, newItemId, 1);

            if (msg == EQUIP_ERR_OK)

            {

                owner->StoreNewItem(dest, newItemId, 1, true);

            }

            else

            {

                // try store new item in any slot

                if (Item* pItem = owner->StoreNewItemInInventorySlot(newItemId, 1))

                    owner->SendNewItem(pItem, 1, true, false);

                // on any case... send error

                else

                    owner->SendEquipError(msg, NULL, NULL);

            }

        }

        return;

    }

KiriX · 28.06.2010, 10:20

Ради двух итемов действительно таблица ни к чему.
xex, спасибо, значит после дестроя итема мы не теряем его ид? Хорошо...

xex · 28.06.2010, 15:10

Здесь он сначала запоминается:
uint32 ItemId = GetProto()->ItemId;

а потом уже удаляется айтем:
owner->DestroyItem(bagSlot, slot, true);

И удаляется он только из сумки. Мы же в Item объекте находимся

Текущий объект будет удалён на следующем тике обновления.

Можно и так написать:
owner->DestroyItem(bagSlot, slot, true);

if (GetProto()->ItemId == 39878 || GetProto()->ItemId == 44717)

но предпочитаю сначала переменные обозначать.

**NeatElves** · 13.05.2011, 18:48

Вот вам еще цепочки по году в реале.))

44623-44625-44627
44632-44626-44629

xex · 15.05.2011, 00:36

Цитата:

Сообщение от NeatElves

Вот вам еще цепочки по году в реале.))

44623-44625-44627
44632-44626-44629

Спасибо, пофиксено:

PHP код:


			
--- Item.cpp

+++ Item.cpp

@@ -275,15 +275,20 @@

 {

     if (!GetUInt32Value(ITEM_FIELD_DURATION))

         return;



     //DEBUG_LOG("Item::UpdateDuration Item (Entry: %u Duration %u Diff %u)",GetEntry(),GetUInt32Value(ITEM_FIELD_DURATION),diff);



-    if (GetUInt32Value(ITEM_FIELD_DURATION)<=diff)

+    if (GetUInt32Value(ITEM_FIELD_DURATION) <= diff)

     {

-        owner->DestroyItem(GetBagSlot(), GetSlot(), true);

+        uint32 itemId = GetEntry();

+        uint8 bagSlot = GetBagSlot();

+        uint8 slot = GetSlot();

+        uint16 pos = GetPos();

+        owner->DestroyItem(bagSlot, slot, true);

+        owner->HandleDestroyItemReplace(itemId, bagSlot, slot, pos);

         return;

     }



     SetUInt32Value(ITEM_FIELD_DURATION, GetUInt32Value(ITEM_FIELD_DURATION) - diff);

     SetState(ITEM_CHANGED, owner);                          // save new time in database

 }



--- Player.cpp

+++ Player.cpp

@@ -11539,12 +11539,69 @@

         pItem->SetGuidValue(ITEM_FIELD_CONTAINED, ObjectGuid());

         pItem->SetSlot( NULL_SLOT );

         pItem->SetState(ITEM_REMOVED, this);

     }

 }



+void Player::HandleDestroyItemReplace(uint32 itemId, uint8 bag, uint8 slot, uint16 pos)

+{

+    if (!itemId || !bag || !slot)

+        return;

+

+    uint32 newItemId;

+    switch(itemId)

+    {

+        case 39878:             // Mysterious Unhatched Egg

+            newItemId = 39883;  // Cracked Egg

+            break;

+        case 44623:             // Bottle of Dalaran Red

+            newItemId = 44625;

+            break;

+        case 44625:             // Bottle of Aged Dalaran Red

+            newItemId = 44627;  // Bottle of Peaked Dalaran Red

+            break;

+        case 44626:             // Cask of Aged Dalaran Red

+            newItemId = 44629;  // Cask of Peaked Dalaran Red

+            break;

+        case 44632:             // Cask of Dalaran Red

+            newItemId = 44626;

+            break;

+        case 44717:             // Disgusting Jar

+            newItemId = 44718;  // Ripe Disgusting Jar

+            break;

+        default:

+            return;

+    }

+

+    ItemPosCountVec dest;

+    if (IsInventoryPos(pos))

+    {

+        uint8 msg = CanStoreNewItem(bag, slot, dest, newItemId, 1);

+        if (msg == EQUIP_ERR_OK)

+        {

+            StoreNewItem(dest, newItemId, 1, true);

+            return;

+        }

+    }

+    else if (IsBankPos(pos))

+    {

+        Item* pNewItem = Item::CreateItem(newItemId, 1, this);

+        if (!pNewItem)

+            return;

+

+        uint8 msg = CanBankItem(bag, slot, dest, pNewItem, true);

+        if (msg == EQUIP_ERR_OK)

+        {

+            BankItem(dest, pNewItem, true);

+            return;

+        }

+        else

+            delete pNewItem;

+    }

+}

+

 void Player::DestroyItemCount( uint32 item, uint32 count, bool update, bool unequip_check)

 {

     DEBUG_LOG( "STORAGE: DestroyItemCount item = %u, count = %u", item, count);

     uint32 remcount = 0;



     // in inventory



--- Player.h

+++ Player.h

@@ -1258,12 +1258,13 @@

         void MoveItemFromInventory(uint8 bag, uint8 slot, bool update);

                                                             // in trade, auction, guild bank, mail....

         void MoveItemToInventory(ItemPosCountVec const& dest, Item* pItem, bool update, bool in_characterInventoryDB = false);

                                                             // in trade, guild bank, mail....

         void RemoveItemDependentAurasAndCasts( Item * pItem );

         void DestroyItem( uint8 bag, uint8 slot, bool update );

+        void HandleDestroyItemReplace(uint32 itemId, uint8 bag, uint8 slot, uint16 pos);

         void DestroyItemCount( uint32 item, uint32 count, bool update, bool unequip_check = false);

         void DestroyItemCount( Item* item, uint32& count, bool update );

         void DestroyConjuredItems( bool update );

         void DestroyZoneLimitedItem( bool update, uint32 new_zone );

         void SplitItem( uint16 src, uint16 dst, uint32 count );

         void SwapItem( uint16 src, uint16 dst );

**Vladimir** · 18.06.2011, 05:09

https://gist.github.com/1032722 обновленная версия с использованием существующего кода аналогичной таблицы. Не тестировал еще.

Если все нормально добавлю вечером.

**Vladimir** · 18.06.2011, 20:16

Патч в форме приведенной в прев. посте добавлен в [11646]

26.06.2010, 15:51	#1
KiriX Умный Старожил Регистрация: 06.03.2010 Сообщений: 886 Сказал(а) спасибо: 698 Поблагодарили 433 раз(а) в 181 сообщениях Записей в дневнике: 4	[patch] Mysterious Egg (item 39878) Итем спустя 7 дней пропадает, но новый не появляется. Должно работать так. Покупаем Таинственное яйцо, через 7 дней оно превращается в Треснутое яйцо у которого есть интересный лут. Патч реализует работу данного итема. PHP код: diff --git a/src/game/Item.cpp b/src/game/Item.cpp index e94bdf3..b5a7655 100644 --- a/src/game/Item.cpp +++ b/src/game/Item.cpp @@ -282,6 +282,19 @@ void Item::UpdateDuration(Player* owner, uint32 diff) if (GetUInt32Value(ITEM_FIELD_DURATION)<=diff) { + if (GetEntry() == 39878) + { + owner->DestroyItem(GetBagSlot(), GetSlot(), true); + uint32 itemId = 39883; + uint8 count = 1; + ItemPosCountVec dest; + if(owner->CanStoreNewItem(NULL_BAG, NULL_SLOT, dest, itemId, count) == EQUIP_ERR_OK) + { + Item* item = owner->StoreNewItem(dest, itemId, true, Item::GenerateItemRandomPropertyId(itemId)); + owner->SendNewItem(item, count, true, false); + } + return; + } owner->DestroyItem(GetBagSlot(), GetSlot(), true); return; } Спасибо LordJZ за подсказку по коду!

26.06.2010, 20:45	#6
Viste Новичок Регистрация: 07.03.2010 Сообщений: 13 Сказал(а) спасибо: 27 Поблагодарили 7 раз(а) в 5 сообщениях	таких вещей всего две, думаю таблица незачем __________________ С недавних времен TrinityCore User

28.06.2010, 15:10	#9
xex Пользователь Регистрация: 08.03.2010 Сообщений: 47 Сказал(а) спасибо: 45 Поблагодарили 29 раз(а) в 13 сообщениях	Здесь он сначала запоминается: uint32 ItemId = GetProto()->ItemId; а потом уже удаляется айтем: owner->DestroyItem(bagSlot, slot, true); И удаляется он только из сумки. Мы же в Item объекте находимся Текущий объект будет удалён на следующем тике обновления. Можно и так написать: owner->DestroyItem(bagSlot, slot, true); if (GetProto()->ItemId == 39878 \|\| GetProto()->ItemId == 44717) но предпочитаю сначала переменные обозначать. Последний раз редактировалось xex; 28.06.2010 в 15:18.

18.06.2011, 05:09	#12
Vladimir MaNGOS Dev Регистрация: 09.02.2010 Сообщений: 594 Сказал(а) спасибо: 315 Поблагодарили 438 раз(а) в 181 сообщениях	https://gist.github.com/1032722 обновленная версия с использованием существующего кода аналогичной таблицы. Не тестировал еще. Если все нормально добавлю вечером. __________________ Так как устал объяснять знайте ICQ не пользуюсь

18.06.2011, 20:16	#13
Vladimir MaNGOS Dev Регистрация: 09.02.2010 Сообщений: 594 Сказал(а) спасибо: 315 Поблагодарили 438 раз(а) в 181 сообщениях	Патч в форме приведенной в прев. посте добавлен в [11646] __________________ Так как устал объяснять знайте ICQ не пользуюсь

26.06.2010, 16:20	#2
LordJZ Супер-модератор Регистрация: 07.03.2010 Сообщений: 647 Сказал(а) спасибо: 100 Поблагодарили 252 раз(а) в 123 сообщениях	Может лучше так PHP код: diff) if (GetUInt32Value(ITEM_FIELD_DURATION)<=diff) { owner->DestroyItem(GetBagSlot(), GetSlot(), true); + if (GetEntry() == 39878) + { + uint32 itemId = 39883; + uint8 count = 1; + ItemPosCountVec dest; + if(owner->CanStoreNewItem(NULL_BAG, NULL_SLOT, dest, itemId, count) == EQUIP_ERR_OK) + { + Item* item = owner->StoreNewItem(dest, itemId, true, Item::GenerateItemRandomPropertyId(itemId)); + owner->SendNewItem(item, count, true, false); + } + } return; } Или может стоит создать для этого отдельную таблицу?

26.06.2010, 16:43	#4
LordJZ Супер-модератор Регистрация: 07.03.2010 Сообщений: 647 Сказал(а) спасибо: 100 Поблагодарили 252 раз(а) в 123 сообщениях	Может и потеряем, не знаю. Ну item entry все-равно не примут в код...

26.06.2010, 17:04	#5
NeatElves YTDB Dev Регистрация: 06.03.2010 Сообщений: 259 Сказал(а) спасибо: 28 Поблагодарили 280 раз(а) в 136 сообщениях	Вот с таблицей..) http://getmangos.com/community/showt...ration-expires

28.06.2010, 04:38	#7
xex Пользователь Регистрация: 08.03.2010 Сообщений: 47 Сказал(а) спасибо: 45 Поблагодарили 29 раз(а) в 13 сообщениях	Мой старый код. Работает с полгода, всё нормально. PHP код: if (GetUInt32Value(ITEM_FIELD_DURATION) <= diff) { uint8 bagSlot = GetBagSlot(); uint8 slot = GetSlot(); uint32 ItemId = GetProto()->ItemId; owner->DestroyItem(bagSlot, slot, true); // Mysterious Egg (39878) & Disgusting Jar (44717) if (ItemId == 39878 \|\| ItemId == 44717) { // Cracked Egg (39883) & Ripe Disgusting Jar (44718) uint32 newItemId = (ItemId == 39878) ? 39883 : 44718; // first try store new item instead of deleted ItemPosCountVec dest; uint8 msg = owner->CanStoreNewItem(bagSlot, slot, dest, newItemId, 1); if (msg == EQUIP_ERR_OK) { owner->StoreNewItem(dest, newItemId, 1, true); } else { // try store new item in any slot if (Item* pItem = owner->StoreNewItemInInventorySlot(newItemId, 1)) owner->SendNewItem(pItem, 1, true, false); // on any case... send error else owner->SendEquipError(msg, NULL, NULL); } } return; }

28.06.2010, 10:20	#8
KiriX Умный Старожил Регистрация: 06.03.2010 Сообщений: 886 Сказал(а) спасибо: 698 Поблагодарили 433 раз(а) в 181 сообщениях Записей в дневнике: 4	Ради двух итемов действительно таблица ни к чему. xex, спасибо, значит после дестроя итема мы не теряем его ид? Хорошо...

13.05.2011, 18:48	#10
NeatElves YTDB Dev Регистрация: 06.03.2010 Сообщений: 259 Сказал(а) спасибо: 28 Поблагодарили 280 раз(а) в 136 сообщениях	Вот вам еще цепочки по году в реале.)) 44623-44625-44627 44632-44626-44629

Опции темы	Поиск в этой теме
Версия для печати Отправить по электронной почте	Поиск в этой теме: Расширенный поиск
Опции просмотра
Линейный вид Комбинированный вид Древовидный вид

Похожие темы
Тема	Автор	Раздел	Ответов	Последнее сообщение
[10883][patch] correct proc - item Heartpierce	Den	Принятые патчи	2	17.12.2010 15:23
[10680] Update item buy code for new way vendor item lists work.	newsbot	CMaNGOS Commits	0	04.11.2010 20:40
[10641] Fixed use item dynamic/item proto flags.	newsbot	CMaNGOS Commits	0	25.10.2010 15:30
item 37098 и item 45785	rsa	Баг-репорты	1	29.05.2010 20:32
[9727] Build equiped item data only for item's owner	newsbot	CMaNGOS Commits	0	11.04.2010 07:50