[4.3.4] Spells/Mage: Mirror Image summoning #9577

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
8 participants
@Souler
Contributor

Souler commented Apr 5, 2013

Fixes the summonig of Mirror Images npcs and implements the Glyph of Mirror Image.

Mirror Image: http://old.wowhead.com/spell=55342
Glyph of Mirror Image: http://old.wowhead.com/spell=63093

Anyways the AI seems to still bugged by some reason (at least on my case).

@killradio

This comment has been minimized.

Show comment Hide comment
@killradio

killradio Apr 5, 2013

Looks good - can you explain a bit more about bugged AI?

Looks good - can you explain a bit more about bugged AI?

@Souler

This comment has been minimized.

Show comment Hide comment
@Souler

Souler Apr 5, 2013

Contributor

@killradio For some reason, on my core, images doesn't cast spells (they target the enemy the mage is attacking, but no spell casting at all).

Contributor

Souler commented Apr 5, 2013

@killradio For some reason, on my core, images doesn't cast spells (they target the enemy the mage is attacking, but no spell casting at all).

@Shauren

This comment has been minimized.

Show comment Hide comment
@Shauren

Shauren Apr 5, 2013

Member

Do not use creature_template.spell fields for AI. These are supposed to be pet/vehicle/control bars only

Member

Shauren commented Apr 5, 2013

Do not use creature_template.spell fields for AI. These are supposed to be pet/vehicle/control bars only

@Souler

This comment has been minimized.

Show comment Hide comment
@Souler

Souler Apr 6, 2013

Contributor

@Shauren So, have TC recently changed the CasterAI? That would explain why Mirror Images (even the old ones) stoped casting stuff.

Contributor

Souler commented Apr 6, 2013

@Shauren So, have TC recently changed the CasterAI? That would explain why Mirror Images (even the old ones) stoped casting stuff.

@Shauren

This comment has been minimized.

Show comment Hide comment
@Shauren

Shauren Apr 6, 2013

Member

No, Combat/Caster AI were left untouched but should be scrapped anyway (do not rely on them)

Member

Shauren commented Apr 6, 2013

No, Combat/Caster AI were left untouched but should be scrapped anyway (do not rely on them)

@saqirmdev

This comment has been minimized.

Show comment Hide comment
@saqirmdev

saqirmdev Apr 7, 2013

@Souler Change unit_class => 8 it fix cast issue

@Souler Change unit_class => 8 it fix cast issue

@Loukie

This comment has been minimized.

Show comment Hide comment
@Loukie

Loukie Jul 6, 2013

works good but some time there is more then 3 mirror images

Loukie commented Jul 6, 2013

works good but some time there is more then 3 mirror images

@jackpoz jackpoz changed the title from Spells/Mage: Mirror Image summoning to [4.3.4] Spells/Mage: Mirror Image summoning May 20, 2014

@msoky

This comment has been minimized.

Show comment Hide comment
@msoky

msoky Jun 18, 2014

Contributor

any news?

Contributor

msoky commented Jun 18, 2014

any news?

@jackpoz

This comment has been minimized.

Show comment Hide comment
@jackpoz

jackpoz Aug 23, 2014

Member

Please apply the requested changes then contact a developer to reopen the pull request.
Commenting back on this PR is fine too.

Member

jackpoz commented Aug 23, 2014

Please apply the requested changes then contact a developer to reopen the pull request.
Commenting back on this PR is fine too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment