Regen AAs (discussion)

[AndMetal]

I didn't realize it until just now, but a lot of these AAs are already being utilized using the aa_effects table in the database, so as a result, we're double dipping the benefit.

They use the ZoneDatabase::FillAAEffects() function in zone/AA.cpp to apply any stats, etc, basically as a spell without any stacking issues. So, it looks like a lot of these AA checks will need to be taken from the source and integrated into the DB, then any missing spell effects will just need to be added into the source.

Anyways, mainly wanted to make note of this so I didn't forget...

Edit: actually, now I'm confused, because the source indicates it's just for the client's benefit:

/*
Every AA can send the client effects, which are purely for client side effects.
Essentially it's like being able to attach a very simple version of a spell to
Any given AA, it has 4 fields:
skill_id = spell effect id
slot = ID slot, doesn't appear to have any impact on stacking like real spells, just needs to be unique.
base1 = the base field of a spell
base2 = base field 2 of a spell, most AAs do not utilize this
example:
	skill_id = SE_STA
	slot = 1
	base1 = 15
	This would if you filled the abilities struct with this make the client show if it had
	that AA an additional 15 stamina on the client's stats
*/

[ChaosSlayer]

yeah it would be nice if i could reattach any given AA to any given spell wihout doing any recoding

[KLS]

Yes those are basically what the client will show. Just because you have it in the db there wont mean it will be implemented. Like if you want the client to have an enduring breath AA you set it to happen there. Yet if you want something like spell casting mastery it has to happen in the server. Some need both clientside and serverside effects so you do both.

[AndMetal]

Ok, that makes sense. I am a little curious though as to way we don't iterate through these on the server side instead of adding manual code changes for each AA. Maybe too much overhead from having to parse through each AA?

[KLS]

I had a plan to do it tbh but never got around to implementing it. Someday we probably will, it will require identifying and implementing a lot of missing spell effects though.

[AndMetal]

That sounds like a good place for me to start

If we can get it moved to a system that iterates through the values, we can always have a default case, just like we do currently in spell_effects.cpp, that doesn't do anything. That way, we can get the AAs in, then focus on the spells effects.

Just thinking out loud, but from what I can see & remember, bonuses for the client are called from Client::CalcBonuses() in zone/bonuses.cpp:

void Client::CalcBonuses()
{
	_ZP(Client_CalcBonuses);
	memset(&itembonuses, 0, sizeof(StatBonuses));
	CalcItemBonuses(&itembonuses);
	CalcEdibleBonuses(&itembonuses);
	
	RecalcWeight();
	
	CalcSpellBonuses(&spellbonuses);
	
	CalcAC();
	CalcATK();
	CalcHaste();
	
	CalcSTR();
	CalcSTA();
	CalcDEX();
	CalcAGI();
	CalcINT();
	CalcWIS();
	CalcCHA();
	
	CalcMR();
	CalcFR();
	CalcDR();
	CalcPR();
	CalcCR();
	
	CalcMaxHP();
	CalcMaxMana();
	CalcMaxEndurance();
	
	rooted = FindType(SE_Root);
}

so we already have functions for the big stuff. I would think we could then start with a similar function for AAs. I'm sure we could have it piggyback spellbonuses, although it might be better to add aabonuses into the Mob class in zone/mob.h, just in case we want to count it separately (but have to change several places in the source to look at itembonuses + spellbonuses + aabonuses).

I've been working on it, but don't have anything usable yet.

P.S. I moved the discussion since it doesn't really apply to just the Regen AA anymore.