[fix] Mirror Image

[Nordway]

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

Второй пост поставили в SD2 ?

Да, в scripts/world/npcs_special.cpp + на базу обновление.
Попробую ещё раз внимательно пересобрать...

[lucky]

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

Сегодня собрал с Мангос 9611 (не чистые) и SD2 1647 (не чистый) старую версию, т.е. без пункта 5. Теперь копируется имя и гильдия. (Спасибо Set).

При появлении нпс реагриует только на вражеских мобов, а так же:
- не ходят за мной;
- не атакуют мою цель.

Такая же проблема со скриптом на SD2, который во 2 посту. А именно у меня он не может получить Owner еще в void Reset(). А так как стоит

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

void Reset() 
{
		Unit *owner = m_creature->GetOwner();
		if (!owner)
			return;

то соответственно они не ходят и не атакуют. Почему не может получить owner не понятно

Не потому ли, что в SpellEffects.cpp нет строки с m_creature->SetOwnerGUID()?

[MaxXx2021]

Сообщение от Дрон01

Не потому ли, что в SpellEffects.cpp нет строки с m_creature->SetOwnerGUID()?

Гыг я сету задавал тот же вопрос!

[xex]

PHP код:

            if(creature_entry == 31216)
            {
+                summon->SetOwnerGUID(m_caster->GetGUID());
                summon->SetLevel(m_caster->getLevel());
                summon->SetMaxHealth(m_caster->GetMaxHealth()); 


[selector]

Небольшая "грамматическая" оЧепятка в Коде :

HandleMirrrorImageDataRequest

Заменить

void WorldSession::HandleMirrrorImageDataRequest( WorldPacket & recv_data )

на

void WorldSession::HandleMirrorImageDataRequest( WorldPacket & recv_data )

Заменить

void HandleMirrrorImageDataRequest(WorldPacket & recv_data);

на

void HandleMirrorImageDataRequest(WorldPacket & recv_data);

Заменить

/*0x401*/ { "CMSG_GET_MIRRORIMAGE_DATA",                    STATUS_LOGGEDIN, &WorldSession::HandleMirrrorImageDataRequest   },

на

/*0x401*/ { "CMSG_GET_MIRRORIMAGE_DATA",                    STATUS_LOGGEDIN, &WorldSession::HandleMirrorImageDataRequest   },

[MadMan]

Fix this @ patch file

               if(forceFaction) 
                   summon->setFaction(forceFaction); 
           }diff --git a/sql/evo-X Update/Spells/evo_X_Spell1_mangos.sql b/sql/evo-X Update/Spells/evo_X_Spell1_mangos.sql 
new file mode 100644 
index 0000000..d908b77

right one:

               if(forceFaction) 
                   summon->setFaction(forceFaction); 
           }
diff --git a/sql/evo-X Update/Spells/evo_X_Spell1_mangos.sql b/sql/evo-X Update/Spells/evo_X_Spell1_mangos.sql 
new file mode 100644 
index 0000000..d908b77

[Vinolentus]

Простите, а зачем нам
WorldPacket data(SMSG_MIRRORIMAGE_DATA, 68)
в EffectSummonWild ?
Мы его создали, заполнили, и?.. %)

[Vinolentus]

И почему, например,

if (Item const* item = ((Player *)m_caster)->GetItemByPos(INVENTORY_SLOT_BAG_0, EQUIPMENT_SLOT_MAINHAND))
    summon->SetUInt32Value(UNIT_VIRTUAL_ITEM_SLOT_ID, item->GetProto()->ItemId);

, а не

summon->SetUInt32Value(UNIT_VIRTUAL_ITEM_SLOT_ID, ((Player *)m_caster)->GetUInt32Value(PLAYER_VISIBLE_ITEM_16_ENTRYID));

?

[MaxXx2021]

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

И почему, например,

if (Item const* item = ((Player *)m_caster)->GetItemByPos(INVENTORY_SLOT_BAG_0, EQUIPMENT_SLOT_MAINHAND))
    summon->SetUInt32Value(UNIT_VIRTUAL_ITEM_SLOT_ID, item->GetProto()->ItemId);

, а не

summon->SetUInt32Value(UNIT_VIRTUAL_ITEM_SLOT_ID, ((Player *)m_caster)->GetUInt32Value(PLAYER_VISIBLE_ITEM_16_ENTRYID));

?

Потому что у героя скажем может и не быть вещи в руке и если начнете использовать ваш вариант будет краш.

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

