- Status Closed
- Percent Complete
- Task Type Feature Request
- Category
-
Assigned To
Glen Kaukola - Operating System
- Severity Low
- Priority
- Reported Version
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#715 - easier way to setskills
The gms wish for an easier way to setskill, working from the chat window is not always easy if we need many skills changed.
To be able to edit stat and skill levels from withing the GUI
ID | Project | Summary | Priority | Severity | Assigned To | Progress | |
---|---|---|---|---|---|---|---|
1985 | PlaneShift | FS#1985 - List of all GM feature requests [or dev lvl requests] | Low | Aresilek Besolez |
Closed by Lanarel
27.11.2009 15:41
Reason for closing:
Additional comments about closing:
27.11.2009 15:41
Reason for closing:
Additional comments about closing:
Fixed by Decker
/setskill me all 200?
that does not work if in events we need certain skills at certain lvls
What kind of event would that be? I can’t imagine a situation in a roleplaying event where your skill in sword making needs to be exactly 56 for the event to succeed. I’m not saying make a GUI for this would be bad, just that I don’t see how it would be more effective and really help events.
Besides, if you need a specific set of skills set before an event for some reason, you could always just make a shortcut beforehand. Or don’t multiple /setskill commands in a shortcut work? If so, that’d be a bug, but it’s known already.
With
FS#2324andFS#2378I don’t think we need skills modifiable via GUI, at least not soon.However, there came up a suggestion that should be taken into account. For testing and events, it might be good to easily set skills equally of a target like
/setskill me match harnquist
whereas the first argument is whose skills get set, “match” is keyword and second argument is the one whose skills get cloned.
Note: both arguments are regular “target”s
assigning to decker.
implement it like this /setskill [targetdestination] [targetorigin] copy
also /setskill [?argetdestination] copy [targetorigin] might work, probably even better
use targetdestination as a generic target but lock it to same target of issuer or put a validate call to a special permission used just for it to be extended to any target.
Here’s what I came up with (from svn diff). Although I’m not sure why you’re asking for a special permission check. There’s already a check in SetSkill to see if the client has “setskill others” access.
Gah! Is there no way to attach files then? the <code> junk didn’t seem to work too well.
Diffs are here:
http://pastebin.ca/1681781 http://pastebin.ca/1681783
Messed up some indentation in adminmanager.cpp. Here’s a fixed up diff:
http://pastebin.ca/1681795
1) you can do a single diff for both the files
2) you should search also with findobjectbystring (and then check you got an actor) else it’s not possible to use pids or eids
All set:
http://pastebin.ca/1682699
committed look ok
works fine. “/setskill me copy someplayer” set most of my skills to 0, and some to different values.