Tradeskill Group Emotes

[seveianrex]

{tradeskills.cpp}

in bool Client::TradeskillExecute, at approx. line 700, locate:

	if(spec->tradeskill == JEWELRY_MAKING){
		switch(GetAA(aaJewelCraftMastery)){
		case 1:
			AAChance = 10;
			break;
		case 2:
			AAChance = 25;
			break;
		case 3:
			AAChance = 50;
			break;
		}
	}

add immediately after:

const Item_Struct* item = NULL;

locate:

		_log(TRADESKILLS__TRACE, "Tradeskill success");

		itr = spec->onsuccess.begin();
		while(itr != spec->onsuccess.end()) {
			//should we check this crap?
			SummonItem(itr->first, itr->second);

add immediately after:

			item = database.GetItem(itr->first);
			if (this->GetGroup())
			{
				entity_list.MessageGroup(this,true,MT_Skills,"%s has successfully fashioned %s!",GetName(),item->Name);
			}

locate:

_log(TRADESKILLS__TRACE, "Tradeskill failed");

add immediately after:

if (this->GetGroup())
		{
			entity_list.MessageGroup(this,true,MT_Skills,"%s was unsuccessful in %s tradeskill attempt.",GetName(),GetGender() == 0 ? "his" : "her");
		}

[janusd]

This is actually emoting as a general emote, not just to group members.

[Derision]

The second parameter to entity_list.MessageGroup determines whether it is group only, or group + other people nearby. Setting it to true will make it group only, if that was what was desired.

[seveianrex]

Whoops. My bad! Thanks Derision for the fix..

[trevius]

I corrected the submitted code with Derision's suggestion. If I didn't do it right, lemme know and we can either fix it or you can post the corrected code.

[cavedude]

Edit: Oh, sorry I thought you meant the SVN. We're good there