Потому что у героя скажем может и не быть вещи в руке и если начнете использовать ваш вариант будет краш.

Человек предложил использовать

((Player *)m_caster)->GetUInt32Value(PLAYER_VISIBLE_ITEM_16_ENTRYID)

вместо

item->GetProto()->ItemId

Если оружия в руках нет, то GetUInt32Value(PLAYER_VISIBLE_ITEM_16_ENTRYID) ноль вернет. Где именно будет краш?

[MaxXx2021]

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

Человек предложил использовать

((Player *)m_caster)->GetUInt32Value(PLAYER_VISIBLE_ITEM_16_ENTRYID)

вместо

item->GetProto()->ItemId

Если оружия в руках нет, то GetUInt32Value(PLAYER_VISIBLE_ITEM_16_ENTRYID) ноль вернет. Где именно будет краш?

лишняя страховка никогда никому не мешала

[Vinolentus]

Нигде краша не будет. Ноль и должен быть, если нпц без оружия. Собственно, он и есть.

Это-то выбросьте из EffectSummonWild

+    WorldPacket data(SMSG_MIRRORIMAGE_DATA, 68);
+    data << (uint32)m_caster->GetDisplayId();
+    if (m_caster->GetTypeId() == TYPEID_PLAYER) 
+    { 
+        Player* pCreator = (Player *)m_caster; 
+        data << (uint8)pCreator->getRace();                         // race 
+        data << (uint8)pCreator->getGender();                       // gender 
+        data << (uint8)pCreator->getClass();                        // class 
+        data << (uint8)pCreator->GetByteValue(PLAYER_BYTES, 0);     // skin 
+        data << (uint8)pCreator->GetByteValue(PLAYER_BYTES, 1);     // face 
+        data << (uint8)pCreator->GetByteValue(PLAYER_BYTES, 2);     // hair 
+        data << (uint8)pCreator->GetByteValue(PLAYER_BYTES, 3);     // haircolor 
+        data << (uint8)pCreator->GetByteValue(PLAYER_BYTES_2, 0);   // facialhair 
+ 
+        data << pCreator->GetName();                    // unknown
+        data << (uint32)pCreator->GetGuildId();
+ 
+        static const EquipmentSlots ItemSlots[] =   
+        { 
+            EQUIPMENT_SLOT_HEAD, 
+            EQUIPMENT_SLOT_SHOULDERS, 
+            EQUIPMENT_SLOT_BODY, 
+            EQUIPMENT_SLOT_CHEST, 
+            EQUIPMENT_SLOT_WAIST, 
+            EQUIPMENT_SLOT_LEGS, 
+            EQUIPMENT_SLOT_FEET, 
+            EQUIPMENT_SLOT_WRISTS, 
+            EQUIPMENT_SLOT_HANDS, 
+            EQUIPMENT_SLOT_BACK, 
+            EQUIPMENT_SLOT_TABARD, 
+            EQUIPMENT_SLOT_END 
+        }; 
+
+        // Display items in visible slots 
+        for (EquipmentSlots const* itr = &ItemSlots[0]; *itr != EQUIPMENT_SLOT_END; ++itr) 
+            if (Item const* item =  pCreator->GetItemByPos(INVENTORY_SLOT_BAG_0, *itr)) 
+                data << (uint32)item->GetProto()->DisplayInfoID;    // display id 
+            else 
+                data << (uint32)0;                    // no item found, so no id 
+    } 
+    else 
+    { 
+        // Skip player data for creatures 
+        data << (uint32)0; 
+        data << (uint32)0; 
+        data << (uint32)0; 
+        data << (uint32)0; 
+        data << (uint32)0; 
+        data << (uint32)0; 
+        data << (uint32)0; 
+        data << (uint32)0; 
+        data << (uint32)0; 
+        data << (uint32)0; 
+        data << (uint32)0; 
+        data << (uint32)0; 
+        data << (uint32)0; 
+        data << (uint32)0; 
+    }

[evilstar]

Ужос, рука -> лицо

Dlink если ты всётаки сделаешь всё ручками, не забудь потом сделать git diff> mirror.patch

[rsa]

Круто... Особенно с учетом того что после 10156 патч требует переписывания более чем на 50%.

[KiriX]

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

Круто... Особенно с учетом того что после 10156 патч требует переписывания более чем на 50%.

Но судя по заверениям 62 поста - у него всё получилось...

