Skip to content

Don't compute the initial value of dynamic properties in the editor#14650

Merged
uckelman merged 1 commit into
vassalengine:release-3.7from
rlament:13609-dynamic-prop-init
Jun 3, 2026
Merged

Don't compute the initial value of dynamic properties in the editor#14650
uckelman merged 1 commit into
vassalengine:release-3.7from
rlament:13609-dynamic-prop-init

Conversation

@rlament

@rlament rlament commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

This prevents evaluation and replacement of expressions for the initial value of a dynamic property while in the editor. Expression evaluation is deferred to the player module.

I did some simple tests initializing from a global property with pieces in the Piece Palette and At-Start stacks. This change may make it harder to debug issues with expression evaluation being deferred to the player.

Closes #13609

This prevents evaluation and replacement of beanshell expressions
for the initial value of a dynamic property while in the editor.
Expression evaluation is deferred to the player module.
@rlament
rlament changed the base branch from master to release-3.7 April 18, 2026 00:20
@uckelman uckelman added this to the 3.7.23 milestone Jun 3, 2026
@uckelman
uckelman merged commit 6aa9dbc into vassalengine:release-3.7 Jun 3, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dynamic Property initial value is evaluated

3 participants