foundryvtt / dnd5e

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

Item Choice Advancement Level Prerequisite: Being able to modify past choices with new options once higher levels are achieved #3471

Closed Zanderaf closed 6 months ago

Zanderaf commented 6 months ago

Sorry if the title of this issue is a bit weird. Did a bit of testing with the https://github.com/foundryvtt/dnd5e/pull/3383 and some stuff I saw from Zhell's test with it yesterday and had a bit of a comment.

With the SRD Eldritch Invocations in the 3.2 branch as an example, leveling up to level 2 and selecting "Armor of Shadows" & "Agonizing Blast" to start off.

From what I understand, the last part of the Eldritch Invocation feature mentions: "Additionally, when you gain a level in this class, you can choose one of the invocations you know and replace it with another invocation that you could learn at that level."

Wouldn't that mean you would be able to slowly change out your "lower level" invocations for higher level ones as you level up instead of the item choice options only being restricted to what you could be taking at the level the option was granted instead of the current level.

arbron commented 6 months ago

Modify Choices is working as intended, it will always modify choices in the state as if you were the level you originally were when making the choice, so level restrictions will still apply.

There is another PR for handling replacing earlier choices made with ItemChoiceAdvancement: https://github.com/foundryvtt/dnd5e/issues/2299