[sorvani]

Originally Posted by lerxst2112

client_process.cpp

void Client::CalcRestState() {

	// This method calculates rest state HP and mana regeneration.
	// The client must have been out of combat for RuleI(Character, RestRegenTimeToActivate) seconds,
	// must be sitting down or mounted, and must not have any detrimental spells affecting them.
	//
	if(!RuleI(Character, RestRegenPercent))
		return;

	RestRegenHP = RestRegenMana = RestRegenEndurance = 0;

        if(AggroCount || !IsSitting() || !GetHorseId())
		return;

IsSitting will be FALSE if GetHorseID = TRUE so the ! will cause it to still do what it was doing before, return if you are not sitting. Should it be more like:

        if(AggroCount || !(IsSitting() || GetHorseId()) )