VladimirMangos

Prof spells recently use built-in level req. table in code and ignore DB settings. Reqlevel = 0 have meaning: use DBC base level requirement. Base at skill/spell dbcs.

In last form patch add in [11646]. Thank you Sorry for long wait review.

In last suggested form added in [11644]. Thank you.

Converted to reuse existed code original table base version: https://gist.github.com/1032722 Not tested yet.

Maybe this way better? https://gist.github.com/1032676 Ofc, if i correct and rage also reset at leave arena

If pissible please provide patches in for master form, because patch anyway will first applied to master and only after backported to One (if this make sense ofc for master).

In [11641]. Thank you

In [11640] implemented options.

and spell.dbc using dbc viewer if you need exactly list for specific client version.

Just for clarify: string placeholder (%s), replace existed command help string in command table by string with unexpected %s and look help for it, if use this way.

spells damage have level dependent debufs and bonuses. Allow cast its before some level can make it work strange. But ofc you can remove level checks for learn by modify code in trainer spell list and buy spell handlers.

you always ca learn profs as GM using .cast console command

Use -d option for point to Data dir explictly. Possible for extraction used wrong version client.

Please test patch: https://gist.github.com/1024064

In [11629]. Thank you. I remove m_caster check, m_caster can't be NULL.

And this create constant problem for us.

If someone interesting ofc he can use. I do like ides use template in this case and use enums.

Maybe note correct but this code style problem and in related files not single existed case. Lot cases infact. So not reason apply only this one case. Also note: in affected lines more code style porblems. Look https://github.com/mangos/mangos/wiki/CodingStandards

Thank you for patch, but sorry, i not agree that selected in suggested patch way so good... So i add to sources own alt. version in [11623].

in [11622]. Thank you

Thank you! in [11621]

It cleary useless attempt use SD2 with 400 branch code. If core not do any usefull in current state then what reason attempt integrate scripts to it.

mangosR2 is fork and not supported. Not mangos team problem.

goname teleport not to player pos, but to calculated _near_ pos. in different go creature, for example. maybe need swtch to exactly pos teleport. This will look not so nice in game but maybe more safe.

Warrning also can point ot possible problem case, but this specific know and not problematic from code work point. It possible can be fixed but noneone look at it for fix it.