[LordJZ]

rsa уже опубликовал, наверное, переписанную версию патча: http://github.com/rsa/mangos/commit/...dc9d7186d4e068

[rsa]

Это совсем другая версия патча, от wowka321, и не вздумайте ее пробовать заводить не с моим SD2 - результат непредсказуем . Причем при вкручивании пришлось дропнуть часть функционала

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

Но судя по заверениям 62 поста - у него всё получилось...

Вот я и говорю - круто...

[partizanes]

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

Круто... Особенно с учетом того что после 10156 патч требует переписывания более чем на 50%.

Кто расскажет ,там коммит длинный,если честно понять смог не оч много((

а ооченннь хочется разобраться со спелом ,уже утро а я все и не продвинулся ,перенести перенес а толку 0 (

[tempura]

Попробуй воспользоваться инструкцией умнющего камрада
http://ru-mangos.ru/showthread.php?t=264

вот переписанный патч для Зеркальное изображение
работает без нареканий,
скрипт для sd2 переписан практически полностью

[rsa]

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

вот переписанный мной патч для Зеркальное изображение
работает без нареканий,
скрипт для sd2 переписан практически полностью

когда указываете что "скрипт для sd2 переписан практически полностью", пожалуйста указывайте, кем он переписан, и откуда взят источник для переписывания. а то создается впечатление, что Вы его сами написали...

PS скрипт Тассадара, переписан мной, в этом варианте только заменены 4 подряд if (owner && !m_creature->HasAura на функцию и даже не убран воткнутый мной не-оффлак ICE_BLOCK

PPS патч на ядро тоже из моего репо. урезанный вариант патча от Wowka321. не переписан ни грамма, строчка в строчку.

чтож я рад что авторы нашлись(авторов к сожалению не знал)

ps

это так и должно быть?

       if (m_creature->IsWithinDistInMap(m_creature->getVictim(),30.0f))
        {
            movement = false;
            if (m_uiFrostboltTimer <= diff)
            {
                DoCastSpellIfCan(m_creature->getVictim(),SPELL_FROSTBOLT);
                m_uiFrostboltTimer = urand(4000,8000);
            } else m_uiFrostboltTimer -= diff;

            if (m_uiFireblastTimer <= diff)
            {
                DoCastSpellIfCan(m_creature->getVictim(),SPELL_FIREBLAST);
                m_uiFireblastTimer = urand(4000,8000);
            } else m_uiFireblastTimer -= diff;

            if (m_uiFrostringTimer <= diff && m_creature->IsWithinDistInMap(m_creature->getVictim(),5.0f))
            {
                DoCastSpellIfCan(m_creature->getVictim(),SPELL_ICERING);
                m_uiFrostringTimer = urand(4000,8000);
            } else m_uiFrostboltTimer -= diff;

            if (!blocked && m_creature->GetHealthPercent() < 10.0f)
            {
                DoCastSpellIfCan(m_creature,SPELL_ICEBLOCK);
                blocked = true;
            }
        }

[partizanes]

Мне кажется чего то не хватает

Pet* FindGuardianWithEntry(uint32 entry);
+GuardianPetList const& GetGuardians() const { return m_guardianPets; }

bool isCharmed() const { return GetCharmerGUID() != 0; }

[rsa]

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

чтож я рад что авторы нашлись(авторов к сожалению не знал)

ps

хе. ошибке уже полгода а ее только заметили... спс

[partizanes]

К слову 3.1.2

* «Зеркальное изображение»: копии мага, создаваемые с помощью этого заклинания, теперь будут атаковать существо, наиболее враждебное к заклинателю. Также они больше не будут применять заклинания «Огненный взрыв» и «Ледяная стрела» против целей, которые находятся под воздействием эффектов контроля, снимаемых при получении целью урона. Если «Огненный взрыв» и «Ледяная стрела» произносились в момент, когда на цель начал воздействовать эффект контроля, применение заклинаний не прервется.


еще наверно стоит обратить внимание на бафы,статы ,показатели брони ,Должны ли они копироваться на иллюзии?

Происходит ли сброс агро при касте заклинания ,вроде бы раньше было превышение агро при касте

также лог

[rsa]

В патче что здесь выложен копирование аур убрано. Кому надо - берите из оригинала у wowka321. Также AI у копий совсем простенький, никакого агролиста у них нет, тупо атакуют цель хозяина.

[Vladimir]

Как я понимаю устарел