MagicSpells icon indicating copy to clipboard operation
MagicSpells copied to clipboard

Suggestion: Check permissions more thoroughly

Open keenerb opened this issue 9 years ago • 0 comments

I haven't been able to compile and test your version, but Magicspells has always suffered from a pretty severe problem with dynamically adding/removing spells with permissions.

If you add a spell with magicspells.grant.spellname and cast.spellname, if you revoke those permissions the player is still able to cast the spells until they relog or you execute /cast reload as an admin.

I believe that permissions should be re-checked at spell cast, at spell selection via item, and at any /cast command, and non-granted/unknown spells should be purged from the characters spell list.

Also, an option to disable spellbook saving/loading would be nice.

keenerb avatar Dec 11 '15 04:12 keenerb