use serverrules to know if you can get rps (so both target is gameplayer and not IsAllowedToAttack - guess you have nothing to change, simply use the method of the server rule
oh, when you implement the RP logic, can you take in consideration the RP multiplier (iirc there is one in SP ?)
For the patch, msn me if you want help : actually your code is certainly working, but if you could both change to what we suggest, and make it testable and safe to integrate (yes this is the why of the patch, and why it'll be except for tiny changes the way to work).
Anw, once again, this is not that your patch is not good, this is simply the SVN criterias are a bit differents
/bow !
