Skip to content

fix: merge property descriptors when getter and setter come from different interfaces (#23939) (CP: 25.1)#23940

Merged
vaadin-bot merged 1 commit into25.1from
cherry-pick-23939-to-25.1-1773847290448
Mar 18, 2026
Merged

fix: merge property descriptors when getter and setter come from different interfaces (#23939) (CP: 25.1)#23940
vaadin-bot merged 1 commit into25.1from
cherry-pick-23939-to-25.1-1773847290448

Conversation

@vaadin-bot
Copy link
Copy Markdown
Collaborator

This PR cherry-picks changes from the original PR #23939 to branch 25.1.

Original PR description

BeanUtil's dedup loop was replacing a read-only descriptor with a write-only one (or vice versa) instead of merging them. This caused BeanPropertySet to lose the getter when a parent interface provided it and a child interface provided the setter.

Fixes #20814

…erent interfaces (#23939)

BeanUtil's dedup loop was replacing a read-only descriptor with a
write-only one (or vice versa) instead of merging them. This caused
BeanPropertySet to lose the getter when a parent interface provided it
and a child interface provided the setter.

Fixes #20814
@vaadin-bot
Copy link
Copy Markdown
Collaborator Author

This PR is eligible for auto-merging policy, so it has been approved automatically. If there are pending conditions, auto merge (with 'squash' method) has been enabled for this PR [Message is sent from bot]

@vaadin-bot vaadin-bot enabled auto-merge (squash) March 18, 2026 15:31
@sonarqubecloud
Copy link
Copy Markdown

@vaadin-bot
Copy link
Copy Markdown
Collaborator Author

This PR is eligible for auto-merging policy, so it has been approved automatically. If there are pending conditions, auto merge (with 'squash' method) has been enabled for this PR[Message is sent from bot]

@github-actions
Copy link
Copy Markdown

Test Results

 1 385 files  ±0   1 385 suites  ±0   1h 25m 4s ⏱️ -59s
 9 901 tests +4   9 830 ✅ +4  71 💤 ±0  0 ❌ ±0 
10 374 runs  +4  10 294 ✅ +4  80 💤 ±0  0 ❌ ±0 

Results for commit 02b218e. ± Comparison against base commit cd1a5d2.

@vaadin-bot vaadin-bot merged commit 9e6f846 into 25.1 Mar 18, 2026
29 of 30 checks passed
@vaadin-bot vaadin-bot deleted the cherry-pick-23939-to-25.1-1773847290448 branch March 18, 2026 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants