Technique of Master Wu AA

**AndMetal** · #1 09-13-2008, 05:46 AM

It looks like support for this was added in a few weeks ago, however, it looks like there's a slight issue: if the check is successful, it adds another regular attack instead of the special attack used. Here's the current code:

Code:

   80 		if(max_damage > 0 && GetClass() == MONK && skill != THROWING)
   81 		{
   82 			int specl = GetAA(aaTechniqueofMasterWu) * 20;
   83 			if(specl == 100 || specl >= MakeRandomInt(0,100))
   84 			{
   85 				Attack(who);
   86 				if(20 > MakeRandomInt(0,100))
   87 					Attack(who);
   88 			}
   89 		}

If it's changed to this:

Code:

	if(max_damage > 0 && GetClass() == MONK && skill != THROWING)
	{
		int specl = GetAA(aaTechniqueofMasterWu) * 20;
		if(specl == 100 || specl >= MakeRandomInt(0,100))
		{
			who->Damage(this, max_damage, SPELL_UNKNOWN, skill, false);
			if(20 > MakeRandomInt(0,100))
				who->Damage(this, max_damage, SPELL_UNKNOWN, skill, false);
		}
	}

that should trigger the additional 1-2 special attacks (flying kick, etc).

So_1337 · #2 09-26-2008, 08:52 PM

Had a pair of monks both mention this lately; I remember it being implemented awhile back, but never heard past that. Didn't even realize it was broken until they'd said so. Great work, hope this makes it in soon =)

So_1337 · #3 10-02-2008, 04:18 PM

I see this stickied still, but I'm getting reports from our monks saying it's working perfectly for them. Did this go in with one of the recent revisions? Running revision 27.

**Derision** · #4 10-02-2008, 04:34 PM

Quote:

Originally Posted by So_1337

I see this stickied still, but I'm getting reports from our monks saying it's working perfectly for them. Did this go in with one of the recent revisions? Running revision 27.

Looking at the ChangeLog, Cavedude incorporated this into Revision 2 on 25th September.

So_1337 · #5 10-02-2008, 04:56 PM

Sorry. I'm blind, I guess. Thanks for the spare set of eyes, Derision.

In any case, it's working and getting great feedback, so great work AndMetal =)

/unsticky =P

**cavedude** · #6 10-02-2008, 05:58 PM

The stickies are handled by KLS to aid in getting the code into the official CVS.

**KLS** · #7 10-02-2008, 06:33 PM

It wasn't meant to go in because it doesn't handle both versions of AC correctly. Apparently we don't even consider such things anymore though.

Edit: Wow that sounded a lot more emo than I meant it too. Anyway I'd like to fix it for the old system. Are the hits supposed to be affected by melee avoidance or are they a guaranteed thing?