Skip to content

fix: make configuration init parameters unmodifiable - #25076

Merged
mcollovati merged 1 commit into
mainfrom
issues/9299-mutable-init-parameters
Jul 30, 2026
Merged

fix: make configuration init parameters unmodifiable#25076
mcollovati merged 1 commit into
mainfrom
issues/9299-mutable-init-parameters

Conversation

@caalador

Copy link
Copy Markdown
Contributor

Configuration properties should
not accept changes after generation.

Now any tries to change property value
is met with an exception.

Closes #9299

Configuration properties should
not accept changes after generation.

Now any tries to change property value
is met with an exception.

Closes #9299
@github-actions

Copy link
Copy Markdown
Contributor

Test Results

 1 445 files  +1   1 445 suites  +1   1h 38m 38s ⏱️ - 3m 12s
10 321 tests +5  10 254 ✅ +5  67 💤 ±0  0 ❌ ±0 
10 775 runs  +5  10 707 ✅ +5  68 💤 ±0  0 ❌ ±0 

Results for commit 5925274. ± Comparison against base commit 690ccfa.

@sonarqubecloud

Copy link
Copy Markdown

@mcollovati
mcollovati added this pull request to the merge queue Jul 30, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 30, 2026
@mcollovati
mcollovati added this pull request to the merge queue Jul 30, 2026
Merged via the queue into main with commit b3c6a36 Jul 30, 2026
35 checks passed
@mcollovati
mcollovati deleted the issues/9299-mutable-init-parameters branch July 30, 2026 10:04
@vaadin-bot

Copy link
Copy Markdown
Collaborator

This ticket/PR has been released with Vaadin 25.3.0-alpha8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PropertyDeploymentConfiguration.getInitParameters() returns internal mutable map

3 participants