Here's what I see:
Code:
// Check for No Drop Hacks
Mob* with = trade->With();
if (((with && with->IsClient() && dst_slot_id>=3000 && dst_slot_id<=3007) || // Trade
(dst_slot_id >= 2500 && dst_slot_id <= 2550)) // Shared Bank
&& GetInv().CheckNoDrop(src_slot_id)
&& RuleI(World, FVNoDropFlag) == 0 || RuleI(Character, MinStatusForNoDropExemptions) < Admin() && RuleI(World, FVNoDropFlag) == 2) {
DeleteItemInInventory(src_slot_id);
WorldKick();
return false;
}
loosely interpreted as:
if
(((<trade>
) ||
(<shbank>
)) &&
(<chknodrop>
) &&
(<rule:fvnodrop==2>
) ||
(<rule:nodropexemption<admin>
) &&
(<rule:fvnodrop==0>
))
There's probably a missing encapsulation somewhere in there somewhere.