[10641][hack fix] Отображение характеристик heroic предметов.

[Warlord123]

Суть проблемы: Вещи с меткой "Heroic" не показывают вставленные камни, бонусы от камней, наложенные энчанты,
прочность.
Идея: Оригинальный патч от Shauren. Суть в том , что в данный момент флаги от шаблона предмета копируются каждому
экземпляру предмета. В итоге в экземпляре храниться сумма флагов предмета и шаблона.
До появления ITEM_FLAGS_HEROIC = 0x00000008 проблема себя не проявляла.
Предлагаемый патч требует дополнительной проверки, пока проблемы не выявлены.

diff --git a/src/game/Bag.cpp b/src/game/Bag.cpp
index c2d3b74..e929796 100644
--- a/src/game/Bag.cpp
+++ b/src/game/Bag.cpp
@@ -74,7 +74,7 @@ bool Bag::Create(uint32 guidlow, uint32 itemid, Player const* owner)
 
     SetUInt32Value(ITEM_FIELD_MAXDURABILITY, itemProto->MaxDurability);
     SetUInt32Value(ITEM_FIELD_DURABILITY, itemProto->MaxDurability);
-    SetUInt32Value(ITEM_FIELD_FLAGS, itemProto->Flags);
+    //SetUInt32Value(ITEM_FIELD_FLAGS, itemProto->Flags);
     SetUInt32Value(ITEM_FIELD_STACK_COUNT, 1);
 
     // Setting the number of Slots the Container has
diff --git a/src/game/Item.cpp b/src/game/Item.cpp
index 52e58a9..68049ce 100644
--- a/src/game/Item.cpp
+++ b/src/game/Item.cpp
@@ -266,7 +266,7 @@ bool Item::Create( uint32 guidlow, uint32 itemid, Player const* owner)
     for(int i = 0; i < MAX_ITEM_PROTO_SPELLS; ++i)
         SetSpellCharges(i,itemProto->Spells[i].SpellCharges);
 
-    SetUInt32Value(ITEM_FIELD_FLAGS, itemProto->Flags);
+    //SetUInt32Value(ITEM_FIELD_FLAGS, itemProto->Flags);
     SetUInt32Value(ITEM_FIELD_DURATION, itemProto->Duration);
 
     return true;
diff --git a/src/game/Item.h b/src/game/Item.h
index 33ac541..a64dbbd 100644
--- a/src/game/Item.h
+++ b/src/game/Item.h
@@ -240,7 +240,7 @@ class MANGOS_DLL_SPEC Item : public Object
 
         void SetBinding(bool val) { ApplyModFlag(ITEM_FIELD_FLAGS,ITEM_FLAGS_BINDED,val); }
         bool IsSoulBound() const { return HasFlag(ITEM_FIELD_FLAGS, ITEM_FLAGS_BINDED); }
-        bool IsBoundAccountWide() const { return HasFlag(ITEM_FIELD_FLAGS, ITEM_FLAGS_BOA); }
+        bool IsBoundAccountWide() const { return (GetProto()->Flags & ITEM_FLAGS_BOA); }
         bool IsBindedNotWith(Player const* player) const;
         bool IsBoundByEnchant() const;
         virtual void SaveToDB();
diff --git a/src/game/ItemHandler.cpp b/src/game/ItemHandler.cpp
index 08b3386..bdb08ae 100644
--- a/src/game/ItemHandler.cpp
+++ b/src/game/ItemHandler.cpp
@@ -1111,7 +1111,7 @@ void WorldSession::HandleWrapItemOpcode(WorldPacket& recv_data)
         return;
     }
 
-    if (!gift->HasFlag(ITEM_FIELD_FLAGS, ITEM_FLAGS_WRAPPER))// cheating: non-wrapper wrapper
+    if (!(gift->GetProto()->Flags & ITEM_FLAGS_WRAPPER))// cheating: non-wrapper wrapper
     {
         _player->SendEquipError( EQUIP_ERR_ITEM_NOT_FOUND, gift, NULL );
         return;

У существующих экземпляров предметов нужно сбросить флаг 0x8; Перед применением запроса необходимо сделать
резервную копию базы characters.

PHP код:

--
update item_instance , character_inventory
set `data`=   CONCAT(SUBSTRING_INDEX( `data`, ' ', 21),' ',SUBSTRING_INDEX(SUBSTRING_INDEX(`data` , ' ', 22),' ',-1) & ~8 ,' ',SUBSTRING_INDEX(`data`, ' ',-43)) 
where item_instance.guid = item and item_template in (select entry from mangos.item_template where Flags & 8) ;
-- 


В файле shauren.7z содержиться патч Shauren`а для Trinity. Там полностью разделены флаги прототипа и самого предмета.

[Vladimir]

Никаких неожиданных эфектов не замечено с вещами с этим патчем?
Флаги копировались в вещь вообщемто исторически - с древних времен и никто сильно не проводил иследование нужно ли это делать для чего либо.
Пример с heroic вроде показывает что это не стоит делать по крайней мере для некоторых флагов точно. Моловероятно чтобы 2 поля флагов частично совпадали по флагам а частично были различны... поэтому возможно это и не хак, но нужна некоторая увеерность...

[Warlord123]

Неожиданные эффекты были. Когда я попробовал просто не копировать флаги возникли проблемы с обернутыми подарками и при проверке IsBoundAccountWide ;
После изменений проблем не замечено. Я чуть позже подниму изменения в патчах, мне говорили что можно пересылать AccountWide вещи на другую фракцию.

[Vladimir]

Я не имею ввиду проверки в коде - там конечно надо исправлять обращения раз мы статические флаги не копируем. Имеются ввиду неадекватность на стороне клинета.

[Warlord123]

После "очистки" флагов SQL-скриптом проблем в клиенте не было.

[Vladimir]

Так как мне все равно надо проверять все случаи с флагами то я не планирую использовать trinity версию. За подтверждение работоспособности для стандартных случаев при некопировании, спасибо.

SQL запрос добавлю отдельно за вашим авторством.

[Vladimir]

В [10641] добавлена моя версия патче.

SQL запрос добавлю отдельно за вашим авторством.

Я добавил проверку wrapped состояния при загрузке вешей поэтому sql update не требуется - флаг будет удален для некоректных случаев при первом логине после коммита.