255 is highest status.
And I'm sorry for going on the tangent I'm about to go on, but I want to point out something that's bugged me for a while now. If you're not serious about this change going into the CVS, you can disregard my entire message :lol: But if you are, please take this into consideration:
I'm not a big fan of hard coding status level requirements into the code when a GetGM() check would do. There are plenty of spots in the existing code where status level requirements are hard coded. Problem is, your requirement for status 250+ may suit your server, but how about my server where I want status 100+, and then there's Joe Bob's Server that wants 50+, and Betty Sue's Server that wants 150+.
More examples of where status checks are hard-coded, and the minimum status was evidently arbitrarily set by the coder:
- GM-only items (ie, non-legit where #si is enabled but you get "You are not a GM to summon this item" when attempting summon the best of items. The minimum status for "being a GM to summon this item" is hard coded at 100. Although each item can be set in the database to either be or not be GM-only-summonable, it requires a C++ coder to get in the code if you want that status requirement of 100 raised/lowered.
- Most, if not all, of the GM slash (/) commands that become enabled when you turn your #gm flag on have a status level check. Now, some kind of check is necessary, otherwise some people know how to trick their client into thinking their GM flag is on when it's not, and then the client unlocks those commands for them. But instead of checking GetGM(), it checks status requirements. The status requirements vary depending on command, and many of them require 80+. Ironically, 80 is higher than Guide status, and guides on EQlive summoned, zoned, goto'd, and all that other stuff as part of their day to day activities. Why did they not use either GetGM(), or at least check the requirement of the # equivalent of the commands where applicable?
Also, I checked the items database for an item id 100001. In PEQ's db, there is no such item. Does the items database from Packetcollector have one?
Don't take my criticism personally, especially if you were just offering the code as a "whoever cares to do this," since that "whoever" can replace 250 with anything they like