-
Notifications
You must be signed in to change notification settings - Fork 240
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add god admin command #1924
add god admin command #1924
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall probably minimal impact, but this implementation deviates from what I recall the real command doing.
Aside from the flourishes at the end of it, the only concerning part is the training/specing of every skill including ones that were depreciated, and possible desync on crash maybe
I won't have time to play test this for a few more days however
any idea what the retail command did specifically? |
from what I remember it just set whatever currently spec/trained skills to a specific level (not sure what), and i think changed the level to match that same number... it's probably not that big of a deal being an admin only function |
this is definitely more fun, and serves the same purpose. i can not figure out why i need two calls to setmaxvitals() to get the hp to update in the client. it was working back when i was first looking at this, now it needs the second call. stam and mana seem to update just fine with the first call. also it appears to only be the client, calling /showstats reports the correct values in the terminal for health - just not in the client. ultimately not a huge deal, just leave the second call to setmaxvitals in there... but it may be something that affects other areas so worth noting |
add /god admin command functionality