Miscellaneous: Summary Of Magician Issues

[Jify]

Originally Posted by guineapig [You must be logged in to view images. Log in or Register.]

SO Mage heals are still in the game? Shit, might as well give me Breeze then.
I can't believe mages are asking for fixes. You should be happy your not getting nerfed.

I just want Alliance to work and my pet to not break mez. =(

Some of the fixes requested are nerfs. Thanks for your input Guinea!

[messiah_b]

Think we are missing a few research recipes from Elemental Grimoir.

I'm going off this site which is post Kunark so maybe someone could correct me.
http://web.archive.org/web/200106280...arch/magician/

Looks like it is missing 2 lvl 16 pet spells

Minor Summoning Fire
Minor Summoning Air

[Finawin]

Pet Heal still doesn't cure anything.
Pets still don't dual wield at the proper level
Pets still have faction(exploitable)
Mage DS duration still messed up.

[Finawin]

Spell: Phantom Plate doesn't increase hp regen.

p.s. Bump the other things

[messiah_b]

If you read the thread you would see that the phantom lines did not increase hp regen until just before Kunark and they won't here.

Also removing the faction from pets is exponentially more exploitable than having it.

[Finawin]

Before Kunark, meaning it was done while still in classic. There's no reason it shouldn't have hp regen now and not one dev responded in this thread about that specifically. So, the banter from others is irrelevant.

The obsession some people have following some arbitrarily chosen timeline to ignore fixes, and changes, etc. is disgusting...

"Hey, AFK leveling was doable in classic! Leave it alone!"

*rolls eyes*

So were a great number of other exploits. That doesn't mean we shouldn't fix them until some retarded time chosen for no logical reason.

[Jify]

Bump-dee-bump

[Aaron]

I'm glad to see Dual Wield and Damage Shield problems addressed (even if it's not completely ironed out yet). Would really like to see the resistances of each pet and the hp regen of 39+ pets to be fixed soon as well.

[incorect]

Originally Posted by Dolalin [You must be logged in to view images. Log in or Register.]

Ok, this source patch should work to fix dual wield but I have no way of testing it.

Currently the source for the DW check is this:

mobai.cpp:1271

//can only dual weild without a weapon if your a monk
if((GetEquipment(MATERIAL_SECONDARY) != 0 && GetLevel() > 39) || myclass == MONK || myclass == MONKGM) {

That means, only if your pet's actual level is over 39 will it dual wield. Well we have to add a special exception for lower level pets.

The situation we aim to achieve is that:

Pets summoned with 29-39th spells will dual-wield if given weapons (if offhand is not empty). Pets with 44th and 49th spells should dual wield without weapons.

Changing the check to this should achieve that goal:

if((GetEquipment(MATERIAL_SECONDARY) != 0 && GetLevel() > 39) || (GetOwner() != NULL && GetEquipment(MATERIAL_SECONDARY) != 0 && GetLevel() > 24) || (GetOwner() != NULL && GetLevel() > 37) || myclass == MONK || myclass == MONKGM)

Added two new conditions based on the return of GetOwner() (if returns non-null, I'm a pet), and GetLevel() (self-explanatory).

Only one question: What are the lowest level 29 and 44 pets? I THINK they're 25 and 38 respectively but I could be wrong.

Nice work on this. However... Can't lvl 44 summons be greater then lvl 39? And to my understanding lvl 44 summons don't dual wield. So there might be another issue going on.

Edit... I am not sure why the > lvl 39 check is there in the first place. CanThisClassDualWield() checks the level of the mob to make sure it should be able to dual wield. I think the > 39 check should probably just be removed.

[Wenai]

Water Pets are now more resistant to Poison.
Air pets are now more resistant to Cold.
Earth pets are now more resistant to Disease.
Fire pets are now more resistant to Fire.

Feedback required for the resist changes. Please remember to take level into consideration (ie. Level 50 mob casting spells on your level 42 pet may still stick spells even with increased resists).

Increased max damage of the level 44 Fire pet.

All these changes will go in next patch.