foundryvtt / dnd5e

An implementation of the 5th Edition game system for Foundry Virtual Tabletop (http://foundryvtt.com).
MIT License
333 stars 222 forks source link

In "Cast" activity, set "Material" as an ignored property by default #4588

Closed krbz999 closed 1 hour ago

krbz999 commented 3 hours ago

Seems like a sensible default.

from the Basic Rules on DDB: "Some magic items allow the user to cast a spell from the item. The spell is cast at the lowest possible spell and caster level, doesn't expend any of the user's spell slots, and requires no components, unless the item's description says otherwise."

Emphasis mine. Whether that includes Verbal and Somatic, I am not entirely sure.

etiquettestartshere commented 3 hours ago

I could be wrong, but fairly sure 'no components' refers to all three (Verbal, Somatic, Material), as indicated by Components in the basic rules, so it would probably be fine to set all three V S M as ignored

krbz999 commented 1 hour ago

I suppose this default should only apply if the parent item is a physical item. A feat that allows you to cast Misty Step for example shouldn't necessarily remove components.

arbron commented 1 hour ago

I suppose this default should only apply if the parent item is a physical item. A feat that allows you to cast Misty Step for example shouldn't necessarily remove components.

Much harder to do defaults that only apply for only one item type since they would have to be handled outside data models. Probably simpler for now to just fill those by default and let people delete them if it isn't on a physical item.