Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

View maps entries are not restored in "finally" block #5234

Merged
merged 1 commit into from
May 20, 2023

Conversation

santoszv
Copy link

View maps entries are not restored in "finally" block of method "ViewMetadataImpl.createMetadataView" because "currentViewMapShallowCopy" is always empty

@BalusC BalusC changed the base branch from master to 4.0 May 20, 2023 12:17
@BalusC BalusC changed the base branch from 4.0 to master May 20, 2023 12:17
@BalusC
Copy link
Contributor

BalusC commented May 20, 2023

Can you the next time please target exactly the branch of the version you want to see this fixed in? This PR is currently targeting master instead of 4.0 where the first occurrence of the bug is.

@BalusC
Copy link
Contributor

BalusC commented May 20, 2023

@arjantijms FYI: this was thus a regression introduced in ebf6a08#diff-c73ecc7df840224c9e86287fdc64900a282fa24f60a2da13d3606d140572a2dfL102 I'll assume that this was just an oversight and thus not intentional.

@BalusC BalusC merged commit 75ab32d into eclipse-ee4j:master May 20, 2023
@BalusC
Copy link
Contributor

BalusC commented May 20, 2023

Cherry picked for 4.0 via 40b8441

@BalusC
Copy link
Contributor

BalusC commented May 20, 2023

All good now. Thank you very much for creating the PR and improving Mojarra!

@arjantijms arjantijms added the 5.0 label Jul 6, 2023
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