Pre-WoD 4.3.4.15595 SpellPower and TalentTab field mistakes about wowdbdefs HOT 8 CLOSED

I'm sorry, but SpellPower does not have a definition for 4.3.4.15595 and neither does https://wowdev.wiki/DB/SpellPower. The same goes for TalentTab, which has no definition on here or at https://wowdev.wiki/DB/TalentTab.

I'm not sure what bug you're reporting here. Would you like to share those files you're reporting on, as those appear not to be files from this repository?

from wowdbdefs.

@bloerwald #27

from wowdbdefs.

@bloerwald you might consider reading the title again. There is a branch specifically for client builds prior to WoD:
https://github.com/wowdev/WoWDBDefs/tree/pre-wod

There you find all 4.x definitions and there's where I've dug up my named concerns

from wowdbdefs.

I see, I forgot about that branch. We may want to move this discussion over to the merge request for it.

On TalentTab I fully agree.

On SpellPower it appears to be a result of trying to match the only float value in there. Your observation is right, the powertype column sure isn’t that. I guess we have a column type change across versions?!

from wowdbdefs.

Keep in mind said branch isn't quite up to date or supported/battle-tested like master is.

from wowdbdefs.

@bloerwald I just got a helping hand and received a reversed spell power cost calculation function from the client and I have the answer now. Field 4 and 8 are both PowerCostPct. Field 4 is the fallback field if 8 is 0.f. I guess that's another of these functions that Blizz had implemented halfway for future expansions. So yeah. I guess field 8 is PowerCostPct and field 4 is PowerCostPct2 or other way arround.

@Marlamin I am aware of that. So I think it's a great opportunity to change that now that I had the pleasure to dig through every single field. And as far as I am aware, most of the namings are actually very accurate.

from wowdbdefs.

Hmk, will see if I can properly diff these and merge in any changes manually because it might have diverged a bit too far from master to be easily mergeable the usual way.

from wowdbdefs.

Thank you for looking into this and letting us know.

RE: SpellPower; you are right, they moved from just a numeric field PowerCost (x/100) to using PowerCostPct as a fallback if PowerCost was 0, to just using PowerCostPct - full change history is here if anyone is interested.

This has all been merged into master now along with the rest of that branch.
Any PRs and issues are more than welcome for any other changes/fixes!

from wowdbdefs.